javaの標準入力された値をその後の入力された値と比べて結果を返すプログラムがうまく作れません。

Question

前提・実現したいこと

標準入力にて以下のように
5 1
3
1 0
5 1
5 3

と言った形で入力した際に
はじめに入力された１行目の２つの数字5 1を軸に
それ以降の
1 0
5 1
5 2
と比較しもし１行目の左の5より小さい場合は負け　大きい場合はは勝ちと表示しさらに、５行目のように１つ目の数字がどちらも同じ場合２つ目の数字で同じよう比較して小さい場合負け　大きい場合勝ちと表示するプログラムを作成しております。

条件
１つ目と２つ目をどちらも同じ数値を入力することはできません。（例　1 1など）
１つ目と２つ目の数字が比較対象である１行目と全く同じ場合はエラーと表示します。（上の例ですと　5 1）

この場合ですと

1 0
負け
5 1
エラー
5 2
勝ち

と結果を返したいです。

因みに２行目の３は　比較するそれよりも下の行数です。今回が３個比較対象があるため３と入力されています。これを活用するべきなのでしょうが現状分からずremoveで削除しております。

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

考え方がそもそも間違っているようですが、ほかに思いつかないため強行して見ましたが、やはりうまく行きませんでした。ただ他のやり方が思い浮かばないためこの方法で行えるように試行錯誤している状況です。私の場合はListを使用しております。

該当のソースコード

未完成となりますが、

        List<String> list = new ArrayList<>();
        try(Scanner sc = new Scanner(System.in)) {
            while(sc.hasNext()) {
                list.add(sc.next());
            }
        }
        
        int a = Integer.parseInt(list.get(0));
        System.out.println(a);
        
        int b = Integer.parseInt(list.get(1));
        System.out.println(b);
        
        list.remove(0);
        list.remove(0);
        
        //数値　入力個数
        int c = Integer.parseInt(list.get(2));
    
        //比べるのに不要部分
        list.remove(0);
        
       
        ArrayList<Integer> array = new ArrayList<Integer>();
        
        for(int i = 0; i < list.size(); i++) {
            Integer value = Integer.parseInt(list.get(i));
            list.remove(i);
            array.add(value);
        }

となっており現状抱えている問題としては、比較していく３行目以降の数値をどのように１行目の数値と比較するかという部分です。

試したこと

現状では　まず比較対象となる１行目を　１つ目の数値はa 2つ目の数値はbとして変数に格納し削除しております。
そして比較個数を一応cに格納し削除しております（現状使えておりません）

その後新しくIntegerリストを作りそこに比較していく３行目以降を格納しているのですが、これで結局どうやって比較を行い勝ち負け判定をするかという部分で完全にお手上げになってしまいました。

補足情報（FW/ツールのバージョンなど）

正直はじめから間違えている可能性が高いのでコードがガラリと変わってしまっても大丈夫なのでご教示いただけると助かります。

Comments

[LouiS0616]

三行目以降入力される各『数字』は、必ず一桁なのでしょうか？具体的には、『10 3』みたいな入力もあり得るのですか？

[uonitarou]

２桁まであり、さらにその数字にダブりはございません。こちらの説明不足で申し訳ありません。

Answer 1

すごく泥臭いですが、こういうやり方もアリじゃないかと思います。

import java.util.*;

class Main {
    static int convertToInt(String src) {
        String[] nums = src.split(" ");
        // return 100 * Integer.parseInt(nums[0]) + Integer.parseInt(nums[1]); でも可
        return Integer.parseInt(
            String.format("%s%02d", nums[0], Integer.parseInt(nums[1]))
        );
    }
    
    public static void main(String[] args) {
        try(Scanner sc = new Scanner(System.in)) {
            int base = convertToInt(sc.nextLine());
            
            int len = Integer.parseInt(sc.nextLine());
            for(int i = 0; i < len; ++i) {
                int input = convertToInt(sc.nextLine());
                if(base > input) {
                    System.out.println("負け");
                }
                else if(base < input) {
                    System.out.println("勝ち");
                }
                else {
                    System.out.println("エラー");
                }
            }
        }
    }
}

入力例

5 1
6
1 0
6 2
5 0
5 1
15 3
3 35

実行結果 Wandbox

