質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.98%

ListFragmentのアイテムを削除すると、一つ下のアイテムが重なって表示される

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,640

tarofess

score 125

ListFragmentの一番上のアイテムを削除すると、一つ下のアイテムが一番上のアイテムに重なって表示されてしまいます。現在リストのアイテム数は2つあり、2つ目のアイテムのみを削除するとアイテム同士が重なったりということはないのですが、2つ目のアイテムの表示が少し薄くなります。
動作を見る限り実際アイテムは消されているが、描画が残ったままのように思います。
ListView#invalidateViews()や、Adapter#clear、Adapter#addAllなどを試したのですがどれもうまくいきません。どうすれば指定したListFragmentのアイテムを削除することができるでしょうか?下記にソースコードを記します。
public class IPCartProductData {
    private Bitmap mProductImage;
    private String mProductName;
    private int mProductPrice;

    public Bitmap getProductImage() {
        return mProductImage;
    }

    public String getProductName() {
        return mProductName;
    }

    public int getProductPrice() {
        return mProductPrice;
    }

    public void setProductImage(Bitmap mProductImage) {
        this.mProductImage = mProductImage;
    }

    public void setProductName(String mProductName) {
        this.mProductName = mProductName;
    }

    public void setProductPrice(int mProductPrice) {
        this.mProductPrice = mProductPrice;
    }
}


public class IPCartFragment extends ListFragment {
    public static IPCartProductAdapter adapter;

    @Override
    public void onViewCreated(View view, Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);

        ListView cartListView = (ListView)view.findViewById(android.R.id.list); {
            List<IPCartProductData> itemList = new ArrayList<>();

            IPCartProductData customCartData1 = new IPCartProductData();
            customCartData1.setProductImage(BitmapFactory.decodeResource(getResources(), R.drawable.ic_launcher));
            customCartData1.setProductName("ハンバーガー");
            customCartData1.setProductPrice(120);

            IPCartProductData customCartData2 = new IPCartProductData();
            customCartData2.setProductImage(BitmapFactory.decodeResource(getResources(), R.drawable.ic_launcher));
            customCartData2.setProductName("コーラ");
            customCartData2.setProductPrice(80);

            itemList.add(customCartData1);
            itemList.add(customCartData2);

            adapter = new IPCartProductAdapter(getActivity(), android.R.layout.simple_expandable_list_item_1, itemList);

            cartListView.setAdapter(adapter);
        }
    }
}


public class IPCartProductAdapter extends ArrayAdapter <IPCartProductData> {
    private static List<IPCartProductData> productData;
    private LayoutInflater mLayoutInflater;

    public IPCartProductAdapter(Activity activity, int textViewResourceId, List<IPCartProductData> objects) {
        super(activity, textViewResourceId, objects);

        productData = objects;
        mLayoutInflater = (LayoutInflater)activity.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
    }

    @Override
    public View getView(final int position, View convertView, final ViewGroup parent) {
        IPCartProductData item = getItem(position);

        if (null == convertView) {
            convertView = mLayoutInflater.inflate(R.layout.list_item_cart, null);
        }

        ImageView productImageView  = (ImageView)convertView.findViewById(R.id.imageViewProductImageInCart); {
            productImageView.setImageBitmap(item.getProductImage());
        }

        TextView productNameTextView = (TextView)convertView.findViewById(R.id.textViewProductNameInCart); {
            productNameTextView.setText(item.getProductName());
        }

        TextView productAmountTextView = (TextView)convertView.findViewById(R.id.textViewProductAmount); {
            productAmountTextView.setText(String.valueOf(item.getAmountOfProduct()));
        }

        TextView productPriceTextView = (TextView)convertView.findViewById(R.id.textViewProductPriceInCart); {
            productPriceTextView.setText(String.valueOf(item.getProductPrice()));
        }

        Button productDeleteButton = (Button)convertView.findViewById(R.id.buttonDeleteProductInCart); {
            productDeleteButton.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View v) {
                    productData.remove(position);
                    IPCartFragment.adapter.notifyDataSetChanged();

                }
            });
        }

        return convertView;
    }
}

ちなみに上記のコードのようにListFragmentを継承したIPCartFragmentのフィールドにstaticなカスタムアダプターの変数を置き、他のクラスでListViewのデータを変更した時にIPCartFragmentのstaticなカスタムアダプターの変数を参照して更新をかけるというやり方は正しいやり方でしょうか?(IPCartProductAdapterクラスのproductDeleteButtonのOnClickListener内で行っています)
色々と質問が多くて申し訳ないのですが、どなたか分かる方がいれば教えていただきたいです。
すみませんが、宜しくお願いします。

