Общий отражающий вспомогательный метод для equals и hashCode - PullRequest
3 голосов
/ 11 декабря 2010

Я подумываю о создании отражающего вспомогательного метода для equals и hashCode.

  • В случае равных вспомогательный метод просматривает API отражения для полей objectA и сравнивает их с полями objectB.
  • В случае hashCode вспомогательный метод просматриваетОтражает API для полей и вычисляет hashCode в цикле итерации.

Хорошо, что мне не нужно беспокоиться о пропущенных полях в моей реализации equals или hashCode.Плохая вещь, я думаю, производительность.Что вы думаете об этой идее?Пожалуйста, поделитесь своим мнением!

Это мой первый черновик для равных:

public final class ReflectiveEqualsHelper {

public static boolean isEqual(final Object a, final Object b) {
    if (!isTypeEqual(a, b)) {
        return false;
    }

    Field[] fields = getFields(a);

    Object valueA;
    Object valueB;
    String fieldName;
    for (int i = 0; i < fields.length; i++) {
        fieldName = fields[i].getName();
        valueA = getValueByFieldName(a, fieldName);
        valueB = getValueByFieldName(b, fieldName);
        if (!compare(valueA, valueB)) {
            return false;
        }
    }
    return true;
}

@SuppressWarnings("rawtypes")
private static Field[] getFields(final Object o) {
    Class clazz = o.getClass();
    Field[] fields = clazz.getDeclaredFields();
    return fields;
}

private static Field getField(final Object o, final String name) {
    try {
        Field field = o.getClass().getDeclaredField(name);
        return field;
    } catch (NoSuchFieldException e) {
        throw new RuntimeException(e);
    }
}

private static Object getValueByFieldName(final Object o, final String name) {
    Field field = getField(o, name);
    field.setAccessible(true);

    try {
        Object value = field.get(o);
        field.setAccessible(false);
        return value;
    } catch (IllegalAccessException e) {
        throw new RuntimeException(e);
    }

}

private static boolean areBothNull(final Object a, final Object b) {
    return (a == null && b == null);
}

private static boolean isTypeEqual(final Object a, final Object b) {
    if (areBothNull(a, b)) {
        return false;
    }

    return a.getClass().equals(b.getClass());
}

private static boolean compare(final Object a, final Object b) {
    if (a == null) {
        return false;
    } else if (b == null) {
        return false;
    }
    return a.equals(b);
}

}

public class ReflectiveEqualsHelperTest {

@Test
public void testIsEqual() {
    Vector a = new Vector(Long.valueOf(1L), 3L);
    Vector b = new Vector(Long.valueOf(1L), 3L);
    Vector c = new Vector(Long.valueOf(2L), 3L);
    boolean testA = ReflectiveEqualsHelper.isEqual(a, b);
    boolean testB = ReflectiveEqualsHelper.isEqual(a, c);
    boolean testC = ReflectiveEqualsHelper.isEqual(b, c);
    assertTrue(testA);
    assertFalse(testB);
    assertFalse(testC);
}

class Vector {
    public static final int STATIC = 1;

    private Long x;
    private long y;

    public Vector(Long x, long y) {
        super();
        this.x = x;
        this.y = y;
    }

    public Long getX() {
        return x;
    }

    public void setX(Long x) {
        this.x = x;
    }

    public long getY() {
        return y;
    }

    public void setY(long y) {
        this.y = y;
    }

    @Override
    public int hashCode() {
        final int prime = 31;
        int result = 1;
        result = prime * result + ((x == null) ? 0 : x.hashCode());
        result = prime * result + (int) (y ^ (y >>> 32));
        return result;
    }

    @Override
    public boolean equals(Object obj) {
        if (this == obj) {
            return true;
        }
        return ReflectiveEqualsHelper.isEqual(this, obj);
    }
}

}

Приветствия, Кевин

Ответы [ 5 ]

4 голосов
/ 11 декабря 2010

Взгляните на EqualsBuilder в Apache Commons и его reflectionEquals методов. В этой библиотеке также есть HashCodeBuilder + много других полезных вещей.

3 голосов
/ 11 декабря 2010

Я бы предложил Objects.hashCode Guava . Например:

public int hashCode() {
     return Objects.hashCode(getX(), getY(), getZ());
}

Метод Objects.equals должен помочь вам создать метод isEquals.

2 голосов
/ 11 декабря 2010

Производительность, безусловно, является большой проблемой из-за использования отражения, но есть и другие.

Иногда вы не хотите использовать все поля. В частности, при использовании самореферентной структуры это может привести к потенциально бесконечной рекурсии.

2 голосов
/ 11 декабря 2010

Это будет слишком дорого. Эти методы вызываются гораздо чаще, чем вы ожидаете. Не делай этого. Скорее используйте немного приличную IDE, такую ​​как Eclipse, IntelliJ или Netbeans, и пусть они автоматически генерируют equals() и hashCode(). Например, в Eclipse вы можете сделать это, щелкнув правой кнопкой мыши где-нибудь в исходном коде> Source> Generate hashCode и равный .

alt text

1 голос
/ 12 декабря 2010

Я передумал использовать рефлексивный подход из-за проблем с производительностью. Сейчас я использую утилиты EqualsBuilder и HashCodeBuilder проекта Apache commons (спасибо за предложения и отзывы), потому что они скрывают сложность методов. Для быстрой генерации метода #equals и #hashCode я использую плагин Fast Code Eclipse с настроенным шаблоном кода:


<template type="EQUALS_AND_HASHCODE_METHOD">
  <variation></variation>
  <variation-field></variation-field>
  <allow-multiple-variation></allow-multiple-variation>
  <class-pattern></class-pattern>
  <allowed-file-extensions>java</allowed-file-extensions>
  <number-required-classes>1</number-required-classes>
  <description>Generates the equals and hashCode method with EqualsBuilder and HashCodeBuilder</description>
  <template-body>
    <![CDATA[
      @Override
      public boolean equals(final Object obj) {
        if (obj == null) {
          return false;
        }
        if (obj == this) {
          return true;
        }
        if (obj.getClass() != getClass()) {
          return false;
        }

        ${class_name} rhs = (${class_name}) obj;
        return new EqualsBuilder().appendSuper(super.equals(obj))
        #foreach ($field in ${fields})
          .append(${field.name}, rhs.${field.name})
        #end
          .isEquals();
      }

      @Override
      public int hashCode() {
        return new HashCodeBuilder(17, 37).appendSuper(super.hashCode())
        #foreach ($field in ${fields})
          .append(${field.name})
        #end
          .toHashCode();
      }
    ]]>
  </template-body>
</template>

Я использую плагин Fast Code, потому что он может захватывать все поля выбранного класса. Но я не доволен юзабилити плагина. Было бы неплохо, если бы движок затмения кода мог это сделать. Если кто-нибудь знает подобный плагин, то дайте мне знать, пожалуйста!

Ура, Кевин

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...