SQLiteのDBから取得したデータをListviewに表示したい

Question

前提・実現したいこと

Androidのアプリを開発しており、DB内の一部情報を取得しリスト表示する機能の実装を考えています。

※アプリの開発はAndroid Studioで開発しています。
DBの作成は、「DB Browser for SQLite」で作成しています。

DBは以下の通りです。

id	name	tag
1	施設A	娯楽
2	施設B	飲食店
3	施設C	娯楽

実現したいことは、

①DBから「tag」を取得してListViewで表示すること
②任意のListを選択すると、選択された「tag」の名前の情報をもって遷移すること

の２点です。

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

プロブラム事態にエラーは発生していないのですが、画面のListViewになにも表示されないという事態が生じています。

public void run()内のリスト表示の処理が原因かなと考えています。

該当のソースコード

以下は、Android Stdioのコードです。

package com.websarva.wings.android.town_infoapp;

//importは長いので割愛します。

public class MainActivity extends AppCompatActivity {

    private InputMethodManager inputMethodManager;
    private static final String DEBUG_TAG = "AsyncSample";
    private static final String URL = "ローカルURL";

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        String urlFull = URL;
        receiveTag(urlFull);

        //　タグを格納するListMapを生成。
        ListView lvtag = findViewById(R.id.lvtag);
        //　クリックリスナーを登録。
        lvtag.setOnItemClickListener(new ListItemClickListener());

    }

    private void receiveTag(final String urlFull){
        //確実にUIスレッドに処理を戻す
        Looper mainLooper = Looper.getMainLooper();
        //スレッド間通信を行うHandlerオブジェクトを登録
        Handler handler = HandlerCompat.createAsync(mainLooper);
        //非同期処理を行うためのインスタンス生成と実行
        MainActivity.TagInfoReceiver tagInfo = new MainActivity.TagInfoReceiver(handler, urlFull);
        ExecutorService executorService = Executors.newSingleThreadExecutor();
        executorService.submit(tagInfo);
    }

    //非同期でAPIにアクセスするためのクラス
    private class TagInfoReceiver implements Runnable{
        //ハンドラオブジェクト
        private final Handler _handler;
        //情報取得のURL
        private final String _urlFull;

        //コンストラクタ(ハンドラオブジェクトの受け取り)
        public TagInfoReceiver(Handler handler , String urlFull){
            _handler = handler;
            _urlFull = urlFull;
        }

        @WorkerThread
        @Override
        public void run() {
            //ここにWEBAPIにアクセスするコード
            //HTTP接続を行うオブジェクト宣言
            HttpURLConnection con = null;
            //レスポンスデータを取得するInputStringオブジェクトを宣言
            InputStream is = null;
            //JSON文字列を格納
            String result = "";

            try {
                //URLオブジェクトを生成
                java.net.URL url = new URL(_urlFull);
                //URLオブジェクトからHttpURLConnectionオブジェクトを取得
                con = (HttpURLConnection) url.openConnection();
                //各種タイム設定
                con.setConnectTimeout(1000);
                con.setReadTimeout(1000);
                //GET通信に指定
                con.setRequestMethod("GET");
                //接続開始
                con.connect();
                //レスポンスデータを取得
                is = con.getInputStream();
                //レスポンスデータのInputStringオブジェクトを文字列に変換
                result = is2String(is);
            }
            catch (MalformedURLException ex) {
                Log.e(DEBUG_TAG, "URL変換失敗", ex);
            }
            catch (SocketTimeoutException ex) {
                Log.w(DEBUG_TAG, "通信タイムアウト", ex);
            }
            catch (IOException ex){
                Log.e(DEBUG_TAG, "通信失敗", ex);
            }
            finally {
                if(con != null) {
                    con.disconnect();
                }
                if(is != null) {
                    try {
                        is.close();
                    }
                    catch (IOException ex) {
                        Log.e(DEBUG_TAG, "InputStream解放失敗", ex);
                    }
                }
            }
            MainActivity.TagInfoPostExecutor postExecutor = new MainActivity.TagInfoPostExecutor(result);
            _handler.post(postExecutor);
        }

        //InputStreamをStringに変換する定型文
        private String is2String(InputStream is) throws IOException {
            BufferedReader reader = new BufferedReader(new InputStreamReader(is, "UTF-8"));
            StringBuffer sb = new StringBuffer();
            char[] b = new char[1024];
            int line;
            while (0 <= (line = reader.read(b))) {
                sb.append(b, 0, line);
            }
            return sb.toString();
        }
    }

    private class TagInfoPostExecutor extends MainActivity implements Runnable {

        //取得したJSON文字列
        private final String _result;
        //コンストラクタ
        public TagInfoPostExecutor(String result) {
            _result = result;
        }

//ここのコードに問題？
        @UiThread
        @Override
        public void run(){
            //ここにUIスレッドで行う処理コード
            String tag = "";
            ArrayList<String> taglist = new ArrayList<String>();
            try {
                JSONArray rootJSON = new JSONArray(_result);
                for(int i=0; i<rootJSON.length(); i++) {
                    tag = rootJSON.getJSONObject(i).getString("tag");
                    taglist.add(tag);
                }
                ArrayAdapter<String> adapter = new ArrayAdapter<String>(this, android.R.layout.simple_list_item_1, taglist);
                ListView lvtag = findViewById(R.id.lvtag);
                lvtag.setAdapter(adapter);
            }
            catch (JSONException ex) {
                Log.e(DEBUG_TAG, "JSON解析失敗", ex);
            }
        }
    }

    class ListItemClickListener implements AdapterView.OnItemClickListener{
        @Override
        public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
            //　リストデータの取得
            
            //tagnameに選択したリストのtagを格納する処理

            //　インテント生成
            Intent intent = new Intent(MainActivity.this, SelectActivity.class);
            //　次画面に送るデータを格納
            intent.putExtra("tagname", tagname);
            //　次画面を起動
            startActivity(intent);
        }
    }
}

