lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chantal Ackermann (Updated) (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (SOLR-3136) FacetField and Count : Override equals() and hashCode()
Date Wed, 15 Feb 2012 17:14:59 GMT

     [ https://issues.apache.org/jira/browse/SOLR-3136?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Chantal Ackermann updated SOLR-3136:
------------------------------------

    Description: 
Overriding equals() and hashCode() of FacetField and Count to provide equality based on content
(on their properties) would allow using these classes in sets and as keys in maps.

Implementation via commons-lang (which is already a dependency) should be fairly straight
forward?


For FacetFields, compare name, gap and end but not the list of counts:

{code:java}
public boolean equals(Object o) {
  if (o != null && o instanceof FacetField) {
    FacetField ff = (FacetField)o;
    return new EqualsBuilder().append(_name, ff._name).append(_gap, ff._gap).append(_end,
ff._end).isEquals();
  }
  return false;
}

public int hashCode() {
  return new HashCodeBuilder(33,11).append(_name).append(_gap).append(_end).toHashCode();
}
{code}


For Count compare FacetField, name and count:

{code:java}
public boolean equals(Object o) {
  if (o != null && o instanceof Count) {
    Count c = (Count)o;
    return new EqualsBuilder().append(_ff, c._ff).append(_name, c._name).append(_count, c._count).isEquals();
  }
  return false;
}

public int hashCode() {
  return new HashCodeBuilder(35,17).append(_ff).append(_name).append(_count).toHashCode();
}
{code}



  was:
Overriding equals() and hashCode() of FacetField and Count to provide equality based on content
(on their properties) would allow using these classes in sets and as keys in maps.

Implementation via commons-lang (which is already a dependency) should be fairly straight
forward?

For FacetFields, compare name, gap and end but not the list of counts:

public boolean equals(Object o) {
  if (o != null && o instanceof FacetField) {
    FacetField ff = (FacetField)o;
    return new EqualsBuilder().append(_name, ff._name).append(_gap, ff._gap).append(_end,
ff._end).isEquals();
  }
  return false;
}
public int hashCode() {
  return new HashCodeBuilder(33,11).append(_name).append(_gap).append(_end).toHashCode();
}

For Count compare FacetField, name and count:
public boolean equals(Object o) {
  if (o != null && o instanceof Count) {
    Count c = (Count)o;
    return new EqualsBuilder().append(_ff, c._ff).append(_name, c._name).append(_count, c._count).isEquals();
  }
  return false;
}
public int hashCode() {
  return new HashCodeBuilder(35,17).append(_ff).append(_name).append(_count).toHashCode();
}


    
> FacetField and Count : Override equals() and hashCode()
> -------------------------------------------------------
>
>                 Key: SOLR-3136
>                 URL: https://issues.apache.org/jira/browse/SOLR-3136
>             Project: Solr
>          Issue Type: Improvement
>          Components: clients - java
>    Affects Versions: 3.5
>            Reporter: Chantal Ackermann
>            Priority: Trivial
>
> Overriding equals() and hashCode() of FacetField and Count to provide equality based
on content (on their properties) would allow using these classes in sets and as keys in maps.
> Implementation via commons-lang (which is already a dependency) should be fairly straight
forward?
> For FacetFields, compare name, gap and end but not the list of counts:
> {code:java}
> public boolean equals(Object o) {
>   if (o != null && o instanceof FacetField) {
>     FacetField ff = (FacetField)o;
>     return new EqualsBuilder().append(_name, ff._name).append(_gap, ff._gap).append(_end,
ff._end).isEquals();
>   }
>   return false;
> }
> public int hashCode() {
>   return new HashCodeBuilder(33,11).append(_name).append(_gap).append(_end).toHashCode();
> }
> {code}
> For Count compare FacetField, name and count:
> {code:java}
> public boolean equals(Object o) {
>   if (o != null && o instanceof Count) {
>     Count c = (Count)o;
>     return new EqualsBuilder().append(_ff, c._ff).append(_name, c._name).append(_count,
c._count).isEquals();
>   }
>   return false;
> }
> public int hashCode() {
>   return new HashCodeBuilder(35,17).append(_ff).append(_name).append(_count).toHashCode();
> }
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message