You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by "Niall Pemberton (JIRA)" <ji...@apache.org> on 2007/05/25 00:40:16 UTC
[jira] Updated: (BEANUTILS-212) [beanutils] Generic implementations
of toString, hashCode, equals() for DynaBean and DynaClass
[ https://issues.apache.org/jira/browse/BEANUTILS-212?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Niall Pemberton updated BEANUTILS-212:
--------------------------------------
Affects Version/s: (was: Nightly Builds)
1.7.0
Fix Version/s: (was: 1.8.0)
LATER THAN 1.8.0
A few comments:
* I've applied the patch from BEANUTILS-233 which adds hashcode() and equals() implementations to DynaProperty.
* DynaUtils.buildHashCode(DynaBean) will throw a null pointer exception if the property value is null
* DynaUtils.buildHashCode(DynaBean) doesn't take into account the DynaBean implementation - seems to me like it should?
* DynaUtils.buildHashCode(DynaClass) doesn't take into account the DynaClass implementation - seems to me like it should?
I also think that implementing toString() is best done by the person using it - since how muchof little information they would want and the format is very much down to how its being used. Lang has a whole set of string builder classes which do this sort of thing so I'm inclinded to not implement any of this. Personally I have my own custom impl. that does this.
The other part of the problem is I don't understand the implications of hashCode() enough and my big fear is that it would screw things up - so I'm going to punt this for now to post 1.8.0. Apologies if, due to my shortcommings, this is annoying.
> [beanutils] Generic implementations of toString, hashCode, equals() for DynaBean and DynaClass
> ----------------------------------------------------------------------------------------------
>
> Key: BEANUTILS-212
> URL: https://issues.apache.org/jira/browse/BEANUTILS-212
> Project: Commons BeanUtils
> Issue Type: Improvement
> Components: DynaBean
> Affects Versions: 1.7.0
> Environment: Operating System: All
> Platform: All
> Reporter: Kris Nuttycombe
> Priority: Minor
> Fix For: LATER THAN 1.8.0
>
> Attachments: AbstractDynaBean.java, AbstractDynaClass.java, DynaUtils.java, patch.txt
>
>
> This patchset includes a new DynaUtils class containing builder methods for hash
> codes and string representations of DynaBeans and DynaClasses, abstract base
> classes for DynaBean and DynaClass that use DynaUtils to implement these
> methods, and patches for existing DynaBean and DynaClass implementations to make
> them extend these abstract base classes.
> Kris
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org