Почему Android Room не использует мой автоматически сгенерированный идентификатор, как только я создаю свой объект? - PullRequest
Новая
       12

Почему Android Room не использует мой автоматически сгенерированный идентификатор, как только я создаю свой объект?

0 голосов
/ 17 февраля 2019

Я создаю приложение, для которого требуется команда игроков.Я использую идентификатор команды в качестве первичного ключа команды и внешнего ключа для каждого игрока.В одном фрагменте я создаю новую команду.Когда команда создается и добавляется в базу данных моей комнаты, она изначально имеет идентификатор 0 или не установлена, хотя для автоматического создания установлено значение true.Затем я перехожу к представлению списка команды, в котором есть возможность добавлять новых игроков в команду.Когда я создаю нового игрока и использую новый идентификатор команды в модели представления команды, идентификатор команды по-прежнему равен 0 или не установлен, поэтому приложение аварийно завершает работу и возникает ошибка ограничения внешнего ключа.После сбоя, если я снова открою приложение или если я избегу сбоя, вернувшись к списку команд и выбрав команду, которая была только что создана с начальным идентификатором 0, когда я создаю игрока на этот раз, у команды будет действительный идентификатор,Почему комната не сразу назначает уникальный идентификатор при создании объекта, и она ожидает навигации в обратном направлении к фрагменту или перезапуска приложения?Соответствующий код ниже, похоже, что я могу давать слишком много кода, но я следую рекомендациям Jetpack, которые я нашел из документации Android, и я не знаю, откуда возникла проблема.https://developer.android.com/jetpack/docs/guide.

База данных 

@Database (entities = {Team.class,
                   Player.class},
       version = 6)
public abstract class AppDatabase
    extends RoomDatabase
{
private static final String DATABASE_NAME = "Ultimate_Stats_Database";
private static volatile AppDatabase instance;

public abstract TeamDAO teamDao ();
public abstract PlayerDAO playerDAO ();

static synchronized AppDatabase getInstance (Context context)
{
    if (instance == null)
    {
        // Create the instance
        instance = create(context);
    }

    // Return the instance
    return instance;
}

private static AppDatabase create (final Context context)
{
    // Create a new room database
    return Room.databaseBuilder(
                context,
                AppDatabase.class,
                DATABASE_NAME)
               .fallbackToDestructiveMigration()    // TODO Add migrations, poor practice to ignore
               .build();
}
}

Сущность команды 

