Не удается отредактировать или обновить записи в базе данных комнат - PullRequest
Новая
       25

Не удается отредактировать или обновить записи в базе данных комнат

0 голосов
/ 10 мая 2019

Я создаю приложение, в котором пользователь может хранить свои имена пользователей и пароли.Приложение имеет простой пользовательский интерфейс.Главный поток имеет список записей, FAB и значок «Удалить все» на панели действий.Моя проблема в том, что я не могу редактировать и обновлять существующие записи.

У меня есть следующий код в onCreate() моего MainActivity.java.Когда пользователь удерживает запись, он запускает действие AddEditEntry.java.Здесь происходит то, что запущенная операция не имеет существующих данных записи в своих полях EditText: 

adapter.setOnItemLongClickListener(new RecyclerViewAdapter.OnItemLongClickListener() {
            @Override
            public void onItemLongClick(Entries entries) {
                Intent intent = new Intent(MainActivity.this, AddEditEntry.class);
                intent.putExtra(AddEditEntry.EXTRA_USERNAME, entry.getUsername());
                intent.putExtra(AddEditEntry.EXTRA_HINT, entry.getHint());
                intent.putExtra(AddEditEntry.EXTRA_PASSWORD, entry.getPassword());
                intent.putExtra(AddEditEntry.EXTRA_ID, entry.getId());
                startActivityForResult(intent, EDIT_ENTRY_REQUEST);
            }
        });

В моей активности AddEditEntry.java у меня есть следующий код в onClickкнопка сохраненияЯ добавляю новые данные в качестве дополнений к цели: 

saveEntry.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent data = new Intent();
                data.putExtra(EXTRA_USERNAME, usernameEditText.getText().toString());
                data.putExtra(EXTRA_HINT, hintEditText.getText().toString());
                data.putExtra(EXTRA_PASSWORD, passwordEditText.getText().toString());

                int id = getIntent().getIntExtra(EXTRA_ID, -1);

                if(id != -1){data.putExtra(EXTRA_ID, id);}

                setResult(RESULT_OK, data);
                finish();
            }
        });

и обратно в мой MainActivity.jav, это мой onActivityResult(): 

@Override
protected void onActivityResult(int requestCode, int resultCode, @Nullable Intent data) {
    super.onActivityResult(requestCode, resultCode, data);

    if(requestCode == ADD_ENTRY_REQUEST && resultCode == RESULT_OK){
        String username = Objects.requireNonNull(data).getStringExtra(AddEditEntry.EXTRA_USERNAME);
        String password = Objects.requireNonNull(data).getStringExtra(AddEditEntry.EXTRA_PASSWORD);
        String hint = Objects.requireNonNull(data).getStringExtra(AddEditEntry.EXTRA_HINT);

        Entries entry = new Entries(username, hint, password);
        viewModel.insert(entry);

        Toast.makeText(this, "Entry added!", Toast.LENGTH_SHORT).show();

    }else if(requestCode == EDIT_ENTRY_REQUEST && resultCode == RESULT_OK){
        int id = Objects.requireNonNull(data).getIntExtra(AddEditEntry.EXTRA_ID, -1);
        String username = Objects.requireNonNull(data).getStringExtra(AddEditEntry.EXTRA_USERNAME);
        String password = Objects.requireNonNull(data).getStringExtra(AddEditEntry.EXTRA_PASSWORD);
        String hint = Objects.requireNonNull(data).getStringExtra(AddEditEntry.EXTRA_HINT);

        if (id == -1){Toast.makeText(addEditEntry, "Something went wrong", Toast.LENGTH_SHORT).show();}

        Entries entry = new Entries(username, hint, password);
        entry.setId(id);
        viewModel.update(entry);

        Toast.makeText(this, "Entry updated", Toast.LENGTH_SHORT).show();

    }
    else{Toast.makeText(this, "Entry not added!", Toast.LENGTH_SHORT).show();}
}

Когда я запускаю приложениеи попробуйте отредактировать запись, сообщение Toast гласит «Запись обновлена!»поэтому он запускает этот код, но изменений не существует.Я попытался остановить приложение и перезапустить его, чтобы обновить его, но оно все еще не существует.

ViewModel.java: 

public class EntryViewModel extends AndroidViewModel {
    private EntryRepository repository;
    private LiveData<List<Entries>> allEntries;


    public EntryViewModel(@NonNull Application application) {
        super(application);
        repository = new EntryRepository(application);
        allEntries = repository.getAllEntries();
    }

    public void insert(Entries entries){repository.insert(entries);}

