Author: mcucchiara Date: Mon Nov 7 09:17:15 2011 New Revision: 1198666 URL: http://svn.apache.org/viewvc?rev=1198666&view=rev Log: OGNL-38 - Simplify conditionals.
Modified: commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/MapPropertyAccessor.java Modified: commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/MapPropertyAccessor.java URL: http://svn.apache.org/viewvc/commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/MapPropertyAccessor.java?rev=1198666&r1=1198665&r2=1198666&view=diff ============================================================================== --- commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/MapPropertyAccessor.java (original) +++ commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/MapPropertyAccessor.java Mon Nov 7 09:17:15 2011 @@ -62,30 +62,21 @@ public class MapPropertyAccessor { result = map.size(); } + else if ( "keys".equals( name ) || "keySet".equals( name ) ) + { + result = map.keySet(); + } + else if ( "values".equals( name ) ) + { + result = map.values(); + } + else if ( "isEmpty".equals( name ) ) + { + result = map.isEmpty() ? Boolean.TRUE : Boolean.FALSE; + } else { - if ( "keys".equals( name ) || "keySet".equals( name ) ) - { - result = map.keySet(); - } - else - { - if ( "values".equals( name ) ) - { - result = map.values(); - } - else - { - if ( "isEmpty".equals( name ) ) - { - result = map.isEmpty() ? Boolean.TRUE : Boolean.FALSE; - } - else - { - result = map.get( name ); - } - } - } + result = map.get( name ); } } else