web-dev-qa-db-ja.com

LiveDataから値を取得するにはどうすればよいですか？

Roomを初めて使用します。私はLiveDataの概念を調べています。 DBからLiveDataにレコードをフェッチし、Observerをアタッチできることは知っています。

@Query("SELECT * FROM users")

<LiveData<List<TCUser>> getAll();

しかし、私はバックグラウンドで同期を実行しています。サーバーからデータをフェッチし、「users」と呼ばれるRoomDatabaseテーブルのデータと比較してから、usersテーブルから挿入、更新、または削除する必要があります。アクションを実行する前に、LiveDataリストをトラバースするにはどうすればよいですか？ forループに入れるとエラーになります。

または、このシナリオではLiveDataを使用しないでください。

私は電話する必要があると思います

<LiveData<List<TCUser>> getAll().getValue()

しかし、それは正しいことですか？ここに私が何をしようとしているのかについてアイデアを与えるためのいくつかのコードがあります：

List<User>serverUsers: Is the data received from a response from an API

private void updateUsers(List<User> serverUsers) {
    List<UserWithShifts> users = appDatabase.userDao().getAllUsers();
    HashMap<String, User> ids = new HashMap();
    HashMap<String, User> newIds = new HashMap();

    if (users != null) {
        for (UserWithShifts localUser : users) {
            ids.put(localUser.user.getId(), localUser.user);
        }
    }

    for (User serverUser : serverUsers) {
        newIds.put(serverUser.getId(), serverUser);

        if (!ids.containsKey(serverUser.getId())) {
            saveShiftForUser(serverUser);
        } else {
            User existingUser = ids.get(serverUser.getId());
            //If server data is newer than local
            if (DateTimeUtils.isLaterThan(serverUser.getUpdatedAt(), existingUser.getUpdatedAt())) {
                deleteEventsAndShifts(serverUser.getId());
                saveShiftForUser(serverUser);
            }
        }
    }

どこ：

@Query("SELECT * FROM users")
List<UserWithShifts> getAllUsers();

UpdateUsers（）の最初の行は、新しいデータを挿入する前にDBからデータをフェッチして処理する正しい方法ですか、それとも代わりにすべきですか

<LiveData<List<User>> getAll().getValue()

おかげで、

androidandroid-sqliteandroid-roomandroid-livedata
9
user12865

私があなたのアーキテクチャを正しく理解している場合、updateUsersはAsyncTaskなどの内部にあります。

これは私の提案するアプローチで、最大の効果を得るためにDaoを微調整する必要があります。データベースに要求できる決定を下すために、多くのコードを記述しました。

これもタイトまたは効率的なコードではありませんが、これらのライブラリのより効果的な使用法を示していることを願っています。

バックグラウンドスレッド（IntentService、AsyncTaskなど）：

/*
 * assuming this method is executing on a background thread
 */
private void updateUsers(/* from API call */List<User> serverUsers) {
    for(User serverUser : serverUsers){
        switch(appDatabase.userDao().userExistsSynchronous(serverUser.getId())){
            case 0: //doesn't exist
                saveShiftForUser(serverUser);
            case 1: //does exist
                UserWithShifts localUser = appDatabase.userDao().getOldUserSynchronous(serverUser.getId(), serverUser.getUpdatedAt());
                if(localUser != null){ //there is a record that's too old
                    deleteEventsAndShifts(serverUser.getId());
                    saveShiftForUser(serverUser);
                }
            default: //something happened, log an error
        }
    }
}

UIスレッド（アクティビティ、フラグメント、サービス）で実行している場合：

/*
 * If you receive the IllegalStateException, try this code
 *
 * NOTE: This code is not well architected. I would recommend refactoring if you need to do this to make things more elegant.
 *
 * Also, RxJava is better suited to this use case than LiveData, but this may be easier for you to get started with
 */
private void updateUsers(/* from API call */List<User> serverUsers) {
    for(User serverUser : serverUsers){
        final LiveData<Integer> userExistsLiveData = appDatabase.userDao().userExists(serverUser.getId());
        userExistsLiveData.observe(/*activity or fragment*/ context, exists -> {
            userExistsLiveData.removeObservers(context); //call this so that this same code block isn't executed again. Remember, observers are fired when the result of the query changes.
            switch(exists){
                case 0: //doesn't exist
                    saveShiftForUser(serverUser);
                case 1: //does exist
                    final LiveData<UserWithShifts> localUserLiveData = appDatabase.userDao().getOldUser(serverUser.getId(), serverUser.getUpdatedAt());
                    localUserLiveData.observe(/*activity or fragment*/ context, localUser -> { //this observer won't be called unless the local data is out of date
                        localUserLiveData.removeObservers(context); //call this so that this same code block isn't executed again. Remember, observers are fired when the result of the query changes.
                        deleteEventsAndShifts(serverUser.getId());
                        saveShiftForUser(serverUser);
                    });
                default: //something happened, log an error
            }
        });
    }
}

Daoを変更して、使用する方法を決定します。

@Dao
public interface UserDao{
    /*
     * LiveData should be chosen for most use cases as running on the main thread will result in the error described on the other method
     */
    @Query("SELECT * FROM users")
    LiveData<List<UserWithShifts>> getAllUsers();

    /*
     * If you attempt to call this method on the main thread, you will receive the following error:
     *
     * Caused by: Java.lang.IllegalStateException: Cannot access database on the main thread since it may potentially lock the UI for a long periods of time.
     *  at Android.Arch.persistence.room.RoomDatabase.assertNotMainThread(AppDatabase.Java:XXX)
     *  at Android.Arch.persistence.room.RoomDatabase.query(AppDatabase.Java:XXX)
     *
     */
    @Query("SELECT * FROM users")
    List<UserWithShifts> getAllUsersSynchronous();

    @Query("SELECT EXISTS (SELECT * FROM users WHERE id = :id)")
    LiveData<Integer> userExists(String id);

    @Query("SELECT EXISTS (SELECT * FROM users WHERE id = :id)")
    Integer userExistsSynchronous(String id);

    @Query("SELECT * FROM users WHERE id = :id AND updatedAt < :updatedAt LIMIT 1")
    LiveData<UserWithShifts> getOldUser(String id, Long updatedAt);

    @Query("SELECT * FROM users WHERE id = :id AND updatedAt < :updatedAt LIMIT 1")
    UserWithShifts getOldUserSynchronous(String id, Long updatedAt);
}

これで問題は解決しましたか？

注：saveShiftForUserまたはdeleteEventsAndShiftsメソッドが見つかりませんでした。挿入、保存、更新は、Roomによって同期的に実行されます。メインスレッドでいずれかのメソッドを実行している場合（これはエラーの原因であると思います）、appDatabaseから返されるdaoWrapperを次のように作成する必要があります。

public class UserDaoWrapper {
    private final UserDao userDao;

    public UserDaoWrapper(UserDao userDao) {
        this.userDao = userDao;
    }

    public LiveData<Long[]> insertAsync(UserWithShifts... users){
        final MutableLiveData<Long[]> keys = new MutableLiveData<>();
        HandlerThread ht = new HandlerThread("");
        ht.start();
        Handler h = new Handler(ht.getLooper());
        h.post(() -> keys.postValue(userDao.insert(users)));
        return keys;
    }

    public void updateAsync(UserWithShifts...users){
        HandlerThread ht = new HandlerThread("");
        ht.start();
        Handler h = new Handler(ht.getLooper());
        h.post(() -> {
            userDao.update(users);
        });
    }

    public void deleteAsync(User... users){
        HandlerThread ht = new HandlerThread("");
        ht.start();
        Handler h = new Handler(ht.getLooper());
        h.post(() -> {
            for(User e : users)
                userDao.delete(e.getId());
        });
    }
}
2
KG6ZVP