データベースに接続してデータを読み込みたい android studio / SQLite

Question

実現したいこと

android studioでデータベースを使ったアプリを作ってみています。既存のデータベースからデータを引っ張り出してきたいです。

前提

javaで書いています。
androidstudioに既存のデータベースをインポート(?)するところまではできました。ちゃんとandroidstudio内でインポートできていることも確認できました。とりあえず、1列目のデータを上から10個表示させようとしましたが2個目から読み込めませんのエラーが出ました。
MainActivity.javaからSearchFragment.javaを呼び出しています
データベースは５１０ｘ５２の大きさです

3日ほど格闘しています。どうかよろしくお願いします。

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

#sql="select colum1 from table1 limit 10"　のエラー
2023-03-07 19:13:53.996 15655-15655/com.example.dicapp E/CursorWindow: Failed to read row 2, column 1 from a CursorWindow which has 10 rows, 1 columns.
2023-03-07 19:13:54.001 15655-15655/com.example.dicapp E/AndroidRuntime: FATAL EXCEPTION: main
~省略
Caused by: android.view.InflateException: Binary XML file line #81: Binary XML file line #81: Error inflating class fragment
Caused by: android.view.InflateException: Binary XML file line #81: Error inflating class fragment
Caused by: java.lang.IllegalStateException: Couldn't read row 2, col 1 from CursorWindow.  Make sure the Cursor is initialized correctly before accessing data from it.
....    

#sql="PRAGMA table_info(table1)" のときのエラー
2023-03-07 19:09:21.315 15481-15481/com.example.dicapp E/CursorWindow: Failed to read row 7, column 6 from a CursorWindow which has 52 rows, 6 columns.
2023-03-07 19:09:21.319 15481-15481/com.example.dicapp E/AndroidRuntime: FATAL EXCEPTION: main
　　～省略
Caused by: android.view.InflateException: Binary XML file line #81: Binary XML file line #81: Error inflating class fragment
Caused by: android.view.InflateException: Binary XML file line #81: Error inflating class fragment
Caused by: java.lang.IllegalStateException: Couldn't read row 7, col 6 from CursorWindow.  Make sure the Cursor is initialized correctly before accessing data from it.
.....      

該当のソースコード

public class SearchFragment extends Fragment {
    private SearchViewModel homeViewModel;
    private FragmentSearchBinding binding;
    private DatabaseHelper helper=null;

    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();

//ここが問題のコードです
        setConj(helper);

        return root;
    }



    public void setConj(DatabaseHelper helper){
        //helperを準備
        helper=new DatabaseHelper(getContext());//元々this
        //データベース取得
        try(SQLiteDatabase db=helper.getWritableDatabase()){
            Toast.makeText(getContext(),"接続済み", Toast.LENGTH_SHORT).show();
        }

        StringBuilder builder = new StringBuilder();
        SQLiteDatabase db = helper.getWritableDatabase();
        try {
            int i =0;
            //何個か試したうちの３つです
            String sql="select colum1 from table1 limit 10";   //error
            //sql="PRAGMA table_info(table1)";     //error
            //sql="SELECT name FROM sqlite_master WHERE type='table';"  //ちゃんとテーブル名が返ってきました

            Cursor cursor = db.rawQuery(sql , null);
            cursor.moveToFirst();
            builder.append(cursor.getString(i));
            while (cursor.moveToNext()) {
//ここがエラーの出る箇所です。
　　　　　//cursor.getString(i)の行をコメントアウトしたらエラーは出なかったので、クエリ自体はうまく処理されている？？
                builder.append(cursor.getString(i));
                i++;
            }
            Log.e("debug", builder.toString());
        }finally{
            db.close();
        }
    }
}

試したこと

sql="SELECT name FROM sqlite_master WHERE type='table';"
でクエリを実行したときは、ちゃんとテーブルの名前が返ってきました。
コメントアウトして、どこでエラーが出ているのか確認したら、cursorからwhileを使って順に値を挿入している部分でエラーが発生しているようでした。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[jimbe]

>//データベース取得
>try(SQLiteDatabase db=helper.getWritableDatabase()){
>Toast.makeText(getContext(),"接続済み", Toast.LENGTH_SHORT).show();
>}

コレは意味が無いでしょう。
try-resource は try の () 内のリソースを {} を抜ける時に close します。
"接続済み" と表示して close してはトーストを出す意味もありません。
SQLiteDatabase は通常の使用においては helper 内で管理されるため、明示的に close する必要はありません。

[dedede914]

そうなんですね。サイトのページコピペが多くてちゃんと意味が分かっていませんでした
ありがとうございます

Answer 1

Cursor の get～ で指定する index は、行では無く列です。

Answer 2

結論から言えば、変数 ｉ を増分しているのが間違いです。

前提として、以下のことを理解している必要があります。
・Cursor は、処理結果表の現在処理中の行を保持し、メソッドで行を移動。
・今回のSQLは、colum1 のみを取得しているので、テーブルの列がいくつあっても、１列。

エラーの発生機序ですが、以下のようになります。
while前の cursor.getString(i) では、i が０なため、先頭行の１列目が取得され、正常に終了します。
cursor.moveToNext() で、現在行が２行目に移りますが、こちらも、i が 0 なので、正常処理。
ですが、次のループでは、i が増加し１になっているため、２列目を取得しようとして、存在しないためにエラーになります。

"SELECT name FROM sqlite_master WHERE type='table';" が正常に処理されたのは、１行しか存在せず、while ループに入らなかったためと思われます。

余談ですが、カーソルではなく、２次元配列などで結果を全て取得し、i に該当する行のデータを取得するような処理だった場合は、１行目が２回取得されることになり、エラーにはならないものの、不具合となるでしょう。

Comments

[dedede914]

詳細に説明ありがとうございました！！！
本当にありがとうございました！！！！！！