36

This is my listview click event:

 lv1.setOnItemClickListener(new OnItemClickListener() {
       @Override
            public void onItemClick(AdapterView<?> a, View v, int position, long id) { 
                Object o = lv1.getItemAtPosition(position);
                ItemDetails obj_itemDetails = (ItemDetails)o;
                Toast.makeText(ListViewImagesActivity.this, "You have chosen : " + " " + obj_itemDetails.getName(), Toast.LENGTH_LONG).show();

        }  
 });

And this is my button click event:

btnNxt = (Button) findViewById(R.id.btnNext);
 btnNxt.setOnClickListener(new OnClickListener() {
 @Override
 public void onClick(View arg0) {
  //Here I need to get that position
});

Now I need to get the position of the clicked listview item..I have declared postion globally but that always gives me position 1.

Can anyone say me how do I get the positon ?

Mycoola
  • 1,135
  • 1
  • 8
  • 29
coder
  • 13,002
  • 31
  • 112
  • 214

5 Answers5

74

do you execute this

    btnNxt = (Button) findViewById(R.id.btnNext);
 btnNxt.setOnClickListener(new OnClickListener() {
 @Override
 public void onClick(View arg0) {
  //Here I need to get that position
});

inside the getView method? if so it's very easy

     btnNxt = (Button) findViewById(R.id.btnNext);
     btnNxt.setTag(position);
     btnNxt.setOnClickListener(new OnClickListener() {
     @Override
     public void onClick(View arg0) {
      int position=(Integer)arg0.getTag();
    });
Pasquale Anatriello
  • 2,355
  • 1
  • 16
  • 16
  • Hello sir, Please tell me how can i get the position of particular element without clicking on that element just basis of element name . – Nikhil Singh Feb 01 '16 at 08:06
  • @NikhilSingh you should open a question on your own, or use the search function. Anyway, the method `public View getView(int position, View convertView, ViewGroup parent)` of Adapter class already containts a parameter indicating the item position. – ocramot Mar 22 '16 at 14:07
  • Using tag is a bad idea. For instance, you can't call setTag on an ImageView if you're loading with Glide. – Tyler Apr 15 '19 at 15:08
33

I know I'm bit late with the answer, but what about this:

private OnClickListener myButtonClickListener = new OnClickListener() {
@Override
public void onClick(View v) {
    View parentRow = (View) v.getParent();
    ListView listView = (ListView) parentRow.getParent();
    final int position = listView.getPositionForView(parentRow);
  }
};

in this case you don't need to set tags and so on.

narancs
  • 5,234
  • 4
  • 41
  • 60
7

If you are using ViewHolder make sure you use setTag() like this.

    public View getView(int position, View convertView, ViewGroup parent) {
        final ViewHolder holder;


        if (convertView == null) {
            holder = new ViewHolder();
            convertView = mLayoutInflater.inflate(R.layout.custom_row, parent, false);
            holder.btnNxt = (Button) convertView.findViewById(R.id.btnNext);
            holder.btnNxt.setOnClickListener(new OnClickListener() {

                @Override
                public void onClick(View arg0) {
                    int position=(Integer)arg0.getTag();
                });
            convertView.setTag(holder);
        } else {
            holder = (ViewHolder) convertView.getTag();
        }
          holder.btnNxt.setTag(position);   // <---- setTag() here 

            return convertView;
        }

        private class ViewHolder {
            Button btnNxt;
        }
    }

If you setTag() before your setOnClickListener() line, you will have problem when you scroll your listview.

Lazy Ninja
  • 22,342
  • 9
  • 83
  • 103
0

declare a global variable and in onItemClick() method of lisview set the view position as variable's value. now use same variable in click method of button.

int global_position =0; // your global variable 

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

btnNxt = (Button) findViewById(R.id.btnNext);
 btnNxt.setOnClickListener(new OnClickListener() {
 @Override
 public void onClick(View arg0) {
    Toast.makeText(<context>,""+global_position,500).show();
});
r4jiv007
  • 2,974
  • 3
  • 29
  • 36
0

// use "top" to get listview items top position

// Listview items top position

    int top;
    listView.setOnItemClickListener(new AdapterView.OnItemClickListener() {
        @Override
        public void onItemClick(AdapterView<?> parent, View lView, final int pos, long id) {
            top = lView.getTop();
            if (top < 1) top = 1;
        }
    });

// Or // Listview items top position Under CustomAdapter's getView method

    int top;
    final View finalConvertView = convertView;
    convertView.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {

            top = finalConvertView.getTop();

            if (top < 1) top = 1;

        }
    });
Ahamadullah Saikat
  • 4,437
  • 42
  • 39