fix nutzam#1289

默认启用mirror池,500w次的简单json字符串解码耗时,从66秒降到26秒,对得起观众了吧

Author: wendal

performance/org/nutz/json/JsonFaster.java

@@ -10,36 +10,45 @@ public class JsonFaster {
 
     @Test
     public void json() {
-        nutzJson(10000);
-        fastJson(10000);
+        String jsonStr = "{name:'wendal'}";
+        nutzJson(100000, jsonStr);
+        fastJson(100000, jsonStr);
+        System.gc();
+        
+        int t = 5000*10000;
 
         Stopwatch sw = Stopwatch.begin();
-        nutzJson(50*10000);
+        nutzJson(t, jsonStr);
         sw.stop();
-        System.out.println("Nutz-Json 50w次耗时: " + sw.getDuration());
+        System.out.println("Nutz-Json "+t+"次耗时: " + sw.getDuration());
+        System.gc();
 
-        sw.start();
-        fastJson(50*10000);
-        System.out.println("Fast-Json 50w次耗时: " + sw.getDuration());
+        sw = Stopwatch.begin();
+        //fastJson(t, jsonStr);
+        sw.stop();
+        System.out.println("Fast-Json "+t+"次耗时: " + sw.getDuration());
+        System.gc();
 
         //-------------------------------------------------------------------
-        sw.start();
-        nutzJson(50*10000);
+        sw = Stopwatch.begin();
+        nutzJson(t, jsonStr);
         sw.stop();
-        System.out.println("Nutz-Json 50w次耗时: " + sw.getDuration());
+        System.out.println("Nutz-Json "+t+"次耗时: " + sw.getDuration());
+        System.gc();
 
-        sw.start();
-        fastJson(50*10000);
+        sw = Stopwatch.begin();
+        //fastJson(t, jsonStr);
         sw.stop();
-        System.out.println("Fast-Json 50w次耗时: " + sw.getDuration());
+        System.out.println("Fast-Json "+t+"次耗时: " + sw.getDuration());
+        System.gc();
     }
 
     public void nutzJson(int time) {
         JsonObject obj = new JsonObject();
         obj.setName("wendal");
         for (int i = 0; i < time; i++) {
             String jsonStr = Json.toJson(obj);
-            obj = Json.fromJson(JsonObject.class, jsonStr);
+            Json.fromJson(jsonStr);
         }
     }
 
@@ -52,6 +61,18 @@ public void fastJson(int time) {
         }
     }
 
+    public void nutzJson(int time, String jsonStr) {
+        for (int i = 0; i < time; i++) {
+            Json.fromJson(JsonObject.class, jsonStr);
+        }
+    }
+    
+    public void fastJson(int time, String jsonStr) {
+        for (int i = 0; i < time; i++) {
+            JSON.parseObject(jsonStr, JsonObject.class);
+        }
+    }
+    
     public static void main(String[] args) throws Throwable {
         Thread.sleep(60*1000);
         new JsonFaster().json();

src/org/nutz/conf/NutConf.java

@@ -134,4 +134,5 @@ public static void clear() {
     }
 
     public static boolean USE_FASTCLASS = !Lang.isAndroid;
+    public static boolean USE_MIRROR_CACHE = true;
 }

src/org/nutz/lang/Mirror.java

@@ -22,6 +22,7 @@
 
 import org.nutz.castor.Castors;
 import org.nutz.castor.FailToCastObjectException;
+import org.nutz.conf.NutConf;
 import org.nutz.lang.born.BornContext;
 import org.nutz.lang.born.Borning;
 import org.nutz.lang.born.BorningException;
@@ -43,6 +44,11 @@
  * @param <T>
  */
 public class Mirror<T> {
+    
+    @SuppressWarnings("rawtypes")
+    static Map<Type, Mirror> mirrorCache = new HashMap<Type, Mirror>();
+    
+    protected BornContext<T> emtryArgsBornContext;
 
     private static class DefaultTypeExtractor implements TypeExtractor {
 
@@ -156,9 +162,16 @@ public static <T> Mirror<T> me(Type type) {
         if (null == type) {
             return null;
         }
-        Mirror<T> mir = (Mirror<T>) Mirror.me(Lang.getTypeClass(type));
-        mir.type = type;
-        return mir;
+        if (NutConf.USE_MIRROR_CACHE) {
+            Mirror<T> mir = mirrorCache.get(type);
+            if (mir == null) {
+                mir = (Mirror<T>) Mirror.me(Lang.getTypeClass(type));
+                mir.type = type;
+                mirrorCache.put(type, mir);
+            }
+            return mir;
+        }
+        return (Mirror<T>) Mirror.me(Lang.getTypeClass(type));
     }
 
     private Class<T> klass;
@@ -983,7 +996,14 @@ public Borning<T> getBorningByArgTypes(Class<?>... argTypes) throws BorningExcep
      * @return 新对象
      */
     public T born(Object... args) {
-        BornContext<T> bc = Borns.eval(klass, args);
+        BornContext<T> bc;
+        if (NutConf.USE_MIRROR_CACHE && args.length == 0) {
+            if (emtryArgsBornContext == null)
+                emtryArgsBornContext = Borns.eval(klass, args);
+            bc = emtryArgsBornContext;
+        }
+        else
+            bc = Borns.eval(klass, args);
         if (null == bc)
             throw new BorningException(klass, args);
 

src/org/nutz/mapl/impl/convert/ObjConvertImpl.java

@@ -8,6 +8,7 @@
 import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.Map.Entry;
 import java.util.Set;
 import java.util.Stack;
 
@@ -192,35 +193,24 @@ private Object injectObj(Object model, Mirror<?> mirror) {
         Map<String, ?> map = (Map<String, ?>) model;
 
         JsonEntity jen = Json.getEntity(mirror);
-        for (String key : map.keySet()) {
+        for (Entry<String, ?> en : map.entrySet()) {
+            Object val = en.getValue();
+            if (val == null)
+                continue;
+            String key = en.getKey();
             JsonEntityField jef = jen.getField(key);
             if (jef == null) {
                 continue;
             }
-
-            Object val = map.get(jef.getName());
-            if (val == null) {
-                continue;
-            }
-
             if (isLeaf(val)) {
                 if (val instanceof El) {
                     val = ((El) val).eval(context);
                 }
-                // zzh@2012-09-14: 暂时去掉 createBy 吧
-                // jef.setValue(obj, Castors.me().castTo(jef.createValue(obj,
-                // val, null), Lang.getTypeClass(jef.getGenericType())));
-                // jef.setValue(obj, jef.createValue(obj, val, null));
                 jef.setValue(obj, Mapl.maplistToObj(val, jef.getGenericType()));
                 continue;
             } else {
                 path.push(key);
-                // jef.setValue(obj, Mapl.maplistToObj(val,
-                // me.getGenericsType(0)));
                 jef.setValue(obj, Mapl.maplistToObj(val, jef.getGenericType()));
-                // zzh@2012-09-14: 暂时去掉 createBy 吧
-                // jef.setValue(obj, jef.createValue(obj, val,
-                // me.getGenericsType(0)));
             }
         }
         return obj;