RecyclerView OnClick funktioniert nicht

Ich habe eine horizontale recyclerview innen ein fragment. Wenn ich nun auf für eine Sache, die ich nicht sehen, die auf Klick-listener arbeiten. Hier ist mein code für die Adapter-Klasse:

public class FeaturedProductsAdapter  extends RecyclerView.Adapter<FeaturedProductsAdapter.CustomViewHolder> {
private List<FeaturedProductInfo> feedItemList;
private Context mContext;

public FeaturedProductsAdapter(Context context, List<FeaturedProductInfo> feedItemList) {
    this.feedItemList = feedItemList;
    this.mContext = context;
}
public class CustomViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener {
    protected ImageView imageView;
    protected TextView textView,priceView;
    private Context context;


    public CustomViewHolder(View view,Context context) {

        super(view);
        this.context=context;
        this.imageView = (ImageView) view.findViewById(R.id.thumbnail);
        this.textView = (TextView) view.findViewById(R.id.prodTitle);
        this.priceView = (TextView) view.findViewById(R.id.prodPrice);
        view.setOnClickListener(this);

    }

    @Override
    public void onClick(View view) {

        int position = getLayoutPosition(); //gets item position
        Log.e("Check", position + "");
        FeaturedProductInfo user = feedItemList.get(position);//[position];
        //We can access the data within the views
        Intent intent = new Intent(context, ProductDescription.class);
        intent.putExtra("id", user.getId());
        mContext.startActivity(intent);


    }

}

@Override
public CustomViewHolder onCreateViewHolder(ViewGroup viewGroup, int i) {
    View view = LayoutInflater.from(mContext).inflate(R.layout.featured_product_list_item_card, null);
    Context context = viewGroup.getContext();

    CustomViewHolder viewHolder = new CustomViewHolder(view,context);

    return viewHolder;
}

@Override
public void onBindViewHolder(CustomViewHolder customViewHolder, int i) {
    FeaturedProductInfo feedItem = feedItemList.get(i);

    //Download image using picasso library
    if(!feedItem.getUrl().contains("."))
    {
        feedItem.setUrl("nothing");
    }
    Picasso.with(mContext).load(feedItem.getUrl())
            .error(R.drawable.unavailable)
            .placeholder(R.drawable.unavailable)
            .resize(110,110)
            .into(customViewHolder.imageView);

    //Setting text view title
    customViewHolder.textView.setText(feedItem.getTitle());
    customViewHolder.priceView.setText(feedItem.getPrice());
    //Log.e("Featured: ","SET");
}

@Override
public int getItemCount() {
    return (null != feedItemList ? feedItemList.size() : 0);
}

}

Ich denke, ich bin nicht immer, wie der view-Halter richtig. Ich habe zwar verwendet den gleichen code für recyclerView in anderen Tätigkeiten, und es funktioniert wie Charme.

Ist es nicht auslösen überhaupt oder ist es nur brennen falsch. Sie sollten die Einstellung der onClick während der bindViewHolder (), da Sie nur die Sicht Halter zunächst einmal und dann "Papierkorb", wenn die Liste gescrollt wird.
seine gar nicht feuern.
Ist der hintergrund der anzeigen unset? fügen Sie auch einen toast für debug.
finden Sie diese stackoverflow.com/a/24933117/4146722

InformationsquelleAutor Akshay Bhasin | 2016-02-23

1.Einfache Click-Handler innerhalb ViewHolder

RecyclerView keine besonderen Bestimmungen für die Befestigung click-Handler, um Elemente, die im Gegensatz zu ListView hat die Methode setOnItemClickListener(). Einen ähnlichen Effekt zu erzielen, können wir anfügen, klicken Sie auf Ereignisse innerhalb der ViewHolder innerhalb unserer adapter:

public class ContactsAdapter extends RecyclerView.Adapter<ContactsAdapter.ViewHolder> {
    //...

    //Used to cache the views within the item layout for fast access
    public class ViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener {
        public TextView tvName;
        public TextView tvHometown;
        private Context context;

        public ViewHolder(Context context, View itemView) {
            super(itemView);
            this.tvName = (TextView) itemView.findViewById(R.id.tvName);
            this.tvHometown = (TextView) itemView.findViewById(R.id.tvHometown);
            //Store the context
            this.context = context;
            //Attach a click listener to the entire row view
            itemView.setOnClickListener(this);
        }

        //Handles the row being being clicked
        @Override
        public void onClick(View view) {
            int position = getLayoutPosition(); //gets item position
            User user = users.get(position);
            //We can access the data within the views
            Toast.makeText(context, tvName.getText(), Toast.LENGTH_SHORT).show();
        }
    }

    //...
}

Anderen Weg ist mein bevorzugter Weg ist.. aber das ist auch eine feine Weise, darüber zu gehen.

Meine onBindViewHolder

