ListViewからアイテムを削除しようとすると、最初のアイテムが常に削除されます

Question

私はカスタムアダプターとボタンクリック時にアイテムを動的に追加するリストを持っています。各リスト行には削除ボタン（または画像）があります。それは私のカスタムアダプタクラスです：

package com.ameer.easycooking; 

import android.content.Context; 
import android.media.Image; 
import android.view.LayoutInflater; 
import android.view.View; 
import android.view.ViewGroup; 
import android.widget.ArrayAdapter; 
import android.widget.ImageView; 
import android.widget.TextView; 
import android.widget.Toast; 

import java.util.List; 

/** 
* Created by ameer on 12/10/2016. 
*/ 
public class IngredientAdapter extends ArrayAdapter<String> { 
    private List<String> ingredientList; 
    private Context context; 

    public IngredientAdapter(List<String> ingredientList, Context context) { 
     super(context, R.layout.ingredient_list_item, ingredientList); 
     this.ingredientList = ingredientList; 
     this.context = context; 
    } 

    public static class IngredientHolder { 
     private TextView name; 
     private ImageView remove; 

     public IngredientHolder(TextView name, ImageView remove) { 
      this.name = name; 
      this.remove = remove; 
     } 

     public TextView getName() { 
      return name; 
     } 

     public ImageView getRemove() { 
      return remove; 
     } 
    } 

    @Override 
    public View getView(int position, View convertView, ViewGroup parent) { 
     View v = convertView; 

     IngredientHolder holder; 

     if (convertView == null) { 
      LayoutInflater inflater = (LayoutInflater) context.getSystemService(Context.LAYOUT_INFLATER_SERVICE); 
      v = inflater.inflate(R.layout.ingredient_list_item, null); 

      holder = new IngredientHolder((TextView) v.findViewById(R.id.ingredientName), (ImageView) v.findViewById(R.id.remove)); 

      holder.getRemove().setTag(position); 

      holder.getRemove().setOnClickListener(new View.OnClickListener() { 
       @Override 
       public void onClick(View v) { 
        int index = (int) v.getTag(); 
        Toast.makeText(context, ingredientList.get(index) + " removed", Toast.LENGTH_SHORT).show(); 
        ingredientList.remove(index); 
        notifyDataSetChanged(); 
       } 
      }); 

      v.setTag(holder); 
     } else { 
      holder = (IngredientHolder) v.getTag(); 
     } 

     String ingredient = ingredientList.get(position); 
     holder.name.setText(ingredient); 

     return v; 
    } 
} 

私の問題は、私が追加または1つ、私はそれは常にリストビュー内の最初の行ではなく、クリック1つを取り外しています、削除を選択するどのように多くのアイテムという問題です。

Answer 1

convertView == nullに一度だけヒットするため、この問題が発生します。 if/elseステートメントの後にholder.getRemove().setTag(position);と設定する必要があります。

Answer 2

変換されたビューをいつ使用するのか分からず、クリックされた位置が間違っている可能性があるため、クリックリスナーをifステートメント内に設定しないでください。だから、ちょうどif文のうち、クリックリスナーを移動し、holder.getRemove().setTag(position)文を削除します。

@Override 
public View getView(int position, View convertView, ViewGroup parent) { 
    View v = convertView; 

    IngredientHolder holder; 

    if (convertView == null) { 
     LayoutInflater inflater = (LayoutInflater) context.getSystemService(Context.LAYOUT_INFLATER_SERVICE); 
     v = inflater.inflate(R.layout.ingredient_list_item, null); 
     holder = new IngredientHolder((TextView) v.findViewById(R.id.ingredientName), (ImageView) v.findViewById(R.id.remove)); 
     v.setTag(holder); 
    } else { 
     holder = (IngredientHolder) v.getTag(); 
    } 

    String ingredient = ingredientList.get(position); 
    holder.name.setText(ingredient); 

    holder.getRemove().setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      int index = (int) v.getTag(); 
      Toast.makeText(context, ingredientList.get(index) + " removed", Toast.LENGTH_SHORT).show(); 
      ingredientList.remove(index); 
      notifyDataSetChanged(); 
     } 
    }); 

    return v; 
} 

EDIT：するか、黒帯が言うように、あなたは単にあればブロックのうち、holder.getRemove().setTag(position);文を移動することができます。

Answer 3

良い方法あなたがgetViewメソッドに渡された位置を（）することができ、最終的な パブリックビューgetViewメソッド（最終int型の位置、表示convertView、のViewGroupの親） とクリックリスナー内のその位置を使用します。