ListView setOnItemClickListener nie działa przez dodanie przycisku


92

Mam widok listy z tekstem i przyciskiem w każdym wierszu, widok listy setOnItemClickListener () nie działa. czy można inaczej obsługiwać zdarzenia kliknięcia elementu i kliknięcia przycisku (kliknięcie elementu powinno wywołać ActivityA, a kliknięcie przycisku powinno wywołać ActivityB). Czy ktoś ma rozwiązanie

    private ArrayList<String> userIDArr = null;
    private ArrayList<String> userNameArr = null;
    private DatabaseHelper dbHelper = null;
    private ListView userListView=null; 


    public void onCreate(Bundle savedInstanceState) 
        {
          super.onCreate(savedInstanceState);         
          setContentView(R.layout.list_view);         
          dbHelper = new DatabaseHelper(this.getApplicationContext());        
          Map<String,ArrayList<String>> displayMap = dbHelper.getUserListToDisplay();
          userIDArr = displayMap.get("UserID");
          userNameArr = displayMap.get("FirstName1");           


          userListView = (ListView) findViewById(R.id.listView2);
          userListView.setAdapter(new UserListAdapter(this,userIDArr));


          userListView.setOnItemClickListener(new AdapterView.OnItemClickListener() {
              @Override
              public void onItemClick(AdapterView<?> arg0, View arg1, int position, long arg3) {

                  Toast.makeText(usersListActivity.this,
                            "Item in position " + position + " clicked", Toast.LENGTH_LONG).show();
              }
            });
     }


    public class UserListAdapter extends ArrayAdapter<String>
    {
        Activity context;
        public UserListAdapter(Activity context, ArrayList<String> names) {
            super(context, R.layout.list_item, names);
            this.context = context;
        }
        private class ViewHolder {
            public TextView UserNameAndID;
            public TextView Description;
            public Button  UploadBtn;
        }

        @Override
        public View getView(int position, View convertView, ViewGroup parent) {
            ViewHolder holder;
            View rowView = convertView;
            if (rowView == null) {
                LayoutInflater inflater = context.getLayoutInflater();
                rowView = inflater.inflate(R.layout.list_item, null, true);
                holder = new ViewHolder();
                holder.UserNameAndID = (TextView) rowView.findViewById(R.id.User_detailsTxt);
                holder.Description = (TextView) rowView.findViewById(R.id.User_status);
                holder.UploadBtn = (Button) rowView.findViewById(R.id.uploadbutton);
                holder.UploadBtn.setOnClickListener(new View.OnClickListener() {  

                        public void onClick(View v) {  
                        Toast.makeText(usersListActivity.this," Button clicked",Toast.LENGTH_SHORT).show();
                        }   
                    }); 
                    rowView.setTag(holder);
            } else {
                holder = (ViewHolder) rowView.getTag();
            }
            String s = userNameArr.get(position)+","+userIDArr.get(position);
            holder.UserNameAndID.setText(s);
            holder.Description.setText("U r in middle");
            return rowView;
        }
    }
}`

Odpowiedzi:


261

Spróbuj ustawić przyciski (lub inne widoki, które chcesz obsługiwać, kliknij wewnątrz elementu listy) w następujący sposób:

android:focusable="false"
android:focusableInTouchMode="false"

@Ben Lee: Dodałem je do moich pól wyboru! Wciąż nie działa. Co mogę zrobić źle?
TDaver,

1
android:focusableInTouchMode="false"rozwiązuje moje. Używam dwóch TextView w niestandardowym wierszu LinearLayout. Yuk android ...
Youngjae,

12
ImageButton nie będzie działać. W widoku elementu listy zawiera ImageButton, ustaw android: descendantFocusability = "blocksDescendants".
ChangUZ,

8
dodanie androida: clickable = "false" może być również wymagane (
włączaj

11
Ostrzeżenie: w przypadku, gdy używasz ImageButton, musisz użyć setFocusable (false), ponieważ konstruktor ImageButton włączy ten atrybut po nadmuchaniu z pliku xml. Ale dobra odpowiedź
Mateu

120

Czasami lista nadal nie może zmusić nasłuchiwania kliknięć do przejścia. W tym przypadku może być konieczne dodanie jeszcze jednego atrybutu.

android:descendantFocusability="blocksDescendants" 

I ten atrybut musi zostać dodany do najwyższego układu Twojego XML, gdzie dostarczyłeś elementy ListView.


Po prostu nikt inny nie robi tego, co właśnie zrobiłem: będzie to niepożądane, jeśli masz coś wymagającego skupienia się w wierszu, na przykład EditText. Teraz czuję się głupio.
Kitalda

W moim przypadku ta rada pomogła. Mam TextView i RadioButton, pierwszy nie kliknął, drugi nie wybrał TextView. Napisałem "android: clickable =" false "" w RadioButton.
CoolMind

To jest rozwiązanie, gdy przycisk to ImageButton
Mahonster,

Co ciekawe, to rozwiązanie zadziałało na mnie. OnItemClickListener działa bez wpływu na zdarzenia podrzędne. Nie wiem, dlaczego to zadziałało, więc ...
ONE

15

Jeśli masz aktywny widok / widok z możliwością zaznaczenia w widoku listy, to wyłączy się onItemClickListener... możesz spróbować uczynić go nieokreślonym, dodając: android:focusable="false"do dowolnego widoku, który zwykle jest aktywowany.


8

ListenerCałość dodajemy zarówno do całości jak rowViewi do Buttonśrodka Adapter. Coś takiego.

public class MyAdapter extends BaseAdapter implements View.OnClickListener{

    @Override
    public View getView(int position, View convertView, ViewGroup parent)
    {
            View rowView = convertView;
            if(rowView == null)
            {
                    //intialize rowView, and set onclick listener only once.
                    rowView = someIntilizationMehhodOrInflatorMethod();
                    //add listener to the button also and also to the row view
                    rowView.setOnClickListener(this);
            }
            //all your inflation and setting values goes here and at the end,
            //set position as tag to get the correct position, rather buggy one.
            rowView.setTag(String.valueOf(position));


            return rowView;
    }
    public void onClick(View v)
    {
            //now get the tag of View v and convert it to integer.
            int pos = Integer.parseInt(v.getTag().toString());
            Toast.makeText(context,"Item in position " + pos + " clicked",Toast.LENGTH_LONG).show();
    }
}

3
Jak to jest nawet odrobinę lepsze? Tworzysz onClickListener dla KAŻDEGO elementu danych w porównaniu z tylko jednym odbiornikiem. Twój przykład nawet nie przetwarza które widoki.
JRomero

@ J.Romero to był tylko przykład, chociaż jest miejsce na optymalizację kodu. Jednak właśnie zaktualizowałem kod.
Adil Soomro,

5

Spróbuj ustawić setClickable(false)dla każdego Button, ImageButtonitp i Viewtak:

view.setClickable(false);
button.setClickable(false);
imagebutton.setClickable(false);

Musisz też dodać

android:descendantFocusability="blocksDescendants"

do układu głównego (pierwszego poziomu)


2
Znalazłem ustawione blokiDescendants wystarczy, nie ma potrzeby anulowania ustawiania ostrości itp. Android 4.1
djdance

2

Zamiast dodawać przycisk dodaj ImageViewi podaj setOnItemClciklistener, który będzie działał dobrze.

public View getView(final int position, View convertView, ViewGroup parent) {
     MyViewHolder mViewHolder;
     if(convertView == null) {
         convertView = inflater.inflate(R.layout.youtubesearchrow, null);
         mViewHolder = new MyViewHolder();
         mViewHolder.alarm = (ImageView)convertView.findViewById(R.id.alarm);
         convertView.setTag(mViewHolder);
     } else {
         mViewHolder = (MyViewHolder) convertView.getTag();
     }

     mViewHolder.tvTitle = detail(convertView, R.id.tvTitle,    
     // mViewHolder.tvDesc  = detail(convertView, R.id.tvDesc,  
     mViewHolder.ivIcon  = detail_image(convertView, R.id.ivIcon,  

     mViewHolder.alarm.setOnClickListener(new OnClickListener() {

        @Override
        public void onClick(View v) {
        // TODO Auto-generated method stub
        }
    });

        return convertView;
}

1

Otrzymuję również ten sam błąd. Jeśli na liście układ masz dowolny przycisk, aby zastąpić go z TextViewIE ImageButtonz ImageView. Dla odniesienia ich jest mój kod:

private Activity activity;
private LayoutInflater inflater;

private List<ItemList> itemListsItems;

private Bookmark_SharedPref pref;

ImageLoader imageLoader = AppController.getInstance().getImageLoader();

public CustomListAdapter_Item(Activity activity,List<ItemList> itemListsItems){
    this.activity=activity;
    this.itemListsItems=itemListsItems;
    pref = new Bookmark_SharedPref(activity);
}


@Override
public int getCount() {
    return itemListsItems.size();
}

@Override
public Object getItem(int position) {
    return itemListsItems.get(position);
}

@Override
public long getItemId(int position) {
    return position;
}

@Override
public View getView(final int position, View convertView, ViewGroup parent) {

    View v = convertView;
    ViewHolder holder;

    if(inflater == null){
        inflater = (LayoutInflater)activity.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
    }
    if(convertView == null){
        convertView = inflater.inflate(R.layout.item_layout,null);

        holder = new ViewHolder();
        convertView.setTag(holder);
    }
    else{
        holder = (ViewHolder)convertView.getTag();
    }


    if(imageLoader == null){
        imageLoader = AppController.getInstance().getImageLoader();
    }




     holder.img = (NetworkImageView)convertView.findViewById(R.id.item_image);
    holder.Title = (TextView)convertView.findViewById(R.id.item_title);
    holder.Cat = (TextView)convertView.findViewById(R.id.item_category);

    holder.id = (TextView)convertView.findViewById(R.id.itemid);

    holder.Desc = (TextView)convertView.findViewById(R.id.item_description);

    holder.book = (ImageView)convertView.findViewById(R.id.bookmark_star);



    // getting blog data or the row
    ItemList il = itemListsItems.get(position);

    //setting image
    holder.img.setImageUrl(il.getImageUrl(), imageLoader);

    // setting title
    holder.Title.setText(il.getTitle());

    //setting category
    holder.Cat.setText(il.getCategory());


    //setting blog id
    holder.id.setText(il.getId());

    //setting description
    holder.Desc.setText(il.getDescription());

    //bookmarking the post
    holder.book.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {

            itemListsItems.get(position).isFav = !itemListsItems.get(position).isFav;
            ((ImageView)v.findViewById(R.id.bookmark_star)).setImageResource(itemListsItems.get(position).isFav ? R.mipmap.active_star : R.mipmap.inactive_star);

            if(itemListsItems.get(position).isFav){
                String data = Integer.toString(itemListsItems.get(position).id);
                Toast.makeText(v.getContext(),"fav hai "+data,Toast.LENGTH_SHORT).show();
                pref.addPref(data);
            } else {
                String data = Integer.toString(itemListsItems.get(position).id);
                Toast.makeText(v.getContext(),"not fav"+ data,Toast.LENGTH_SHORT).show();
                pref.removePref(data);
            }
            //notifyDataSetChanged();
        }
    });

    ((ImageView)convertView.findViewById(R.id.bookmark_star)).setImageResource(itemListsItems.get(position).isFav ? R.mipmap.active_star : R.mipmap.inactive_star);

    /*
    if(itemListsItems.get(position).isFav){
        String data = Integer.toString(itemListsItems.get(position).id);
        Toast.makeText(convertView.getContext(),"fav hai "+data,Toast.LENGTH_SHORT).show();
        pref.addPref(data);
    } else {
        String data = Integer.toString(itemListsItems.get(position).id);
        //Toast.makeText(convertView.getContext(),"not fav"+ data,Toast.LENGTH_SHORT).show();
        pref.removePref(data);
    }
    */
    return convertView;
}

private static class ViewHolder {
   public TextView Title, Desc,Cat,id;
    ImageView book;
    NetworkImageView img;
}

Pamiętaj również, aby układ listy był taki sam, jak android:focussable="true"i dla tego przycisku / parametru tekstowego android:focussable="false".


poprawiono , powinien być android: focusable = "false" nie można
focussable

1

Dodaj android:focusable="false"atrybut do przycisku, a do widoku nadrzędnego dodaj android:descendantFocusability="blocksDescendants"atrybut podobny do tego.

<RelativeLayout
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:descendantFocusability="blocksDescendants">

    <ImageButton
        android:id="@+id/sell_button"
        android:layout_width="wrap_content"
        android:focusable="false"
        android:layout_height="wrap_content"
        android:src="@drawable/ic_sell" />
</RelativeLayout>

0

To niesamowite, że zauważyłem, że zmiana XML powoduje, że setonitemclicked w java działa dla mnie ..

android:descendantFocusability="blocksDescendants"
android:focusable="false"
android:focusableInTouchMode="false"

0

Zmień wartość atrybutu nadrzędnego, który można kliknąć, na false w ten sposób android:clickable="false".

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:clickable="false"
    android:orientation="vertical">

Ponownie w obliczu problemu, do wszystkich widoków potomnych dodaj to klikalne fałsz.


0

Nie wiem dokładnie dlaczego, ale czasami kiedy ustawiasz adapter, słuchacze są czyszczeni. Następnie musisz zadzwonić setOnClickListenerpo setAdapterlub setListAdapter.


Korzystając z naszej strony potwierdzasz, że przeczytałeś(-aś) i rozumiesz nasze zasady używania plików cookie i zasady ochrony prywatności.
Licensed under cc by-sa 3.0 with attribution required.