Java if文を多様しないで組みたい

Question

現在Javaにて名前と科目を入力したら登録した点数がでるプログラムを組んでいます。

エラーは無いのですがif文を多様してしまっているのでif文をあまり使わずに作りたいです。
アドバイスいただけるとありがたいです。

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class test33 {

        public static void main(String[] args) throws IOException {

                String name[] = { "田中", "加藤", "森田", "北国" };
String test[] = { "国語", "英語", "理科", "社会" };

                int point[][] = { { 100, 80, 50, 30 }, { 30, 40, 70, 90 }, {
50, 90, 100, 0 }, { 0, 10, 70, 30 } };

                BufferedReader br = new BufferedReader(new
InputStreamReader(System.in));

                System.out.println("名前を入力してください");
String strx = br.readLine();

                System.out.println("科目を入力してください");
String stry = br.readLine();

                System.out.println("名前:"+ name);
System.out.print(test);
System.out.print(point);

/*              if (strx.equals(name[0])) {
System.out.println("名前:" + name[0]);
if (stry.equals(test[0])) {
System.out.println(test[0] + ":" +
point[0][0]);
} else if (stry.equals(test[1])) {
System.out.println(test[1] + ":" +
point[0][1]);
} else if (stry.equals(test[2])) {
System.out.println(test[2] + ":" +
point[0][2]);
} else if (stry.equals(test[3])) {
System.out.println(test[3] + ":" +
point[0][3]);
}
}

                if (strx.equals(name[1])) {
System.out.println("名前:" + name[1]);
if (stry.equals(test[0])) {
System.out.println(test[0] + ":" +
point[1][0]);
} else if (stry.equals(test[1])) {
System.out.println(test[1] + ":" +
point[1][1]);
} else if (stry.equals(test[2])) {
System.out.println(test[2] + ":" +
point[1][2]);
} else if (stry.equals(test[3])) {
System.out.println(test[3] + ":" +
point[1][3]);
}
}

                if (strx.equals(name[2])) {
System.out.println("名前:" + name[2]);
if (stry.equals(test[0])) {
System.out.println(test[0] + ":" +
point[2][0]);
} else if (stry.equals(test[1])) {
System.out.println(test[1] + ":" +
point[2][1]);
} else if (stry.equals(test[2])) {
System.out.println(test[2] + ":" +
point[2][2]);
} else if (stry.equals(test[3])) {
System.out.println(test[3] + ":" +
point[2][3]);
}
}

                if (strx.equals(name[3])) {
System.out.println("名前:" + name[3]);
if (stry.equals(test[0])) {
System.out.println(test[0] + ":" +
point[3][0]);
} else if (stry.equals(test[1])) {
System.out.println(test[1] + ":" +
point[3][1]);
} else if (stry.equals(test[2])) {
System.out.println(test[2] + ":" +
point[3][2]);
} else if (stry.equals(test[3])) {
System.out.println(test[3] + ":" +
point[3][3]);
}
}*/
}

}

Comments

[otftrough]

コードを書くときに、コード全体を「```java」と「```」で囲むと、より見やすくなります。

Answer 1

if文を一切使いたくないのですね、わかります。

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.stream.IntStream;

public class test33 {
  public static void main(String[] args) throws IOException {
    String name[] = { "田中", "加藤", "森田", "北国" };
    String test[] = { "国語", "英語", "理科", "社会" };

    int point[][] = {
      { 100, 80, 50, 30 },
      { 30, 40, 70, 90 },
      { 50, 90, 100, 0 },
      { 0, 10, 70, 30 } };

    BufferedReader br = new BufferedReader(new InputStreamReader(System.in));

    System.out.println("名前を入力してください");
    String strx = br.readLine();

    System.out.println("科目を入力してください");
    String stry = br.readLine();

    IntStream.range(0, name.length)
      .filter(i -> strx.equals(name[i]))
      .forEach(i -> {
        System.out.println("名前:" + name[i]);
        IntStream.range(0, test.length)
          .filter(j -> stry.equals(test[j]))
          .forEach(j -> {
            System.out.println(test[j] + ":" + point[i][j]);
          });
      });
  }
}

Answer 2

