SQLiteデータベースの使い方がわからない

Question

SQLiteOpenHelperを継承したCustomOpenHelperを使って、
日付ごとにメモを登録しようとしているのですが、
データの検索、更新がうまくいっておりません。

String型のdisplayをキーにして、
MemoActivityの生成と同時にデータ検索、表示、
onUpdateメソッドでデータの追加、更新をしようとしています。

データ更新時、onUpdateメソッドのtryまで、
変数が正しく入っており、アプリが落ちたりもしません。
Toastの表示も正常に動作しております。

しかし、再度アクティビティーを生成するとデータが登録されていないことを判明します。
データベースの登録、検索を正常にする方法を教えていただければ幸いです。
Android Studio:3.6.1
build:grandle:3.6.3

import androidx.appcompat.app.AppCompatActivity;

import android.content.Context;
import android.content.Intent;
import android.os.Bundle;
import android.view.Gravity;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.view.inputmethod.InputMethodManager;
import android.widget.Button;
import android.widget.EditText;
import android.widget.ImageView;
import android.widget.LinearLayout;
import android.widget.TextView;
import android.widget.Toast;

import java.util.Calendar;

import datebase.CustomOpenHelper;

public class MemoActivity extends AppCompatActivity implements View.OnClickListener{
//EditTextを生成
EditText editText;
//データベース
CustomOpenHelper customOpenHelper;
//表示する日付
String display;
//表示する年月日
int displayYear;
int displayMonth;
int displayDay;

TextView textView;
//Calendarクラス
Calendar calendar = null;
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_memo);

//MainActivityからインテントを取得
Intent intent = this.getIntent();
//値を取得
String year = intent.getStringExtra("year"); //年
String month = intent.getStringExtra("month"); //月
String day = intent.getStringExtra("day");
//表示する文字列を作成
display = year+"年"+month+"月"+day+"日";

//文字列を登録
displayYear = Integer.parseInt(year);
displayMonth = Integer.parseInt(month);
displayDay = Integer.parseInt(day);

//CalendarMakerの生成
this.calendar = Calendar.getInstance();
calendar.set(displayYear,displayMonth,displayDay);

//画面に表示
textView = ((TextView)findViewById(R.id.year_month_day));
textView.setText(display);

//EditTextを生成
editText = findViewById(R.id.memo);
//Focusの設定
editText.setOnFocusChangeListener(new View.OnFocusChangeListener(){
@Override
public void onFocusChange(View v, boolean hasFocus) {
//EditTextのフォーカスが外れた場合
if(!hasFocus){
//ソフトキーボードを非表示にする
InputMethodManager imm = (InputMethodManager)
getSystemService(Context.INPUT_METHOD_SERVICE);

imm.hideSoftInputFromWindow(v.getWindowToken(),InputMethodManager.HIDE_NOT_ALWAYS);
}
}
});
LinearLayout linearLayout = ((LinearLayout)findViewById(R.id.layout));
linearLayout.setOnClickListener(this);

//データベースを生成する
this.customOpenHelper = new CustomOpenHelper(this);

//EditTextに文字列をセットする
String memo = customOpenHelper.getPlan(display);
editText.setText(memo);
}
//EditTextに入力された文字列を取得する
public void onUpdate(View v){
//EditTextのテキストを取得
String text = editText.getText().toString();
//メモに入力がなかった場合
if(text.equals("")){
ImageView imageView = new ImageView(this);
imageView.setImageResource(R.drawable.no);
Toast toast = new Toast(this);
toast.setDuration(Toast.LENGTH_SHORT);
toast.setView(imageView);
toast.setGravity(Gravity.CENTER,0,0);
toast.show();
}else{//メモに入力があった場合
//データベースに登録する
this.customOpenHelper.addMemo(display,text);
Toast toast = new Toast(this);
toast.makeText(this,"登録しました",Toast.LENGTH_SHORT).show();
}
}
//日付を前にずらす
public void onPrevious(View v){
this.calendar.add(Calendar.DAY_OF_MONTH,-1);
this.getDate();
String date = displayYear+"年"+displayMonth+"月"+displayDay+"日";
textView.setText(date);
}
//日付を後ろにずらす
public void onNext(View v){
this.calendar.add(Calendar.DAY_OF_MONTH,1);
this.getDate();
String date = displayYear+"年"+displayMonth+"月"+displayDay+"日";
textView.setText(date);
}
//戻るボタン""
public void onDelete(View v){
//アクティビティーの破棄
finish();
}
@Override
public void onClick(View v) {
//ボタンにフォーカスを移動させる
Button button = ((Button)findViewById(R.id.update));
button.setFocusable(true);
button.setFocusableInTouchMode(true);
button.requestFocus();
}
//年月日の取得
private void getDate(){
this.displayYear = calendar.get(Calendar.YEAR);
this.displayMonth = calendar.get(Calendar.MONTH);
this.displayDay = calendar.get(Calendar.DATE);
}
}

