Use Boolean.valueOf(b) instead of new Boolean(b)
------------------------------------------------

                 Key: IBATIS-509
                 URL: https://issues.apache.org/jira/browse/IBATIS-509
             Project: iBatis for Java
          Issue Type: Improvement
          Components: SQL Maps
    Affects Versions: 2.3.2
            Reporter: Derek Alexander


A performance improvement could be realised by replacing "new Boolean(b)" with 
"Boolean.valueOf(b)" in BooleanTypeHandler.

>From the JDK API:

public static Boolean valueOf(boolean b)

    Returns a Boolean instance representing the specified boolean value. If the 
specified boolean value is true, this method returns Boolean.TRUE; if it is 
false, this method returns Boolean.FALSE. If a new Boolean instance is not 
required, this method should generally be used in preference to the constructor 
Boolean(boolean), as this method is likely to to yield significantly better 
space and time performance.  (Since: 1.4)

I can't think of a good reason why a new Boolean instance should be required 
from BooleanTypeHandler (rather than one of the two static instances mentioed 
above). 

NOTE: Above method is since JDK 1.4 only, couldn't find details of minimum JVM 
targetted. If it is <1.4 you can always use "(b) ? Boolean.TRUE : 
Boolean.FALSE" instead.





-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to