ListViewの画面が表示されない

Question

ListViewに選択した日付に格納されたデータを表示するという処理なのですが、エラーも起きず、画面が表示されません。表示の位置を変えたり、レイアウトファイルをまとめるなどやってみたのですが、全くわからなかったのでよろしければご助言いただきたいです。

@Override
    public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
        Intent intent = new Intent(getApplication(), DailyData.class);
        mDateManager = new DateManager();
        dateArray = mDateManager.getDays();
        String date = titleText.getText().toString().substring(0,7);
        date = date + "-" + dateArray.get(position).toString().substring(8,10);
        intent.putExtra("CalendarID", date);
        startActivity(intent);
    }

public class DailyData extends AppCompatActivity {

    private TextView mTextMessage;

    private BottomNavigationView.OnNavigationItemSelectedListener mOnNavigationItemSelectedListener
            = new BottomNavigationView.OnNavigationItemSelectedListener() {

        @Override
        public boolean onNavigationItemSelected(@NonNull MenuItem item) {
            switch (item.getItemId()) {
                case R.id.navigation_input:
                    mTextMessage.setText(R.string.title_home);
                    return true;
                case R.id.navigation_calendar:
                    mTextMessage.setText(R.string.title_dashboard);
                    return true;
                case R.id.navigation_config:
                    mTextMessage.setText(R.string.title_notifications);
                    return true;
            }
            return false;
        }
    };

    private KakeiboDatabase kDB;
    ListView listView;

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

        listView = new ListView(this);

        kDB = new KakeiboDatabase(getApplicationContext());

        ArrayList<Daily> list = new ArrayList<>();
        DailyAdapter dailyAdapter = new DailyAdapter(DailyData.this);

        Intent intent = getIntent();
        date = intent.getStringExtra("CalendarID");

        Daily daily = new Daily();

        SQLiteDatabase db = kDB.getReadableDatabase();

        Cursor cursor = db.query(
                "kakeibodb",
                new String[] {"category", "memo", "price"},
                "date=?",
                new String[] {date},
                null,
                null,
                null);

        boolean next = cursor.moveToFirst();
        while (next) {
            // 取得したカラムの順番(0から始まる)と型を指定してデータを取得する
            daily.setCategory(cursor.getString(0));// categoryを取得
            daily.setMemo(cursor.getString(1));// memoを取得
            daily.setPrice(cursor.getInt(2));// priceを取得

            // 次の行が存在するか確認
            next = cursor.moveToNext();
            list.add(daily);
            dailyAdapter.notifyDataSetChanged();
        }
        cursor.close();

        dailyAdapter.setDailyList(list);
        listView.setAdapter(dailyAdapter);
    }
}

public class DailyAdapter extends BaseAdapter {

    Context context;
    LayoutInflater layoutInflater = null;
    ArrayList<Daily> dailyList;

    public DailyAdapter(Context context) {
        this.context = context;
        this.layoutInflater = (LayoutInflater)context.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
    }

    public void setDailyList(ArrayList<Daily> dailyList) {
        this.dailyList = dailyList;
    }

    @Override
    public int getCount() {
        return dailyList.size();
    }

    @Override
    public Object getItem(int position) {
        return dailyList.get(position);
    }

    @Override
    public long getItemId(int position) {
        return dailyList.get(position).getId();
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        convertView = layoutInflater.inflate(R.layout.list_cell,parent,false);

        ((TextView)convertView.findViewById(R.id.dCategory)).setText(dailyList.get(position).getCategory());
        ((TextView)convertView.findViewById(R.id.dMemo)).setText(dailyList.get(position).getMemo());
        ((TextView)convertView.findViewById(R.id.dPrice)).setText(String.valueOf(dailyList.get(position).getPrice()));

        return convertView;
    }
}

package com.example.a162105.kakeibo;

public class Daily {
    long id;
    private String category;
    private String memo;
    private int price;

    public long getId() {
        return id;
    }

    public void setId(long id) {
        this.id = id;
    }

    public String getCategory(){
        return category;
    }

    public void setCategory(String category){
        this.category = category;
    }

    public String getMemo() {
        return memo;
    }

    public void setMemo(String memo) {
        this.memo = memo;
    }

    public int getPrice(){
        return price;
    }

    public void setPrice(int price){
        this.price=price;
    }

}

Answer 1

ListViewよりもRecyclerViewのほうが新しい上に拡張性があるらしく、RecyclerViewを使用したらできました。

Answer 2

layout が無かったりデータベースを使ったりと再現が面倒な感じなので, ぱっと見た感じということで失礼します.

まず, 呼ばれたパラメータ "CalendarID" によるデータベースの query でちゃんと想定した行数分のデータが出てきているでしょうか.
1行以上データがあれば, なにかしらリストに表示されそうです.

また, 2行以上データがある場合, おそらく最期のデータだけがずらずらと並ぶのではないでしょうか.
それは Daily daily = new Daily(); が while (next) {} の外にあるためです. while (next) { の後, daily.setCategory(cursor.getString(0)); の前に無ければなりません.

ついでですが, dailyAdapter.notifyDataSetChanged(); は while (next) {} の中では意味がありません. アダプタのデータが更新された時に '自身を設定されたListView' に表示を更新するよう通知するモノですが, while (next) {} でループしている時点でアダプタはまだ ListView に設定されていませんので.

さらに蛇足ですが, cursor のループは while 文では長くなってしまうので, for 文で書くとコンパクト(?)にまとまります.

boolean next = cursor.moveToFirst();
while (next) {
:
next = cursor.moveToNext();
}
↓
for(boolean next=cursor.moveToFirst(); next; next=cursor.moveToNext) {
:
}

Comments

[Aies]

変えてみたのですが、理解が浅く、構文の理解が進まなく、治る感じがしなかったのでListViewをやめてRecyclerViewに変えたらできました。

[jimbe]

出来たという事で, おめでとうございます.
可能でしたら, 変更したコードを回答として公開頂ければ, 他に悩んでいる方のヒントになると思います.

[jimbe]

遅くて申し訳ないですが, コードを見直したら
listView = new ListView(this);
と ListView を (xmlからの取得でなく) コード内で作っていたんですね.
listView = (ListView) findViewById(R.id.list);
のようにxmlから取得しないと, そのままでは画面に繋がっていないです.
RecyclerView に変えた際にこの辺りを直された, もしくは RecyclerView を表示するように作りこまれたのでしょう.