Android Studio カレンダーの日付を選択して開いたときに、同じデータが開かれてしまう

Question

前提・実現したいこと

初めて投稿させていただきます。よろしくお願いします。
Android Studioの勉強のため他のサイト様などを参考に、カレンダーUIのメモ帳を作成しています。

・カレンダーが開く
・カレンダーの日付を押すとメモ帳が開く
・メモ帳の保存ボタンを押すとデータベースにデータが保存され、カレンダー画面に戻る

こういったものを作ろうとしているのですが、データを保存した後に、どこの日付を押しても同じデータが読み込まれた状態で開かれてしまいます。一つ一つ日付毎にデータを開きたいのですがやり方が分からず詰まってしまっています。どのようにすれば一つ一つ開くことができるのでしょうか？
どうかご教授お願いいたします。

該当のソースコード

public class MainActivity extends AppCompatActivity implements AdapterView.OnItemClickListener,View.OnClickListener{

    private TextView titleText;
    private Button prevButton, nextButton;
    private CalendarAdapter mCalendarAdapter;
    private GridView calendarGridView;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        findViewById(R.id.hButton).setOnClickListener(this);//上ボタン用追加
        findViewById(R.id.setButton).setOnClickListener(this);//上ボタン用追加

        titleText = findViewById(R.id.titleText);
        prevButton = findViewById(R.id.prevButton);
        prevButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                mCalendarAdapter.prevMonth();
                titleText.setText(mCalendarAdapter.getTitle());
            }
        });
        nextButton = findViewById(R.id.nextButton);
        nextButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                mCalendarAdapter.nextMonth();
                titleText.setText(mCalendarAdapter.getTitle());
            }
        });
        calendarGridView = findViewById(R.id.calendarGridView);
        mCalendarAdapter = new CalendarAdapter(this);
        calendarGridView.setAdapter(mCalendarAdapter);
        titleText.setText(mCalendarAdapter.getTitle());

        calendarGridView.setOnItemClickListener(this);
    }

    @Override
    public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
        Intent intent = new Intent(getApplication(), TextActivity.class);
        intent.putExtra("date", mCalendarAdapter.getItem(position).toString());
        startActivity(intent);
    }

    @Override
    public void onClick(View view){
        //ここに遷移するための処理を追加する
        switch (view.getId()){
            case R.id.hButton:
                Intent intent = new Intent(this,Main2Activity.class);
                startActivity(intent);
                break;

            case R.id.setButton:
                intent = new Intent(this,Main3Activity.class);
                startActivity(intent);
                break;
        }

        /*Intent intent = new Intent(this, Main2Activity.class);
        startActivity(intent);
        */
    }

public class TextActivity extends AppCompatActivity {

    private String currentDate;
    private TextView tex;
    private SQLiteDatabase db;
    private TextOpenHelper helper = null;
    private EditText weightText, bodyText;
    boolean newFlag = false;
    String id = "";
    String youbi;
    String month;
    String day;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_text);

        //データベース作成
        weightText = findViewById(R.id.weightText);
        bodyText = findViewById(R.id.body);
        tex = findViewById(R.id.tex);

        if (helper == null) {
            helper = new TextOpenHelper(TextActivity.this);
        }
        Intent intent = getIntent();
        currentDate = intent.getStringExtra("date");
        youbi = currentDate.substring(0,3);
        month = currentDate.substring(4,7);
        day = currentDate.substring(8,10);
        tex.setText(month+","+day+","+youbi);
        id = intent.getStringExtra("id");
        if (month ==("") && day == ("") && id == ("")) {
            newFlag = true;
        } else {
            db = helper.getWritableDatabase();
            try {
                // rawQueryというSELECT専用メソッドを使用してデータを取得する
                Cursor c = db.rawQuery("select id, weight, body from text_table", null);
                // Cursorの先頭行があるかどうか確認
                boolean next = c.moveToFirst();
                // 取得した全ての行を取得
                while (next) {
                    // 取得したカラムの順番(0から始まる)と型を指定してデータを取得する
                    Double dispWeight = c.getDouble(1);
                    String dispBody = c.getString(2);
                    //EditText body = (EditText)findViewById(R.id.body);
                    weightText.setText(String.valueOf(dispWeight));
                    bodyText.setText(dispBody, TextView.BufferType.NORMAL);
                    next = c.moveToNext();
                }
            } finally {
            }
        }


        //セーブボタン
        Button saveButton = findViewById(R.id.save);
        saveButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                if (helper == null) {
                    helper = new TextOpenHelper(getApplicationContext());
                }
                if (db == null) {
                    db = helper.getWritableDatabase();
                }

                String weight = weightText.getText().toString();
                String body = bodyText.getText().toString();

                insertData(db, month, day, Double.valueOf(weight), body);
                finish();

            }
        });
    }

    //データベースinsertData
    private void insertData(SQLiteDatabase db,String month, String day, Double weight, String body) {

        ContentValues iValues = new ContentValues();
        iValues.put("month",month);
        iValues.put("day",day);
        iValues.put("weight", weight);
        iValues.put("body", body);
        db.insert("text_table", null, iValues);

    }
    //データベースinsertData
}

