I want to update the listview in the activation every time I click the checkbox, but when I call the updateTodo () method from the checkbox listener, the application does not work.

MainActivity

@Override protected void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.activity_main); this.getListView().setDividerHeight(2); dbHelper = new TodoDatabase(this); fillData(); registerForContextMenu(getListView()); mEditText = (EditText)findViewById(R.id.editTextNewTask); RowId = null; Bundle extras = getIntent().getExtras(); RowId = (savedInstanceState == null) ? null : (Long) savedInstanceState .getSerializable(TodoDatabase.COLUMN_ID); if (extras != null) { RowId = extras.getLong(TodoDatabase.COLUMN_ID); } startService(new Intent(this, TimeService.class)); } ......... ......... public void updateTodo(){ fillData(); } private void fillData() { cursor = dbHelper.getAllTodosListView(); if (Build.VERSION.SDK_INT < Build.VERSION_CODES.HONEYCOMB) { startManagingCursor(cursor); } String[] from = new String[] {TodoDatabase.COLUMN_ACHIEVED, TodoDatabase.COLUMN_SUMMARY, TodoDatabase.COLUMN_TIMESECONDS}; int[] to = new int[] {R.id.checkBoxLabel, R.id.textViewLabel }; // Π’Π΅ΠΏΠ΅Ρ€ΡŒ создадим Π°Π΄Π°ΠΏΡ‚Π΅Ρ€ массива ΠΈ установим Π΅Π³ΠΎ для отобраТСния Π½Π°ΡˆΠΈΡ… // Π΄Π°Π½Π½Ρ‹Ρ… SimpleCursorAdapter notes = new MySimpleCursorAdapter(getApplicationContext(), R.layout.list_item, cursor, from, to, 0); notes.changeCursor(cursor); notes.notifyDataSetChanged(); setListAdapter(notes); }

SimpleCursorAdapter

 public class MySimpleCursorAdapter extends SimpleCursorAdapter { private Context context; private LayoutInflater mInflater; private CheckBox mCheckBox; private long xId; public MySimpleCursorAdapter(Context context, int layout, Cursor c, String[] from, int[] to, int flags) { super(context, layout, c, from, to, flags); mInflater = LayoutInflater.from(context); } @Override public View newView(Context context, Cursor cursor, ViewGroup parent) { final View view = mInflater.inflate(R.layout.list_item, parent, false); return view; } @Override public void bindView(View view, final Context context, final Cursor cursor) { final TodoDatabase mDbHelper = new TodoDatabase(context); final TextView textView = (TextView)view.findViewById(R.id.textViewLabel); mCheckBox = (CheckBox)view.findViewById(R.id.checkBoxLabel); xId =Long.parseLong(cursor.getString(cursor .getColumnIndex(TodoDatabase.COLUMN_ID))); textView.setText(cursor.getString(cursor.getColumnIndex(TodoDatabase.COLUMN_SUMMARY))); if (cursor.getString(cursor.getColumnIndex(TodoDatabase.COLUMN_TIMESECONDS)) != null) { if (cursor.getLong(cursor.getColumnIndex(TodoDatabase.COLUMN_TIMESECONDS)) > 0) { if (cursor.getLong(cursor.getColumnIndex(TodoDatabase.COLUMN_TIMESECONDS)) > System.currentTimeMillis()) { } else if (cursor.getInt(cursor.getColumnIndex(TodoDatabase.COLUMN_ACHIEVED)) != 1){ textView.setTextColor(Color.parseColor("#FF0000")); } } } if (cursor.getInt(cursor.getColumnIndex(TodoDatabase.COLUMN_ACHIEVED)) == 1) { mCheckBox.setChecked(true); textView.setPaintFlags(textView.getPaintFlags() | Paint.STRIKE_THRU_TEXT_FLAG); final Long mRowId = xId; mCheckBox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() { @Override public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { Long mXRowId = mRowId; if (mCheckBox.isChecked()) { TodoDatabase mDbHelper = new TodoDatabase(context); mDbHelper.updateTodoAchieved(mXRowId, "1"); MainActivity ma = new MainActivity(); ma.updateTodo(); // Ошибка } else if (!mCheckBox.isChecked()) { TodoDatabase mDbHelper = new TodoDatabase(context); mDbHelper.updateTodoAchieved(mXRowId, "0"); MainActivity ma = new MainActivity(); ma.updateTodo(); // Ошибка } } }); } else { mCheckBox.setChecked(false); textView.setPaintFlags(textView.getPaintFlags() & (~ Paint.STRIKE_THRU_TEXT_FLAG)); final Long mRowId = xId; mCheckBox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() { Long mXRowId = mRowId; @Override public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { if (mCheckBox.isChecked()){ TodoDatabase mDbHelper = new TodoDatabase(context); mDbHelper.updateTodoAchieved(mXRowId, "1"); MainActivity ma = new MainActivity(); ma.updateTodo(); // Ошибка } else if (!mCheckBox.isChecked()){ TodoDatabase mDbHelper = new TodoDatabase(context); mDbHelper.updateTodoAchieved(mXRowId, "0"); MainActivity ma = new MainActivity(); ma.updateTodo(); // Ошибка } } }); } }

}

  • If my answer is below, you are approached then mark it (accept) as correct. - Leo K
  • one
    So I could not figure it out unfortunately. As soon as I deal with your option, I will definitely note the result. - AndroidDev2015
  • The notifydatasetchanged () method came up to me or I can use INTENT. - AndroidDev2015
  • In general, yes, as I said, there are many ways. You asked and why you have a fall / error, and not the best way or different possible ways to update the list. - Leo K

1 answer 1

There are a lot of problems in your code. Specifically on your question, you do not need to create activity on a new one, it already exists, you have an error and for this new MainActivity() . There are different options, for example, you can pass an interface with an onUpdateTodo onUpdateTodo through the constructor and implement your interface to your MainActivity, and pass this : in the MySimpleCursorAdapter construct:

 new MySimpleCursorAdapter(getApplicationContext(), R.layout.list_item, cursor, from, to, 0, this);

Save this listener as a member class (field) and call it when clicking on the checkbox mListener.onUpdateTodo()