リストの1行ごとに背景を変えたい

Question

前提

androidstudioについて質問します。
以下のプログラムにおいて、リストを1行ごとに色を変えたいと思っていますが、レイアウトの1行の取り方がわからずに困っています。どうしたら1行取得できますか

実現したいこと

リストの1行ごとに背景を変えたい

該当のソースコード

import android.content.Context;
import android.database.sqlite.SQLiteDatabase;
import android.database.sqlite.SQLiteOpenHelper;

public class DatabaseHelper extends SQLiteOpenHelper {
    private static final String DATABASE_NAME = "Sample.db";
    private static final int DATABASE_VERSION = 1;

    /**
     * コンストラクタ。
     *
     * @param context コンテキスト
     */
    public DatabaseHelper(Context context){
        super(context, DATABASE_NAME, null, DATABASE_VERSION);
    }

    @Override
    public void onCreate(SQLiteDatabase db){
        //ユーザー情報
        StringBuffer sb = new StringBuffer();
        sb.append("CREATE TABLE sample (");
        sb.append("_id INTEGER PRIMARY KEY AUTOINCREMENT,");
        sb.append("name TEXT NOT NULL,");
        sb.append("age INTEGER NOT NULL");
        sb.append(");");
        String sql = sb.toString();

        db.execSQL(sql);
    }

    @Override
    public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion){
    }
}

import android.database.Cursor;
import android.database.sqlite.SQLiteDatabase;
import android.database.sqlite.SQLiteStatement;

public class SampleDAO {
    public static Cursor findAll(SQLiteDatabase db){
        String sql = "SELECT * FROM sample";
        Cursor cursor = db.rawQuery(sql, null);
        return cursor;
    }

    public static long insert(SQLiteDatabase db, String name, int age){
        String sql = "INSERT INTO sample (name, age) VALUES (?, ?)";
        SQLiteStatement stmt = db.compileStatement(sql);
        stmt.bindString(1, name);
        stmt.bindLong(2, age);
        long insertedId = stmt.executeInsert();
        return insertedId;
    }
}

import androidx.appcompat.app.AppCompatActivity;

import android.database.Cursor;
import android.database.sqlite.SQLiteDatabase;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.ListView;
import android.widget.SimpleCursorAdapter;

import java.util.ArrayList;

public class MainActivity extends AppCompatActivity {
    /**
     * データベースヘルパーオブジェクト。
     */
    private DatabaseHelper _helper;
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        _helper = new DatabaseHelper(this);
        SQLiteDatabase db = _helper.getWritableDatabase();

        ArrayList<ArrayList<String>> list = new ArrayList<>();

        ArrayList<String> record = new ArrayList<>();
        record.add("赤松");
        record.add("20");
        list.add(record);
        record = new ArrayList<>();
        record.add("河田");
        record.add("30");
        list.add(record);
        record = new ArrayList<>();
        record.add("鈴木");
        record.add("40");
        list.add(record);
        record = new ArrayList<>();
        record.add("山田");
        record.add("20");
        list.add(record);
        record = new ArrayList<>();
        record.add("石田");
        record.add("20");
        list.add(record);

        for(int i=0; i<list.size(); i++){
            SampleDAO.insert(db, list.get(i).get(0), Integer.parseInt(list.get(i).get(1)));
        }

        Cursor cursor = SampleDAO.findAll(db);
        ListView flashcardList = findViewById(R.id.listView);
        //単語帳を表示
        String[] from = {"name", "age", "_id"};
        int[] to = {R.id.textView1, R.id.textView2, R.id.edit};
        SimpleCursorAdapter adapter = new SimpleCursorAdapter(this, R.layout.list, cursor, from, to, 0);
        adapter.setViewBinder(new ViewBinder());
        flashcardList.setAdapter(adapter);
    }

    private class ViewBinder implements SimpleCursorAdapter.ViewBinder {
        @Override
        public boolean setViewValue(View view, Cursor cursor, int columnIndex) {
long id = Long.parseLong(cursor.getString(columnIndex));
            switch(view.getId()) {
                case R.id.edit:
                    Button button = view.findViewById(R.id.edit);
                    long id = Long.parseLong(cursor.getString(columnIndex));
                    if(id % 2 == 1){
                        ((ViewGroup)button.getParent()).setBackgroundColor(Color.rgb(127, 127, 255));
                    }
                    return true;
            }
            return false;
        }
    }
}

<?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">

    <ListView
        android:id="@+id/listView"
        android:layout_width="0dp"
        android:layout_height="0dp"
        android:layout_marginStart="8dp"
        android:layout_marginTop="8dp"
        android:layout_marginEnd="8dp"
        android:layout_marginBottom="8dp"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent" />
</androidx.constraintlayout.widget.ConstraintLayout>

