お絵描きアプリで消しゴムとペンを切り替えられるようにしたい

Question

前提・実現したいこと

現在、本体ストレージに保存されている画像を開きそれにお絵描きができるようなアプリを開発しています。
下部に「消しゴム」「黒ペン」「赤ペン」のボタンを配置し、それを押すと各機能に切り替えられるようにしたいのですが、消しゴムだけうまく機能せず実装できません。

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

「消しゴム」ボタンを押すとペンの色が黒で描画され、それから「赤ペン」ボタンを押しても色が変わらない

該当のソースコード

public class DrawActivity extends AppCompatActivity implements View.OnClickListener {

    String mDrawUri;
    Uri mUri;

    private PaintView paintView;
    Button eraser,black_button,red_button;

    protected void onCreate(Bundle savedInstanceState) {

        // OnCreateで前画面から選択された画像のパスをインテントで渡し、
        // imageviewに渡す処理をしていますがここでは割愛

        // 以下がお絵描きに関わる部分のコードです

        eraser= findViewById(R.id.eraser);
        black_button= findViewById(R.id.black_button);
        red_button= findViewById(R.id.red_button) ;
        eraser.setOnClickListener(this);
        black_button.setOnClickListener(this);
        red_button.setOnClickListener(this);

        // 表示したimageviewの上に描画用のviewをFrameLayoutで重ねて表示しています
        this.paintView = findViewById(R.id.PaintView);

        findViewById(R.id.delete_button).setOnClickListener(deleteDrawing);

    }

    View.OnClickListener deleteDrawing = new View.OnClickListener(){
        @Override
        public void onClick(View view){
            paintView.delete();
        }
    };


    public void onClick(View v){
        switch(v.getId()){
            case R.id.eraser:
                paintView.setPen(Color.TRANSPARENT);
                Toast.makeText(this,"消しゴムで消去できます",Toast.LENGTH_SHORT).show();
                break;
            case R.id.black_button:
                paintView.setPen(Color.BLACK);
                Toast.makeText(this,"黒色で描画します",Toast.LENGTH_SHORT).show();
                break;
            case R.id.red_button:
                paintView.setPen(Color.RED);
                Toast.makeText(this,"赤色で描画します", Toast.LENGTH_SHORT).show();
                break;
        }
    }

    @Override
    public void onPointerCaptureChanged(boolean hasCapture) {

    }
}

public class PaintView extends View {
    // 履歴
    private List<DrawLine> lines;
    // 現在、描いている線の情報
    private Paint paint;
    private Path path;

    // 線の履歴(座標＋色)
    class DrawLine {
        private Paint paint;
        private Path path;

        DrawLine(Path path, Paint paint) {
            this.paint = new Paint(paint);
            this.path = new Path(path);
        }

        void draw(Canvas canvas) {
            canvas.drawPath(this.path, this.paint);
        }
    }

    public PaintView(Context context) {
        super(context);
    }

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

        this.path = new Path();

        this.paint = new Paint();
        this.paint.setStyle(Paint.Style.STROKE);
        this.paint.setAntiAlias(true);
        this.paint.setStrokeWidth(5);
        // 配列の設定
        // 線の色ごとの保存とかしてる（いずれUndoとか実装したい）
        this.lines = new ArrayList<DrawLine>();
    }

    @Override
    protected void onDraw(Canvas canvas) {
        super.onDraw(canvas);

        // キャンバスをクリア
        canvas.drawColor(Color.TRANSPARENT);
        // 履歴から線を描画
        for(DrawLine line : this.lines) {
            line.draw(canvas);
        }
        // 現在、描いている線を描画
        canvas.drawPath(this.path, this.paint);
    }

    @Override
    public boolean onTouchEvent(MotionEvent event) {
        float x = event.getX();
        float y = event.getY();

        switch (event.getAction()) {
            case MotionEvent.ACTION_DOWN:
                this.path.moveTo(x, y);
                break;
            case MotionEvent.ACTION_MOVE:
                this.path.lineTo(x, y);
                break;
            case MotionEvent.ACTION_UP:
                this.path.lineTo(x, y);
                // 指を離したので、履歴に追加する
                this.lines.add(new DrawLine(this.path, this.paint));
                // パスをリセットする
                // これを忘れると、全ての線の色が変わってしまう
                this.path.reset();
                break;
        }
        invalidate();
        return true;
    }

    public void delete() {
        // 履歴をクリア
        this.lines.clear();
        // 現在の線をクリア
        this.path.reset();
        invalidate();
    }

    // 色の設定
    // ここが原因で色が変わらなくなっている？
    public void setPen(int color){
        if (color == 0) {
            paint.setXfermode(new PorterDuffXfermode(PorterDuff.Mode.CLEAR));
        }else {}
        this.paint.setColor(color);
    }

試したこと

「消しゴム」ボタンが押されたことを判定するためにif式などを使ってみるも改善しなかった

Answer 1

setPen で, color が 0 に設定されたときに実行されております setXfermode を, color が 0 以外に(再)設定されたときに戻さなければいけないのではないでしょうか.

Comments

[tery0203]

回答ありがとうございます。
elseの中に
paint.setXfermode(new PorterDuffXfermode(PorterDuff.Mode.SRC_OVER));
を追加することで黒ペン、赤ペンを押した時はいつでも該当の機能が使えるようになりました。

ですが、消しゴムを押した時に黒の軌跡になってしまう現象は改善しませんでした…
PorterDuff.Mode.CLEARだとおそらく下の画像ごと消えてしまっており、背景の黒が表示されてしまっているのではないかと予測できるのですが、他のモードに変えても消しゴムになるものはありませんでした。

[jimbe]

color == 0 時に設定している CLEAR を SRC_OUT にしても変わらなかったでしょうか.
また, 調べていると「ハードウェアアクセラレータが有効の場合に CLEAR の動作が定義と異なる場合がある」という記事が見受けられました.
PaintView のコンストラクタで ```setLayerType(View.LAYER_TYPE_SOFTWARE, null);``` とするとハードウェアアクセラレータが無効になるそうなので, お試し頂けますでしょうか.

[tery0203]

コンストラクタにsetLayerType(View.LAYER_TYPE_SOFTWARE, null);を入れ、モードをCLEARにすることで消しゴムのように線が消えました！
ハードウェアアクセラレータ周りはまだ勉強不足でしたので、これを機に色々調べてみようと思います。
ありがとうございました！