DBのデータの取り扱い方

[labeneko]

• 自分ではとてもわかりやすいと思っているが、多分これは他の人からすると気持ち悪くて仕方ないと思う
• helper、model以外になにを作るべきだったのだろう
  • dao？

[labeneko]

• 多分実運用上ではActiveRecordを扱うライブラリを使用すると思いますが、今回SQLiteを生で実装するにあたり、自分で設計しないといけなくなったので結構つまづきました
• 他人のコードを読んでも、たいていActivityに直書きであてにならない感じだったです
• なかなか自分で良い設計を思いつかなかったので、自分なりによく分かる方法で書いたつもりです
• 改めて見るとmodelのコードが本来daoですることなのかなって思いました
• 今はgetListとかaddとかdeleteとか単純なものしか扱っていないのでこれで良いかもですが、それ以外のSQLを発行したい時は、別途modelとdaoに分けるべきだなあと感じました

[chuross]

NoteModelがどのような概念のクラスなのかにもよりますが、構造体の扱い方が一貫していないので個人的には気持ち悪いですね。

• getListでNoteのリストを返すのに、updateではidとタイトルと説明を別々に渡さなきゃいけないのは微妙。
  • Noteの一部だけ変更したい場合でも呼び出し側はNoteの構造に対して参照しないといけない

こことか

• https://github.com/labeneko/Notepad/blob/master/app/src/main/java/com/example/takahiro_tsuno/notepad/EditActivity.java#L173

僕が書くとしたらこんな感じ

public class Note {
    private long id = -1;
    private String title;
    private description;

    // コンストラクタ
    public Note(String title, String description);

    // コンストラクタ
    public Note(long id, String title, String description);

    public long getId();

    public void setId(long id);

    public String getTitle();

    public void setTitle(String title);

    public String getDescription();

    public void setDescription(String description);
}

public class NoteDao{

    public List<Note> findAll();

    // @return idが振られたNoteを返す
    // 受け取ったNoteに追加時のidをセットして返す
    public Note add(Note note);

    // @return 変更したNoteを返す
    // updatedAt等の値が処理前と変わってるはず
    public Note update(Note note);

    public boolean delete(Note note);
}

この書き方だと呼び出し側はNoteDaoからくるNoteだけ取り扱えばいいし、更新の時はNoteの中身だけ変更してNoteDao#updateに渡すだけで良くなる。