Menu

#1301 DataBinder.fetchValue throws unnecessary exception for map

3.5.2
closed
nobody
General (233)
5
2009-02-24
2009-02-23
No

ZK 3.5.2
org.zkoss.zkplus.databind.DataBinder
line 919

Code catches NoSuchMethodException then tests if it is a Map and handles things accordingly. The Java Exception handling process is expensive and should be avoided except for exceptional circumstances. The code should check first whether the bean is a map to avoid throwing the exception.

Discussion

  • Henri Chen

    Henri Chen - 2009-02-24
    • status: open --> closed-fixed
     
  • Henri Chen

    Henri Chen - 2009-02-24

    Agree. Fixed since 2/24.

     
  • Henri Chen

    Henri Chen - 2009-02-24
    • milestone: --> 3.5.2
    • status: closed-fixed --> closed
     

Log in to post a comment.