    public void update(Entries entries){repository.update(entries);}

    public void delete(Entries entries){repository.delete(entries);}

    public void deleteAll(){repository.deleteAllEntries();}

    public LiveData<List<Entries>> getAllEntries() {return allEntries;}
}

EntryRepository.java: 

public class EntryRepository {
    private EntryDAO entryDAO;
    private LiveData<List<Entries>> allEntries;

    public EntryRepository(Application application){
        EntryDatabase database = EntryDatabase.getInstance(application);
        entryDAO = database.generateDao();
        allEntries = entryDAO.getAllEntries();
    }

    public void insert(Entries entries){new InsertEntryAsyncTask(entryDAO).execute(entries);}

    public void update(Entries entries){new UpdateEntryAsyncTask(entryDAO).execute(entries);}

    public void delete(Entries entries){new DeleteEntryAsyncTask(entryDAO).execute(entries);}

    public void deleteAllEntries(){new DeleteAllEntriesAsyncTask(entryDAO).execute();}

    public LiveData<List<Entries>> getAllEntries(){return allEntries;}


    public static class InsertEntryAsyncTask extends AsyncTask<Entries, Void, Void>{
        private EntryDAO entryDAO;

        private InsertEntryAsyncTask(EntryDAO entryDAO){this.entryDAO = entryDAO;}

        @Override
        protected Void doInBackground(Entries... entries) {
            entryDAO.insert(entries[0]);
            return null;
        }
    }

    public static class UpdateEntryAsyncTask extends AsyncTask<Entries, Void, Void>{
        private EntryDAO entryDAO;

        private UpdateEntryAsyncTask(EntryDAO entryDAO){
            this.entryDAO = entryDAO;
        }
        @Override
        protected Void doInBackground(Entries... entries) {
            entryDAO.update(entries[0]);
            return null;
        }
    }

    public static class DeleteEntryAsyncTask extends AsyncTask<Entries, Void, Void>{
        private EntryDAO entryDAO;

        private DeleteEntryAsyncTask(EntryDAO entryDAO){this.entryDAO = entryDAO;}

        @Override
        protected Void doInBackground(Entries... entries) {
            entryDAO.delete(entries[0]);
            return null;
        }
    }

    public static class DeleteAllEntriesAsyncTask extends AsyncTask<Void, Void, Void>{
        private EntryDAO entryDAO;

        private DeleteAllEntriesAsyncTask(EntryDAO entryDAO){this.entryDAO = entryDAO;}

        @Override
        protected Void doInBackground(Void... voids) {
            entryDAO.deleteAllEntries();
            return null;
        }
    }
}

EntryDAO.java: 

@Dao
public interface EntryDAO {

    @Insert
    void insert(Entries entries);

    @Update
    void update(Entries entries);

    @Delete
    void delete(Entries entries);

    @Query("DELETE FROM entries_table")
    void deleteAllEntries();

    @Query("SELECT * FROM entries_table")
    LiveData<List<Entries>> getAllEntries();
}

Entries.java: 

@Entity(tableName = "entries_table")
public class Entries {
    @PrimaryKey(autoGenerate = true)
    private int id;

    private String username, hint, password;

    public Entries(String username, String hint, String password){
        this.username = username;
        this.hint = hint;
        this.password = password;
    }

    public Entries(){}

    public int getId() {return id;}

    public void setId(int id) {this.id = id;}

    public String getUsername() {return username;}

    public void setUsername(String username) {this.username = username;}

    public String getHint() {return hint;}

    public void setHint(String hint) {this.hint = hint;}

    public String getPassword() {return password;}

    public void setPassword(String password) {this.password = password;}
}

