如何在Android中为新菜单项设置onclicklistener

Question

I am adding a menu item for texts selection. normally it shows cut,copy,share, etc.. I added a one more item to this menu and named the item as "Mark".

For this I added the following code in my Activity.Java

@Override
public void onActionModeStarted(ActionMode mode) {
    if (mActionMode == null) {
        mActionMode = mode;
        Menu menu = mode.getMenu(); 
        menu.add("Mark");
        mode.getMenuInflater().inflate(R.menu.main, menu);
    }
    super.onActionModeStarted(mode);
}

I am able to get my menu item while long press on texts.. Below is the screen, which reflects my menu item.

For this menu item, I want to do something while select it. So, I used the following code.

@Override
public boolean onOptionsItemSelected(MenuItem item) {
    if( item.getTitle().equals("Mark")){
        System.out.println("MenuItem Mark clicked----");
        Log.d("MenuItem  clicked----", "Mark");          
    }
    return super.onOptionsItemSelected(item);
}

Here, I am not able to get the message "MenuItem Mark clicked----" or "MenuItem clicked---- Mark" in my Logcat.

How may I do this?

update

Followed Elitz's answer, but still no luck. my Changes below

@Override
public void onActionModeStarted(ActionMode mode) {
    if (mActionMode == null) {
        mActionMode = mode;
        Menu menu = mode.getMenu();  
        menu.add(0,1000,0,"Mark");
        mode.getMenuInflater().inflate(R.menu.main, menu);
    }
    super.onActionModeStarted(mode);
}

@Override
public boolean onOptionsItemSelected(MenuItem item) {
    if (item.getItemId() == 1000) {
         System.out.println("MenuItem Mark clicked----");
         Log.d("MenuItem  clicked----", "Mark");
    }         
    return super.onOptionsItemSelected(item);
}

Added updated answer and still no messages came in Logcat

private ActionMode.Callback startActionMode = (new ActionMode.Callback() {

    @Override
    public boolean onPrepareActionMode(ActionMode mode, Menu yourMenu) {
        return false;
    }

    @Override
    public void onDestroyActionMode(ActionMode mode) {

    }       
    @Override
    public boolean onCreateActionMode(ActionMode mode, Menu yourMenu) {
        return false;
    }       
    @Override
    public boolean onActionItemClicked(ActionMode mode, MenuItem yourMenu) {
        Log.d("MenuItem  clicked----303", "Mark");
        if (yourMenu.getItemId() == R.attr.actionModeSelectAllDrawable) {
             System.out.println("MenuItem Mark clicked----305");
             Log.d("MenuItem  clicked----", "Mark");
        }         
        return true;
    }
});

update2

Followed Ankesh's answer also,

@Override
public void onActionModeStarted(ActionMode mode) {
    if (mActionMode == null) {
        mActionMode = mode;
        Menu menu = mode.getMenu();  
        menu.add(R.id.privateText);
        mode.getMenuInflater().inflate(R.menu.main, menu);
    }
    super.onActionModeStarted(mode);
}

@Override
public boolean onOptionsItemSelected(MenuItem item) {
    if (item.getItemId() == R.id.privateText) {
         System.out.println("MenuItem Mark clicked----");
         Log.d("MenuItem  clicked----", "Mark");
    }         
    return super.onOptionsItemSelected(item);
}

Both attempt there is no logs available...

Update3

For test the other menu, I found id for selectAll menu item in R,java, and tried the following code,

@Override
public boolean onOptionsItemSelected(MenuItem item) {           if (item.getItemId() == R.attr.actionModeSelectAllDrawable) {
         System.out.println("MenuItem All clicked----");
         Log.d("MenuItem  clicked----", "All");
    }         
    return super.onOptionsItemSelected(item);
}

this is also not showing the message in Logcat. Is this function is right one for menu items selection. or what else I am missing in this?

Answer 1

use this instead of your menu.add("Mark");

add(int groupId, int itemId, int order, CharSequence title);

now you have your Id now you can check for that.

groupId = 0; and order = 0; , or any number that fits your choice, but since in your example you have only 1 group, just put 0.

hmm.. i think you got us all fooled :) yes it will not work, because you are using ActionMode right and with ActionMode you need to specify a callback for it, so you should put your code here

    @Override
    public boolean onActionItemClicked(ActionMode mode, MenuItem arg1) {
        return false;
    }

in its callback. like something like this, when you call startActoinMode

startActionMode(new ActionMode.Callback() {

    @Override
    public boolean onPrepareActionMode(ActionMode mode, Menu yourMenu) {
    }

    @Override
    public void onDestroyActionMode(ActionMode mode) {

    }       
    @Override
    public boolean onCreateActionMode(ActionMode mode, Menu yourMenu) {
        return false;
    }       
    @Override
    public boolean onActionItemClicked(ActionMode mode, MenuItem yourMenu) {
        //put your item click here
        return false;
    }
});

Answer 2

Use

if (item.getItemId() == R.id.xyz) { 

}

Answer 3

Can you just add your menu item inside your xml menu(I mean inside R.menu.main )? And after that just detect the item click inside this method:

 @Override
    public boolean onOptionsItemSelected(MenuItem item) {
        switch (item.getItemId()) {
            case R.id.menu_item_id:
            //TODO click detected!!
            break
            default:
                return super.onOptionsItemSelected(item);
        }
    }

Its that or I don't understand your question :)