import android.content.ContentValues;
import android.content.Context;
import android.database.Cursor;
import android.database.sqlite.SQLiteDatabase;
import android.database.sqlite.SQLiteOpenHelper;


public class CustomOpenHelper extends SQLiteOpenHelper {
    //データベースの名前
    private static final String DB_NAME ="calendar";
    //バージョン管理
    private static final int DB_VERSION = 1;
    //テーブル作成SQL
    private static final String CREATE_TABLE_MESSAGES =""+
            "CREATE TABLE calendar("+
            "date TEXT PRIMARY KEY,"+
            "memo TEXT"+
            ")";
    //データベース
    private SQLiteDatabase db = null;

    public CustomOpenHelper(Context context){
        super(context,DB_NAME,null,DB_VERSION);
    }

    @Override
    public void onCreate(SQLiteDatabase db) {
        //テーブル作成
        db.execSQL(CREATE_TABLE_MESSAGES);
    }

    @Override
    public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {

    }
    //メモを保存する
    public void addMemo(String date,String memo){
        //メモを取得
        String addOrUpdate = this.getMemo(date);
        this.db = this.getWritableDatabase();
        //ContentValuesにデータを入れる
        ContentValues val = new ContentValues();
        val.put("date",date);
        val.put("memo",memo);
        try{
            //メモの登録がされていなかった場合
            if(addOrUpdate.equals("")){
                //メモを追加する
                db.insert(this.DB_NAME,null,val);
            }else{
                //メモを更新する
                db.update(this.DB_NAME,val,"date = ?",new String[]{memo});
            }
        }finally {
            db.close();
        }
    }
    //メモを返す
    public String getPlan(String date){
        return this.getMemo(date);
    }
    //データが保存してあるかを確認する
    private String getMemo(String date){
        //返すmemoの箱
        String memo = "";
        //読み込み用のデータベース
        this.db = this.getReadableDatabase();
        try{
            String sql = "";
            Cursor cursor = db.query(
                    this.DB_NAME,
                    new String[]{"memo"},
                    "date == ?",
                    new String[]{"memo"},
                    null,null,null,null
            );
            //データが存在する場合
            if(cursor.moveToFirst()){
                memo = cursor.getString(0);
            }
        }finally {
            db.close();
        }
        return memo;
    }

    public SQLiteDatabase getDb() {
        return db;
    }
}

Comments

[m.ts10806]

「機能しません」や「動きません」や「うまくいきません」
これは誰にも何も伝えない表現です。
実際に起きている現象を記載してください。
あと、何が分かっていて何が分かっていないかも明確にしてください。

[m.ts10806]

特にDBなど外部の仕組みを利用していると、コードだけが原因とは限りませんからね。
環境情報全て提示すること、ここまでの手順詳細
これらも追記してください。

[Shibou]

失礼しました。
これの変数displayをキーとして使って、アクティビティーの生成と同時に、
データの引き出し、onUpdateメソッドを使ってのデータ登録をしようとしております。
メモ登録時のonUpdateメソッドのToastは表示されますが、データの登録が失敗しているみたいで、
再度アクティビティーを生成するときにメモデータを引き出すことができていません。
デバッグの結果、addMemoメソッドのtryまで変数がうまく入っています。
アプリが落ちたりもしません。

使っているAndroid Studioのバージョンは
3.6.1
grandleのbuildは3.6.3です。
import androidx.appcompat.app.AppCompatActivity;

import android.content.Context;
import android.content.Intent;
import android.os.Bundle;
import android.view.Gravity;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.view.inputmethod.InputMethodManager;
import android.widget.Button;
import android.widget.EditText;
import android.widget.ImageView;
import android.widget.LinearLayout;
import android.widget.TextView;
import android.widget.Toast;

import java.util.Calendar;

import datebase.CustomOpenHelper;

public class MemoActivity extends AppCompatActivity implements View.OnClickListener{
//EditTextを生成
EditText editText;
//データベース
CustomOpenHelper customOpenHelper;
//表示する日付
String display;
//表示する年月日
int displayYear;
int displayMonth;
int displayDay;

TextView textView;
//Calendarクラス
Calendar calendar = null;
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_memo);

