Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.2
    • Component/s: None
    • Labels:
      None
    • Environment:

      Operating System: other
      Platform: Other

      Description

      I have modified reflectionEquals. The new signature adds the ability to exclude
      a list of fields from testing.

      reflectionEquals(Object lhs, Object rhs, String[] excludeFields)

      This is useful when the occasional field is to be excluded when many fields need
      to be tested. The alternative is to hard-code each field in the equals method
      and hope that if an additional field is added, the equals method is updated.

        Activity

        Hide
        Pete Gieser added a comment -

        Created an attachment (id=18101)
        Add new reflectionEquals signatures

        Show
        Pete Gieser added a comment - Created an attachment (id=18101) Add new reflectionEquals signatures
        Hide
        Pete Gieser added a comment -

        Created an attachment (id=18102)
        junit test for new reflectionEquals behavior

        Show
        Pete Gieser added a comment - Created an attachment (id=18102) junit test for new reflectionEquals behavior
        Hide
        Pete Gieser added a comment -

        Created an attachment (id=18139)
        New reflectionHashCode signature with excludeFields

        Corresponding reflectionHashCode signature

        Show
        Pete Gieser added a comment - Created an attachment (id=18139) New reflectionHashCode signature with excludeFields Corresponding reflectionHashCode signature
        Hide
        Pete Gieser added a comment -

        Created an attachment (id=18140)
        junit test for new reflectionHashCode behavior

        Show
        Pete Gieser added a comment - Created an attachment (id=18140) junit test for new reflectionHashCode behavior
        Hide
        Henri Yandell added a comment -

        Valueable addition, and perusing the diffs I can't see anything wrong with
        Pete's code.

        svn ci -m "Adding new excludeFields functionality for EqualsBuilder and
        HashCodeBuilder, as submitted by Pete Gieser, Bugzilla: #39315"
        Sending src/java/org/apache/commons/lang/builder/EqualsBuilder.java
        Sending src/java/org/apache/commons/lang/builder/HashCodeBuilder.java
        Sending src/test/org/apache/commons/lang/builder/EqualsBuilderTest.java
        Sending src/test/org/apache/commons/lang/builder/HashCodeBuilderTest.java
        Transmitting file data ....
        Committed revision 396461.

        Show
        Henri Yandell added a comment - Valueable addition, and perusing the diffs I can't see anything wrong with Pete's code. svn ci -m "Adding new excludeFields functionality for EqualsBuilder and HashCodeBuilder, as submitted by Pete Gieser, Bugzilla: #39315" Sending src/java/org/apache/commons/lang/builder/EqualsBuilder.java Sending src/java/org/apache/commons/lang/builder/HashCodeBuilder.java Sending src/test/org/apache/commons/lang/builder/EqualsBuilderTest.java Sending src/test/org/apache/commons/lang/builder/HashCodeBuilderTest.java Transmitting file data .... Committed revision 396461.

          People

          • Assignee:
            Unassigned
            Reporter:
            Pete Gieser
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development