Jazzepi:
効率的にオブジェクトの新しいコピーが任意の異なるフィールドを持っているかどうかを確認しようとすると、彼らがしなければ、地元のものを更新し、それをメモします。フィールドのいずれかが変更された場合、私は、データベースにオブジェクトを永続化する必要があります。私は、それゆえにブール値を持っていない場合、その呼び出しを行う必要はありません。
私は、リフレクションを使用せずにこれを行うには良い方法を考えることができませんでしたが、私は(フィールド名に文字列の参照を持っているでしょう)ので、コンパイラ担保安全性の不足のため、ここでリフレクションを使用したくない、といないすべてのフィールド同じタイプ(私はそこにいくつかのJava 8インスタントフィールドを持っている)です。
私が本当に避けたいと、フィールドが変更されたときの同期方法から/に加算または減算することを忘れないように持つの記帳です。明らかに引い方法が解除されますので、大したことではありませんが、誰かが新しいフィールドを更新するために覚えていない場合は追加することは怖いです。
public boolean syncWithFieldsFrom(User currentUser) {
boolean doesUserNeedUpdating = false;
if (!StringUtils.equals(email, currentUser.email)) {
email = currentUser.email;
doesUserNeedUpdating = true;
}
if (!StringUtils.equals(firstName, currentUser.firstName)) {
firstName = currentUser.firstName;
doesUserNeedUpdating = true;
}
if (!StringUtils.equals(lastName, currentUser.lastName)) {
lastName = currentUser.lastName;
doesUserNeedUpdating = true;
}
if (!StringUtils.equals(fullName, currentUser.fullName)) {
fullName = currentUser.fullName;
doesUserNeedUpdating = true;
}
return doesUserNeedUpdating;
}
shmosel:
これはちょっとやりすぎかもしれないが、あなたはフィールドを抽出し、それらに対してループを実行するためにラムダを使用することができます。私はあなたが簡単のためにゲッターとセッターを持っていると仮定します。
private static class Field<T> {
final Function<User, T> getter;
final BiConsumer<User, T> setter;
Field(Function<User, T> getter, BiConsumer<User, T> setter) {
this.getter = getter;
this.setter = setter;
}
boolean sync(User src, User dst) {
T srcField = getter.apply(src);
if (!Objects.equal(srcField, getter.apply(dst))) {
setter.accept(dst, srcField);
return true;
}
return false;
}
}
private static final List<Field<?>> FIELDS = Arrays.asList(
new Field<>(User::getEmail, User::setEmail),
new Field<>(User::getFirstName, User::setFirstName),
new Field<>(User::getLastName, User::setLastName),
new Field<>(User::getFullName, User::setFullName));
public boolean syncWithFieldsFrom(User currentUser) {
boolean updated = false;
for (Field<?> f : FIELDS) {
updated |= f.sync(currentUser, this);
}
return updated;
}