public static void main(String[] args) throws Exception {

	String name[] = { "田中", "加藤", "森田", "北国" };
	String test[] = { "国語", "英語", "理科", "社会" };

	int point[][] = { { 100, 80, 50, 30 }, { 30, 40, 70, 90 }, { 50, 90, 100, 0 }, { 0, 10, 70, 30 } };

	BufferedReader br = new BufferedReader(new InputStreamReader(System.in));

	System.out.println("名前を入力してください");
	String strx = br.readLine();

	System.out.println("科目を入力してください");
	String stry = br.readLine();

	int nameIndex = index(name, strx);
	int testIndex = index(test, stry);

	System.out.println("名前:" + strx);
	System.out.println(stry + ":" + point[nameIndex][testIndex]);
}

private static int index(String[] array, String value) {
	int index = -1;
	for (int i = 0; i < array.length; i++) {
		if (value.equals(array[i])) {
			index = i;
			break;
		}
	}
	if (index == -1) {
		throw new IllegalArgumentException(value);
	}

	return index;
}

Comments

[退会済みユーザー]

この方法は考えられなかったので実践してみます。ありがとうございます。

[swordone]

indexという変数を使わずに、breakのところでreturn iすればもっと簡単になります。

[退会済みユーザー]

そうですね、あまり変数を増やすと自分でもごっちゃになることもあるのでその辺りは勉強しながら工夫できるようにしていきたいです。

[root_jp]

IllegalArgumentException 投げたいので、そこで return していないだけですね。

Answer 3

与えられたデータをそのままつかうより、処理がしやすいようなデータを構築すると良さそうです。

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.HashMap;

public class Test33 {

    public static void main(String[] args) throws IOException {

        String name[] = { "田中", "加藤", "森田", "北国" };
        String test[] = { "国語", "英語", "理科", "社会" };
        int point[][] = { { 100, 80, 50, 30 },
                          { 30, 40, 70, 90 },
                          { 50, 90, 100, 0 },
                          { 0, 10, 70, 30 } };

        // 処理しやすくする為に連想配列を構築する
        HashMap<String, Integer> map = new HashMap<String, Integer>();
        for (int n = 0; n < name.length; n++) {
            for (int t = 0; t < test.length; t++) {
                map.put(name[n] + " " + test[t], point[n][t]);
            }
        }

        BufferedReader br = new BufferedReader(new InputStreamReader(System.in));

        System.out.println("名前を入力してください");
        String strx = br.readLine();
        System.out.println("科目を入力してください");
        String stry = br.readLine();

        System.out.println(map.get(strx + " " + stry));
    }
}

Answer 4

Java、十数年ぶりに書いてみました。
コードを書いているうちに、他のいい回答が出てきちゃったので、解説とかはもういいかな、と。

import java.io.BufferedReader; 
import java.io.IOException; 
import java.io.InputStreamReader;
import java.util.HashMap;

public class Main {
    protected class Student {
        public String name;
        public HashMap<String,Integer> points = new HashMap<String,Integer>();
        public Student(String name) {
            this.name = name;
        }
    }
    protected HashMap<String,Student> students;
    
    protected Main() {
       this.students = new HashMap<String,Student>();
    }
    
    protected Main setup(String exams[]) {
        for (String exam: exams) {
            String ar[] = exam.split(" ", 0);
            String studentName = ar[0];
            for (String pair: ar[1].split(",", 0)) {
                String val[] = pair.split(":", 0);
                this.addPoint(studentName, val[0], Integer.parseInt(val[1]));
            }
        }
        return this;
    }

    protected Main addPoint(String studentName, String subject, int point) {
        Student student = this.students.get(studentName);
        if (student == null) {
            student = new Student(studentName);
            this.students.put(studentName, student);
        }
        student.points.put(subject, point);
        return this;
    }
    protected int get(String studentName, String subject) throws Exception {
        Student student = this.students.get(studentName);
        if (student == null) {
            throw new Exception("「"+ studentName + "」ってゆー生徒はいねーよ"); // 適当に
        }
        Integer point = student.points.get(subject);
        if (point == null) {
            throw new Exception("「" + subject + "」なんちゅー教科はないぞよ"); // 適当に
        }
        return point;
    }
    