//MainActivityからインテントを取得
Intent intent = this.getIntent();
//値を取得
String year = intent.getStringExtra("year"); //年
String month = intent.getStringExtra("month"); //月
String day = intent.getStringExtra("day");
//表示する文字列を作成
display = year+"年"+month+"月"+day+"日";

//文字列を登録
displayYear = Integer.parseInt(year);
displayMonth = Integer.parseInt(month);
displayDay = Integer.parseInt(day);

//CalendarMakerの生成
this.calendar = Calendar.getInstance();
calendar.set(displayYear,displayMonth,displayDay);

//画面に表示
textView = ((TextView)findViewById(R.id.year_month_day));
textView.setText(display);

//EditTextを生成
editText = findViewById(R.id.memo);
//Focusの設定
editText.setOnFocusChangeListener(new View.OnFocusChangeListener(){
@Override
public void onFocusChange(View v, boolean hasFocus) {
//EditTextのフォーカスが外れた場合
if(!hasFocus){
//ソフトキーボードを非表示にする
InputMethodManager imm = (InputMethodManager)
getSystemService(Context.INPUT_METHOD_SERVICE);

imm.hideSoftInputFromWindow(v.getWindowToken(),InputMethodManager.HIDE_NOT_ALWAYS);
}
}
});
LinearLayout linearLayout = ((LinearLayout)findViewById(R.id.layout));
linearLayout.setOnClickListener(this);

//データベースを生成する
this.customOpenHelper = new CustomOpenHelper(this);

//EditTextに文字列をセットする
String memo = customOpenHelper.getPlan(display);
editText.setText(memo);
}
//EditTextに入力された文字列を取得する
public void onUpdate(View v){
//EditTextのテキストを取得
String text = editText.getText().toString();
//メモに入力がなかった場合
if(text.equals("")){
ImageView imageView = new ImageView(this);
imageView.setImageResource(R.drawable.no);
Toast toast = new Toast(this);
toast.setDuration(Toast.LENGTH_SHORT);
toast.setView(imageView);
toast.setGravity(Gravity.CENTER,0,0);
toast.show();
}else{//メモに入力があった場合
//データベースに登録する
this.customOpenHelper.addMemo(display,text);
Toast toast = new Toast(this);
toast.makeText(this,"登録しました",Toast.LENGTH_SHORT).show();
}
}
//日付を前にずらす
public void onPrevious(View v){
this.calendar.add(Calendar.DAY_OF_MONTH,-1);
this.getDate();
String date = displayYear+"年"+displayMonth+"月"+displayDay+"日";
textView.setText(date);
}
//日付を後ろにずらす
public void onNext(View v){
this.calendar.add(Calendar.DAY_OF_MONTH,1);
this.getDate();
String date = displayYear+"年"+displayMonth+"月"+displayDay+"日";
textView.setText(date);
}
//戻るボタン""
public void onDelete(View v){
//アクティビティーの破棄
finish();
}
@Override
public void onClick(View v) {
//ボタンにフォーカスを移動させる
Button button = ((Button)findViewById(R.id.update));
button.setFocusable(true);
button.setFocusableInTouchMode(true);
button.requestFocus();
}
//年月日の取得
private void getDate(){
this.displayYear = calendar.get(Calendar.YEAR);
this.displayMonth = calendar.get(Calendar.MONTH);
this.displayDay = calendar.get(Calendar.DATE);
}
}

[m.ts10806]

質問は編集できますし、コメント欄ではマークダウン使えませんので、
原則質問本文を編集して加筆修正してください

[退会済みユーザー]

getなのにsetしてるが、設計は大丈夫か？

Answer 1

メモの状態(空だったり空白のみだったり)とデータベースの状態をどう整合性を取るかがまだ不十分ですが, 単体で動作します.

package com.teratail.q256872;

import androidx.appcompat.app.AppCompatActivity;

import android.content.*;
import android.os.Bundle;
import android.text.*;
import android.view.*;
import android.view.inputmethod.InputMethodManager;
import android.widget.*;

import java.text.SimpleDateFormat;
import java.util.Calendar;

public class MemoActivity extends AppCompatActivity {
  private Calendar calendar;
  private TextView textView;
  private Button delete;
  private String before;
  private EditText editText;
  private CustomOpenHelper customOpenHelper;

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

    //データベースを生成する
    customOpenHelper = new CustomOpenHelper(this);

