Android / Java中的OOP

Question

我目前正在开发一个flashcard-app，因为我不喜欢现有的，第二个原因是我想提高我的java / android技能。

在我的应用程序中，我试图使用OOP。 我的卡存储在sqlite数据库中。

现在我的问题：目前我的对象“卡”代表一个单独的Flashcard有自己的DatabaseAdapter，它可以用它来更新自己的状态，即它的点。

 Card card = new Card();
 card.setPoints(10); //<-- Also update its row in the database

或者做这样的事情会更好：

 DBAdapter dbAdapter = new DBAdapter();
 Card card = new Card();
 dbAdapter.setPoints(Card);//<-- Updates Card and Database!

你觉得什么更好？

Thx提前！

Answer 1

您在第二个示例中所做的是不可取的，因为您正在创建副作用。 更新数据库也在更新卡，这是错误的。 更新数据库应该做一件事，那就是更新数据库。

我更喜欢第一种方法，因为卡知道如何更新自己，但你也犯了同样的错误，更新数据库。 让数据库知道如何更新自己，并且卡知道如何更新自己。 我认为它实现了关注的清晰分离。

例如：

card.setPoints(points)//update card and only card
dbAdapter.setPoints(card) //update the database and nothing else.

Answer 2

绝对是第一个。 设置点的类应该不知道如何设置（或存储）点。 这是一种“责任分离”。

想象一下，如果您决定从数据库更改为ACME Dilithium Crystal Accumulator（TM）。 在选项1和选项2中需要更改多少代码？

Answer 3

我猜不是两个。

 dbAdapter.update(card.getId(),card.getPoint());

语法应该是

  public boolean update(int id, int value)

您的DbAdapter应该只有存储或检索的方法，删除不应该特定于卡功能的数据。

在上面的示例中， card.getId()返回与Card关联的URI ，然后它只更新整数值，因此您的代码具有职责分离和隔离。

Answer 4

如果上面提到的两个代码片段都有效，我宁愿选择第一个代码片段，因为代码看起来更具可读性，更直观且更短！

我希望这有帮助。