    protected static void test(Main m, String studentName, String subject) {
        String message;
        try {
            message = "名前:" + studentName + ",教科:" + subject + "=" + m.get(studentName, subject);
        } catch (Exception e) {
            message = e.getMessage();
        }
        System.out.println(message);
    }
    public static void main(String[] args) throws IOException {
        String exams[] = {
            "田中 国語:100,英語:80,理科:50,社会:30",
            "加藤 国語:30,英語:40,理科:70,社会:90",
            "森田 国語:50,英語:90,理科:100,社会:0",
            "北国 国語:0,英語:10,理科:70,社会:30"
        };
        Main m = new Main().setup(exams);
        test(m, "田中", "国語");
        test(m, "森田", "仏語");
        test(m, "山下", "英語");
   }
}

Comments

[退会済みユーザー]

ありがとうございます。
動作確認はしてますが、配列要素以外を打ち込まれた場合の対策はしてなかったので、参考にさせていただきます。

Answer 5

すでに解決済みではありますが、、

ロジックが複雑になりすぎると感じたときは、そのロジックで扱っているデータの構造を見直してみることをお勧めします。

例えば、名前と点数を以下のような構造の2次元のMapオブジェクトに格納してやると、簡潔に書けます。

[
  田中 => [
    国語 => 100,
    英語 => 80,
    理科 => 50,
    社会 => 30
  ],

  加藤 => [
    国語 => 30,
    英語 => 40,
    理科 => 70,
    社会 => 90
  ],

  ...
]

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.HashMap;
import java.util.Map;

public class test33 {

    public static void main(String[] args) throws IOException {

        String name[] = { "田中", "加藤", "森田", "北国" };
        String test[] = { "国語", "英語", "理科", "社会" };

        int point[][] = { { 100, 80, 50, 30 }, { 30, 40, 70, 90 }, { 50, 90, 100, 0 },
                { 0, 10, 70, 30 } };

        // 変換後の名前と点数を格納する2次元Mapオブジェクト。
        Map<String, Map<String, Integer>> scores = new HashMap<String, Map<String, Integer>>();

        // データを変換するロジック。
        for (int i = 0; i < name.length; i++) {
            Map<String, Integer> score = new HashMap<String, Integer>();
            for (int j = 0; j < test.length; j++) {
                score.put(test[j], point[i][j]);
            }
            scores.put(name[i], score);
        }

        BufferedReader br = new BufferedReader(new InputStreamReader(System.in));

        System.out.println("名前を入力してください");
        String strx = br.readLine();

        System.out.println("科目を入力してください");
        String stry = br.readLine();

        System.out.println(scores.get(strx).get(stry));
    }
}

ただし、上のプログラムは、存在しない名前（例えば"鈴木"とか）を入力するとNullPointerExceptionで落ちます。
適切にNullチェックを入れてください。

Comments

[退会済みユーザー]

回答ありがとうございます。
次はマップデータで同じものを作る予定でしたので詰まったときに参考にさせていただきます。

Answer 6

swordone さん、otftrough さんとあまり変わりませんが。。。

if 文の外側に for 文を書くことで、これまで配列に数字を直接指定していたところを、
i や j などの変数にして、繰り返し確認することが出来ます。

for(int i = 0; i < name.length; i++) {

  if (strx.equals(name[i]) {
    System.out.println("名前:" + name[i]);
    
    for(int j = 0; j < test.length; j++) {
    
      if (stry.equals(test[j])) {
        System.out.println(test[j] + ":" + point[i][j]);
        return;
      }
    
    }
  }
}

動作未確認なので、参考までに・・・。

Comments

[退会済みユーザー]

自分が考えていたのと似ていて分かりやすかったためベストにしました、ありがとうございます。

Answer 7

swordoneさんのコードを使った例です。参考に。

int n = 0;
for(; n < name.length; n++){
    if(strx.equals(name[n]))
        break;
}

if(n < name.length){

    System.out.println("名前:" + name[n]);
    int t = 0;
    for(; t < test.length; t++){
        if(stry.equals(test[t]))
            break;
    }

    if(t < test.length)
        System.out.println(test[t] + ":" + point[n][t] + "点");
    else
        System.out.println("not found.");

} else
    System.out.println("not found.");

動作確認済

Comments

[退会済みユーザー]

具体的にソースまで書いていただきありがとうございます。

Answer 8

各配列内でループで一致する要素を探せばいいでしょう。

int i = 0;
for(; i < name.length; i++){
    if(strx.equals(name[i]))
        break;
}

あるいは各配列をArrays.asListでList型に変換して、indexOfメソッドを使えばループも不要です。

Comments

[退会済みユーザー]

ありがとうございます。
ループ処理で行ったら簡単にできました。