質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.48%

  • Java

    14153questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Android

    6638questions

    Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

  • Android Studio

    3805questions

    Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。

Android Studioで他のクラスのメソッド

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 701

surakku

score 8

Android Studioでお絵かきアプリを作っています。ボタンを押すとMyViewクラスのメソッドで色を変えるようにしたいのですがエミュレータで確認すると、ボタンを押すと同時にアプリが落ちてしまいます。
何が原因か教えてくださいませんでしょうか。
以下のプログラムではボタンを押すとMainActivityのonClickがmyviewのcolorBlueメソッドを呼ぶシステムにしたいと思っています。

public class MainActivity extends AppCompatActivity implements View.OnClickListener {

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

        this.myview = (MyView)findViewById(R.id.drawing_view);
        findViewById(R.id.buttonB).setOnClickListener(this);

    }
        @Override
        public void onClick(View v) {
            myview.colorBlue();
        }
}

MyView全文

public class MyView extends View {
// イベント発生時の X 座標、Y 座標を保存するための動的配列
private ArrayList array_x, array_y;
private ArrayList array_status;

private Paint p;

// コンストラクタ(ここではこの引数のものを追加)
public MyView(Context context) {
        super(context);
        array_x = new ArrayList();
        array_y = new ArrayList();
        array_status = new ArrayList();
        p = new Paint();
        }
public MyView(Context context, AttributeSet attrs) {
        super(context, attrs);
        array_x = new ArrayList();
        array_y = new ArrayList();
        array_status = new ArrayList();
        p = new Paint();
        }

// ビューの描画を行うときに呼ばれるメソッド
@Override
protected void onDraw(Canvas canvas) {
        super.onDraw(canvas);

// 背景を白色で塗りつぶす
        p = new Paint();
        p.setStyle(Paint.Style.FILL);
        p.setColor(Color.WHITE);
        canvas.drawRect(new Rect(0, 0,
        canvas.getWidth(), canvas.getHeight()), p);
// 描画用の Paint オブジェクトを用意
        //p = new Paint();
        p.setStyle(Paint.Style.STROKE);
        p.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, p);
                }
        }
}

// タッチパネルを操作した時に呼ばれるメソッド
@Override
public boolean onTouchEvent(MotionEvent event) {
// 座標を取得
        int x = (int) event.getX();
        int y = (int) event.getY();
// イベントに応じて動作を変更
        switch (event.getAction()) {
        case MotionEvent.ACTION_DOWN: // タッチパネルが押されたとき
        case MotionEvent.ACTION_POINTER_DOWN:
        array_x.add(new Integer(x)); // 座標を配列に保存
        array_y.add(new Integer(y)); // 線の描画はしない(false)
        array_status.add(new Boolean(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;
        }

public void colorBlue() {     // 色を青に変えるメソッド
p = new Paint();
        p.setColor(Color.BLUE);
}
}

追記
皆さん有難うございます。ボタンを押すと強制終了する問題は何故か解決しました。しかしペンの色を変えることができません。そこを教えていただけませんでしょうかm(__)m
MyViewの全文を記載させていただきます。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • yona

    2016/05/11 12:14

    ログキャットのエラーログを追記してください。 ソースコードは専用の記述方法があるのでそちらに修正してください。

    キャンセル

  • surakku

    2016/05/11 16:49

    有難うございます。強制終了の問題は他のPCでプログラムすることで解決いたしました。

    キャンセル

  • abs123

    2016/05/11 17:31

    途中で質問の趣旨をかえることはやめたほうがよいです。古い問題への解答と、新しい問題への解答がごちゃ混ぜになってしまいます。別途質問を上げてください。また、他のPCで解決できたとのことですが、なぜ解決できたのか分かる場合は、自己回答の形で、解決までの流れを書くことをお勧めします。

    キャンセル

  • surakku

    2016/05/12 10:22

    分かりにくいことになってしまい申し訳ありません。新しい質問を上げさせていただきます。

    キャンセル

回答 3

+1

colorBlue()

このメソッドはそもそも自クラスMyViewでも機能していますか?

デフォルトで色を決めている部分のコードも開示しましょう

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/05/11 16:50

    有難うございます。MyView全文を記載させていただきました。初期色の赤では問題なく機能しております。色を変えるBLUEボタンが機能していない状態です。

    キャンセル

+1

findViewById(R.id.drawing_view)がnullを返しているのでは?レイアウトを確認すべきかと。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/05/11 16:49 編集

    有難うございます。取り敢えず別のPCで動作させることで強制終了問題は解決いたしました。

    キャンセル

checkベストアンサー

0

onDrawの中でpを新しく作り直し、色を設定していますよね。
これでは色を変えることはできません。
背景用のPaintとライン用のPaintを別にし、ライン用のPaintはonDrawの中で色を設定してない必要があります。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.48%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Java

    14153questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Android

    6638questions

    Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

  • Android Studio

    3805questions

    Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。