負け
勝ち
負け
エラー
勝ち
負け

Comments

[uonitarou]

迅速なご回答ありがとうございます。splitで分けて〜といった所が私が最初に試し挫折したやり方に少し似ておりました。（笑）この書き方もどうやればうまくいくのか気になっていたのでとても勉強になります。
本当にありがとうございます。

Answer 2

※回答ではありませんが、問題についていくつか質問したうえで回答したと思います。
1.すべてデータは1桁ですか?
2.入力で最初の数字２つと　後で比べる２つの数字がそれぞれ等しいときの判定はいらないですか?
(引き分けなど)
特に1番についてが知りたいです。
理由は、すべて１桁の時、スペースなしの入力で簡単なプログラムが作れるからです。

以上　よろしくお願いします。

<追記>
いくつか説明させてください。
1.入力の２行目で比較したいデータの個数を入力するのでListを使わなくても大丈夫です。
データを格納したい配列の要素数がわかるので(下のソースではn)その変数を使って宣言しましょう!
2.配列にそれぞれデータを格納することで　後は基本に沿ってif文で評価するだけで済みます。
3.最初の質問で「データはすべて１桁ですか?」と質問しましたが　
データがすべて１桁ですと　もっと簡単にプログラムが書けるからです。
例えば　1 3と　入力したとき　1 3を 13と2桁の数字に置き換えることで、
簡単に比較ができます。

以上で説明を終わりたいと思います。
以下　ソースコードです。

import java.util.Scanner;

public class Main {

    public static void main(String[] args) {
        
        Scanner sc = new Scanner(System.in);
        String line = sc.nextLine();
        String []resultArray = line.split(" ");
        int []sample = new int[2];
        String line2 = sc.nextLine();
        int n = Integer.parseInt(line2);
        int [][]data = new int[n][2];
        int flag = 0; /*勝ったら1になる*/
        for(int i = 0; i < 2; i++){
            sample[i] = Integer.parseInt(resultArray[i]);
        }
        
        for(int i = 0; i < n; i++){
            String line3 = sc.nextLine();
            String []resultArray2 = line3.split(" ");
            for(int j = 0; j < 2; j++){
                data[i][j] = Integer.parseInt(resultArray2[j]);
            }
            
        }
        for(int i = 0; i < n; i++){
            flag = 0;
           if(data[i][0] > sample[0]){
               flag = 1;
           }
           else if(data[i][0] == sample[0]){
               if(data[i][1] > sample[1]){
                   flag = 1;
               }
               else if(data[i][1] == sample[1]){
                   flag = 2;
               }
               else{
                   flag = 0;
               }
           }
           else{
               flag = 0;
           }
            switch (flag) {
                case 0:
                    System.out.println(data[i][0] + " " + data[i][1] + ":負け");
                    break;
                case 1:
                    System.out.println(data[i][0] + " " + data[i][1] + ":勝ち");
                    break;
                default:
                    System.out.println(data[i][0] + " " + data[i][1] + ":エラー");
                    break;
            }
        }
    }
        
  
}

Comments

[uonitarou]

書き忘れがあり申し訳ありませんでした。

1.すべてデータは1桁ですか?
データは２桁までですので９９までとなります。

2.入力で最初の数字２つと　後で比べる２つの数字がそれぞれ等しいときの判定はいらないですか?
こちらのミスで同じ数字は入力できないようにする予定でした。そのため等しい場合がなくその判定が不要のようです。失礼しました。こちらに関して質問文章の修正をさせていただきます。

[退会済みユーザー]

分かりました。今からJavaでプログラムを構築いたしますので少々お待ちください。

[退会済みユーザー]

上記の回答の欄にソースコードを載せました。どこかわからないところ　または
エラーがありましたらまたご連絡ください。

[uonitarou]

とても素早いご対応本当にありがとうございます。
コードは一度動かしてみて動きましたのであとは１行ずつ何を行ってるかを理解し改良して問題に取り組んでいこうと思います。（問題通りですと勉強にならないと思い数値の条件等を変えて質問させていただいておりましたので）私の頭では時間がかかると思いましたので先にお礼を書かせていただきました。
この度は本当にありがとうございました。

[退会済みユーザー]

質問がありましたら　いつでもどうぞ!!

[uonitarou]

