Re: Object retval = (Object)bindings.get(var); // why not OK?
metaperl wrote:
I'm wondering why I cannot simply cast the return result of .get() to
satisfy the compiler expectation that the method return an object.
Instead I have to assign the result of .get() to a variable and then
return that.
You never have to upcast, so a cast to Object is never needed.
package redick;
import java.util.*;
import java.util.Iterator;
public class Environment {
Please do not embed TABs in Usenet posts.
/*
* map from a variable name to an Object - which may be a datum or a
* procedure
*/
public Map<String,Object> bindings = new HashMap<String, Object>();
public Environment parent;
public void put(String var, Object value) {
bindings.put(var, value);
}
public Object get(String var) {
Object retval = bindings.get(var); // cast not enough
There is no cast here, and indeed, none is needed.
/* Cannot comment this section out */
if (retval == null) {
return (Object)null;
} else {
return retval;
}
/* End required section */
}
public String toString() { return bindings.toString(); }
}
You could simply "return retval;" or even "return bindings.get(var);"
public Object get(String var)
{
return bindings.get(var);
}
You haven't shown us a complete (i.e., "SSCCE") example of the erroneous code
that gives you trouble, or what the error is. I don't see what your
difficulty could have been.
There are times when cover methods delegate with one-line bodies, but
sometimes that's an indicator that you can use the Map directly instead of
wrapping it.
--
Lew