ListView's OnItemClick does not work

Question

I made a listview. Using this tutorial link It works fine. But the problem is when i try to add onitemclick listener in its activity class the click action does not work.i just want to show a toast message.And here it the code for onItemClickListener

listViewCity.setOnItemClickListener(new OnItemClickListener() {
                @Override
                public void onItemClick(AdapterView<?> a, View v, int position, long id) { 

                    String item = ((TextView)v).getText().toString();

                    Toast.makeText(getApplicationContext(), item, Toast.LENGTH_LONG).show();

            }  
        });

Why it doesn't work anybody can tell me???

Answer 1

If you are getting an exception or a force close, you should have mentioned it.

First of all, your OnItemClickListener is wrong, because the View that is passed to your OnClickListener is not a TextView so You can not just cast it to a TextView.

You can fix it using the following:

listViewCity.setOnItemClickListener(new OnItemClickListener() {
    @Override
    public void onItemClick(AdapterView<?> a, View v, int position, long id) { 
        String item = ((TextView)v.findViewById(R.id.cityName)).getText().toString();
            Toast.makeText(getApplicationContext(), item, Toast.LENGTH_LONG).show();

    }  
});

---

Try what I just did and it should work. If it doesn't work, you may need to remove the line android:autoLink="web" from your xml file.

---

The attribute android:autoLink="web" is making your View's TextView Clickable. As I mentioned in my comment, if any of your subviews is clickable, your OnItemClickListener will not receive events, because it will be consumed by the children.

Answer 2

Use this

Since you are using a customadapter you will have to do this in your customadapter class to handle onclicks of the listview...

convertView.setClickable(true);
    convertView.setOnClickListener(new OnClickListener() {

                    @Override
                    public void onClick(View paramView)
                    {
                      //if you want you can use the "position" value to get the desired row items.
                    }
                    });

Answer 3

Try this...

listViewCity.setOnItemClickListener(new OnItemClickListener() {
                @Override
                public void onItemClick(AdapterView<?> a, View v, int position, long id) { 
                     String item = listViewCity.getItemAtPosition(position).toString();

                    Toast.makeText(getApplicationContext(), item, Toast.LENGTH_LONG).show();

            }  
        });

Answer 4

I had the same problem and solved it by setting each view I needed to click to clickable and then added the listener. I put the following method inside the customized adapter (in getView method). Something like this:

public void setViewsClickable(int position) {
        myView.button.setClickable(true);
        myView.radioButton.setClickable(true);
        myView.button.setOnClickListener(new OnClickListener() {

            @Override
            public void onClick(View v) {
                // what this button should do
            }
        });

        myView.radioButton.setOnClickListener(new OnClickListener() {

            @Override
            public void onClick(View v) {
                // what clicking this radio button should do
            }
        });

    }