public class CalendarAdapter extends BaseAdapter {
    private List<Date> dateArray = new ArrayList();
    private Context mContext;
    private DateManager mDateManager;
    private LayoutInflater mLayoutInflater;

    //カスタムセルを拡張したらここでWidgetを定義
    private static class ViewHolder {
        public TextView dateText;
    }

    public CalendarAdapter(Context context){
        mContext = context;
        mLayoutInflater = LayoutInflater.from(mContext);
        mDateManager = new DateManager();
        dateArray = mDateManager.getDays();
    }

    @Override
    public int getCount() {
        return dateArray.size();
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        ViewHolder holder;
        if (convertView == null) {
            convertView = mLayoutInflater.inflate(R.layout.calendar_cell, null);
            holder = new ViewHolder();
            holder.dateText = convertView.findViewById(R.id.dateText);
            convertView.setTag(holder);
        } else {
            holder = (ViewHolder)convertView.getTag();
        }

        //セルのサイズを指定
        float dp = mContext.getResources().getDisplayMetrics().density;
        AbsListView.LayoutParams params = new AbsListView.LayoutParams(parent.getWidth()/7 - (int)dp, (parent.getHeight() - (int)dp * mDateManager.getWeeks() ) / mDateManager.getWeeks());
        convertView.setLayoutParams(params);

        //日付のみ表示させる
        SimpleDateFormat dateFormat = new SimpleDateFormat("d", Locale.US);
        holder.dateText.setText(dateFormat.format(dateArray.get(position)));

        //当月以外のセルをグレーアウト
        if (mDateManager.isCurrentMonth(dateArray.get(position))){
            convertView.setBackgroundColor(Color.WHITE);
        }else {
            convertView.setBackgroundColor(Color.GRAY);
        }

        //当日の背景をライトグレーに
        if (mDateManager.isToday(dateArray.get(position))) {
            convertView.setBackgroundColor(Color.LTGRAY);
        }

        //日曜日を赤、土曜日を青に
        int colorId;
        switch (mDateManager.getDayOfWeek(dateArray.get(position))){
            case 1:
                colorId = Color.RED;
                break;
            case 7:
                colorId = Color.BLUE;
                break;

            default:
                colorId = Color.BLACK;
                break;
        }
        holder.dateText.setTextColor(colorId);

        return convertView;
    }

    @Override
    public long getItemId(int position) {
        return position;
    }

    @Override
    public Object getItem(int position) {
        return dateArray.get(position);
    }

    //表示月を取得
    public String getTitle(){
        SimpleDateFormat format = new SimpleDateFormat("yyyy.MM", Locale.US);
        return format.format(mDateManager.mCalendar.getTime());
    }

    //翌月表示
    public void nextMonth(){
        mDateManager.nextMonth();
        dateArray = mDateManager.getDays();
        this.notifyDataSetChanged();
    }