先のURLで処理しているコードです

from flask import Flask, app, jsonify, g
import sqlite3

app = Flask(__name__)
#JSONでの日本語の文字化け阻止
app.config['JSON_AS_ASCII'] = False

#DBに接続
def get_db():
    if 'db' not in g:
        g.db = sqlite3.connect("sample.sqlite3")
    return g.db
#DBとの接続を終了
def close_db(e=None):
    db = g.pop('db', None)
    if db is not None:
        db.close()


@app.route('/', methods=["GET"])
def index():
    db = get_db()
    cur = db.execute("select * from mydata")
    cur.row_factory = sqlite3.Row
    mydata = [dict(row) for row in cur.fetchall()]
    return jsonify(mydata)

if __name__ == "__main__":
    app.run(host='ローカルURL', port=5000, debug=True)

分からないこと

現状、起きている不具合は「処理は動くがListviewに要素が表示されていない」という事象です。

表示されていないということは、taglistに要素が追加されていないと思うのですが、どのように改良すればよいのかわからない状態です。

これに加え、Listviewを選択した際に選択したtagの名前の情報を次の画面へ渡したいのですが、tagnameに選択したリストのtagを格納するコードがわかりません。。

上記の2点、ご教授いただければ幸いです。

Comments

[hoshi-takanori]

ListView になにも表示されない原因はいくつか考えられます。
- Web API が結果を返してない
- Android 側で API を叩く処理に失敗している、または結果をうまく受け取れてない
- Android 側で結果を受け取ったが、表示に失敗している
ので、まずをれを切り分けましょう。

とりあえず、TagInfoPostExecutor の run メソッドの for ループ終了後の taglist の中身を、デバッガや Logcat などを使って確認してみては。

[tasojiro]

返信遅くなり申し訳ありません。

原因について調査しましたが、WEBAPIは結果を返していました。
2番目のAndroid側で結果をうまく受け取れていない可能性を探りましたが、
正直うまく調べられたか分かりません。が、「taglist」の内容はログで確認できませんでした。
（ログのタグ属性につけた名前も確認できなかったので、うまく調べられていないかも）

2番の可能性として原因究明と解決にあたっていますが、いまだにわかりません。。

[hoshi-takanori]

Logcat の見方が分からないってことでしょうか？ まずバッファが溢れてる場合はゴミ箱アイコンを押して消すと良いかも。あとはフィルタの使い方を覚えてください。
https://developer.android.com/studio/debug/am-logcat?hl=ja

[tasojiro]

いえ、使い方が分からなかったのです。。
しかし、調べて適応してみましたが、Logcatを使ってログを確認したところ、自分のつけたタグ情報が確認できませんでした。

これは、そもそも何らかの原因でrun()メソッド自体が実行されていないのでしょうか？

(以下、追記です。)
デバッガで確認したところ、TagInfoReceiverクラスのrun()メソッドでresultにWEBAPIから受け取ったJSONの文字列は取得しているようです。
しかし、
MainActivity.TagInfoPostExecutor postExecutor = new MainActivity.TagInfoPostExecutor(result);
からステップアウトを_handler.post(postExecutor);にはいかずに急にデバッガが追えなくなってしまいました。（デバッグログを見ていても突然ステップイン/アウトが出来なくなった状態）
やはり、TagInfoPostExecutorのrun()メソッドが実行できていないように思えます。

[hoshi-takanori]

動かしてみましたが、new MainActivity.TagInfoPostExecutor(result) で止まってますね。で、よく見てみたら、TagInfoPostExecutor が extends MainActivity してますが、これはありえないですね。
extends MainActivity を削除して、new ArrayAdapter の最初の引数を MainActivity.this にしましょう。

[tasojiro]

extend MainActivityを削除し、MainActivity.thisにしたところ表示されました！ありがとうございます。
確かに、MainActivity自身をextendするのはおかしかったですね。。

当たり前ですが、onItemClickで//tagnameに選択したリストのtagを格納する処理」を入力していないので、リストを押下しても遷移しない状態です。（②の質問）
「tagname」にリストの文字情報（run()でtagに格納した文字列）を格納する方法はありますでしょうか。taglistをonItemClickで利用できるようにするところからだと思いますが、よければよろしくお願いいたします。

[hoshi-takanori]

taglist を MainActivity のフィールドにして、onItemClick でその position 番目を取得すれば良いのでは。

[tasojiro]

positionを使う考えに至らなかったです。。。正確に動作を確認しました。
本当にありがとうございました！

Answer 1

hosi-takanori様の改善案により、解決しました。

修正箇所を以下に示します。
->private class TagInfoPostExecutor extends MainActivity implements Runnable
+>private class TagInfoPostExecutor implements Runnable

->//tagnameに選択したリストのtagを格納する処理
+>String tagname = (String)parent.getItemAtPosition(position);

ありがとうございました。