android room に関するエラー

Question

実現したいこと

Room を使用して、データベースに LocalDateTime 型を保存したいです。
Converter を使用するとの記載を見つけてやってみたのですが、
https://developer.android.com/training/data-storage/room/referencing-data?hl=ja
以下のエラーが出て、実現できませんでした。
何が間違えているのか詰まってしまっているため、アドバイスをいただけませんでしょうか？

また、もし知っていれば、スキーマを書き換えたときに
バージョンを上げるなどの処理をしなければならないと思うのですが、
その処理を実施しなくてもいい方法がもしあれば教えていただけませんでしょうか。
この辺のこと↓です。
https://qiita.com/kazuma_f/items/8c15e7087623e8f6706b

ご回答をいただけますととても嬉しいです。
どうぞよろしくお願いいたします。

発生している問題・エラーメッセージ

FATAL EXCEPTION: Thread-2
Process: com.example.RecordTime, PID: 13832
java.lang.IllegalArgumentException: Unexpected type converter com.example.RecordTime.Rooms.Converters@8399701. Annotate TypeConverter class with @ProvidedTypeConverter annotation or remove this converter from the builder.
at androidx.room.RoomDatabase.init(RoomDatabase.kt:284)
at androidx.room.RoomDatabase$Builder.build(RoomDatabase.kt:1352)
at com.example.RecordTime.DateFragment$ConnectDB.run(DateFragment.java:90)
at java.lang.Thread.run(Thread.java:1012)

該当のソースコード

public class DateFragment extends Fragment {

    Adapter adapter = new Adapter();

    TimeTableDao timeTableDao;

    List<TimeTableEntity> timeTableEntities = new ArrayList<>();

    View view;

    int year;
    int month;
    int date;

    @Override
    public void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        new Thread(new ConnectDB()).start();

        getParentFragmentManager().setFragmentResultListener("date", this, new FragmentResultListener() {

            @Override
            public void onFragmentResult(@NonNull String key, @NonNull Bundle bundle) {
                year = bundle.getInt("year");
                month = bundle.getInt("month");
                date = bundle.getInt("date");

                // 〇年・〇月・〇日をセット
                setDateText();
            }
        });
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {
        // Inflate the layout for this fragment
        return inflater.inflate(R.layout.date_fragment, container, false);
    }

    @Override
    public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {

        this.view = view;

        new Thread(new SelectRec()).start();

        // RecyclerView をセット
        RecyclerView recyclerView = view.findViewById(R.id.time_table_recycler_view);
        recyclerView.setHasFixedSize(true);
        recyclerView.setLayoutManager(new LinearLayoutManager(getContext()));
        recyclerView.setAdapter(adapter);
    }

    class ConnectDB implements Runnable {
        @Override
        public void run() {
            AppDatabase database = Room.databaseBuilder(getActivity().getApplicationContext(),
                    AppDatabase.class, "TimeTable")
                    .addTypeConverter(new Converters())
                    .build();
            timeTableDao = database.timeTableDao();
        }
    }

    class SelectRec implements Runnable {
        @Override
        public void run() {
            timeTableDao.insert(new TimeTableEntity("time1"));
            timeTableDao.insert(new TimeTableEntity("time2"));
            timeTableEntities = timeTableDao.getAll();
            Log.d("================>", timeTableEntities.get(0).title);
        }
    }

    public void setDateText() {

        // 〇年をセット
        TextView year_text = view.findViewById(R.id.selected_year);
        year_text.setText(year + " 年");

        // 〇月をセット
        TextView month_text = view.findViewById(R.id.selected_month);
        month_text.setText(month + " 月");

        // 〇日をセット
        TextView date_text = view.findViewById(R.id.selected_date);
        date_text.setText(date + " 日");
    }

    class Adapter extends RecyclerView.Adapter<Adapter.ViewHolder> {

        public class ViewHolder extends RecyclerView.ViewHolder {

            TextView textView;

            public ViewHolder(@NonNull View itemView) {
                super(itemView);

                this.textView = itemView.findViewById(R.id.time_table_view_holder);
            }
        }

        @NonNull
        @Override
        public ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
            View view = LayoutInflater.from(parent.getContext()).inflate(R.layout.time_table_viewholder, parent, false);
            return new ViewHolder(view);
        }