    calendar = getCalendar(getIntent());
    textView = findViewById(R.id.year_month_day);
    final Button update = findViewById(R.id.update);
    delete = findViewById(R.id.delete);

    editText = findViewById(R.id.memo);
    editText.setOnFocusChangeListener(new View.OnFocusChangeListener(){
      @Override
      public void onFocusChange(View v, boolean hasFocus) {
        //EditTextのフォーカスが外れた場合
        if(!hasFocus){
          //ソフトキーボードを非表示にする
          InputMethodManager imm = (InputMethodManager)getSystemService(Context.INPUT_METHOD_SERVICE);
          imm.hideSoftInputFromWindow(v.getWindowToken(),InputMethodManager.HIDE_NOT_ALWAYS);
        }
      }
    });
    editText.addTextChangedListener(new TextWatcher() {
      @Override
      public void beforeTextChanged(CharSequence s, int start, int count, int after) {}
      @Override
      public void onTextChanged(CharSequence s, int start, int before, int count) {}
      @Override
      public void afterTextChanged(Editable s) { update.setEnabled(!s.toString().equals(before)); }
    });

    //画面を表示
    showCurrent();
  }
  private Calendar getCalendar(Intent intent) {
    calendar = Calendar.getInstance();
    if(intent != null && intent.hasExtra("year") && intent.hasExtra("month") && intent.hasExtra("day")) {
      //値を取得
      int year = Integer.parseInt(intent.getStringExtra("year"));
      int month = Integer.parseInt(intent.getStringExtra("month"));
      int day = Integer.parseInt(intent.getStringExtra("day"));
      calendar.set(year, month, day);
    }
    return calendar;
  }

  public void onUpdate(View v){
    //EditTextのテキストを取得
    String text = editText.getText().toString();
    //メモに入力がなかった場合
    if(text.equals("")){
      ImageView imageView = new ImageView(this);
      imageView.setImageResource(R.drawable.no);
      Toast toast = new Toast(this);
      toast.setDuration(Toast.LENGTH_SHORT);
      toast.setView(imageView);
      toast.setGravity(Gravity.CENTER,0,0);
      toast.show();
    }else{//メモに入力があった場合
      //データベースに登録する
      customOpenHelper.addMemo(getKey(), text);
      Toast.makeText(this,"登録しました",Toast.LENGTH_SHORT).show();
      showCurrent();
    }
  }
  //日付を前にずらす
  public void onPrevious(View v){
    calendar.add(Calendar.DAY_OF_MONTH,-1);
    //画面を表示
    showCurrent();
  }
  //日付を後ろにずらす
  public void onNext(View v){
    calendar.add(Calendar.DAY_OF_MONTH,1);
    //画面を表示
    showCurrent();
  }
  //メモを削除
  public void onDelete(View v){
    customOpenHelper.deleteMemo(getKey());
    //画面を表示
    showCurrent();
    //アクティビティーの破棄
    //finish();
  }
  //対象の表示
  private void showCurrent(){
    SimpleDateFormat sdf = new SimpleDateFormat("yyyy年MM月dd日");
    textView.setText(sdf.format(calendar.getTime()));

    String memo = customOpenHelper.getMemo(getKey());
    delete.setEnabled(memo != null);
    if(memo == null) memo = "";
    before = memo;
    editText.setText(memo);
  }
  private String getKey() {
    return textView.getText().toString();
  }
}

package com.teratail.q256872;

import android.content.ContentValues;
import android.content.Context;
import android.database.*;
import android.database.sqlite.SQLiteDatabase;
import android.database.sqlite.SQLiteOpenHelper;

public class CustomOpenHelper extends SQLiteOpenHelper {
  //データベースの名前
  private static final String DB_NAME ="calendar";
  //バージョン管理
  private static final int DB_VERSION = 1;
  //テーブル作成SQL
  private static final String CREATE_TABLE_MESSAGES =""+
          "CREATE TABLE calendar("+
          "date TEXT PRIMARY KEY,"+
          "memo TEXT"+
          ")";

  public CustomOpenHelper(Context context){
    super(context,DB_NAME,null,DB_VERSION);
  }

  @Override
  public void onCreate(SQLiteDatabase db) {
    //テーブル作成
    db.execSQL(CREATE_TABLE_MESSAGES);
  }