現在Javaを触り始めて１ヶ月ほどなのですが詰まることが多く、地道に少しずつ進めているのでとても心強いです。また今回のようなどうしてもわからない場合はこちらで質問させて頂くかもしれません。
よろしくお願い致します。

[LouiS0616]

@Stars1024 さん
ローカル変数が多いことが非常に気になります。ローカル変数はいわば『メモ』の働きをしますから、あまりに多いと本当に重要な情報が埋もれてしまうのです。

ローカル変数を減らすためには、メソッドチェーンやそれに準ずるもの(一時変数の積極的利用)、適切なサブルーチン化などが考えられます。
特にフラグを必要とする処理は、アーリーリターンを駆使すれば簡潔に書ける場合が多いです。

また、変数をスコープの頭でまとめて宣言するのも一般にはアンチパターンとして知られますので、避けておいた方が良いかと思います。

---
@uonitarou さん
ひと月でここまで書けるのでしたら、いいペースで学習できていそうですね。
質問本文も非常に丁寧かつ的確でしたし、順調に伸びていきそうな気がします。頑張ってください。

[退会済みユーザー]

「変数をスコープの頭でまとめて宣言するのも一般にはアンチパターンとして知られます」について
ある変数を使おうと考えたときその行に変数を宣言するという意味でしょうか?

[LouiS0616]

新たな質問を拝見させていただきました。https://teratail.com/questions/120426
有益な回答が多くみられ、私自身非常に勉強になりました。

---
変数をスコープ先頭で宣言『しなければならなかった』のはC99までです。
先頭で宣言しなくても良い以上、よりメリットが大きい方法を選ぶべきかと思います。

また『メモ』に例えてみます。
本にメモを貼る際、付箋を使ってどの箇所への言及かわかるようにしませんか？
ページ数が少ないリーフレットならば、最初にまとめて貼った方が見通しがつくかもしれません。
しかし、ある程度ページ数が増えてくると、気にかけるべきことが増えすぎて、集中して本を読むことが出来ません。

---
『スコープの広さ』と『ネストの深さ』はしばしば『精神的スタック』だと言われます。
これに注意することでコードは綺麗に、分かりやすく、デバッグしやすくなりますよ。

[退会済みユーザー]

回答ありがとうございます。私が最初に勉強したのはJavaではなくCだったので
先頭で宣言する癖がJavaにも響いたと思います。

[LouiS0616]

学校というのは得てして専門外の教授がプログラミング言語の講義を持っていることもあり、情報が古かったり正確でなかったりしますので、それを補う調査は自力でする必要があります。

実は、『メソッドチェーン』とか『アーリーリターン』とか、意識して横文字を使ったコメントをしました。まずはとにかく調べてみてほしかったからです。
モダンなコードに積極的に触れてみてください。学ぶものがあるはずです。

Answer 3

◇謝辞
LouiS0616さんの回答を参考にさせて頂きました。

以下は**※11 0←のような入力はないと仮定した回答です。**

1. 区切り文字のスペースをreplace後にInteger.parseInt。
2. 判定はJudge関数を作成して比較はInteger.compareを使うように

import java.util.Scanner;

public class A119051 {

	public static void main(String[] args) {
		try (Scanner sc = new Scanner(System.in)) {
			int base = Integer.parseInt(sc.nextLine().replace(" ", ""));
			int length = Integer.parseInt(sc.nextLine().replace(" ", ""));
			for (int i = 0; i < length; ++i) {
				int input = Integer.parseInt(sc.nextLine().replace(" ", ""));
				Judge(base, input);
			}
		}
	}

	private static void Judge(int x, int y) {
		int copareted = Integer.compare(x, y);
		// Swinch文を使った判定
//		 switch (copareted) {
//		 case 0:
//		 System.out.println("エラー");
//		 break;
//		 case -1:
//		 System.out.println("勝ち");
//		 break;
//		 case 1:
//		 System.out.println("負け");
//		 break;
//		 }
		// ◇Integer.compareのAPI仕様に合わせる形の判定
		if (copareted == 0) {
			// 最初に同じ値どうかを判定
			System.out.println("エラー");
		}
		if (copareted < 0) {
			System.out.println("勝ち");
		} else {
			System.out.println("負け");
		}
	}
}