        @Override
        public void onBindViewHolder(@NonNull ViewHolder holder, int position) {
//            holder.textView.setText(timeTableRoomEntities.get(position).title);
        }


        @Override
        public int getItemCount() {
            return 0;
        }
    }
}

@ProvidedTypeConverter
public class Converters {

    @TypeConverter
    public static LocalDateTime toLocalDateTime(String val) {
        return LocalDateTime.parse(val);
    }

    @TypeConverter
    public static String fromLocalDateTime(LocalDateTime val) {
        return val.toString();
    }
}

@Database(entities = {TimeTableEntity.class}, version = 1)
@TypeConverters(Converters.class)
public abstract class AppDatabase extends RoomDatabase {
    public abstract TimeTableDao timeTableDao();
}

@Dao
public interface TimeTableDao {

    @Insert
    void insert(TimeTableEntity timeTableEntity);

    @Query("SELECT * FROM TimeTable")
    List<TimeTableEntity> getAll();
}

@Entity(tableName = "TimeTable")
public class TimeTableEntity {

    @PrimaryKey
    public int id;

    @ColumnInfo(name = "dateTime")
    public LocalDateTime dateTime;

    @ColumnInfo(name = "title")
    public String title;

    public TimeTableEntity(String title) {
        this.dateTime = LocalDateTime.parse(title);
    }
}

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="match_parent">

    <TextView
        android:id="@+id/time_table_view_holder"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:layout_weight="1"
        android:ems="10"
        android:text="@string/time_view_holder" />

</LinearLayout>

Comments

[jimbe]

リンクが開けないです。
![] の中に URL を書いてその後ろの () 内には書いてないのでは無いでしょうか。 [] 内の文字列は表示のみで、クリックした時に実際に飛ぶ先は () 内の URL です。

[tetejiro]

すみません、間違えておりました。
修正いたしました。
どうぞよろしくお願いします。

[jimbe]

ありがとうございます、リンク先見えました。

Answer 1

公式のサンプルコードに質問の TypeConverter を入れて試してみました。
手抜きで EmptyActivity のプロジェクトの MainActivity にベタ書きして確認は logcat です ^^;

build.gradle(Module:app) の dependencies に以下を追加

    implementation "androidx.room:room-runtime:2.6.1"
    annotationProcessor "androidx.room:room-compiler:2.6.1"

MainActivity.java

import androidx.appcompat.app.AppCompatActivity;
import androidx.room.Room;

import android.os.*;
import android.util.Log;

import java.time.LocalDateTime;
import java.util.concurrent.*;

public class MainActivity extends AppCompatActivity {
  private ExecutorService executor = Executors.newSingleThreadExecutor();

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

    executor.submit(() -> {
      try {
        AppDatabase db = Room.databaseBuilder(getApplicationContext(), AppDatabase.class, "database").build();
        UserDao userDao = db.userDao();
        userDao.insertAll(new User("AAA", "aaa", LocalDateTime.now()));
        userDao.getAll().forEach(user -> Log.d("MainActivity", user.toString()));
      } catch(Throwable t) {
        t.printStackTrace();
      }
    });
  }
}

User.java

import androidx.room.*;

import java.time.LocalDateTime;

@Entity
public class User {
  @PrimaryKey(autoGenerate = true)
  public int uid;

  @ColumnInfo(name = "first_name")
  public String firstName;

  @ColumnInfo(name = "last_name")
  public String lastName;

  @ColumnInfo(name = "date_time")
  public LocalDateTime datetime;

  User(String firstName, String lastName, LocalDateTime datetime) {
    this.firstName = firstName;
    this.lastName = lastName;
    this.datetime = datetime;
  }

  @Override
  public String toString() {
    return new StringBuilder("User")
            .append("[uid=").append(uid)
            .append(",firstName=").append(firstName)
            .append(",lastName=").append(lastName)
            .append(",datetime=").append(datetime.toString())
            .append("]").toString();
  }
}

Converters.java

import androidx.room.TypeConverter;

import java.time.LocalDateTime;

public class Converters {
  @TypeConverter
  public static LocalDateTime fromDateTime(String value) {
    return value == null ? null : LocalDateTime.parse(value);
  }

  @TypeConverter
  public static String toDateTime(LocalDateTime datetime) {
    return datetime == null ? null : datetime.toString();
  }
}