イメージ説明
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+1

リストをあえてフィールドで保持する必要ないと思います。
ArrayAdapter#removeを使うことで対象データは削除はされます。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/10/14 13:31

    カートだけにすると正常に削除することができました!
    やはりTabHostが問題だったのですね。
    しかしタブは使いたいので、自分でも少し調べてみます。

    キャンセル

  • 2015/10/14 13:48

    最初に考えるべきでした。申し訳ない

    フラグメントをセットしているところが怪しいです。二重になっていないかを確認するといいでしょう。

    キャンセル

  • 2015/10/14 13:57

    いえいえ、お力添え頂き感謝しています。
    ご指摘の通りFragmentをセットしているところが原因でした。
    FragmentTabHostをセットする前にFragmentTransaction#addでタブに表示される最初の画面を追加していました。addしなくても最初にセットしたタブをFragmentTabHostでは表示してくれるのですね。
    無事解決することができて大変うれしいです。誠にありがとうございました。

    キャンセル

+1

どうすれば指定したListFragmentのアイテムを削除することができるでしょうか?
ソースコードを拝見しましたが,superクラス(ArrayAdapter)のメソッド,自分のクラス(IPCartProductAdapter)のメソッドを使い分けられていないようです.
superクラスの要素を参照するのか,IPCartProductAdapterの要素を参照するのか明確にしましょう.

ちなみに上記のコードのようにListFragmentを継承したIPCartFragmentのフィールドにstaticなカスタムアダプターの変数を置き、他のクラスでListViewのデータを変更した時にIPCartFragmentのstaticなカスタムアダプターの変数を参照して更新をかけるというやり方は正しいやり方でしょうか?
正しくありません.今回のように Adapter 内のクラス View.OnClickListener などから,notifyDataSetChanged() を呼ぶときには,親を参照するために IPCartProductAdapter.this を使って参照します.

superクラスの要素を参照する形で書き直したものを載せます.
public class IPCartFragment extends ListFragment {
    // static にすべきではありません
    // 参照されないなら不要です
    // public IPCartProductAdapter adapter;

    // 明確な理由がなければ onCreate() で setAdapter() してよいでしょう
/*
    @Override
    public void onViewCreated(View view, Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);

        ListView cartListView = (ListView)view.findViewById(android.R.id.list); {
            List<IPCartProductData> itemList = new ArrayList<>();

            IPCartProductData customCartData1 = new IPCartProductData();
            customCartData1.setProductImage(BitmapFactory.decodeResource(getResources(), R.drawable.ic_launcher));
            customCartData1.setProductName("ハンバーガー");
            customCartData1.setProductPrice(120);

            IPCartProductData customCartData2 = new IPCartProductData();
            customCartData2.setProductImage(BitmapFactory.decodeResource(getResources(), R.drawable.ic_launcher));
            customCartData2.setProductName("コーラ");
            customCartData2.setProductPrice(80);

            itemList.add(customCartData1);
            itemList.add(customCartData2);

            adapter = new IPCartProductAdapter(getActivity(), android.R.layout.simple_expandable_list_item_1, itemList);

            cartListView.setAdapter(adapter);
        }
*/

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        List<IPCartProductData> itemList = new ArrayList<>();

        IPCartProductData customCartData1 = new IPCartProductData();
        customCartData1.setProductImage(BitmapFactory.decodeResource(getResources(), R.drawable.ic_launcher));

        customCartData1.setProductName("ハンバーガー");
        customCartData1.setProductPrice(120);

        IPCartProductData customCartData2 = new IPCartProductData();
        customCartData2.setProductImage(BitmapFactory.decodeResource(getResources(), R.drawable.ic_launcher));
        customCartData2.setProductName("コーラ");
        customCartData2.setProductPrice(80);

        itemList.add(customCartData1);
        itemList.add(customCartData2);

        // adapter = new IPCartProductAdapter(getActivity(), android.R.layout.simple_expandable_list_item_1, itemList);

        // cartListView.setAdapter(adapter);

        // ListFragment を使うのであれば 便利なメソッドがあります
        // 外部から参照しないのであればこれで十分でしょう
        setListAdapter(new IPCartProductAdapter(getContext(), R.layout.list_item_cart, itemList));
    }
}


