fix #710

Author: wendal

src/org/nutz/ioc/impl/ObjectMakerImpl.java

@@ -114,13 +114,10 @@ public ObjectProxy make(IocMaking ing, IocObject iobj) {
                 IocField ifld = iobj.getFields()[i];
                 try {
                     ValueProxy vp = ing.makeValue(ifld.getValue());
-                    fields[i] = FieldInjector.create(mirror, ifld.getName(), vp);
+                    fields[i] = FieldInjector.create(mirror, ifld.getName(), vp, ifld.isOptional());
                 }
                 catch (Exception e) {
-                	if (!ifld.isOptional()) {
-                        throw Lang.wrapThrow(e, "Fail to eval Injector for field: '%s'", ifld.getName());
-                	}
-            		log.info("ioc field create fail, and it is optional, ignore error", e);
+                	throw Lang.wrapThrow(e, "Fail to eval Injector for field: '%s'", ifld.getName());
                 }
             }
             dw.setFields(fields);
@@ -135,11 +132,8 @@ public ObjectProxy make(IocMaking ing, IocObject iobj) {
         }
         // 当异常发生，从 context 里移除 ObjectProxy
         catch (Throwable e) {
-            if (log.isWarnEnabled())
-                log.warn(String.format("IobObj: \n%s", iobj.toString()), e);
             ing.getContext().remove(iobj.getScope(), ing.getObjectName());
-            throw new IocException("create ioc bean fail name="
-                                   + ing.getObjectName(), e);
+            throw new IocException("create ioc bean fail name=" + ing.getObjectName(), e);
         }
 
         // 返回

src/org/nutz/ioc/weaver/FieldInjector.java

@@ -2,25 +2,40 @@
 
 import org.nutz.ioc.IocMaking;
 import org.nutz.ioc.ValueProxy;
+import org.nutz.lang.Lang;
 import org.nutz.lang.Mirror;
 import org.nutz.lang.inject.Injecting;
+import org.nutz.log.Log;
+import org.nutz.log.Logs;
 
 public class FieldInjector {
+	
+	private static final Log log = Logs.get();
 
-    public static FieldInjector create(Mirror<?> mirror, String fieldName, ValueProxy vp) {
+    public static FieldInjector create(Mirror<?> mirror, String fieldName, ValueProxy vp, boolean optional) {
         FieldInjector fi = new FieldInjector();
         fi.valueProxy = vp;
         fi.inj = mirror.getInjecting(fieldName);
+        fi.optional = optional;
         return fi;
     }
 
     private ValueProxy valueProxy;
     private Injecting inj;
+    private boolean optional;
 
     private FieldInjector() {}
 
     void inject(IocMaking ing, Object obj) {
-        Object value = valueProxy.get(ing);
-        inj.inject(obj, value);
+    	try {
+    		Object value = valueProxy.get(ing);
+            inj.inject(obj, value);
+    	} catch (Throwable e) {
+			if (optional) {
+				log.info("field inject fail, but this field is optional, ignore error", e);
+				return;
+			}
+			throw Lang.wrapThrow(e);
+		}
     }
 }