InflateException: Couldn't resolve menu item onClick handler

Question

I asked this question 6 years ago. In the meantime Android development best practices have changed, and I have become a better developer. Since then, I have realized that using the `onClick` XML attribute is a bad practice, and have removed it from any code base I work on. All of my click handlers are now defined in the code of the app, not the XML layouts! My reasons for never using `onClick` are - it is easy to make a mistake in the value of the `onClick` XML attribute, which will then result in a run-time error - a developer might refactor the name of the click handler method, without realizing it is called from a layout (see reason 1) - finding out which method is actually being called is not always obvious. Especially if the layout is being used by a Fragment - separating the concerns of layout vs behavior is good. Using `onClick` mixes them up, which is bad! I hope I have convinced you to never use `onClick` in a layout :) ! Below is my original question, which is a pretty good…

Accepted Answer

I found a solution that worked for me. Usually the `onClick` attribute in a layout has the following method

```
public void methodname(View view) { 
    // actions
}
```

On a menu item (in this case Sherlock menu) it should follow the following signature:

```
public boolean methodname(MenuItem item) { 
    // actions
}
```

So, your problem was that your method returned `void` and not `boolean`.

InflateException: Couldn't resolve menu item onClick handler

Solution

Problem

Related problems