DAO・DTOクラスのコードに不足部分がないか

Question

英単語の学習アプリケーションを作っています。
このサイトでアドバイスを頂き、だいぶ遅くはなったのですがDAO・DTOクラスを作ることにしました。
私が今認識しているそれぞれの役割は、
DAOクラス･･･DBより値を呼び出し、list等にする
DTOクラス･･･DBより取り出した値を保持する
です。まずこの認識が合っているかも少し心配なところではあります。

そして自分なりに調べてそれぞれのクラスを作成してみました。
DAOで別で用意しているDataBaseHelperと接続し値を取り出しています。
DTOではgetした値を返しています。
しかし私が記述したコードでは宣言や記述必須なものなど抜けがあるかもしれません。指摘していただければ幸いです。

それと、このクラスをActivityで使うようにするには、
宣言したうえで、newするだけで、リストごと取ってこれるのでしょうか？それをActivityで用意する空のlistに入れればよいのでしょうか。

public class WordDataInfo {

    public class WordDao {
        //データベースより値を取り出す

        private DataBaseHelper dbhelper;

        private ArrayList<SparseArrayCompat<String>> wordList() {
            ArrayList<SparseArrayCompat<String>> data = new ArrayList<SparseArrayCompat<String>>();

            SQLiteDatabase databaseObject = dbhelper.getReadableDatabase();

            String query = "SELECT * " + "FROM WordData";
            Log.d("query", "query = " + query);

            Cursor cursor = databaseObject.rawQuery(query, null);
            cursor.moveToFirst();
            for (int i = 0; i < cursor.getCount(); i++) {
                SparseArrayCompat<String> word = new SparseArrayCompat<String>();
                word.append(0, cursor.getString(0));
                Log.d("word1","word1 = " + cursor.getString(0));
                word.append(1, cursor.getString(1));
                word.append(2, cursor.getString(2));
                word.append(3, cursor.getString(3));
                word.append(4, cursor.getString(4));
                word.append(5, cursor.getString(5));
                word.append(6, cursor.getString(6));

                data.add(word);
                cursor.moveToNext();
                Log.d("data","data = " + data);
                Log.d("word2","data = " + word);
            }
            cursor.close();
            databaseObject.close();

            return data;
        }
    }

    public class WordDto {
        //データベースより取り出した値を保持

        private String word;
        private String pron1;
        private String pron2;
        private String mean1;
        private String mean2;
        private String description;
        private String part;
        private int grade;

        public WordDto (String word, String pron1, String pron2, String mean1, String mean2, String description, String part, int grade) {
            this.word = word;
            this.pron1 = pron1;
            this.pron2 = pron2;
            this.mean1 = mean1;
            this.mean2 = mean2;
            this.description = description;
            this.part = part;
            this.grade = grade;
        }

        public String getWord() {
            return word;
        }

        public String getPron1() {
            return pron1;
        }

        public String getPron2() {
            return pron2;
        }

        public String getMean1() {
            return mean1;
        }

        public String getMean2() {
            return mean2;
        }

        public String getDescription() {
            return description;
        }

        public String getPart() {
            return part;
        }

        public int getGrade() {
            return grade;
        }

        @Override
        public String toString() {
            return "WordDto [word=" + word + ",pron1=" + pron1 + ",pron2=" + pron2 + ",mean1=" + mean1
                    + ",mean2=" + mean2 + ",description=" + description + ",part="+ part + ",grade=" + grade + "]";
        }
    }
}

Answer 1

・DAOとDTOは別のクラスファイルに分けましょう。
・DAOから取り出したデータはDTOに変換するべきです。

Comments

[Haru_T]

クラス分けた方がいいんですね！
DAOで取り出したデータをsetしてDTOでgetするということですよね？ありがとうございます！

[yona]

setというかコンストラクタで初期化しているようなのでコンストラクタに渡しましょう。

[Haru_T]

返答遅くなりました????
調べて実践してみます！

[Haru_T]

DBより取り出してきたデータをfor文でListに入れるときに一緒にこのような記述してみたのですが、このままではcursor.getString(0)にエラーが出てしまいます。
値の渡し方が違っているのでしょうか、、
WordDataDto word = new WordDataDto(cursor.getString(0));
word.getWord();

[yona]

はい、間違っています。
コンストラクタの引数と実際に渡している引数が異なります。Javaの基本ですね。

また、エラーが出ていますじゃわかりません。質問するならどのようなエラーが出ているかを記載するようにしてください。時間が無駄になります。

[Haru_T]

調べてこのように訂正しました。
渡すときにgetを使うわけがないし、渡すものを記述するところを間違えていました。
WordDataDto word = new WordDataDto();
word.setWord(cursor.getString(0));
何度もすみません。ありがとうございます。

[yona]

値を後で変更しないなら、コンストラクタで渡した方がいいですね。