質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.49%

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

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 4,084

score 13

前提・実現したいこと

初めて投稿させていただきます。よろしくお願いします。
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を使用しています。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

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

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/05 13:32

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

    キャンセル

  • 2019/02/05 20:34

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

    キャンセル

  • 2019/02/05 20:49

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る