UserDao.java

import androidx.room.*;

import java.util.List;

@Dao
public interface UserDao {
  @Query("SELECT * FROM user")
  List<User> getAll();

  @Query("SELECT * FROM user WHERE uid IN (:userIds)")
  List<User> loadAllByIds(int[] userIds);

  @Query("SELECT * FROM user WHERE first_name LIKE :first AND last_name LIKE :last LIMIT 1")
  User findByName(String first, String last);

  @Insert
  void insertAll(User... users);

  @Delete
  void delete(User user);
}

AppDatabase.java

import androidx.room.*;

@Database(entities = {User.class}, version = 1)
@TypeConverters({Converters.class})
public abstract class AppDatabase extends RoomDatabase {
  public abstract UserDao userDao();
}

起動する度に insert を 1 回実行して全部を表示するので 3 回目には以下のようになり、問題無さそうです。

D  User[uid=1,firstName=AAA,lastName=aaa,datetime=2024-02-17T23:26:25.888]
D  User[uid=2,firstName=AAA,lastName=aaa,datetime=2024-02-17T23:26:41.826]
D  User[uid=3,firstName=AAA,lastName=aaa,datetime=2024-02-17T23:27:44.631]

TypeConverter の所は @ProvidedTypeConverter を使わない書き方で十分だろうというくらいで他に問題があるのか分かりませんが、構造の部分で onCreate で別スレッドでデータベースに接続を開始して onViewCreated でまた別スレッドでデータベースを使うというのは、接続と使用とぶつかって異常動作する可能性が高いと思います。
確実に接続が終わってから使用する ように書いたほうが良いでしょう(というよりそう書かないといけない事案です)。なお、 ConnectDB の処理は別スレッドにしなくても良い(それによる例外は出ない)ようです。

スキーマを書き換えたときに
バージョンを上げるなどの処理をしなければならないと思うのですが、
その処理を実施しなくてもいい方法

実行環境(エミュレータとか実機とか) の アプリ をアンインストールすればデータベースごと消えますので再インストールすればバージョンを変えなくても再生成されますね。

Comments

[tetejiro]

いつも回答をいただき、本当にありがとうございます。
全然手抜きではないです。大変助かっております。

無事、こちらでも保存ができました。
前々回、回答をいただいた質問のページと今回の質問は同じ箇所でしたが、以下になりました。

熟練した方が自分と同じ機能を実装してくださり、
質問の解消だけでなく、多くを学ばせていただいております。
ありがとうございます。

＞接続と使用とぶつかって異常動作する可能性が高いと思います。
こちらご指摘ありがとうございました。対応をしてみました。

＞アンインストールすればデータベースごと消えますので
android studio のアンインストール・再インストールをやってみたのですが、再現できず、プロジェクトを作り直して対応しました。
（きちんとした手順を勉強したいと思います。時間がかかりそうですが、、、）

public class DateFragment extends Fragment {

List<TimeTableEntity> timeTableEntities;
List<TimeTableEntity> returnedTimeTableEntities = new ArrayList<>();
TimeTableDao timeTableDao;

View view;

int year;
int month;
int date;

@Override
public void onCreate(@Nullable Bundle savedInstanceState) {
super.onCreate(savedInstanceState);

timeTableEntities = new ArrayList<>();
timeTableEntities.add(new TimeTableEntity("0番名のレコード"));
timeTableEntities.add(new TimeTableEntity("1番名のレコード"));
timeTableEntities.add(new TimeTableEntity("2番名のレコード"));

getParentFragmentManager().setFragmentResultListener("date", this, new FragmentResultListener() {

@Override
public void onFragmentResult(@NonNull String key, @NonNull Bundle bundle) {
year = bundle.getInt("year");
month = bundle.getInt("month");
date = bundle.getInt("date");

// 〇年・〇月・〇日をセット
setDateText();
}
});
}

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
Bundle savedInstanceState) {
// Inflate the layout for this fragment
return inflater.inflate(R.layout.fragment_date, container, false);
}

