elseへのifがありません。

Question

２つの値を入力して、大小比較により大きい値と小さい値を分けて表示し、同値の場合には別で表示を出すというプログラムです。

import java.util.Scanner;

class P314{
public static void main(String［］ args){
Scanner stdIn = new Scanner(System.in);

System.out.print("整数a:");
int a = stdIn.nextInt();
System.out.print("整数b:");
int b = stdIn.nextInt();

if(a==b){
System.out.print("２つの値は同じ値です。");
}else {
 if(a＞b){
 int max,min;
 max = a;
 min = b;

 else 
 max = b;
 min = a;

 System.out.print("小さい方の値は"+min+"で す。")
 System.out.print("大きい方の値は"+max+"で す。");
}}}
}}

3-14.java:22:エラー:'else'への'if'がありません
else{
^

エラーの原因がよくわからないです。
まだifelseの理解が浅いがゆえのことではありますがご助言お願いします。

Answer 1

elseに対応するifは、elseと同じ階層で、直前にないといけません。

Comments

[tak]

もっとも近いifとelseが関連付けされるという認識で良いでしょうか？
階層というのはよくわからないです…

[otn]

同じ階層というのは、一方が { } の外で、一方が { } の中とかではだめだと言う事です。

[tak]

回答ありがとうございます。
なるほど、覚えておきます。

Answer 2

わかり易いようにきちんと字下げする手間は惜しまないこと。

以下のようにちょっとしたかゆいところに手が届く機能があるエディタもあります。
8行目の } にカーソルがあるとき、対応する5行目の { が強調表示されるから、{ } のペアがわかる。

Comments

[tak]

アドバイスありがとうございます。
やっていて自分でも見にくいと感じたので
eclipseをインストールしました。
見やすいコードを書くように努めます。

Answer 3

{}がぐちゃぐちゃです。
eclipsやVSCodeをインストールして、コードの自動整形機能を活用するとよいですよ。

Comments

[tak]

アドバイスありがとうございます。
eclipseインストールしたので活用していきたいと思います。

Answer 4

中括弧'{'の対応が取れていませんｄ＾＾

 if(a＞b){
 int max,min;
 max = a;
 min = b;

 else 
 max = b;
 min = a;

↓↓↓↓

 int max,min;
 if(a＞b){
   max = a;
   min = b;
 }else{ 
   max = b;
   min = a;
 }

「追記」
ソースの中に全角文字が含まれているようですが・・・・

Comments

[tak]

申し訳ないです、携帯で直打ちしていたので
全角だと気づきませんでした…

Answer 5

elseの後ろに二行以上書くときは{}を入れてください。

import java.util.Scanner;

class P314{
public static void main(String［］ args){
Scanner stdIn = new Scanner(System.in);

System.out.print("整数a:");
int a = stdIn.nextInt();
System.out.print("整数b:");
int b = stdIn.nextInt();

if(a==b){
System.out.print("２つの値は同じ値です。");
}else {
 if(a＞b){
 int max,min;
 max = a;
 min = b;

 }else{
 max = b;
 min = a;
 }
 System.out.print("小さい方の値は"+min+"で す。")
 System.out.print("大きい方の値は"+max+"で す。");
}}}
}}

Comments

[tak]

回答ありがとうございます。
括弧の対応付けを今一度見直し、上手くいきました。

Answer 6

 else 

これだとelseになってません

if(条件式){

}else if(条件式){

}else{

}

ご自身でも最初に　}else {　と書かれてますよね。

また、{に対する}の対応が非常にわかりづらいコードになっています。
コードフォーマットなど利用して、インデントをきちんとつけれられたほうが良いかと。

Comments

[tak]

回答ありがとうございます。
メモ帳を使ってコードを打っていたため
自分でも括弧の対応がよくわからないときがあったので、別のアプリを使ってコードを作成するようにします。

[m.ts10806]

メモ帳でJavaは良くないですね。
せめて構文チェックが自動で行われるIDE(Eclipseなど)を利用しましょう。
プログラムを組むところ以外で悩んでるのは勿体ないです