検索ボックスでフィルターの効果を有効にした場合のコンテンツ内容取得方方法 listview/searchview

Question

実現したいこと

検索ボックスに文字を入力するとフィルター機能によって、その下にあるリストボックスに表示されるコンテンツが変化します。いずれかのリストをクリックするとそのリスト特有の値をもって画面を遷移させます。　onItemClick()を使用しています
よろしくお願いします

あと検索ボックスに入力したときに入力した文字がトーストみたいに下に出てくるのもなくしたいです。できればこちらもお助けいただければ
質問→検索ボックスに文字を入力するとトーストで文字表示される　android studio / SearchView

よろしくお願いします

発生している問題・エラーメッセージ

フィルター機能によって表示される順番が変化するので、実際のインデックスとは異なってしまいます。

入力アリ	入力ナシ

該当のソースコード

public class SearchFragment extends Fragment implements AdapterView.OnItemClickListener {

    private SearchViewModel homeViewModel;
    private FragmentSearchBinding binding;
    private DatabaseHelper helper=null;
    private Cursor cursor=null;
    private GlobalVariables gl;

    public View onCreateView(@NonNull LayoutInflater inflater,
                             ViewGroup container, Bundle savedInstanceState) {
        homeViewModel =new ViewModelProvider(this).get(SearchViewModel.class);
        binding = FragmentSearchBinding.inflate(inflater, container, false);
        View root = binding.getRoot();

        searchfunc(helper);

        return root;
    }


    public void searchfunc(DatabaseHelper helper){
        //配列アダプターを制作、ListViewに登録
        final ListView list = binding.list;
        list.setAdapter(get_wordlist_fromDB(helper));
        //フィルター機能有効か
        list.setTextFilterEnabled(true);

        SearchView sv=binding.searchingbox ;
        //検索ボックスdefoult入力状態にする
        sv.requestFocus();
        //検索ボックス入力時の動作
        sv.setOnQueryTextListener(new SearchView.OnQueryTextListener(){
            public boolean onQueryTextChange(String text){
                if(text==null || text.equals("")){
                    list.clearTextFilter();
                }else{
                    list.setFilterText(text);
                }
                return false;
            }

            @Override
            public boolean onQueryTextSubmit(String s) {
                return false;
            }
        });

        list.setOnItemClickListener(this);
    }



    public void onItemClick(AdapterView<?> parent, View v, int position, long id) {
        Intent intent = new Intent(this.getContext(), ResultsActivity.class);
//positionがフィルターによって変わるので本来のインデックスと異なる
        this.cursor.moveToPosition(position);
        //golobalvariablesにwordをセット
        gl=(GlobalVariables) getContext().getApplicationContext();
        gl.setWord(cursor.getString(0));
        // Activity をスイッチする
        startActivity(intent);
        Log.e("debug","word: "+gl.getWord());
    }



    //データをリストに入れるための準備
    //cursorに全部のデータを入れてます
    public ArrayAdapter<String> get_wordlist_fromDB(DatabaseHelper helper){
        //helperを準備
        helper=new DatabaseHelper(getContext());//元々this
        try {
            helper.createDatabase();
        }
        catch (IOException e) {
            throw new Error("Unable to create database");
        }
        //データベース取得
        SQLiteDatabase db=helper.getWritableDatabase();
        try{
            int i =0;
            String sql="QUERY"; 
            this.cursor=db.rawQuery(sql , null);
            cursor.moveToFirst();
            ArrayList<String> data =new ArrayList<>();
            data.add(cursor.getString(i));
            while (cursor.moveToNext()) {
                data.add(cursor.getString(i));
            }
            ArrayAdapter<String> adapter=new ArrayAdapter<>(getContext(), android.R.layout.simple_list_item_1, data);
            return adapter;
        }finally{
            db.close();
        }
    }


    @Override
    public void onDestroyView() {
        super.onDestroyView();
        binding = null;
    }


}

Comments

[jimbe]

かなり切り貼り感があるコードですが、 DB あり ViewModel ありの上でさらに GlobalVariables とか必要なのでしょうか。

>//cursorに全部のデータを入れてます

cursor にはデータは入りません。データが入っているレコードを指しているのが cursor です。

>db.close();

通常使用では SQLiteDatabase は close する必要はありません。

[dedede914]

初めてアプリを作っていて、何度か挫折して、今また取り組み始めたところです
とりあえず自分の欲しい機能が備わったしっかり動くアプリが早くほしくて、意味をしっかり理解せずコピペ改変をしまくっています。ので、ぐちゃぐちゃだとは思います！とりあえず完成させてから体系的に学んでいこうと思っています。
データベースはよく使いそうなのでcursorについてはしっかり勉強してみます
globalvariablesを使っている理由としては、
インテントした先がtab表示のあるページなのでpageradapterを一回かましてから、値を受け渡したいfragmentにたどり着きます。なので何度も値を受け渡すよりも一発で値にアクセスできた方が楽だなと思いそうしました。activityからpageradapter、pageradapterからfragmentに値を受け渡していった方がいいですか？

[jimbe]

>インテントした先がtab表示のあるページなのでpageradapterを一回かましてから ～

まず、フラグメントが使えるなら、アクティビティを分ける必要はあるのでしょうか。
アクティビティの代わりに土台となるフラグメントを用意すれば、リストから選択したらインテントでアクティビティを呼ぶ代わりにフラグメントを切り替えるだけです。
そしてアプリ全体でアクティビティが1つで済めば、ビューモデルによって階層構造を跨いでもフラグメント間でデータの共有が可能です。

