Android Studio お絵かきアプリで「描画」ができない

Question

PCスペック
Windows7 64bit
メモリ　8GB
プロセッサ　Intel Core i7-3610

デモ用端末(XPERIA OS:7.0 API 24)
Android Studio 3.4

Android Studioにて、今後、撮影した写真の必要な部分だけなぞって保存するアプリを開発するために、OnTouchListenerについて詳しく調べています。

そこで、お絵かきアプリなら「描画」を「なぞる」ことによって実現させていると思うので、以下のサイトを参考に同じアプリを実現したいと思いました。
https://qiita.com/InoueDaiki/items/3b51aaabdaad1c2227a3

ですが、上記のサイトの方は、ソースコードを全て載せているのですが、断片的でございまして、(Javaなら特にメソッドの書く順番なんか関係ない？)それっぽくソースコードを書いた結果、
アプリを実行して画面を指でなぞってみても線が描けません。

なぞっているのに、画面が飯能しないことについて、どんな原因が考えられるでしょうか・・？？
↓MainActivity.java

package com.example.paintview;

import android.os.Bundle;
import android.support.design.widget.FloatingActionButton;
import android.support.design.widget.Snackbar;
import android.support.v7.app.AppCompatActivity;
import android.support.v7.widget.Toolbar;
import android.view.View;
import android.view.Menu;
import android.view.MenuItem;

public class MainActivity extends AppCompatActivity {

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


        Toolbar toolbar = findViewById(R.id.toolbar);
        setSupportActionBar(toolbar);

        FloatingActionButton fab = findViewById(R.id.fab);
        fab.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                //Snackbar.make(view, "Replace with your own action", Snackbar.LENGTH_LONG).setAction("Action", null).show();
                PaintView paintView = (PaintView)findViewById(R.id.view);
                paintView.clear();
            }
        });
    }

    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        // Inflate the menu; this adds items to the action bar if it is present.
        getMenuInflater().inflate(R.menu.menu_main, menu);
        return true;
    }

    @Override
    public boolean onOptionsItemSelected(MenuItem item) {
        // Handle action bar item clicks here. The action bar will
        // automatically handle clicks on the Home/Up button, so long
        // as you specify a parent activity in AndroidManifest.xml.
        int id = item.getItemId();

        //noinspection SimplifiableIfStatement
        if (id == R.id.action_settings) {
            return true;
        }

        return super.onOptionsItemSelected(item);
    }
}

↓PaintView.java

package com.example.paintview;

import android.content.Context;
import android.graphics.Canvas;
import android.graphics.Paint;
import android.graphics.Path;
import android.util.AttributeSet;
import android.view.MotionEvent;
import android.view.View;

public class PaintView extends View {

    private Paint paint;
    private Path path;



    public PaintView(Context context) {
        this(context, null);

    }

    public PaintView(Context context, AttributeSet attrs) {

        super(context, attrs);
        path = new Path();
        paint = new Paint();
        paint.setColor(0xFF000000);
        paint.setStyle(Paint.Style.STROKE);
        paint.setStrokeJoin(Paint.Join.ROUND);
        paint.setStrokeCap(Paint.Cap.ROUND);
        paint.setStrokeWidth(10);
    }

    @Override
    protected void onDraw(Canvas canvas) {
        canvas.drawPath(path, paint);
    }

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

        switch (event.getAction()) {
            case MotionEvent.ACTION_DOWN:
                path.moveTo(x, y);
                invalidate();
                break;
            case MotionEvent.ACTION_MOVE:
                path.lineTo(x, y);
                invalidate();
                break;
            case MotionEvent.ACTION_UP:
                path.lineTo(x, y);
                invalidate();
                break;
        }
        return true;
    }

    public void clear(){
        path.reset();
        invalidate();
    }

}

↓activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<android.support.design.widget.CoordinatorLayout 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">

    <android.support.design.widget.AppBarLayout
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:theme="@style/AppTheme.AppBarOverlay">

        <android.support.v7.widget.Toolbar
            android:id="@+id/toolbar"
            android:layout_width="match_parent"
            android:layout_height="?attr/actionBarSize"
            android:background="?attr/colorPrimary"
            app:popupTheme="@style/AppTheme.PopupOverlay" />

    </android.support.design.widget.AppBarLayout>

    <include layout="@layout/content_main" />

    <android.support.design.widget.FloatingActionButton
        android:id="@+id/fab"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_gravity="bottom|end"
        android:layout_margin="@dimen/fab_margin"
        app:srcCompat="@android:drawable/ic_dialog_email" />



</android.support.design.widget.CoordinatorLayout>

↓content_main/xml

<?xml version="1.0" encoding="utf-8"?>
<android.support.constraint.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"
    app:layout_behavior="@string/appbar_scrolling_view_behavior"
    tools:context=".MainActivity"
    tools:showIn="@layout/activity_main">


    <view
        class="com.example.paintview.PaintView"
        id="@+id/view"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        tools:layout_editor_absoluteX="45dp"
        tools:layout_editor_absoluteY="421dp" />