@Override
    public void onBindViewHolder(CategoryViewHolder holder, int position) {
        Category category = mCategories.get(position);

        holder.tvTitle.setText(category.getTitle());
        holder.tvDescription.setText(category.getDescription());

        holder.rlContainer.setOnClickListener(mClickListener);
        holder.rlContainer.setTag(holder);
    }

Meiner Klasse (Adapter-Objekt Anzeigen.OnClickListner)

View.OnClickListener mClickListener = new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            CategoryViewHolder holder = (CategoryViewHolder) view.getTag();
            int position = holder.getAdapterPosition();

            startAppointmentBookingFor(mCategories.get(position));
        }
    };

also im Grunde befestigen Sie die Zuhörer zu einer beliebigen Ansicht in Ihrem Halter (ich versuche es auf den container), dann entpacken Sie es heraus auf der onclick - und Griffpositionen etc.

hey ich habe nur verwendet die gleiche Methode, die da oben, und seine nicht funktioniert.
das ist, warum ich war die Eingabe, die andere Weise, meine bevorzugte Weise :), das ausprobieren @AkshayBhasin
was ist das rlContainer?
es ist im Grunde ein container, der viewHolder... ie:<RelativeLayout><TextView><Bildansicht></RelativeLayout>.....also habe ich nur angebracht, eine id und voila der Eltern wird mein container für die Ansicht ein, die verwendet wird, von meinem adapter, kann für jetzt nur noch Anhängen, dass zu Ihrer priceView oder TextView.
Ich bin ziemlich sicher, Sie müssen vergessen haben, fügen Sie das Klick-listener. Dies ist die Zeile, die Sie müssen haben verpasst diese Zeile: itemView.setOnClickListener(this);

InformationsquelleAutor yUdoDis

Machen Sie die folgenden änderungen an Ihrem Adapter:

public class FeaturedProductsAdapter  extends RecyclerView.Adapter<FeaturedProductsAdapter.CustomViewHolder> {
private List<FeaturedProductInfo> feedItemList;
private Context mContext;
private OnItemClickListener onItemClickListener;

public FeaturedProductsAdapter(Context context, List<FeaturedProductInfo,OnItemClickListener onItemClickListener> feedItemList) {
    this.feedItemList = feedItemList;
    this.mContext = context;
    this.onItemClickListener = onItemClickListener;

}
public class CustomViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener {
    protected ImageView imageView;
    protected TextView textView,priceView;
    private Context context;


    public CustomViewHolder(View view,Context context) {

        super(view);
        this.context=context;
        this.imageView = (ImageView) view.findViewById(R.id.thumbnail);
        this.textView = (TextView) view.findViewById(R.id.prodTitle);
        this.priceView = (TextView) view.findViewById(R.id.prodPrice);
        view.setOnClickListener(this);

    }

    @Override
    public void onClick(View view) {

        onItemClickListener.onItemClick(getLayoutPosition());
        Log.e("Check", position + "");
        FeaturedProductInfo user = feedItemList.get(position);//[position];
        //We can access the data within the views
        Intent intent = new Intent(context, ProductDescription.class);
        intent.putExtra("id", user.getId());
        mContext.startActivity(intent);


    }

}

public interface OnItemClickListener{
    void onItemClick(int position);
}

@Override
public CustomViewHolder onCreateViewHolder(ViewGroup viewGroup, int i) {
    View view = LayoutInflater.from(mContext).inflate(R.layout.featured_product_list_item_card, null);
    Context context = viewGroup.getContext();

    CustomViewHolder viewHolder = new CustomViewHolder(view,context);

    return viewHolder;
}

@Override
public void onBindViewHolder(CustomViewHolder customViewHolder, int i) {
    FeaturedProductInfo feedItem = feedItemList.get(i);

    //Download image using picasso library
    if(!feedItem.getUrl().contains("."))
    {
        feedItem.setUrl("nothing");
    }
    Picasso.with(mContext).load(feedItem.getUrl())
            .error(R.drawable.unavailable)
            .placeholder(R.drawable.unavailable)
            .resize(110,110)
            .into(customViewHolder.imageView);

    //Setting text view title
    customViewHolder.textView.setText(feedItem.getTitle());
    customViewHolder.priceView.setText(feedItem.getPrice());
    //Log.e("Featured: ","SET");
}

@Override
public int getItemCount() {
    return (null != feedItemList ? feedItemList.size() : 0);
}

Funktioniert nicht. Jetzt kann ich auch nicht sehen, das onClick-Effekt auf die Elemente.
Zeige code der MainActivity wo Sie Ihre Adapter
Überprüfen Sie die aktualisierte Antwort
hey, warum hast du pass die itemClickListener an den adapter? Seine Angabe ist ein Fehler.

InformationsquelleAutor Veeresh Charantimath

0

In 'CustomViewHolder' unten 'super(view)" hinzufügen
Blick.setOnClickListener(this)

Du fertig bist
Sollte funktionieren.

InformationsquelleAutor

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.