我正在开发一个允许用户管理帐户的应用程序.所以,假设我有一个Account类,代表用户的一个帐户:
class Account
{
public int id;
public String accountName;
public String accountIdentifier;
public String server;
public String notes;
}
我的equals方法如下所示:
public boolean equals(Object o)
{
if (this == o)
return true;
if (o == null || !(o instanceof Account))
return false;
Account other = (Account) o;
if (!accountIdentifier.equals(other.accountIdentifier))
return false;
if (!server.equals(other.server))
return false;
return true;
}
如您所见,我只是比较accountIdentifier和服务器,而不是其他字段.我选择这种方法有几个原因.
>我将帐户保存在列表中.当用户更新帐户时,通过更改帐户名称(这只是用户指定的名称来识别帐户)或备注,我可以执行accountList.set(accountList.indexOf(account),account);更新列表中的帐户.如果equals比较所有属性,这种方法将无法工作,我必须解决它(例如通过迭代列表并手动检查这些属性).
>这可能实际上更重要,但在考虑了一段时间之后才会出现在我脑海中.帐户由accountIdentifier及其所属的服务器唯一标识.用户可能决定重命名帐户,或更改备注,但它仍然是同一帐户.但如果服务器发生变化,我想我会认为它是一个不同的帐户. id只是一个内部ID,因为帐户存储在数据库中.即使更改了,如果accountIdentifier和服务器保持不变,该帐户仍被视为同一帐户.
我想说的是我基本上都是这样实现的,以便在应用程序的其余部分中使用更简洁,更简洁的代码.但是我不确定我是否在这里违反了一些规则,或者我是否正在做一些可能导致其他开发人员头痛的事情,如果有人正在使用我的应用程序的API.
是否可以只比较equals方法中的某些字段,还是应该比较所有字段?