@Entity (tableName = "teams")
public class Team
    implements Parcelable
{
@PrimaryKey (autoGenerate = true)
private long id;
private String name;


public Team ()
{
    this.name = "";
}


public Team (String name)
{
    this.name = name;
}
...

Команда DAO 

@Dao
public abstract class TeamDAO
{

@Insert (onConflict = OnConflictStrategy.REPLACE)
public abstract long insert (Team team);


@Delete
public abstract int deleteTeam (Team team);


@Query ("SELECT * FROM teams")
public abstract LiveData<List<Team>> getAllTeams ();
}

Репозиторий команды (только вставка) 

private TeamDAO teamDao;
private LiveData<List<Team>> teams;

public TeamRepository (Application application)
{
    AppDatabase db = AppDatabase.getInstance(application);
    teamDao = db.teamDao();
    teams = teamDao.getAllTeams();
}

private static class insertAsyncTask
        extends AsyncTask<Team, Void, Void>
{

    private TeamDAO asyncTeamTaskDao;


    insertAsyncTask (TeamDAO teamDao)
    {
        asyncTeamTaskDao = teamDao;
    }


    @Override
    protected Void doInBackground (final Team... params)
    {
        // Trace entry
        Trace t = new Trace();

        // Insert the team into the database
        asyncTeamTaskDao.insert(params[0]);

        // Trace exit
        t.end();

        return null;
    }
}

Team View Model 

public class TeamViewModel
    extends AndroidViewModel
{
private TeamRepository teamRepository;
private LiveData<List<Team>> teams;
private MutableLiveData<Team> selectedTeam;

public TeamViewModel (Application application)
{
    super(application);
    teamRepository = new TeamRepository(application);
    teams = teamRepository.getAllTeams();
    selectedTeam = new MutableLiveData<Team>();
}

public LiveData<Team> getSelectedTeam()
{
    return selectedTeam;
}

public void selectTeam(Team team)
{
    selectedTeam.setValue(team);
}

public LiveData<List<Team>> getTeams ()
{
    return teams;
}

public void insert (Team team)
{
    teamRepository.insert(team);
}
...

Сущность игрока 

@Entity(tableName = "players",
        foreignKeys = @ForeignKey(entity = Team.class,
                              parentColumns = "id",
                              childColumns = "teamId"),
        indices = {@Index(value = ("teamId"))})
public class Player
    implements Parcelable
{

@PrimaryKey (autoGenerate = true)
private long id;
private String name;
private int line;
private int position;
private long teamId;

public Player ()
{
    this.name = "";
    this.line = 0;
    this.position = 0;
    this.teamId = 0;
}


public Player(String name,
              int line,
              int position,
              long teamId)
{
    this.name = name;
    this.line = line;
    this.position = position;
    this.teamId = teamId;
}
....

Игрок DAO 

@Dao
public abstract class PlayerDAO
{

@Insert (onConflict = OnConflictStrategy.REPLACE)
public abstract void insert (Player player);


@Delete
public abstract int deletePlayer (Player player);


@Query ("SELECT * FROM players WHERE teamId = :teamId")
public abstract LiveData<List<Player>> getPlayersOnTeam (long teamId);


@Query ("SELECT * FROM players")
public abstract LiveData<List<Player>> getAllPlayers();


@Query ("SELECT * FROM players WHERE id = :id")
public abstract LiveData<Player> getPlayerById (long id);
}

Хранилище игрока (только для вставки) 

private PlayerDAO playerDAO;
private LiveData<List<Player>> players;

public PlayerRepository(Application application)
{
    AppDatabase db = AppDatabase.getInstance(application);
    playerDAO = db.playerDAO();
    players = playerDAO.getAllPlayers();
}

public void insert (Player player)
{
    new PlayerRepository.insertAsyncTask(playerDAO).execute(player);
}

private static class insertAsyncTask
        extends AsyncTask<Player, Void, Void>
{
    private PlayerDAO asyncTaskDao;

    insertAsyncTask (PlayerDAO dao)
    {
        asyncTaskDao = dao;
    }

    @Override
    protected Void doInBackground (final Player... params)
    {
        // Get the player being inserted by its id
        LiveData<Player> player = asyncTaskDao.getPlayerById(((Player) params[0]).getId());

        if (player != null)
        {
            // Delete the old record of the player
            asyncTaskDao.deletePlayer(params[0]);
        }

        // Insert the player into the database
        asyncTaskDao.insert(params[0]);

        return null;
    }
}
...

Модель просмотра игрока 

public class PlayerViewModel
    extends AndroidViewModel
{
private PlayerRepository playerRepository;
private LiveData<List<Player>> players;
private MutableLiveData<Player> selectedPlayer;

public PlayerViewModel(Application application)
{
    super(application);
    playerRepository = new PlayerRepository(application);
    players = playerRepository.getAllPlayers();
    selectedPlayer = new MutableLiveData<Player>();
}

public LiveData<Player> getSelectedPlayer()
{
    return selectedPlayer;
}

public void selectPlayer(Player player)
{
    selectedPlayer.setValue(player);
}

public LiveData<List<Player>> getPlayers ()
{
    return players;
}

public void insert (Player player)
{
    playerRepository.insert(player);
}
...

Где я создаю команду (в TeamListFragment и когда фрагмент диалога завершен) 

public void onDialogPositiveClick (String teamName)
{
    // Trace entry
    Trace t = new Trace();

    // Create a new team object
    Team newTeam = new Team();

    // Name the new team
    newTeam.setName(teamName);

    // Insert the team into the database and set it as the selected team
    teamViewModel.insert(newTeam);
    teamViewModel.selectTeam(newTeam);

    // Trace exit
    t.end();

    // Go to the player list view
    routeToPlayerList();
}

В playerListFragment при его создании

    /*------------------------------------------------------------------------------------------------------------------------------------------*
     *  If the view model has a selected team                                                                                                   *
     *------------------------------------------------------------------------------------------------------------------------------------------*/
    if (sharedTeamViewModel.getSelectedTeam().getValue() != null)
    {
        // Set the team to the team selected
        team = sharedTeamViewModel.getSelectedTeam().getValue();

        // Set the team name fields default text
        teamNameField.setText(team.getName());
    }

В playerFragment при нажатии кнопки сохранения 

        @Override
        public void onClick (View v)
        {
            // Trace entry
            Trace t = new Trace();

            // Update the player object with the info given by the user
            boolean success = getUserInput();

            /*------------------------------------------------------------------------------------------------------------------------------*
             *  If the input was valid                                                                                                      *
             *------------------------------------------------------------------------------------------------------------------------------*/
            if (success)
            {
                // Set the player id to the team that is selected
                player.setTeamId(sharedTeamViewModel.getSelectedTeam()
                                                    .getValue()
                                                    .getId());

                // Input the the player into the player view model
                sharedPlayerViewModel.insert(player);

                // Remove this fragment from the stack
                getActivity().onBackPressed();
            }

            // Trace exit
            t.end();
        }

Если есть какой-либо другой код, который требуется, сообщите мне

1 Ответ

0 голосов
/ 17 февраля 2019

Это ожидаемое поведение.Room не будет напрямую обновлять поле id в newTeam.

. Room не имеет смысла изменять входной объект, не говоря уже о том, что Room не предполагает, чтополя сущностей являются изменяемыми.Вы можете сделать все свои поля Entity неизменяемыми, и я считаю хорошей практикой делать классы сущностей неизменяемыми, когда это возможно.

Если вы хотите получить id вставленной строки, проверьте этоSO link: Android Room - Получить идентификатор новой вставленной строки с помощью функции автоматического создания

...