Added simple inverted String conversion.

When binding a primitive to an EditText, a common
pattern is to use '@{"" + value}'. This, however,
doesn't allow for a two-way data binding expressions.
To mitigate the need for conversion functions, a simple
inversion for this expression wsa implemented that
just converts value from a String when possible.

This CL also fixes a bug in which a method matching
the first parameter was always chosen, reguardless of
the second and further parameters.

Change-Id: I36828d9f54d2073965358fceb140b2d5e6328919

5 files changed, 105 insertions, 42 deletions

diff --git a/compiler/src/main/java/android/databinding/tool/reflection/InjectedBindingClass.java b/compiler/src/main/java/android/databinding/tool/reflection/InjectedClass.java
index 4759a9a6..45857991 100644
--- a/compiler/src/main/java/android/databinding/tool/reflection/InjectedBindingClass.java
+++ b/compiler/src/main/java/android/databinding/tool/reflection/InjectedClass.java
@@ -18,6 +18,7 @@ package android.databinding.tool.reflection;
 
 import android.databinding.tool.util.StringUtils;
 
+import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
 import java.util.Map;
@@ -29,18 +30,23 @@ import java.util.Map;
  *
  * @see ModelAnalyzer#injectViewDataBinding(String, Map, Map)
  */
-public class InjectedBindingClass extends ModelClass {
+public class InjectedClass extends ModelClass {
     private final String mClassName;
     private final String mSuperClass;
-    private final Map<String, String> mVariables;
-    private final Map<String, String> mFields;
+    private final List<InjectedMethod> mMethods = new ArrayList<InjectedMethod>();
+    private final List<InjectedField> mFields = new ArrayList<InjectedField>();
 
-    public InjectedBindingClass(String className, String superClass, Map<String, String> variables,
-            Map<String, String> fields) {
+    public InjectedClass(String className, String superClass) {
         mClassName = className;
         mSuperClass = superClass;
-        mVariables = variables;
-        mFields = fields;
+    }
+
+    public void addField(InjectedField field) {
+        mFields.add(field);
+    }
+
+    public void addMethod(InjectedMethod method) {
+        mMethods.add(method);
     }
 
     @Override
@@ -183,12 +189,11 @@ public class InjectedBindingClass extends ModelClass {
     protected ModelField[] getDeclaredFields() {
         ModelClass superClass = getSuperclass();
         final ModelField[] superFields = superClass.getDeclaredFields();
-        final int fieldCount = superFields.length + mFields.size();
+        final int initialCount = superFields.length;
+        final int fieldCount = initialCount + mFields.size();
         final ModelField[] fields = Arrays.copyOf(superFields, fieldCount);
-        int index = superFields.length;
-        for (String fieldName : mFields.keySet()) {
-            final String fieldType = mFields.get(fieldName);
-            fields[index++] = new InjectedBindingClassField(fieldName, fieldType);
+        for (int i = 0; i < mFields.size(); i++) {
+            fields[i + initialCount] = mFields.get(i);
         }
         return fields;
     }
@@ -197,15 +202,11 @@ public class InjectedBindingClass extends ModelClass {
     protected ModelMethod[] getDeclaredMethods() {
         ModelClass superClass = getSuperclass();
         final ModelMethod[] superMethods = superClass.getDeclaredMethods();
-        final int methodCount = superMethods.length + (mVariables.size() * 2);
+        final int initialCount = superMethods.length;
+        final int methodCount = initialCount + mMethods.size();
         final ModelMethod[] methods = Arrays.copyOf(superMethods, methodCount);
-        int index = superMethods.length;
-        for (String variableName : mVariables.keySet()) {
-            final String variableType = mVariables.get(variableName);
-            final String getterName = "get" + StringUtils.capitalize(variableName);
-            methods[index++] = new InjectedBindingClassMethod(this, getterName, variableType, null);
-            final String setterName = "set" + StringUtils.capitalize(variableName);
-            methods[index++] = new InjectedBindingClassMethod(this, setterName, "void", variableType);
+        for (int i = 0; i < mMethods.size(); i++) {
+            methods[i + initialCount] = mMethods.get(i);
         }
         return methods;
     }
diff --git a/compiler/src/main/java/android/databinding/tool/reflection/InjectedBindingClassField.java b/compiler/src/main/java/android/databinding/tool/reflection/InjectedField.java
index d15cc908..85719f1c 100644
--- a/compiler/src/main/java/android/databinding/tool/reflection/InjectedBindingClassField.java
+++ b/compiler/src/main/java/android/databinding/tool/reflection/InjectedField.java
@@ -24,11 +24,11 @@ import java.util.Map;
  *
  * @see ModelAnalyzer#injectViewDataBinding(String, Map, Map)
  */
-public class InjectedBindingClassField extends ModelField {
+public class InjectedField extends ModelField {
     private final String mType;
     private final String mName;
 
-    public InjectedBindingClassField(String name, String type) {
+    public InjectedField(String name, String type) {
         mName = name;
         mType = type;
     }
diff --git a/compiler/src/main/java/android/databinding/tool/reflection/InjectedBindingClassMethod.java b/compiler/src/main/java/android/databinding/tool/reflection/InjectedMethod.java
index 20e24824..f47442b7 100644
--- a/compiler/src/main/java/android/databinding/tool/reflection/InjectedBindingClassMethod.java
+++ b/compiler/src/main/java/android/databinding/tool/reflection/InjectedMethod.java
@@ -25,18 +25,22 @@ import java.util.Map;
  *
  * @see ModelAnalyzer#injectViewDataBinding(String, Map, Map)
  */
-public class InjectedBindingClassMethod extends ModelMethod {
-    private final InjectedBindingClass mContainingClass;
+public class InjectedMethod extends ModelMethod {
+    private final InjectedClass mContainingClass;
     private final String mName;
-    private final String mReturnType;
-    private final String mParameter;
-
-    public InjectedBindingClassMethod(InjectedBindingClass containingClass, String name, String returnType,
-            String parameter) {
+    private final String mReturnTypeName;
+    private final String[] mParameterTypeNames;
+    private ModelClass[] mParameterTypes;
+    private ModelClass mReturnType;
+    private boolean mIsStatic;
+
+    public InjectedMethod(InjectedClass containingClass, boolean isStatic, String name,
+            String returnType, String... parameters) {
         mContainingClass = containingClass;
         mName = name;
-        mReturnType = returnType;
-        mParameter = parameter;
+        mIsStatic = isStatic;
+        mReturnTypeName = returnType;
+        mParameterTypeNames = parameters;
     }
 
     @Override
@@ -46,11 +50,18 @@ public class InjectedBindingClassMethod extends ModelMethod {
 
     @Override
     public ModelClass[] getParameterTypes() {
-        if (mParameter != null) {
-            ModelClass parameterType = ModelAnalyzer.getInstance().findClass(mParameter, null);
-            return new ModelClass[] { parameterType };
+        if (mParameterTypes == null) {
+            if (mParameterTypeNames == null) {
+                mParameterTypes = new ModelClass[0];
+            } else {
+                mParameterTypes = new ModelClass[mParameterTypeNames.length];
+                ModelAnalyzer modelAnalyzer = ModelAnalyzer.getInstance();
+                for (int i = 0; i < mParameterTypeNames.length; i++) {
+                    mParameterTypes[i] = modelAnalyzer.findClass(mParameterTypeNames[i], null);
+                }
+            }
         }
-        return new ModelClass[0];
+        return mParameterTypes;
     }
 
     @Override
@@ -60,8 +71,10 @@ public class InjectedBindingClassMethod extends ModelMethod {
 
     @Override
     public ModelClass getReturnType(List<ModelClass> args) {
-        ModelClass returnType = ModelAnalyzer.getInstance().findClass(mReturnType, null);
-        return returnType;
+        if (mReturnType == null) {
+            mReturnType = ModelAnalyzer.getInstance().findClass(mReturnTypeName, null);
+        }
+        return mReturnType;
     }
 
     @Override
@@ -81,7 +94,7 @@ public class InjectedBindingClassMethod extends ModelMethod {
 
     @Override
     public boolean isStatic() {
-        return false;
+        return mIsStatic;
     }
 
     @Override
@@ -108,4 +121,26 @@ public class InjectedBindingClassMethod extends ModelMethod {
     public boolean isVarArgs() {
         return false;
     }
+
+    @Override
+    public String toString() {
+        StringBuilder sb = new StringBuilder("public ");
+        if (mIsStatic) {
+            sb.append("static ");
+        }
+        sb.append(mReturnTypeName)
+                .append(' ')
+                .append(mName)
+                .append("(");
+        if (mParameterTypeNames != null) {
+            for (int i = 0; i < mParameterTypeNames.length; i++) {
+                if (i != 0) {
+                    sb.append(", ");
+                }
+                sb.append(mParameterTypeNames[i]);
+            }
+        }
+        sb.append(')');
+        return sb.toString();
+    }
 }
diff --git a/compiler/src/main/java/android/databinding/tool/reflection/ModelAnalyzer.java b/compiler/src/main/java/android/databinding/tool/reflection/ModelAnalyzer.java
index 8943200b..995ae219 100644
--- a/compiler/src/main/java/android/databinding/tool/reflection/ModelAnalyzer.java
+++ b/compiler/src/main/java/android/databinding/tool/reflection/ModelAnalyzer.java
@@ -18,6 +18,7 @@ package android.databinding.tool.reflection;
 import android.databinding.tool.reflection.annotation.AnnotationAnalyzer;
 import android.databinding.tool.util.L;
 import android.databinding.tool.util.Preconditions;
+import android.databinding.tool.util.StringUtils;
 
 import java.util.HashMap;
 import java.util.Map;
@@ -83,8 +84,8 @@ public abstract class ModelAnalyzer {
     private ModelClass mViewStubType;
 
     private static ModelAnalyzer sAnalyzer;
-    private final Map<String, InjectedBindingClass> mInjectedClasses =
-            new HashMap<String, InjectedBindingClass>();
+    private final Map<String, InjectedClass> mInjectedClasses =
+            new HashMap<String, InjectedClass>();
 
     protected void setInstance(ModelAnalyzer analyzer) {
         sAnalyzer = analyzer;
@@ -234,10 +235,33 @@ public abstract class ModelAnalyzer {
 
     public abstract TypeUtil createTypeUtil();
 
+    public ModelClass injectClass(InjectedClass injectedClass) {
+        mInjectedClasses.put(injectedClass.getCanonicalName(), injectedClass);
+        return injectedClass;
+    }
+
     public ModelClass injectViewDataBinding(String className, Map<String, String> variables,
             Map<String, String> fields) {
-        InjectedBindingClass injectedClass = new InjectedBindingClass(className,
-                ModelAnalyzer.VIEW_DATA_BINDING, variables, fields);
+        InjectedClass injectedClass = new InjectedClass(className,
+                ModelAnalyzer.VIEW_DATA_BINDING);
+
+        if (fields != null) {
+            for (String name : fields.keySet()) {
+                String type = fields.get(name);
+                injectedClass.addField(new InjectedField(name, type));
+            }
+        }
+        if (variables != null) {
+            for (String name : variables.keySet()) {
+                String type = variables.get(name);
+                String capName = StringUtils.capitalize(name);
+                String setName = "set" + capName;
+                String getName = "get" + capName;
+                injectedClass.addMethod(new InjectedMethod(injectedClass, false, getName, type));
+                injectedClass.addMethod(new InjectedMethod(injectedClass, false, setName, "void",
+                        type));
+            }
+        }
         mInjectedClasses.put(className, injectedClass);
         return injectedClass;
     }
diff --git a/compiler/src/main/java/android/databinding/tool/reflection/ModelMethod.java b/compiler/src/main/java/android/databinding/tool/reflection/ModelMethod.java
index 5bd214e3..3ec7ff8d 100644
--- a/compiler/src/main/java/android/databinding/tool/reflection/ModelMethod.java
+++ b/compiler/src/main/java/android/databinding/tool/reflection/ModelMethod.java
@@ -96,6 +96,9 @@ public abstract class ModelMethod {
             final ModelClass arg = args.get(i);
             final ModelClass thisParameter = getParameter(i, parameterTypes);
             final ModelClass thatParameter = other.getParameter(i, otherParameterTypes);
+            if (thisParameter.equals(thatParameter)) {
+                continue;
+            }
             final int diff = compareParameter(arg, thisParameter, thatParameter);
             if (diff != 0) {
                 return diff < 0;