Reputation: 4101
I want to change the background of a selected list item in my adapter.
@Override
public View getView(int position, View convertView, ViewGroup parent) {
final Card card = getItem(position);
final ViewHolder viewHolder;
if (convertView != null) {
viewHolder = (ViewHolder) convertView.getTag();
} else {
convertView = from(context).inflate(R.layout.card_item, parent, false);
viewHolder = new ViewHolder(convertView);
convertView.setTag(viewHolder);
}
viewHolder.select.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View v) {
viewHolder.cardLayout.setBackgroundColor(ContextCompat.getColor(context, R.color.md_grey_600));
}
});
return convertView;
}
private class ViewHolder {
Button select;
RelativeLayout cardLayout;
public ViewHolder(View view) {
select = (Button) view.findViewById(R.id.carditem_btn_upvote);
cardLayout = (RelativeLayout) view.findViewById(R.id.cardlist_item);
}
}
The problem is that when I select for ex. the first item of the list, the 7th item is also selected. I have noticed that there is a kind of a patron. When I select the second item, the 8th item is also selected and so on.
Upvotes: 2
Views: 2253
Reputation: 7087
Let' say you have 10 item in list and initially you will store all your item color as white .
HashMap<Integer,Integer> mhashColorselected=new HashMap();
HashMap<Integer,Integer> mHashBtnVisibility=new HashMap();
for(i=0;i<10;i++){
//Put Default Color of your All list item will be here
mhashColorselected.put(i,R.color.white)
mHashBtnVisibility.put(i,View.Invisible);
}
Now in your adapter getView();
@Override
public View getView(final int position, View convertView, ViewGroup parent) {
final Card card = getItem(position);
final ViewHolder viewHolder;
if (convertView != null) {
viewHolder = (ViewHolder) convertView.getTag();
} else {
convertView = from(context).inflate(R.layout.card_item, parent, false);
viewHolder = new ViewHolder(convertView);
convertView.setTag(viewHolder);
}
viewHolder.select.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View v) {
viewHolder.cardLayout.setBackgroundColor(ContextCompat.getColor(context, R.color.md_grey_600));
//Here we are storing Selected item By User
mhashColorselected.put(position,R.color.md_grey_600);
mHashBtnVisibility.put(i,View.Visible);
notifitydataSetChange();
}
});
//Here it's will take fault value or else it will set color as selected item
viewHolder.cardLayout.setBackgroundColor(ContextCompat.getColor(context, mhashColorselected.getValue(position)));
if(View.VISIBLE==mHashBtnVisibility.get(position)){
viewHolder.select.setVisibility(View.VISIBLE)
}else{
viewHolder.select.setVisibility(View.INVISIBLE)
}
return convertView;
}
Upvotes: 3