Это onCreate() моего AddEditEntry.java класса.Я добавил следующие сообщения Toast, чтобы увидеть, получает ли он данные вообще, и оказывается, что нет.Тост сообщения были пустыми: 

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

        usernameEditText = findViewById(R.id.username_field);
        passwordEditText = findViewById(R.id.password_field);
        hintEditText = findViewById(R.id.hint_field);

        passwordABCD = findViewById(R.id.upp_checkbox);
        passwordabcd = findViewById(R.id.low_checkbox);
        password0123 = findViewById(R.id.num_checkbox);
        passwordSymbols = findViewById(R.id.sym_checkbox);

        radio4 = findViewById(R.id.four);
        radio8 = findViewById(R.id.eight);
        radio12 = findViewById(R.id.twelve);
        radio16 = findViewById(R.id.sixteen);

        Button generatePassword = findViewById(R.id.btn_password_generate);
        Button saveEntry = findViewById(R.id.btn_save);

        Intent intent = getIntent();

        if(intent.hasExtra(EXTRA_ID)){
        setTitle("Edit Entry");
        usernameEditText.setText(Objects.requireNonNull(getIntent().getExtras()).getString(EXTRA_USERNAME));
        passwordEditText.setText(Objects.requireNonNull(getIntent().getExtras()).getString(EXTRA_PASSWORD));
        hintEditText.setText(Objects.requireNonNull(getIntent().getExtras()).getString(EXTRA_HINT));

        Toast.makeText(this, "Info Received!!!", Toast.LENGTH_SHORT).show();
        Toast.makeText(this, Objects.requireNonNull(getIntent().getExtras()).getString(EXTRA_USERNAME), Toast.LENGTH_SHORT).show();
        Toast.makeText(this, Objects.requireNonNull(getIntent().getExtras()).getString(EXTRA_PASSWORD), Toast.LENGTH_SHORT).show();
        Toast.makeText(this, Objects.requireNonNull(getIntent().getExtras()).getString(EXTRA_HINT), Toast.LENGTH_SHORT).show();
    }
        else{setTitle("Add Entry");}

        generatePassword.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {passwordEditText.setText(generatedPassword());}});

        saveEntry.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent data = new Intent();
                data.putExtra(EXTRA_USERNAME, usernameEditText.getText().toString());
                data.putExtra(EXTRA_HINT, hintEditText.getText().toString());
                data.putExtra(EXTRA_PASSWORD, passwordEditText.getText().toString());

                int id = getIntent().getIntExtra(EXTRA_ID, -1);

                if(id != -1){data.putExtra(EXTRA_ID, id);}

                setResult(RESULT_OK, data);
                finish();
            }
        });
    }

1 Ответ

1 голос
/ 10 мая 2019

Сделайте это так

В вашем MainActivity.java

    ....
    ....
    adapter.setOnItemLongClickListener(new RecyclerViewAdapter.OnItemLongClickListener() {
        @Override
        public void onItemLongClick(Entries entries) {
            entry = entries;  // this is very important, entry holds the current edited item
            Intent intent = new Intent(MainActivity.this, AddEditEntry.class);
            intent.putExtra(AddEditEntry.EXTRA_USERNAME, entry.getUsername());
            intent.putExtra(AddEditEntry.EXTRA_HINT, entry.getHint());
            intent.putExtra(AddEditEntry.EXTRA_PASSWORD, entry.getPassword());
            // no need to pass the id, it's a autogenerated field 
            // intent.putExtra(AddEditEntry.EXTRA_ID, entry.getId());
            startActivityForResult(intent, EDIT_ENTRY_REQUEST);
        }
    });

    ....
    ...

    @Override
    protected void onActivityResult(int requestCode, int resultCode, @Nullable Intent data) {

        if(requestCode == ADD_ENTRY_REQUEST && resultCode == RESULT_OK){
            ...
            ... 
        } else if(requestCode == EDIT_ENTRY_REQUEST && resultCode == RESULT_OK) {
            // in an Edit operation, id should not be modified, so, no need to pass this parameter 
            // int id = 
            // Objects.requireNonNull(data).getIntExtra(AddEditEntry.EXTRA_ID, -1);
            String username = Objects.requireNonNull(data).getStringExtra(AddEditEntry.EXTRA_USERNAME);
            String password = Objects.requireNonNull(data).getStringExtra(AddEditEntry.EXTRA_PASSWORD);
            String hint = Objects.requireNonNull(data).getStringExtra(AddEditEntry.EXTRA_HINT);

            // entry already exists, so, no need to create a new one            
            //Entries entry = new Entries(username, hint, password);
            //entry.setId(id);
            entry.setUsername(username);
            entry.setPassword(password);
            entry.setHint(hint);
            viewModel.update(entry);

            Toast.makeText(this, "Entry updated", Toast.LENGTH_SHORT).show();

        } else {
            Toast.makeText(this, "Entry not added!", Toast.LENGTH_SHORT).show();
        }
        super.onActivityResult(requestCode, resultCode, data);
    }

Другие замечания ...

В вашем RecyclerViewAdapter.java

    // This is not needed. Your list is already created in your Room query
    //private List<Entries> entries = new ArrayList<>();

    private List<Entries> entries;

В вашем MainActivity.java

    // This is not needed
    // AddEditEntry addEditEntry;
    ....
    ....
    // addEditEntry = new AddEditEntry();
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.

Похожие темы

...