Java 无法编辑或更新文件室数据库条目

我正在建立一个应用程序，用户可以存储他/她的用户名和密码。该应用程序有一个简单的用户界面。主线程在操作栏上有一个条目列表、一个FAB和一个deleteall图标。我的问题是我无法编辑和更新现有条目

我在MainActivity.java的onCreate中有以下代码。当用户持有条目时，它将启动addedEntry.java活动。此处发生的情况是，已启动的活动的EditText字段中没有现有的条目数据：

在AddedEntry.java活动中，单击save按钮可以看到以下代码。我将新数据作为额外数据添加到意图中：

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();
            }
        });

回到我的主要活动。jav，这是我的onActivityResult：

当我运行应用程序并尝试编辑条目时，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;}
}

这是我的AddedEntry.java类的onCreate。我添加了下面的Toast消息，以查看它是否接收到了数据，结果却没有。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();
            }
        });
    }

像这样做

在MainActivity.java中

其他评论

在您的RecycleServiceAdapter.java中

---

请分享你的想法viewmodel@Qasim就这么做了。检查编辑。另外，请粘贴您的EntryRepository，也请粘贴完毕。检查编辑。@oo92如何知道数据库已更新。您的更新DAO返回void，因此将其返回类型更改为intCheck编辑以查看Entries.java类。这种方法可能不起作用，为什么不呢？只是实现了可序列化到Entries类，从何处获取Entries.getposition？我假设Entries是传递给recyclerView适配器的查询列表。条目只是提交到不同的EditText字段中。它们不是作为列表传递的。

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;}
}

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;
        }
    }
}

@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();
}

@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;}
}

 @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();
            }
        });
    }

    ....
    ....
    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);
    }

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

    private List<Entries> entries;

    // This is not needed
    // AddEditEntry addEditEntry;
    ....
    ....
    // addEditEntry = new AddEditEntry();