    //前月表示
    public void prevMonth(){
        mDateManager.prevMonth();
        dateArray = mDateManager.getDays();
        this.notifyDataSetChanged();
    }

補足情報（FW/ツールのバージョンなど）

Android Studio3.1.2
データベースはSQLiteを使用しています。

Answer 1

// rawQueryというSELECT専用メソッドを使用してデータを取得する
Cursor c = db.rawQuery("select id, weight, body from text_table", null);

データベースから読み込む時に何の条件も指定せず全部読み込んでいるようです.
月日を条件として読み込む必要があるのではないでしょうか.

Comments

[K-K-K]

回答ありがとうございます。
なるほど、どこを直せばいいのかがイマイチわかっていませんでした。
試してみます、ありがとうございます！

[jimbe]

ついでですが,

> if (month ==("") && day == ("") && id == ("")) {

という文字列 ("" という空文字列も文字列です ) との比較は java では機能致しません.
空文字列かを調べるのでしたら, month.equals("") とするか, 長さを調べる month.length() == 0 とする必要があります.

[K-K-K]

別の箇所の補足までありがとうございます。

Cursor c = db.rawQuery("select id, month, day, weight, body from text_table WHERE month='Feb' AND day='02'", null);

例としてこのような形で指定するのかと思い試してみると、２月２日に保存したデータがどの日を選んでも抽出されていました。
変数と比較するのかと考えまして、

String sql = "select id, month, day, weight, body from text_table WHERE month="+month+"& day="+day;
Cursor c = db.rawQuery(sql, null);

このように書き換えたところ、
E/SQLiteLog: (1) no such column: Feb
とエラーが出てしまいます。変数の場合何か別の方法で無いと無理なのでしょうか？

[jimbe]

遅くまでお疲れさまです.
rawQuery は文字列で SQL を受けるだけなので, 変数から作られた SQL かまでは関係ありません.
エラーの出る SQL と, 抽出ができた SQL とを比較して見てください. 寸分も違いが無ければ動くはずです.

[K-K-K]

夜分遅くの返信ありがとうございます。
エラーが出ているSQLも抽出出来ているSQLもWHEREまでは一緒だということは確認できました。条件をdayのみにした場合、なぜか１日〜９日のみ保存されませんでしたが、１０日〜２８日まではそれぞれに保存できていることも確認できました。
変数monthの書き方に何かあるということはわかったのですが、変数monthは文字列なのでシングルクォーツで囲った'Feb'と同じだと思えるのですが、違っているのでしょうか。。？

[jimbe]

先に書きました通り, rawQuery は SQL のみで動作致します. 「変数monthは文字列なので」は rawQueryには関係ありませんので, エラーになるのは SQL に異常があるためです.
「寸分も違いが無ければ」とコメントさせて頂いております. シングルクオットが有って動作し, 無くてエラーになるならば, 「シングルクオットが必要なのだ」との認識に改められては如何でしょうか.
また, SQL において文字列を直接指定する場合にどう書かなければいけないのかを調査すれば, トライ＆エラーを繰り返す必要は無いはずです. 落ち着いて, 一つ一つ調査・確認をされたほうが良いかと思います.

日付の条件につきましても, 2月2日を固定条件としていた時, 日付の比較値はどうされていましたでしょうか?
文字列の場合はスペース1つ, '0'1つ違っても条件に一致致しません.
人間には「数'値'として考えれば同じ」と見えても, コンピュータは数'字'か数'値'かは関係ありません. 文字列として定義された column を文字列として比較しろと SQL で指示されれば, 文字列として比較致します.

[jimbe]

なお, 現在 null とされている rawQuery の第二引数を使用すると, 文字列変数を(変数として)渡すことが出来ます. その使用が本来でして, そうしないとセキュリティ的な問題(SQLインジェクション)が発生する場合があります. お時間がありましたら調査をお勧めします.

[K-K-K]

色々とご教授ありがとうございます。
頭を固くしてしまっていました。rawQueryの意味がわかり、第2引数なども利用して全て抽出することができました。お付き合い下さりありがとうございました。もっと精進します。

[jimbe]

お考えの動作をするように記述出来た様で, 良かったです.
最初に SQL に WHERE 節を追加する際に, まず固定値で記述して動作を確認してから変数化するという手順を踏まれたのは, 一つ一つ確認していく手法として正しいやり方です.
今後に期待できると思います.