<?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"
    android:layout_width="match_parent"
    android:layout_height="match_parent">

    <Button
        android:id="@+id/edit"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_marginTop="8dp"
        android:layout_marginEnd="8dp"
        android:text="編集"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintTop_toTopOf="parent" />

    <TextView
        android:id="@+id/textView2"
        android:layout_width="0dp"
        android:layout_height="wrap_content"
        android:layout_marginStart="8dp"
        android:layout_marginEnd="8dp"
        app:layout_constraintBottom_toBottomOf="@+id/edit"
        app:layout_constraintEnd_toStartOf="@+id/edit"
        app:layout_constraintStart_toStartOf="@+id/guideline2"
        app:layout_constraintTop_toTopOf="@+id/edit" />

    <androidx.constraintlayout.widget.Guideline
        android:id="@+id/guideline2"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:orientation="vertical"
        app:layout_constraintGuide_percent="0.4" />

    <TextView
        android:id="@+id/textView1"
        android:layout_width="0dp"
        android:layout_height="wrap_content"
        android:layout_marginStart="8dp"
        android:layout_marginEnd="8dp"
        app:layout_constraintBottom_toBottomOf="@+id/edit"
        app:layout_constraintEnd_toStartOf="@+id/guideline2"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="@+id/edit" />
</androidx.constraintlayout.widget.ConstraintLayout>

Comments

[jimbe]

>switch(view.getId()) {
>case R.id.edit:
>Button button = view.findViewById(R.id.edit);

view.getId() が R.id.edit なら view 自体が Button ですので findViewById は不要でしょう。
Button button = (Button)view;

そして button の親が ConstraintLayout ですから、 button.getParent() で行が取れると思います。

[syosinsya109]

コメントありがとうございます。
バックグラウンドを指定するメソッドはsetBackgroundColorだと思うのですが、button.getParent()にはないみたいで、背景色が付与することができません。書き方、考え方が間違えているのでしょうか

[jimbe]

button.getParent() が返す ViewParent を ViewGroup にキャストすれば setBackgroundColor は使えます。

他に、行レイアウトの root に id を付け

<androidx.constraintlayout.widget.ConstraintLayout
～ android:id="@+id/row" ～

from/to に指定を追加すれば

String[] from = {"name", "age", "_id", "_id"}; //rowに対応するのは何でも
int[] to = {R.id.textView1, R.id.textView2, R.id.edit, R.id.row};

setViewValue でボタンと同じように得られます。

switch(view.getId()) {
case R.id.edit: ～
case R.id.row: ～
}

[syosinsya109]

(ViewGroup)button.getParent().setBackgroundColor(Color.rgb(127, 127, 255));
こういうことでしょうか。それでもエラーが出てきます。
setBackgroundColorを使っているサイトがあり、Viewをキャストして試してみましたが、同様にエラーが出てきました。

[jimbe]

>button.getParent() が返す ViewParent を ViewGroup にキャスト
です。

((ViewGroup)button.getParent()).setBackgroundColor(Color.rgb(127, 127, 255));

[syosinsya109]

ありがとうございました！おかげで色を変えることができました！
追加で質問してもいいでしょうか。良くてもダメでもベストアンサーに選びたいので回答をしてくださるとうれしいです。

質問
自分は1行ごとの色を変化させたくてこの質問をしました。そして先ほどのやり方で色は変更できましたが、スクロールするたび色が変動してしまいます。id % 2 == 1のようなif文を書いても変動してしまいます。
多分ですがスクロールするたびにViewBinderを経由するのでずれるものと思われます。どのようにしたら対処できるのでしょうか

[jimbe]

"スクロールするたび色が変動" してしまう時のコードをご提示ください。

setViewValue は 1 行につき from/to の件数分 view と columnIndex を変えて呼ばれます。
もし setViewValue の最初で
long id = Long.parseLong(cursor.getString(columnIndex));
としているなら、ボタンの時以外(R.id.name や R.id.age) では例外が発生したり年齢が id になったりで、id % 2 == 1 の判定も意味無いでしょう。

コードを移動させるなら、各変数値がどうなっているのかを注意してください。
cursor.getString(columnIndex) が _id 列の値を返すのは、

String[] from = {"name", "age", "_id"};
int[] to = {R.id.textView1, R.id.textView2, R.id.edit};

なら view.getId() が R.id.edit の時だけです。

[syosinsya109]

すいません。ただいま修正しました。
editボタンを全行配置していますので、表示した時にはid % 2 == 1がtrueの時に入っていると思っています。

[jimbe]

long id = ～ が switch の中にあるのでしたら、外の同処理は削除してください。余計な処理を残しておくと、後で問題になる可能性をも残します。

ListView は行のビューを再利用します。
背景色を特定の場合だけ変更するのなら、特定で無い場合の背景色も設定してください。

[syosinsya109]

無事できました！！ありがとうございます！

Answer 1

こんな感じでしょうか?

            switch(view.getId()) {
                case R.id.edit:
                    Button button = view.findViewById(R.id.edit);
                    long id = Long.parseLong(cursor.getString(columnIndex));
                    int bgcolor = id % 2 == 1 ? Color.rgb(127, 127, 255) : Color.rgb(255, 255, 255);
                    ((ViewGroup)button.getParent()).setBackgroundColor(bgcolor);

なおこの実装では、データベースの id が連続した番号かどうか、データベースからデータを削除することは無いかが問題になるかもしれません。
表示上1行置きに色を変えるだけであれば、 id % 2 では無く curosr.getPosition() % 2 を使う手もあるかもしれません。

Comments

[syosinsya109]

ありがとうございます！