2

У меня в классе куча полей и каждый раз писать такого рода проверки кажется мне не правильным, вот:

if (tmpProfileModel != null) {
                    if (tmpProfileModel.getLastName().equals(profileModel.getLastName()) &&
                            tmpProfileModel.getFirstName().equals(profileModel.getFirstName()) &&
                            tmpProfileModel.getBirthday().equals(profileModel.getBirthday()) &&
                            tmpProfileModel.getGender().equals(profileModel.getGender()) &&
                            tmpProfileModel.getFacebook().equals(profileModel.getFacebook()) &&
                            tmpProfileModel.getInstagram().equals(profileModel.getInstagram()) &&
                            tmpProfileModel.getEducation().equals(profileModel.getEducation()) &&
                            tmpProfileModel.getImage().equals(profileModel.getImage())) {
                        intent.putExtra(ProfileActivity.IS_EDITED_KEY, false);
                    } else {
                        intent.putExtra(ProfileActivity.IS_EDITED_KEY, true);
                        intent.putExtra(ProfileActivity.EDITED_OBJECT_KEY, profileModel);
                    }
                }

Еще дюжина полей, которое урезал. Теперь мне нужно проверить все эти поля на Null, будут еще много проверок и уже путаюсь.

Как вы решаете такого рода задачи?

1
  • Можно загнать проверяемые значения в списки, напр. Arrays.asList( some.foo(), some.bar(), some.baz() ).equals( Arrays.asList( other.foo(), other.bar(), other.baz() ) ).
    – zRrr
    12 фев 2018 в 12:24

2 ответа 2

3

Кроме выноса логики сравнения в отдельный метод (о чем написал @Serodv) можно использовать Objects.equals, который сравнивает два значения принимая во внимание null:

boolean isPartiallyEqual(Model other) {
    if(other==null) return false;
    return Objects.equals(getFirstName(), other.getFirstName()) &&
           Objects.equals(getBirthday(), other.getBirthday()) &&
           ...
}

Для большего упрощения можно подключить Apache Commons Lang и использовать EqualsBuilder.

С помощью этого класса можно проверить все поля с помощью рефлексии:

return EqualsBuilder.reflectionEquals(this, other);

либо построить свой объект сравнения с заданным списком полей:

if(other==null) return false;
return new EqualsBuilder()
             .append(getFirstName(), other.getFirstName())
             .append(getBirthday(), other.getBirthday())
             ...
             .isEquals();
6
  • Какой бы вы использовали, Objects или Apache?
    – DevOma
    12 фев 2018 в 13:52
  • @DevOma Для простого сравнения Objects, т.к. для простой задачи хочется обойтись стандартными классами. Но Apache Commons поддерживает сценарий с динамическим набором полей (рефлексией). 12 фев 2018 в 14:04
  • Еще вопрос в Apache Common можно ли проверить не поля а обьекты типа такого EqualsBuilder.reflectionEquals(newInstance, oldInstance)? То есть не прописывая все поля?
    – DevOma
    12 фев 2018 в 15:30
  • @DevOma Ну так для этого reflectionEquals и нужен. Метод динамически сравнивает все имеющиеся поля. Имейте ввиду, что рефлексия медленнее обычного доступа к полю. Также если имеются поля унаследованные от класса-родителя, то может потребоваться их включить/исключить. В общем, почитайте документацию к методам reflectionEquals (их несколько с разными аргументами) и тщательно протестируйте код. 12 фев 2018 в 15:35
  • Спасибо большое, помогло! А вот Objects.equals работает оказывается начиная с API19
    – DevOma
    12 фев 2018 в 15:45
2

Реализуйте в своём классе методы, обеспечивающие проверки

class SomeClass {
...
    boolean someEquals(SomeClass someObj) {
        return getFirstName().equals(someObj.getFirstName()) &&
               getBirthday().equals(someObj.getBirthday()) &&
               getGender().equals(someObj.getGender()) &&
               .....;
    }
}

ну и проверка

if (tmpProfileModel.someEquals(profileModel))
2
  • Я думал о таком, но хотелось бы еще короче..)
    – DevOma
    12 фев 2018 в 12:02
  • Могу предложить сгруппировать методы в какие-то группы, и затем пользоваться комбинациями сравнения. Но опять же, изменение структуры класса либо логики приведёт к неизбежному хаосу
    – Serodv
    12 фев 2018 в 12:06

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.