在setOnCheckedChangeListener方法内执行Toast消息

Question

I have an array of togglebuttons. There is a 1-to-1 ratio of toggleButtons with number of rows in a cursor. I assign an setId() to each toggleButton in the array with returned data from the cursor.

My update is always updating 1 row up in the cursor and I'm trying to troubleshoot this issue using some toast messages. The problem is that I can't get the toast message to display. I know the method is listener is firing because my sql update is running, it's just that it's the wrong row. Here is my code.

toggleButton[i].setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener()
        {
            ContentValues values = new ContentValues(); //Create the new content values object for data input
            @Override
            public void onCheckedChanged(CompoundButton buttonView, boolean isChecked)
            {
                if(isChecked)
                {
                    Toast.makeText(Cart.this.getBaseContext(), "hello", Toast.LENGTH_SHORT).show();
                    toggleButton[buttonView.getId()].setTextOn("Yes"); //Set the text when the button changes state to on
                    values.put(Contract.ItemsTable.COLUMN_ONE, 1); //Update the record on the 'status' column to true
                    String[] itemId = {String.valueOf(buttonView.getId())}; //
                    db.update(Contract.ItemsTable.TABLE_NAME, values, SmartCartContract.ItemsTable.COLUMN_ITEM_ID + " LIKE ?", itemId); //Update the DB record
                } else
                {
                    Toast.makeText(Cart.this.getBaseContext(), "hello", Toast.LENGTH_SHORT).show();
                    toggleButton[buttonView.getId()].setTextOff("No"); //Set the text when the button changes state to off
                    values.put(SmartCartContract.ItemsTable.COLUMN_TWO, 0); //Update the record on the 'status' column to false
                    String[] itemId = {String.valueOf(buttonView.getId())}; //
                    db.update(Contract.ItemsTable.TABLE_NAME, values, Contract.ItemsTable.COLUMN_ITEM_ID + " LIKE ?", itemId); //Update the DB record
                }
            }
        });

Cart.this.getBaseContext() is my last attempt of the options I've seen to use. Is this what I'm doing wrong?

I'm building those buttons dynamically with:

toggleButton[i] = new ToggleButton(this); //Create new ToggleButton control for this row    
toggleButton[i].setGravity(Gravity.CENTER); //Align the text in the middle 
toggleButton[i].setId(cursor.getInt(cursor.getColumnIndex("itemId"))); 
if (cursor.getInt(cursor.getColumnIndex("itemInCart")) == 1) //Check if marked 'in Cart'

Answer 1

Not sure what the problem with the database is, but here is a minimal example of some ToggleButtons with a working Toast.

public class CartActivity extends AppCompatActivity {

    private CompoundButton.OnCheckedChangeListener listener = new CompoundButton.OnCheckedChangeListener() {

        ContentValues cv;

        @Override
        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
            cv = new ContentValues();
            String key;
            int val;

            if (isChecked) {
                key = "col1";
                val = 1;
            } else {
                key = "col2";
                val = 0;
            }

            cv.put(key, val);
            String[] itemId = {String.valueOf(buttonView.getId())};
            Toast.makeText(CartActivity.this,
                    String.format("[%d] %s {%s}", buttonView.getId(), isChecked, cv),
                    Toast.LENGTH_SHORT).show();
        }
    };

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        LinearLayout view = new LinearLayout(this);
        view.setOrientation(LinearLayout.VERTICAL);

        LinearLayout.LayoutParams lp = new LinearLayout.LayoutParams(
                    ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.WRAP_CONTENT);

        ToggleButton[] buttons = new ToggleButton[3];
        for (int i = 0; i < buttons.length; i++) {
            ToggleButton toggle = new ToggleButton(this);

            toggle.setLayoutParams(lp);
            toggle.setId(i);
            toggle.setGravity(Gravity.CENTER);            
            toggle.setTextOn(String.format("On [%d]", i));
            toggle.setTextOff(String.format("Off [%d]", i));
            toggle.setChecked(false);
            toggle.setOnCheckedChangeListener(listener);

            buttons[i] = toggle;
            view.addView(toggle);
        }

        setContentView(view);
    }
}

Answer 2

Not sure what kind of object Cart is. But you could extend the Application class and save off the application context, which you could then use in your toasts.

import android.app.Application;
import android.content.Context;
import android.content.pm.PackageInfo;
import android.content.pm.PackageManager;

public class BaseApplication extends Application
{
    protected static Context context;
    protected static PackageInfo packageInfo;
    protected static BaseApplication baseApplication;

    @Override
    public void onCreate()
    {
        try
            {

                baseApplication = this;

                context = getApplicationContext();

                PackageManager manager = context.getPackageManager();
                String packageName = context.getPackageName();

                packageInfo = manager.getPackageInfo(packageName, 0);
            }
        catch (Exception e)
            {
                // ignore
            }
    }

    public static Context getContext()
    {
        return context;
    }

    public static PackageInfo getPackageInfo()
    {
        return packageInfo;
    }

    public static BaseApplication getApplication(){
        return baseApplication;
    }
}