Question

I have a bean whose properties I want to access via reflection. I receive the property names in String form. The beans have getter methods for their private fields.

I am currently getting the field using getDeclaredField(fieldName), making it accessible by using setAccessible(true) and then retrieving its value using get.

Another way to go about it would be to capitalize the field name and add get to the front of it, and then get the method by that name from the class and finally invoke the method to get the value of the private field.

Which way is better?

EDIT

Perhaps I should explain what I mean by “better”… By “better”, I mean in the sense of best-practices. Or, if there are any subtle caveats or differences.

Answer 1

It depends of your use, but in general I would prefer to use the getter as this is the “normal” way and will in more cases do the thing the developer of the class expects gets done.

In principle, if the developer of the class has made the field private he is free to do as he pleases, like for instance removing it later if it can be calculated in another way. Then the fieldaccess will break, hopefully immediately, if you are unlucky 3 months later when nobody remembers anymore.

Note that there a libraries like apache commons BeanUtils (I believe there is one in Spring too) which does that for you and offer a more sane interface, like a hash map for example.