Javaのタートルグラフィックで亀の総移動距離や回転を表示する

Question

亀を移動させ、終了命令と共に総移動距離と平均移動距離を出力したい

タートルグラフィックスの亀を動かし回転させ、終了とともに総移動距離と平均移動距離を出力したいです。
命令は２つあります

命令1(op1)：quit, go, rotate　の３つです
命令2(op2)：taro(亀t1), hanako(亀t2) の２つです

ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー

1.命令1でquitと入力した場合：それまでのそれぞれの亀の総移動距離と平均移動距離を出力する

2.命令1でgoと入力した場合：
①続けて命令2でtaroと入力した場合→taroは乱数m1(10.0以上50.0未満)で前に進み、インスタンスの位置と向きの角度を出力する
②続けて命令2でhanakoと入力した場合→hanakoは乱数m2(10.0以上50.0未満)で前に進み、インスタンスの位置と向きの角度を出力する

3.命令1でrotateと入力した場合：
①続けて命令2でtaroと入力した場合→taroは乱数k(0.0以上180.0未満)で右に回転し、インスタンスの位置と向きの角度を出力する
②続けて命令2でhanakoと入力した場合→hanakoは乱数k(0.0以上180.0未満)で右に回転し、インスタンスの位置と向きの角度を出力する

ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー

という内容のプログラムを組みたいです

しかし、私の書いたプログラムでは、以下の問題が生じてしまいます

・go ~ で前に進む距離が毎回変化せず、一回目の乱数のみで固定されてしまう

・rotate ~ と入力すると
【Exception in thread "main" java.lang.NullPointerException
at homework06_02.main(homework06_02.java:48)】
と表示されてしまう

（上記の問題が解決できないため、総移動距離や平均移動距離を出力するプログラムはまだ書いていません）

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

Exception in thread "main" java.lang.NullPointerException
	at homework06_02.main(homework06_02.java:48)

該当のソースコード

import java.util.Scanner;

import javafx.scene.paint.Color;
import tg.Turtle;
import tg.TurtleFrame;

public class homework06_02 {

	public static void main(String[] args) {
		// TODO 自動生成されたメソッド・スタブ
		Scanner sc = new Scanner(System.in);
		TurtleFrame f = new TurtleFrame();
		Turtle t1 = new Turtle(150, 200, 180); //taro
		Turtle t2 = new Turtle(250, 200, 0); //hanako
		f.add(t1); f.add(t2);

		t1.setTColor(Color.BLUE);
		t2.setTColor(Color.RED);

		double m = 10.0 + Math.random() * 40.0;
		double m1, m2, sum1 = 0, sum2 = 0;
		double k = Math.random() * 180.0;

		while(true) {
			String op1, op2;
			op1 = sc.next();
			if (op1.equals("quit")) {
				System.out.println("終了します");
				System.out.println("taroの総移動距離=" + sum1);
				System.out.println("hanakoの総移動距離=" + sum2);
				break;
				
			} else if(op1.equals("go")) {
				op2 = sc.next();
				if(op2.equals("taro")) {
					t1.fd(m);
					m1 = m;
					sum1 = sum1 + m1;
					System.out.println("taroは" + m1 + "移動して(150.0, " + sum1 + ")にいます");
				} else if(op2.equals("hanako")) {
					t2.fd(m);
					m2 = m;
					sum2 = sum2 + m2;
					System.out.println("hanakoは" + m2 + "移動して(250.0, " + sum2 + ")にいます");
				}

			} else if(op1.equals("rotate")) {
				if(op2.equals("taro")) {
					t1.rt(k);
					System.out.println("taroは(150.0, "+ sum1 + ")で" + t1.getAngle() + "を向きました");
				}else if(op2.equals("hanako")) {
					t2.rt(k);
					System.out.println("hanakoは(250.0, " + sum2 + ")で" + t2.getAngle() + "を向きました");
				}
			}
		}sc.close();
	}
}

試したこと

・if文に不備がないか紙に書き出し、一度中身を削除して範囲を見直し書き直した

###最後に
長々と分かりづらい説明でも仕分けございません。
コメントをいただければ分かる限りお答えしたいと思います。
当方かなりの初心者で、調べようにもどう調べたら良いか分からず最終手段として質問させていただきました。どうかよろしくお願いいたします。

Answer 1

まずはエラーメッセージに従って、48行目をチェックしましょう。

if(op2.equals("taro")) {

この行のどこかで、NullPointerExceptionが生じています。
...と言っても、この中でnull値を取り得るのはop2しかありませんね。

それでは、op2に着目してコードを追ってみましょう。

String op1, op2;
op1 = sc.next();
if (op1.equals("quit")) {
    ...
    break;
} else if(op1.equals("go")) {
    op2 = sc.next();
    ...
} else if(op1.equals("rotate")) {
    if(op2.equals("taro")) {
        ...

rotateと入力された際は、op2を読み込んでいないことが分かります。

乱数の取り扱いについて

根本的な勘違いをしていると思います。

double m = 10.0 + Math.random() * 40.0;

このmはただのdouble型変数であって、たまたまその値が乱数で決定されているだけです。
mを評価する度に新たな乱数が生成されるわけではありません。

乱数が必要なときは再度Math.random()を使って下さい。

Comments

[退会済みユーザー]

迅速な回答ありがとうございます。

rotateの条件下でop2を読み込んでいない理由や読み込ませる修正方法などご教授いただけないでしょうか？

エラーの出ている48,51行目のop2にカーソルを当てると「変数を初期化します」と表示され、クリックすると25行目の String op1, op2; が String op1, op2 = null; となりました。
またその状態で実行しrotate ~ を入力しても【Exception in thread "main" java.lang.NullPointerException
at homework06_02.main(homework06_02.java:48)】と表示されてしまいました。。。

[退会済みユーザー]

回答に追加していただきありがとうございます。
乱数は毎回生成されるわけではないのですね！

[LouiS0616]

> rotateの条件下でop2を読み込んでいない理由や読み込ませる修正方法

if(op1.equals("go")) のときと同じように入力を受けてやれば良いです。
あるいはquitしなかったときに常にop2を読み取るようにします。

[swordone]

この時って、ヌルポじゃなくて｢変数が初期化されていない｣コンパイルエラーになりそうだが…

[LouiS0616]

@swordone さん
確かに。でも48行目そこなんですよね...

[退会済みユーザー]

解決いたしました！
ありがとうございます！