  @Override
  public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
  }

  //メモを保存する
  public void addMemo(String date, String memo) throws SQLException {
    ContentValues val = new ContentValues();
    val.put("date",date);
    val.put("memo",memo);
    try (SQLiteDatabase db = getWritableDatabase();) {
      db.replaceOrThrow(DB_NAME, null, val);
    }
  }
  //メモを返す
  String getMemo(String date){
    try (SQLiteDatabase db = getReadableDatabase();
      Cursor cursor = db.query(
              DB_NAME, new String[]{"memo"},
              "date == ?", new String[]{date},
              null,null,null,null
      );) {
      if(cursor.moveToFirst()){
        return cursor.getString(0);
      }
    }
    return null;
  }
  //メモを削除する
  public void deleteMemo(String date) throws SQLException {
    try (SQLiteDatabase db = getWritableDatabase();) {
      db.delete(DB_NAME, "date == ?", new String[]{date});
    }
  }
}

layout/activity_memo.xml

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MemoActivity">
  <TextView
      android:id="@+id/year_month_day"
      android:layout_width="match_parent"
      android:layout_height="wrap_content"
      android:text="9999年99月99日"
      android:textSize="32dp"
      android:gravity="center"
      app:layout_constraintLeft_toLeftOf="parent"
      app:layout_constraintRight_toRightOf="parent"
      app:layout_constraintTop_toTopOf="parent" />
  <EditText
      android:id="@+id/memo"
      android:layout_width="match_parent"
      android:layout_height="0dp"
      android:hint="メモを入力"
      android:layout_margin="2dp"
      android:padding="5dp"
      android:background="@drawable/memo_background"
      android:inputType="textMultiLine"
      android:gravity="top|left"
      app:layout_constraintBottom_toTopOf="@id/layout"
      app:layout_constraintLeft_toLeftOf="parent"
      app:layout_constraintRight_toRightOf="parent"
      app:layout_constraintTop_toBottomOf="@id/year_month_day" />
  <LinearLayout
      android:id="@+id/layout"
      android:layout_width="match_parent"
      android:layout_height="wrap_content"
      android:orientation="horizontal"
      android:gravity="center"
      app:layout_constraintBottom_toBottomOf="parent"
      app:layout_constraintLeft_toLeftOf="parent"
      app:layout_constraintRight_toRightOf="parent">
    <Button
        android:id="@+id/previous"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="＜"
        android:onClick="onPrevious" />
    <Button
        android:id="@+id/update"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="Update"
        android:onClick="onUpdate" />
    <Button
        android:id="@+id/delete"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="Delete"
        android:onClick="onDelete" />
    <Button
        android:id="@+id/next"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="＞"
        android:onClick="onNext" />
  </LinearLayout>
</androidx.constraintlayout.widget.ConstraintLayout>

drawable/memo_background.xml

<?xml version="1.0" encoding="utf-8"?>
<shape xmlns:android="http://schemas.android.com/apk/res/android">
  <stroke android:width="2dp"
      android:color="#232323" />
  <corners android:radius="5dp" />
</shape>

Comments

[Shibou]

うまく動かすことができました!ありがとうございました。

Answer 2

データが登録されていないのではなく、取り出し方に誤りがあるのではないでしょうか。

CustomOpenHelperクラスのgetMemo()メソッドが

            Cursor cursor = db.query(
                    this.DB_NAME,
                    new String[]{"memo"},
                    "date == ?",
                    new String[]{"memo"},
                    null,null,null,null
            );

のようになっていますが、これだとdateというカラムの内容が"memo"となっているレコードを探すことになってしまい、一致するのが無いと判断されているのではありませんか？

メソッドの引数dateと一致するレコードを探すのならば、

            Cursor cursor = db.query(
                    this.DB_NAME,
                    new String[]{"memo"},
                    "date = ?",
                    new String[]{date},
                    null,null,null,null
            );

とするのかなと思えます。

Comments

[Shibou]

試してみたのですが動作に変化はなく、データの登録はうまくいきませんでした...。一度ゼロから作り直してみようと思います。ありがとうございました。

[keicha_hrs]

データベースに正常に情報が入っているのか、Android StudioのDevice Explorerを使ってDBファイルをPCにダウンロードし、PC上でSQLiteのDBファイルが閲覧できるソフト（DB Browser for SQLiteなど）を使って確認してみてはどうでしょうか。DBファイルの中身が見えない状態では、どこまでうまくいっているのか判別するのは困難でしょう。

この辺り参考になると思います。
https://www.virment.com/view-sqlite-database-by-android-studio/

[Shibou]

DB Browser for SQLiteをダウンロードしてみましたが、データを登録するところから失敗しているみたいです...。このソフトのことを知らなかったので、とても勉強になりました。また機会があればアドバイスいただければ幸いです。ありがとうございました。