Adding an onclicklistener to listview (android)

Question

I've managed to implement a great listview that I found here http://www.learn-android.com/2011/11/22/lots-of-lists-custom-adapter/comment-page-1/ but I can't seem to add an onclicklistener I just want to be able to do an action when I click on the row, with the data that the row contains of course any ideas? thanks

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.liste);
    this.requestWindowFeature(Window.FEATURE_NO_TITLE);
    // Setup the list view
    final ListView prestListView = (ListView) findViewById(R.id.list);
    final prestationAdapterEco prestationAdapterEco = new prestationAdapterEco(this, R.layout.prestation);

    prestListView.setAdapter(prestationAdapterEco);

    // Populate the list, through the adapter
    for(final prestationEco entry : getPrestations()) {
        prestationAdapterEco.add(entry);
    }
    prestListView.setClickable(true);
    prestListView.setOnItemClickListener(new AdapterView.OnItemClickListener() {

        @Override
        public void onItemClick(AdapterView<?> arg0, View arg1, int position, long arg3) {
            Object o = prestListView.getItemAtPosition(position);
            String str=(String)o;//As you are using Default String Adapter
            Toast.makeText(getApplicationContext(),str,Toast.LENGTH_SHORT).show();
        }
    });
}

What is failing - or how is it failing? Do you see behavior like http://stackoverflow.com/questions/4579754/listview-no-longer-reacts-to-onclick-after-a-call-to-setselection ? — Heiko Rupp, Feb 01 '12 at 14:44
@MikeBryant saying "It crashes" isn't very informative. Can you please post the stacktrace that is produced when the app crashes? — slayton, Feb 01 '12 at 15:08
I don't know where that would be but I added a try catch statement and I get "java.lang.ClassCastException: com.packagename.prestationEco" — Mike Bryant, Feb 01 '12 at 15:29

score 60 · Accepted Answer · edited Aug 06 '17 at 07:45

60

listView.setOnItemClickListener(new OnItemClickListener() {

    public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
        Object o = prestListView.getItemAtPosition(position);
        prestationEco str = (prestationEco)o; //As you are using Default String Adapter
        Toast.makeText(getBaseContext(),str.getTitle(),Toast.LENGTH_SHORT).show();
    }
});

edited Aug 06 '17 at 07:45

spuente

314
5
16

answered Feb 01 '12 at 14:45

Maulik J

2,745
19
22

1

that's exactly what I tried, how would I get the item's details? – Mike Bryant Feb 01 '12 at 14:49
I always add a vector into my custom listview and to get data i create a new object, eg Object obj = (Object)Vector.get(position); – Maulik J Feb 01 '12 at 14:54
i have created a custom list adapter wherin i pass the vector to the class and it creates a listview with the total vector size and data in it... – Maulik J Feb 01 '12 at 15:08

Jean-Philippe Roy · Answer 2 · 2015-07-10T19:02:22.800

9

If your Activity extends ListActivity, you can simply override the OnListItemClick() method like so:

/** {@inheritDoc} */
@Override  
protected void onListItemClick(ListView l, View v, int pos, long id) {  
    super.onListItemClick(l, v, pos, id);

    // TODO : Logic
}

edited Jul 10 '15 at 19:02

answered Feb 01 '12 at 14:45

Jean-Philippe Roy

4,752
3
27
41

the listview just extends activity not listactivity, http://www.learn-android.com/2011/11/22/lots-of-lists-custom-adapter/comment-page-1/ – Mike Bryant Feb 01 '12 at 14:50
For me in a ListFragment this was the ONLY way to get onClick to work. setOnItemClickListener just doesn't work. I guess the same would be true for ListActivity. See http://stackoverflow.com/questions/14080332/listfragment-onitemclicklistener-not-working – Dale Apr 28 '15 at 21:22

score 1 · Answer 3 · answered Nov 11 '13 at 16:12

list.setOnItemSelectedListener(new OnItemSelectedListener() {

        @Override
        public void onItemSelected(AdapterView<?> arg0, View arg1,
                int arg2, long arg3) {
            // TODO Auto-generated method stub

        }

        @Override
        public void onNothingSelected(AdapterView<?> arg0) {
            // TODO Auto-generated method stub

        }
    });

score 1 · Answer 4 · answered Feb 01 '12 at 16:04

The prestListView.getItemAtPosition(position); returns the UI widget: Text, Icon, ...

Try this instead:

Object o = prestationAdapterEco.getItemAtPosition(position);

or

Object o = arg0.getItemAtPosition(position);

Get the object from the adapter. Not from the list-view.

2. Object o is a prestationEco object. Not a String.

score 0 · Answer 5 · edited Nov 21 '16 at 10:46

0

Try this:

    list.setOnItemSelectedListener(new OnItemSelectedListener() {

    @Override
    public void onItemSelected(AdapterView<?> arg0, View arg1,
            int arg2, long arg3)
    }

    @Override
    public void onNothingSelected(AdapterView<?> arg0) {

    }
});

edited Nov 21 '16 at 10:46

mixel

25,177
13
126
165

answered Nov 21 '16 at 09:26

Tamil Selvan S

21
2

1

Please always add a comment to your answer. – Maxim Makhun Nov 21 '16 at 09:40

score 0 · Answer 6 · answered Feb 01 '12 at 16:05

0

You are doing

Object o = prestListView.getItemAtPosition(position);
String str=(String)o;//As you are using Default String Adapter

The o that you get back is not a String, but a prestationEco so you get a CCE when doing the (String)o

answered Feb 01 '12 at 16:05

Heiko Rupp

30,426
13
82
119

Adding an onclicklistener to listview (android)

6 Answers6

Linked

Related