@Override
public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {

this.view = view;

Thread thread = new Thread(new Query());
thread.start();

try {
thread.join();

// RecyclerView をセット
RecyclerView recyclerView = view.findViewById(R.id.time_table_recycler_view);
recyclerView.setHasFixedSize(true);
recyclerView.setLayoutManager(new LinearLayoutManager(getContext()));
recyclerView.setAdapter(new Adapter());

} catch (InterruptedException e) {
throw new RuntimeException(e);
}
}

public class Query implements Runnable {

@Override
public void run() {
AppDatabase database = Room.databaseBuilder(getActivity().getApplicationContext(),
AppDatabase.class, "TimeTable").build();
timeTableDao = database.timeTableDao();
timeTableDao.insertAll(timeTableEntities);
returnedTimeTableEntities = timeTableDao.getAll();
}
}

public void setDateText() {

// 〇年をセット
TextView year_text = view.findViewById(R.id.selected_year);
year_text.setText(year + " 年");

// 〇月をセット
TextView month_text = view.findViewById(R.id.selected_month);
month_text.setText(month + " 月");

// 〇日をセット
TextView date_text = view.findViewById(R.id.selected_date);
date_text.setText(date + " 日");
}

class Adapter extends RecyclerView.Adapter<Adapter.ViewHolder> {

public class ViewHolder extends RecyclerView.ViewHolder {

TextView textView;

public ViewHolder(@NonNull View itemView) {
super(itemView);

this.textView = itemView.findViewById(R.id.every_time_table);
}
}

@NonNull
@Override
public ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
View view = LayoutInflater.from(parent.getContext()).inflate(R.layout.viewholder_time_table, parent, false);
return new ViewHolder(view);
}

@Override
public void onBindViewHolder(@NonNull ViewHolder holder, int position) {
holder.textView.setText(timeTableEntities.get(position).title + " / " + timeTableEntities.get(position).datetime.toString());
if (timeTableEntities.get(position).done) holder.textView.setBackgroundColor(Color.rgb(124,252,0)); // 赤
else holder.textView.setBackgroundColor(Color.rgb(249,247,57)); // 黄色
}

@Override
public int getItemCount() {
return timeTableEntities.size();
}
}
}

[jimbe]

>＞アンインストールすればデータベースごと消えますので
>android studio のアンインストール・再インストールを(ry

あ、ごめんなさい！
作成中の、実行している先(エミュレータとか実機とか)にインストールしている *アプリの* 再インストールのつもりでした。
android studio を再インストールするのは大変だったと思います。手間と時間を掛けさせてしまって申し訳ないです。

[jimbe]

回答のコードを作っていて思ったのですが、 LocalDateTime の文字列化は toString() では無く DateTimeFormatter で "yyyy-MM-dd HH:mm:ss" の形式と変換したほうが、 SQLite の datetime 関数で SELECT 文内で直接 日時による比較(範囲選択とか)が出来て便利かもしれません。 (試してないです。)

[tetejiro]

＞*アプリの* 再インストールのつもりでした。
とんでもないです。試してみたら、確かにできました。
書き方がよく分からず詰まっており、
また今回は必要ないと思って困っていたので本当に助かりました。
時間のロスがかなり減りました。ありがとうございました。

＞toString() では無く DateTimeFormatter で
ご指摘ありがとうございます。
その通りですね。少し試してみたのですがまだ実現できていません。
進めながら気にかけるようにして気長に解消させていこうと思います。

今回もご回答いただき、ありがとうございました。

[jimbe]

取り合えず回答にはなっていたようで…ﾖｶｯﾀﾃﾞｽ…

気を取り直して(汗
DateFragment の onCreate で FragmentManager を妙に使われていたり、 DAO で LiveData を使えばスレッドを使わなくても済むようですので、そろそろ ViewModel や LiveData を使うことも視野に入れていっても良いかと思います。

https://github.com/Jimbe-github/tetemalu_RecordTime/tree/q_uvkoxkqhp3qe8u

[tetejiro]

＞onCreate で FragmentManager を妙に使われていたり
こちら onViewCreated 内に書き直しました。 ありがとうございます。
追記：すみません。勘違いしていました。
newInstance の方で対応します。ありがとうございます。

＞ViewModel や LiveData
手探りでやっているもので、初めて見た単語でした。
github でこちらにも対応していただいているんですね。
何から何まで本当にありがとうございました。

本当にいつもありがとうございます。
github も必ず目を通して自分の糧とさせていただきます。