コンストラクタ名があると出てしまいます。

Question

前提・実現したいこと

キーボードから１が入力されたらスピードが1上がるspeedUpメソッドを呼び出ししたいです。
また
キーボードから２が入力されたらスピードが1下がるspeedDownメソッドを呼び出ししたいです。
そしてキーボードから3の入力で終了するようにしたいです。

発生している問題・エラーメッセージに内容は書いたのですが、どのようにすれば上記のように動作するのか教えてほしいです。

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

3が入力されるまで何度も聞かれるようにするにはどうすれば良いですか？
今の状態だと一度1～3の数字を押すとspeedShow()が一度起動後終了してしまいます。
すいません............

修正点：クラスをCarとTestのみへ変更。

import java.util.Scanner;
public class Test{

	public static void main(String[] args) {

		Car no01=new Car();
		
		System.out.println("スピードを上げるなら[1]、"
				+ "下げるなら[2]を入力([3]は終了)");

		Scanner sc=new Scanner(System.in);
		int n=sc.nextInt();

		if(n==1) {
				//スピードup
				no01.speedUp();
				no01.speedShow();

		}
		if(n==2) {
			//スピードDown
				no01.speedDown();
				no01.speedShow();

	}
		if(n==3) {
			//終了
			System.out.println("終了します");
		}
		sc.close();
}
}

public class Car {
	final static int LIMIT=50;
	
	private int speed;
	
	Car(){
		this.speed=0;
	}

		void speedDown() {
			if(this.speed>0) {
				this.speed--;
			}
		}

		void speedUp() {
			if(this.speed<80) {
				this.speed++;
			}
		}

	void speedShow() {
		if(LIMIT>this.speed) {
		System.out.println("制限速度"+LIMIT+"km/h");
		System.out.println("現在速度"+this.speed+"km/h");
		System.out.println("制限速度内です(あと"+(LIMIT-this.speed)+"km/hです)");
	}
		if(LIMIT==this.speed) {
			System.out.println("制限速度"+LIMIT+"km/h");
			System.out.println("現在速度"+this.speed+"km/h");
			System.out.println("制限速度限界");
	}
		if(LIMIT<this.speed) {
			System.out.println("制限速度"+LIMIT+"km/h");
			System.out.println("現在速度"+this.speed+"km/h");
			System.out.println("制限速度を"+(this.speed-LIMIT)+"km/hオーバーしています");
}
}
}

Answer 1

3 が入力されるまで繰り返すという言葉通りコーディングすれば良いです。
具体的には int n=sc.nextInt() から sc.close() までの形を次のイメージでしょうか。

int n;
while ((n = sc.nextInt()) != 3) {
  if ( n == 1 ) {
     // 1が入力された時の処理をここへ
  } else if ( n == 2 ) {
     // 2が入力された時の処理をここへ
  }
}
System.out.println("終了します");
sc.close();

if ~ else if のところは switch ~ case での書き換えも考えられますが混乱させないために if 文のままとしてあります。ただ、n が　2 であるかどうかは n が　1 の時は調べる意味がないので、 else if に返させてもらいました。
使われているJavaのバージョンによっては try-with-resource 構文にするなど工夫の余地もありますね。そうしないと、Eclilpse などでは sc に対して警告が出ます(警告:「sc は try-with-resource で管理する必要があります」)。もっとも、sc をクローズすると包含されている System.in がクローズされるので、sc.close() 後は System.in も使えなくなるのでご注意を。

Comments

[nubba]

詳しく教えていただいてありがとうございました！
ifで書いていましたがswitchでも書けるように勉強します！
ちなみにですがironyaさんならこのプログラムを書くならswitchで書きますか？

[ironya]

このコードの規模感なら if 文のままで済ませてしまうかな。

Answer 2

クラス名と同じ名前のメソッドを定義することはできません。

慣習的にクラス名はCamelCaseで命名されるので、
クラス名をそれぞれ SpeedUp, SpeedDown に変えると良いと思います。

そもそも

SpeedUpクラスとSpeedDownクラスを別途設ける必要はあるのでしょうか？
Carクラスがスピードを管理しているなら、速度に関するメソッドは直接持たせてやれば良いのでは。

Comments

[nubba]

クラス分けをやめて、CarクラスにspeedUpとspeedDownを持たせてやってみます！

[LouiS0616]

ああ、質問が追加されていたのですね。
質問を編集しても回答者に通知は無いので、コメントなど入れるようにすると良いでしょう。気付かなかったら当然対応もできませんから。

Answer 3

クラス名と同じ名前を持ったメソッドはコンストラクタとして扱われ、返り値の型を書くことが出来ません。