replaced Class#newInstance with Constructor#newInstance

ozlerhakan · Nov 6, 2017 · 8055200 · 8055200
1 parent 940bb43
commit 8055200
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/main/java/com/poiji/bind/mapping/PoijiHandler.java b/src/main/java/com/poiji/bind/mapping/PoijiHandler.java
@@ -4,6 +4,7 @@
 import com.poiji.exception.IllegalCastException;
 import com.poiji.option.PoijiOptions;
 import com.poiji.util.Casting;
+import java.lang.reflect.InvocationTargetException;
 import org.apache.poi.ss.util.CellAddress;
 import org.apache.poi.xssf.usermodel.XSSFComment;
 
@@ -40,8 +41,8 @@ List<T> getDataset() {
     private <T> T newInstanceOf(Class<T> type) {
         T newInstance;
         try {
-            newInstance = type.newInstance();
-        } catch (IllegalAccessException | InstantiationException e) {
+            newInstance = type.getDeclaredConstructor().newInstance();
+        } catch (NoSuchMethodException| InvocationTargetException | IllegalAccessException | InstantiationException e) {
             throw new IllegalCastException("Cannot create a new instance of " + type.getName());
         }
 
@@ -65,7 +66,6 @@ private void setValue(String content, Class<? super T> type, int column) {
 
                 if (column == index.value()) {
 
-                    if (!field.isAccessible())
                         field.setAccessible(true);
 
                     Object o = Casting.castValue(fieldType, content, options);