お絵かきアプリ android studio

Question

前提・実現したいこと

お絵かきアプリにボタンを追加したい

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

xmlでMyView.javaを動かしたいです。ビルドは成功するのですが、runするとアプリが一瞬で落ちてしまいます。xmlの<com.example.DrawingApp.MyView〜/>を消し、MainActivity.javaでsetContentView(R.layout.activity_main);とするとボタンだけ表示され、正しく動作します。また、MainActivity.javaでsetContentView(new MyView(this));とすると描画が正しく動作します。xmlに2,3の引数が使われていないとのメッセージがあるのですがどうしたらいいかわかりません。

Custom view MyView is not using the 2- or 3-argument View constructors; XML attributes will not work 

該当のソースコード

activity_main.xml
<Button
        android:id="@+id/button"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="@string/button"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintLeft_toLeftOf="parent"
        app:layout_constraintRight_toRightOf="parent"
        app:layout_constraintTop_toTopOf="parent" />

    <com.example.DrawingApp.MyView
        android:layout_width="340dp"
        android:layout_height="420dp"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintLeft_toLeftOf="parent"
        app:layout_constraintRight_toRightOf="parent"
        app:layout_constraintTop_toTopOf="parent" />

MyView.java
public class MyView extends View {
    // イべント発生時の X 座標、Y 座標を保存するための動的配列
    private ArrayList array_x, array_y;
    private ArrayList array_status;
    // コンストラクタ
    public MyView(Context context){
        super(context);
        array_x      = new ArrayList();
        array_y      = new ArrayList();
        array_status = new ArrayList();
    }
    // ビューの描画を行うときに呼ばれるメソッド
    @Override
    protected void onDraw(Canvas canvas){
        super.onDraw(canvas);
        // 背景を白色で塗りつぶす
        Paint paint = new Paint();
        paint.setStyle(Paint.Style.FILL);
        paint.setColor(Color.WHITE);
        canvas.drawRect(0, 0, canvas.getWidth(), canvas.getHeight(), paint);
        // 描画用の Paint オブジェクトを用意
        paint = new Paint();
        paint.setStyle(Paint.Style.STROKE);
        paint.setColor(Color.RED);
        // 配列内の座標を読み出して線(軌跡)を描画
        for (int i = 1; i < array_status.size(); i++) {
            // 描画するように(true)状態値が与えられているとき
            // 一度離してしてから次に押されるまでの移動分は描画しない
            if ((Boolean) array_status.get(i)) {
                // 開始点の終了点の座標の値を取得
                int x1 = (Integer) array_x.get(i - 1);
                int x2 = (Integer) array_x.get(i);
                int y1 = (Integer) array_y.get(i - 1);
                int y2 = (Integer) array_y.get(i);
                // 線を描画
                canvas.drawLine(x1, y1, x2, y2, paint);
            }
        }
    }
    @Override
    public boolean onTouchEvent(MotionEvent event) {
        //return super.onTouchEvent(event);
        // 座標を取得
        int x = (int)event.getX();
        int y = (int)event.getY();
        // イベントに応じて動作を変更
        int a = event.getAction();
        switch (a) {
            case MotionEvent.ACTION_DOWN: //タッチパネルが押された時
            case MotionEvent.ACTION_POINTER_DOWN:
                array_x.add(new Integer(x)); // 座標を配列に保存
                array_y.add(new Integer(y));
                array_status.add(new Boolean(false));// 線の描画はしない(false)
                invalidate();// 画面を強制的に再描画
                break;
            case MotionEvent.ACTION_MOVE:
                array_x.add(new Integer(x)); // 座標を配列に保存
                array_y.add(new Integer(y)); // 線の描画をする(true)
                array_status.add(new Boolean(true));
                invalidate(); // 画面を強制的に再描画
                break;
            case MotionEvent.ACTION_UP: // タッチパネルから離れたとき
            case MotionEvent.ACTION_POINTER_UP:
                array_x.add(new Integer(x)); // 座標を配列に保存
                array_y.add(new Integer(y)); // 線の描画をする(true)
                array_status.add(new Boolean(true));
                invalidate(); // 画面を強制的に再描画
                break;
        }
        return true;
    }
}

Comments

[keicha_hrs]

activity_main.xmlはConstraintLayoutなどで囲っているのでしょうか？それも省略せずご提示いただきたいです。

[1236]

<?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"
android:background="#000"
tools:context=".MainActivity">

<Button
android:id="@+id/button"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:text="@string/button"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintLeft_toLeftOf="parent"
app:layout_constraintRight_toRightOf="parent"
app:layout_constraintTop_toTopOf="parent" />

<com.example.DrawingApp.MyView
android:id="@+id/myView"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintLeft_toLeftOf="parent"
app:layout_constraintRight_toRightOf="parent"
app:layout_constraintTop_toTopOf="parent"/>

</androidx.constraintlayout.widget.ConstraintLayout>

ご回答ありがとございます。

[keicha_hrs]

すでに回答が出ていますが、MyViewのコンストラクターを引数2つのものに置き換えれば解決できるようですね。

[1236]

それで解決できました。ご回答ありがとうございました！

Answer 1

カスタムビューのコンストラクタは、引数が 1 〜 4 個の 4 種類 (そのうち、引数が 4 個のものは API level 21 以上) があって、layout XML に配置するには引数が 2 個のものが必要なようです。
参考: ビュークラスを作成する | Android デベロッパー | Android Developers

public class MyView extends View {

    public MyView(Context context, AttributeSet attrs) {
        super(context, attrs);

        // ここに初期化コードを書く。
    }
}

一般的にはコンストラクタを 3 つ (または 4 つ) 書いてる例が多いですが、正直どこまで必要なのかはよく知りません。
参考: Guide to Android custom views: constructors - Marcin Korniluk - Medium
参考: Viewの4つのコンストラクタ - Qiita

public class MyView extends View {

    public MyView(Context context) {
        super(context);
        init();
    }

    public MyView(Context context, AttributeSet attrs) {
        super(context, attrs);
        init();
    }

    public MyView(Context context, AttributeSet attrs, int defStyleAttr) {
        super(context, attrs, defStyleAttr);
        init();
    }

    private void init() {
        // ここに初期化コードを書く。
    }
}

Comments

[1236]

正しく動作しました。ご回答ありがとうございました！