few cleanings for more redeable code

chennaione · Oct 26, 2015 · 01be162 · 01be162
1 parent 0150451
commit 01be162
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/library/src/main/java/com/orm/util/ReflectionUtil.java b/library/src/main/java/com/orm/util/ReflectionUtil.java
@@ -159,7 +159,7 @@ public static void setFieldValueFromCursor(Cursor cursor, Field field, Object ob
 
             if (colName.equalsIgnoreCase("id")) {
                 long cid = cursor.getLong(columnIndex);
-                field.set(object, Long.valueOf(cid));
+                field.set(object, cid);
             } else if (fieldType.equals(long.class) || fieldType.equals(Long.class)) {
                 field.set(object,
                         cursor.getLong(columnIndex));
@@ -212,7 +212,8 @@ public static void setFieldValueFromCursor(Cursor cursor, Field field, Object ob
                     Log.e("Sugar", "Enum cannot be read from Sqlite3 database. Please check the type of field " + field.getName());
                 }
             } else
-                Log.e("Sugar", "Class cannot be read from Sqlite3 database. Please check the type of field " + field.getName() + "(" + field.getType().getName() + ")");
+                Log.e("Sugar", "Class cannot be read from Sqlite3 database. Please check the type of field "
+                        + field.getName() + "(" + field.getType().getName() + ")");
         } catch (IllegalArgumentException e) {
             Log.e("field set error", e.getMessage());
         } catch (IllegalAccessException e) {
@@ -316,15 +317,15 @@ private static List<String> getAllClasses(Context context) throws PackageManager
                 }
             }
         } finally {
-            if (null != dexfile) dexfile.close();
+            if (dexfile != null) dexfile.close();
         }
         return classNames;
     }
 
     private static void populateFiles(File path, List<String> fileNames, String parent) {
         if (path.isDirectory()) {
             for (File newPath : path.listFiles()) {
-                if ("".equals(parent)) {
+                if (parent.isEmpty()) {
                     populateFiles(newPath, fileNames, path.getName());
                 } else {
                     populateFiles(newPath, fileNames, parent + "." + path.getName());
@@ -335,7 +336,7 @@ private static void populateFiles(File path, List<String> fileNames, String pare
             String classSuffix = ".class";
             pathName = pathName.endsWith(classSuffix) ?
                     pathName.substring(0, pathName.length() - classSuffix.length()) : pathName;
-            if ("".equals(parent)) {
+            if (parent.isEmpty()) {
                 fileNames.add(pathName);
             } else {
                 fileNames.add(parent + "." + pathName);