コンパイルエラーの修正について

＃java 　#コンパイルエラー
java言語のBMI計算の下記のコードから間違いを探して修正しなさいというプログラミングについての課題で

import java.util.Scanner;
public class Program {

public void input (){
Scanner scan = new Scanner (System.in);
}
public void compute(){
}

public void output (){
double height = 1.70;
double height = 60.0;
System.out.println (weight / (height * height;
}

public static void main (String[] args){
Program p = new Program ();
p.input();
p.compute();
p.output();
}
}

というコードの13行目のSystem.out.println (weight / (height * height;を
System.out.println (weight / (height * height));に修正して

import java.util.Scanner;
public class Program {

public void input (){
Scanner scan = new Scanner (System.in);
}
public void compute(){
}

public void output (){
double height = 1.70;
double height = 60.0;
System.out.println (weight / (height * height));
}

public static void main (String[] args){
Program p = new Program ();
p.input();
p.compute();
p.output();
}
}

コメント
Program.java:12: error: variable height is already defined in method output()
double height = 60.0;
^
Program.java:13: error: cannot find symbol
System.out.println (weight / (height * height));
^
symbol: variable weight
location: class Program
2 errors

と、コメントが出ました
上記の修正であっているはずなのですがコンパイルエラーだと表示されます
これはどう修正すればいいのでしょうか

ps_aux_grep

2023/03/22 04:50

エラー文 error: variable height is already defined in method output() を翻訳/理解するだけで修正可能なエラーです

maisumakun

2023/03/22 04:51

> 上記の修正であっているはずなのですがどうしてそう考えたのですか？

otn

2023/03/22 05:06 編集

初心者にありがちですが、「あってるはずなのにエラーになる」「あってるはずなのに結果がおかしい」という発想がおかしいです。「エラーになるので、どこか間違っている」「結果がおかしいので、どこか間違っている」という方向で物を考えないと自力解決は出来ません。短いプログラムなので、あと1-2回見直せば間違いが見つかるでしょう。（今回は関係ないですが、操作の間違いもありますね。見ているファイル（質問に掲載したファイル）と実行やコンパイルしたファイルが違う状態で質問している人も何人かいます。そういう風にプログラムの外側かも知れませんが、必ずどこか間違えています）

sis

2023/03/22 05:10

maisumakunさん上記の修正であっていると考えたのは一番目のコードに Program.java:12: error: ')' expected System.out.println(weight / (height * height; 　　　　　　 ^ と、コメントがついたので間違っているポイントはその一箇所だけだと考えました otnさんはい、わかりました。

TakaiY

2023/03/22 06:37 編集

プログラムのエラーってのは1つとは限りまｓん。また、コンパイルしたときに全てのエラーが検出されるわけでもありません。直してコンパイルすると別の問題がみつかる場合もあるということです。ヒットを1つエラーメッセージには重要な情報が含まれています。英語だからと言って敬遠せずに読みましょう。 google翻訳とかでも充分役にたつはずです。 Program.java:12: error: variable height is already defined in method output() 訳すと、「Program.javaの12行目。エラー: 変数heightはメソッドoutput()の中ですでに定義済みです。」です。既に定義済みということは、2回以上定義しちゃっているということですね。

sis

2023/03/22 05:24

TakaiYさんおっしゃるとうり敬遠して読んでいませんでした翻訳かけて読みます