[dedede914]

すみません一個抜けていました。流れは、activityからactivity２,acticity２からpageradapter、pageradapterからfragmentです。
ちなみに日西辞書のアプリを作っています。どんな感じになっているか、ここに詳細を書かせていただきます
MainActivity：navigationbarで３つのfragmentをもっています（その内の一つがSearchFragmentです）
SearchFragment：検索ボックスのある問題のやつです。検索結果のリストをクリックするとresultActivityをインテントします。その際に、検索した単語をStringとしてAFragment,BFragmentに受け渡したいです。
ResultActivity：tab表示をします(AFragment,BFragment)。そのためここではSectionsPagerAdapterをViewPagerにsetAdapterするだけです。
SectionsPagerAdapter：AFragment,BFragmentのインスタンスを制作しています
AFragment,BFragment:ここで、Activityのstringを使ってデータを表示させたい。
といった感じです。

[jimbe]

どのような階層であれ、アクティビティを1つにすればビューモデルでデータを共有できることに変わりはありません。

SearchView 等を乗せている土台としての MainActivity の機能と ViewPager の土台としての ResultActivity の機能をそれぞれ(例えば)MainFragment と ResultFragment を新規に作成して移し、この二つの土台として(そしてアプリの唯一のアクティビティとして) MainActivity を再設計し、全体的なデータの保持役として(例えば) MainViewModel を作成します。
MainFragment 上の SearchFragment でリストを選択したら MainViewModel にその情報を設定すると共に MainFragment を ResultFragment に切り替え、 ViewPager 内の各フラグメントは MainViewModel からデータを取得する…ということが出来ます。

[dedede914]

クラスとかの関係性でいろいろ勘違いしていることが多かったたみたいで、調べてみていると
jimbeさんの言おうとしていることが何となくわかりました
ResultActivityをresult fragmentに変更し、うまく動作させることができました
viewmodelに関しては今調べている最中です
詳しくありがとうございました！

Answer 1

行クリックで呼ばれる onItemClick の position はフィルタリングされた表示上の位置ですが、同時に ListView#getItemAtPosition のパラメータもその範囲になりますので、そのまま指定してデータを取り出すことが出来ます。

データの管理をどのオブジェクトがするのか/しているのかを決める/判断することは、オブジェクト指向としては大事なことです。
リストのデータはデータベースから読んでいますが、リストとして表示する為にそのデータを保持・管理しているのは Adapter ですし、 Adapter のデータを表示しているのは ListView です。
フィルタリングは ListView に指定しているのですから、 ListView も Adapter も飛び越えてデータベースから選択データを得るのでは無く、 ListView から得ることをまず考えられると良いと思います。

import androidx.appcompat.app.AppCompatActivity;
import androidx.appcompat.widget.SearchView;

import android.os.Bundle;
import android.text.TextUtils;
import android.util.Log;
import android.widget.*;

import java.util.*;

public class MainActivity extends AppCompatActivity {
  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    ArrayAdapter<String> adapter = new ArrayAdapter<>(this, android.R.layout.simple_list_item_1,
            Arrays.asList("111","222","333","444","555"));
    Filter filter = adapter.getFilter();

    ListView listView = findViewById(R.id.listView);
    listView.setAdapter(adapter);
    listView.setTextFilterEnabled(true);
    listView.setOnItemClickListener((parent, view, position, id) -> {
      Log.d("onItemClick ***", "position="+position);
      String text = (String)listView.getItemAtPosition(position);
      Log.d("onItemClick ***", "text="+text);
    });

    SearchView searchView = findViewById(R.id.searchView);
    searchView.requestFocus();
    searchView.setOnQueryTextListener(new SearchView.OnQueryTextListener(){
      public boolean onQueryTextChange(String text) {
        Log.d("onQueryTextChange ***", "text="+text);
        if(TextUtils.isEmpty(text)) {
          //listView.clearTextFilter();
          filter.filter(null);
        } else {
          //listView.setFilterText(text);
          filter.filter(text);
        }
        return true;
      }
      @Override
      public boolean onQueryTextSubmit(String text) {
        Log.d("onQueryTextSubmit ***", "text="+text);
        return false;
      }
    });
  }
}

res/layout/activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity">

  <androidx.appcompat.widget.SearchView
      android:id="@+id/searchView"
      android:layout_width="0dp"
      android:layout_height="wrap_content"
      app:iconifiedByDefault="false"
      app:layout_constraintBottom_toTopOf="@id/listView"
      app:layout_constraintEnd_toEndOf="parent"
      app:layout_constraintStart_toStartOf="parent"
      app:layout_constraintTop_toTopOf="parent" />
  <ListView
      android:id="@+id/listView"
      android:layout_width="0dp"
      android:layout_height="0dp"
      app:layout_constraintBottom_toBottomOf="parent"
      app:layout_constraintEnd_toEndOf="parent"
      app:layout_constraintStart_toStartOf="parent"
      app:layout_constraintTop_toBottomOf="@id/searchView" />
</androidx.constraintlayout.widget.ConstraintLayout>

実行時ログ(SearchView に "2" を入力、"222" の行を選択)

D/onQueryTextChange ***: text=2
D/onItemClick ***: position=0
D/onItemClick ***: text=222

Comments

[dedede914]

期待通りに動きましたありがとうございました！
これからはデータを持っているオブジェクトから直接データを得られるメゾットがあるかしっかり確認するようにします！！