Androidアプリのリストのフィルタリングについて

Question

リストビューにRSSから取得したデータを表示するアンドロイドアプリを作っています。
titleとdescriptionに検索文字を含むリストだけを1文字入力するたびに
フィルタリングしていきたいのですが、フィルタリングがうまくいきません。
(下記のコードはとりあえずタイトルだけ比較してます。)
CustomAdapter.javaのgetFilterメソッド内のコメントアウトされているclear()を実行すると
元のobjects(リストデータ)が全部消えてしまって、検索文字を削除したときや2文字目以降の
入力したときの動きがうまくないです。(リストが復帰しない)
どうすればよろしいでしょうか？

CustomAdapter.java

public class CustomAdapter extends ArrayAdapter<RssLine> {
    private LayoutInflater layoutInflater_;
    private List<RssData> objects;
    private List<RssData> mDisplayedValues;
    private Context context;
    private int layoutId;

    public CustomAdapter(Context context, int textViewResourceId, List<RssData> objects) {
        super(context, textViewResourceId, objects);
        layoutInflater_ = (LayoutInflater) context.getSystemService(Context.LAYOUT_INFLATER_SERVICE);

        this.objects = objects;
        this.context = context;
        layoutId = textViewResourceId;
    }
    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        //省略
    }

    @Override
    public Filter getFilter() {
        Filter filter = new Filter() {

            @SuppressWarnings("unchecked")
            @Override
            protected void publishResults(CharSequence constraint,FilterResults results) {
                if(results.count > 0) {
                    mDisplayedValues = (ArrayList<RssData>) results.values; // has the filtered values
//                  clear();
                    addAll(mDisplayedValues);
                    notifyDataSetChanged();  // notifies the data with new filtered values
                }
                else {
                    notifyDataSetInvalidated();
                 }
            }

            @Override
            protected FilterResults performFiltering(CharSequence constraint) {
                FilterResults results = new FilterResults();        // Holds the results of a filtering operation in values
                ArrayList<RssData> FilteredArrList = new ArrayList<RssData>();

                if (objects == null) {
                    objects = new ArrayList<RssLine>(mDisplayedValues); // saves the original data in mOriginalValues
                }

                /********
                 *
                 *  If constraint(CharSequence that is received) is null returns the mOriginalValues(Original) values
                 *  else does the Filtering and returns FilteredArrList(Filtered)
                 *
                 ********/
                if (constraint == null || constraint.length() == 0) {

                    // set the Original result to return
                    results.count = objects.size();
                    results.values = objects;
                }
                else {
                    constraint = constraint.toString().toLowerCase();
                    for (int i = 0; i < objects.size(); i++) {
                        RssData item = getItem(i);
                        String data = objects.get(i).getTitle();
                          if((data != null && data.contains(constraint))) {
                              FilteredArrList.add(item);
                        }
                    }
                    // set the Filtered result to return
                    results.count = FilteredArrList.size();
                    results.values = FilteredArrList;
                }
                return results;
            }
        };
        return filter;
    }
}

MainActivity.java

    public CustomAdapter adapter;
    @SuppressWarnings("unchecked")
    @Override
    public boolean onQueryTextChange(String newText) {
        // 文字が入力されるたびにフィルタを更新します
        adapter.getFilter().filter(newText);
        return true;
    }

以下はリストに持たせるクラスです。
RssData.java

public class RssData {
    private Bitmap imageData;
    private String title;
    private String description;

    public void setImagaData(Bitmap image) {
        imageData = image;
    }

    public Bitmap getImageData() {
        return imageData;
    }

    public void setTitle(String text) {
        title = text;
    }

    public String getTitle() {
        return title;
    }

    public void setDescription(String text) {
        description = text;
    }

    public String getDescription() {
        return description;
    }
}

Answer 1

Adapter#clearはAdapter内で保持している情報を全て削除するメソッドです。フィルターは内部に保持する情報を絞り込むための機能なのでclearをしてはいけません。
また、Adapter#addAllも同様に内部のリストを書き換えてしまうのでしてはいけません。

実装はAdptet#getViewのなかでgetItemをしているところをフィルター済みリストから取得するようになっているかを確認してください。

Comments

[luckyclock]

@Override
public DeadLine getItem(int position) {
return objects.get(position);
}

@Override
public View getView(int position, View convertView, ViewGroup parent) {
// 特定の行(position)のデータを得る
RssData item = (RssData)getItem(position);
　　　//以下省略
}

getItemしているところはフィルタ前のオリジナルのアレイから取得するようになっています。
ただgetItemの戻り値をフィルタされたリストのmDisplayedValuesとした場合、positionはオリジナルのアレイの位置が入ってくるため、mDisplayedValuesのとんでもない位置を参照してしまいアプリが落ちてしまいます。
おしいところまで来ている気がするのですが。後一歩うまくいきません。

[yona]

とりあえず、filterをフィールドにしたほうがいいでしょう。
最初のコメントは誤っていました、申し訳ない。clear()する必要がありますね。

条件が変わる前と後でobjectsとmDisplayedValuesの内容を確認してみてください。

[luckyclock]

やはりclear()を実行するとリスト復帰用に保存していたはずのobjectsがmDisplayedValuesと同じ値になってリストが復帰しません。

試したコードです。

protected void publishResults(CharSequence constraint, FilterResults results) {
// Adapterのメソッドでデータの内容を更新する
if(results.count > 0) {
List<RssData> items = (List<RssData>) results.values;
clear();
addAll(items);
notifyDataSetChanged();
} else {
notifyDataSetInvalidated();
}
}
　@Override
public RssData getItem(int position) {
//フィルタしたアイテムを返す
return mDisplayedValues.get(position);
}

[yona]

具体的にどのタイミングでリストの内容が同じになるか特定できますか？
下記のコードが怪しいです
if (objects == null) {
objects = new ArrayList<RssLine>(mDisplayedValues); // saves the original data in mOriginalValues
}

[luckyclock]

今日いろいろやってみてなんとか動きました。
修正したところは
protected void publishResults(CharSequence constraint,FilterResults results) {
mDisplayedValues = (ArrayList<DeadLine>) results.values; // has the filtered values
notifyDataSetChanged(); // notifies the data with new filtered values
}
clear()とかはいらなかったみたいです。
さらにCustomAdapterクラスのgetCountがオリジナルのリストのサイズを返すように
なっていたため、フィルタしたリストのサイズを返すように変更しました。
@Override
public int getCount() {
// return objects.size();　←よくなかった。
return mDisplayedValues.size();
}

さらに一番最初に指摘を受けたようにgetItemはフィルタしたリストを返すようにしました。最初に試したときgetCountも修正すべきでしたが気づきませんでした。
public RssData getItem(int position) {
return mDisplayedValues.get(position);
}

いろいろ親身になっていただきありがとうございました。

[yona]

力になれず申し訳ないです。