</android.support.constraint.ConstraintLayout>

Answer 1

activity_main.xml内の、

<view
class="com.example.paintview.PaintView"

id="@+id/view"
android:layout_width="match_parent"
android:layout_height="match_parent"
tools:layout_editor_absoluteX="45dp"
tools:layout_editor_absoluteY="421dp" />

idに「android:」がついていなかったため、ひも付けができていなかったことがわかった。これを修正したところ、正常にアプリが動作しました。

Answer 2

コードの間違いはなさそうですね。

一つ考えられるのはこのコードはどこのレイアウトファイルとも紐づけされていないことです。
例えばMainActivity.javaなら activity_main.xmlと紐づけされてますが、このクラスではそのコードの部分が見当たりません。

MainActivityでこのクラスを呼び出すか、MainAcitivityのほうでこのコードを書くとよいと思います。

Comments

[sekaikan_ozaki]

コメントありがとうございます。
よくサイトを見たら、描画するためのviewが、PaintView.javaで使うように指定されています。

そうすると、MainActivity.javaで
PaintView paintView = (PaintView)findViewById(R.id.view);がCannot resolve symbol 'view'としてエラーが発生してしまいました。。
せっかくですので、オブジェクト指向を使って開発したいので、MainActivtyでPaintViewクラスを呼ぶ形にしたいです。
質問を修正して、ソースコードをすべて載せました。
ここがおかしいという部分があればご指摘お願い致します

[退会済みユーザー]

エラー内容は、「viewと名付けたidの場所がわからないよ！」という意味だと思うのですが、コードを拝見したところ書いてありますね…
一度content.xmlではなく、activity. XmlにダイレクトにViewを配置してみてください。

[sekaikan_ozaki]

＞一度content.xmlではなく、activity. XmlにダイレクトにViewを配置してみてください。

このようにしたところ、(viewが赤字のままになってるままなのに)なぜかビルドと実行が出来て描画ができました。

しかし、
https://qiita.com/InoueDaiki/items/3b51aaabdaad1c2227a3
で後半に紹介されている、お絵かきを消す機能も記述しているのですが、起動させたアプリの右下のメールマークのボタンでpaintView.clear();が呼ばれるのですが、その直前のPaintView paintView = (PaintView)findViewById(R.id.view);がエラーになっているせいで、メールボタンをタップするとアプリが停止してしまいます。。。

サイトの通りに記述しているはずなのに、エラーが出る原因としてはどのようなことが考えられるのでしょうか...?

[退会済みユーザー]

普通にボタンを配置して、そこからsetonclicklister、またはonclickをすればいいと思います。
あなたのしたい、削除ができるようになります。

[退会済みユーザー]

viewのエラーがでるのは、レイアウトファイルとコードの紐付がうまく行っていないからです。そこの部分は消しちゃってください。

追記
ボタンにオンクリックを設定して、実行したい内容を打ってください。

[sekaikan_ozaki]

＞普通にボタンを配置して
メールマークのついた特殊なボタンではなくて、普通のボタンを新しく作るという認識でよろしいですか？

＞viewのエラーがでるのは、レイアウトファイルとコードの紐付がうまく行っていないからです。そこの部分は消しちゃってください。

そこの部分というのは具体的にどの部分になりますでしょうか？

[退会済みユーザー]

申し訳ありません、外でスマホで回答したためややこしい回答をしてしまいました。

ボタン配置は、別にfabボタンである必要はありません。
普通の長方形のボタンでも同じ動作はします。
fabボタンと長方形のボタンの違いは形だけです。

ですので新規にボタンを作成してください。

viewのエラーがでるのは レイアウトファイルのactivity_viewのidがうまく紐づけされていないからだと思われます。
ということは、新たに作成したViewをコード上で紐づけしなければならないということです。

現在ActivtyMain.xmlにViewを配置していると思うのでそのViewのIdを"viewchan"にしてください。

このコードをonCreate()内に書いてください。

PaintView paintView = (PaintView)findViewById(R.id.viewchan);

これでレイアウトファイルとコードで紐づけ完了です。

このfindViewByIdは大切なので覚えておきましょう。

[退会済みユーザー]

追記
ボタン処理について
新規にボタンを追加した場合もfindViewByIdで紐づけ作業をしたあとに、

Button button = (Button)findViewById(R.id.button1);

button.setOnClickListner(new View...
//この辺りにコード補充が勝手に出現する。
//画面をクリアしたいコードを打つ

}
といった流れで打つとあなたの思う動作をすると思います。

[sekaikan_ozaki]

丁寧なご回答ありがとうございます。
色々アドバイスしていただき、ものすごくありがたいのですが、ほかに原因があることがわかりました。

<view
class="com.example.paintview.PaintView"

id="@+id/view"
android:layout_width="match_parent"
android:layout_height="match_parent"
tools:layout_editor_absoluteX="45dp"
tools:layout_editor_absoluteY="421dp" />

idに「android:」がついていなかったため、ひも付けができていなかったようでした。これを修正したところ、正常にアプリが動作しました。