Issue #2636 - provide overloadings of ObjectReader.readValue() taking Class valueType

Author: robinroos

src/main/java/com/fasterxml/jackson/databind/ObjectReader.java

@@ -1148,6 +1148,14 @@ public <T> T readValue(InputStream in) throws IOException
                 _considerFilter(_parserFactory.createParser(ctxt, in), false));
     }
 
+    /**
+     * Overloading to alert compiler as to the valueType
+     */
+    public <T> T readValue(InputStream src, Class<T> valueType) throws IOException
+    {
+        return (T) forType(valueType).readValue(src);
+    }
+
     /**
      * Method that binds content read from given input source,
      * using configuration of this reader.
@@ -1163,6 +1171,14 @@ public <T> T readValue(Reader r) throws IOException
                 _considerFilter(_parserFactory.createParser(ctxt, r), false));
     }
 
+    /**
+     * Overloading to alert compiler as to the valueType
+     */
+    public <T> T readValue(Reader src, Class<T> valueType) throws IOException
+    {
+        return (T) forType(valueType).readValue(src);
+    }
+
     /**
      * Method that binds content read from given JSON string,
      * using configuration of this reader.
@@ -1178,6 +1194,14 @@ public <T> T readValue(String content) throws IOException
                 _considerFilter(_parserFactory.createParser(ctxt, content), false));
     }
 
+    /**
+     * Overloading to alert compiler as to the valueType
+     */
+    public <T> T readValue(String src, Class<T> valueType) throws IOException
+    {
+        return (T) forType(valueType).readValue(src);
+    }
+
     /**
      * Method that binds content read from given byte array,
      * using configuration of this reader.
@@ -1193,6 +1217,14 @@ public <T> T readValue(byte[] content) throws IOException
                 _considerFilter(_parserFactory.createParser(ctxt, content), false));
     }
 
+    /**
+     * Overloading to alert compiler as to the valueType
+     */
+    public <T> T readValue(byte[] content, Class<T> valueType) throws IOException
+    {
+        return (T) forType(valueType).readValue(content);
+    }
+
     /**
      * Method that binds content read from given byte array,
      * using configuration of this reader.
@@ -1207,7 +1239,15 @@ public <T> T readValue(byte[] content, int offset, int length) throws IOExceptio
         return (T) _bindAndClose(ctxt,
                 _considerFilter(_parserFactory.createParser(ctxt, content, offset, length), false));
     }
-    
+
+    /**
+     * Overloading to alert compiler as to the valueType
+     */
+    public <T> T readValue(byte[] content, int offset, int length, Class<T> valueType) throws IOException
+    {
+        return (T) forType(valueType).readValue(content, offset, length);
+    }
+
     @SuppressWarnings("unchecked")
     public <T> T readValue(File f) throws IOException
     {
@@ -1217,6 +1257,14 @@ public <T> T readValue(File f) throws IOException
                 _considerFilter(_parserFactory.createParser(ctxt, f), false));
     }
 
+    /**
+     * Overloading to alert compiler as to the valueType
+     */
+    public <T> T readValue(File src, Class<T> valueType) throws IOException
+    {
+        return (T) forType(valueType).readValue(src);
+    }
+
     /**
      * Method that binds content read from given input source,
      * using configuration of this reader.
@@ -1239,6 +1287,14 @@ public <T> T readValue(URL url) throws IOException
                 _considerFilter(_parserFactory.createParser(ctxt, url), false));
     }
 
+    /**
+     * Overloading to alert compiler as to the valueType
+     */
+    public <T> T readValue(URL src, Class<T> valueType) throws IOException
+    {
+        return (T) forType(valueType).readValue(src);
+    }
+
     /**
      * Convenience method for converting results from given JSON tree into given
      * value type. Basically short-cut for:
@@ -1255,6 +1311,14 @@ public <T> T readValue(JsonNode node) throws IOException
                 _considerFilter(treeAsTokens(node, ctxt), false));
     }
 
+    /**
+     * Overloading to alert compiler as to the valueType
+     */
+    public <T> T readValue(JsonNode src, Class<T> valueType) throws IOException
+    {
+        return (T) forType(valueType).readValue(src);
+    }
+
     @SuppressWarnings("unchecked")
     public <T> T readValue(DataInput input) throws IOException
     {
@@ -1264,6 +1328,14 @@ public <T> T readValue(DataInput input) throws IOException
                 _considerFilter(_parserFactory.createParser(ctxt, input), false));
     }
 
+    /**
+     * Overloading to alert compiler as to the valueType
+     */
+    public <T> T readValue(DataInput content, Class<T> valueType) throws IOException
+    {
+        return (T) forType(valueType).readValue(content);
+    }
+
     /*
     /**********************************************************
     /* Deserialization methods; JsonNode ("tree")

src/test/java/com/fasterxml/jackson/databind/ObjectReaderTest.java

@@ -198,6 +198,24 @@ public void testPointerLoading() throws Exception {
         assertEquals(1234, pojo.name.get("value"));
     }
 
+
+    public void testCanPassResultToOverloadedMethod() throws Exception {
+        final String source = "{\"foo\":{\"bar\":{\"caller\":{\"name\":{\"value\":1234}}}}}";
+
+        ObjectReader reader = MAPPER.readerFor(POJO.class).at("/foo/bar/caller");
+
+        process(reader.readValue(source, POJO.class));
+    }
+
+    void process(POJO pojo) {
+        // do nothing - just used to show that the compiler can choose the correct method overloading to invoke
+    }
+
+    void process(String pojo) {
+        // do nothing - just used to show that the compiler can choose the correct method overloading to invoke
+    }
+
+
     public void testPointerLoadingAsJsonNode() throws Exception {
         final String source = "{\"foo\":{\"bar\":{\"caller\":{\"name\":{\"value\":1234}}}}}";