class IPCartProductAdapter extends ArrayAdapter<IPCartProductData> {
    // static にすべきではありません
    // private static List<IPCartProductData> productData;

    // -> 書くならこう書くべきです
    // private List<IPCartProductData> productData;

    // 今回は ArrayAdapter を継承しているため スーパークラスのメソッドを上手く使ってあげると
    // メンバ変数を用意せずとも きれいに書くことができます

    private LayoutInflater mLayoutInflater;

    // ArrayAdapter クラスは Context を 要求しているので Activity を渡す必要もないです
/*
    public IPCartProductAdapter(Activity activity, int textViewResourceId, List<IPCartProductData> objects) {
        super(activity, textViewResourceId, objects);

        // productData = objects;
        mLayoutInflater = LayoutInflater.from(context);
    }
*/
    // こちらが適切でしょう
    public IPCartProductAdapter(Context context, int textViewResourceId, List<IPCartProductData> objects) {
        super(context, textViewResourceId, objects);

        // productData = objects;
        mLayoutInflater = (LayoutInflater) context.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
    }

    @Override
    public View getView(final int position, View convertView, final ViewGroup parent) {
        // このメソッドは ArrayAdapter が持っているメソッドです
        // あなたの productData から消しても ArrayAdapter のリストからは上手く消えてくれません
        // もちろん ArrayAdapter の中のリストを適切に管理すればこれで十分です
        IPCartProductData item = getItem(position);

        // もしメンバ変数として productData を利用するなら 上のgetItem()の代わりに こう書きます
        // IPCartProductData item =productData.get(position);



        if (null == convertView) {
            convertView = mLayoutInflater.inflate(R.layout.list_item_cart, null);
        }

        ImageView productImageView = (ImageView) convertView.findViewById(R.id.imageViewProductImageInCart);
        //{
        productImageView.setImageBitmap(item.getProductImage());
        // }


        // わざわざ {} でブロック分けする必要もなさそうです

        TextView productNameTextView = (TextView) convertView.findViewById(R.id.textViewProductNameInCart);
        //{
        productNameTextView.setText(item.getProductName());
        //}

        TextView productAmountTextView = (TextView) convertView.findViewById(R.id.textViewProductAmount);
        //{
        // 今回のソースコードではこのメソッドが呼べないので コメントアウト
        //    productAmountTextView.setText(String.valueOf(item.getAmountOfProduct()));
        //}

        TextView productPriceTextView = (TextView) convertView.findViewById(R.id.textViewProductPriceInCart);
        //{
        productPriceTextView.setText(String.valueOf(item.getProductPrice()));
        //}

        Button productDeleteButton = (Button) convertView.findViewById(R.id.buttonDeleteProductInCart);
        //{
        productDeleteButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                // これでは IPCartProductAdapter の productData からは消去されますが
                // ArrayAdapter に渡してあるリストからは消えません
                // productData.remove(position);

                // 素直に親のメソッドを呼んであげましょう
                IPCartProductAdapter.this.remove(IPCartProductAdapter.this.getItem(position));

                // IPCartFragment.adapter.notifyDataSetChanged();
                // notifyDataSetChanged() を呼ぶのは正しいです
                IPCartProductAdapter.this.notifyDataSetChanged();
            }
        });
        //}

        return convertView;
    }
}

class IPCartProductData {
    private Bitmap mProductImage;
    private String mProductName;
    private int mProductPrice;

    public Bitmap getProductImage() {
        return mProductImage;
    }

    public String getProductName() {
        return mProductName;
    }

    public int getProductPrice() {
        return mProductPrice;
    }

    public void setProductImage(Bitmap mProductImage) {
        this.mProductImage = mProductImage;
    }

    public void setProductName(String mProductName) {
        this.mProductName = mProductName;
    }

    public void setProductPrice(int mProductPrice) {
        this.mProductPrice = mProductPrice;
    }
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/10/14 15:17

    はい、無事FragmentTabHost周りを修正して問題が解決できました。
    IPCartProductAdapter.thisは外部クラスを参照するという意味合いだったのですね。理解することができました。
    自分のコードではstaticを多用していたり無駄な書き方をしているので、設計についても見直したいと思います。この度は誠にありがとうございました。

    キャンセル

  • 2015/10/14 16:12

    解決してよかったです.ViewHolderについて調べるともっと考慮のよいAdapterが作れます.ぜひ改良を加えていってください.

    キャンセル

  • 2015/10/14 17:43

    ViewHolder、調べてみます。色々と教えてくださりありがとうございます。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.98%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる