ループ化の方法

Question

public class Gohkaku {
public static void main(String[] args){
int math = new java.util.Random().nextInt(51);
int japanese = new java.util.Random().nextInt(51);
int sience = new java.util.Random().nextInt(51);
int social = new java.util.Random().nextInt(51);
int[] score = { math,japanese,sience,social };
int sum = score[0] + score[1] + score[2] + score[3];
if( sum >= 100 ){
System.out.println(sum + "なので合格");
} else if ( sum <= 100 ) {
System.out.println(sum + "なので不合格");
}
}
}

この文を合格とするまでループ化させたいのですがどのように変えたらよろしいでしょうか？
whileの条件にsumを入れたかったのですが、sumを入れるためにグローバル変数にすると乱数が再生成されないということで困っています。
まだ入門本を読んでるレベルなのでwhileや、for等の簡単な文でご教示頂けると助かります。

Answer 1

whileの条件をtrue（無限ループ）にし、合格したらbreakでループを抜ける、というのはどうでしょうか。

Comments

[atm_t]

ありがとうございます。
早速試してみましたところ無事できました。
break文の使い道がいまいちイメージ出来ていなかったのでとても為になりました。
また機会がありましたら宜しくお願い致します。

Answer 2

はじめてのStream APIプログラミングです。swordoneさんの回答をアレンジしました。

import java.util.stream.IntStream;

public class IntStream_ {
    public static void main(String[] args) {
        IntStream.generate(() -> 
                    IntStream.generate(
                        () -> (int)(Math.random() * 51)).limit(4).sum()
                    )
                 .peek(s -> 
                    System.out.println(
                        s + ((s >= 100) ? "なので合格" : "なので不合格")
                    )
                 )
                 .anyMatch(x -> x >= 100);       
    }
}

peekとanyMatchを発見するまでが長かった。やっと眠れる。
sipadan2003さん、swordoneさん、ありがとうございました。

###Random.inits()を利用する(追記)
さらに改良しました。

import java.util.Random;
import java.util.stream.IntStream;

public class IntStream_ {
    public static void main(String[] args) {
        IntStream.generate(
            () -> (new Random()).ints(4l, 0, 51)
            .peek( s -> System.out.print( s + " ") )
            .sum()
        )
        .peek(
            s -> System.out.println(
                "+++p : " + s + ((s >= 100) ? " なので合格" : " なので不合格")
            )
        )
        .anyMatch( s -> s >= 100 );       
    }   
}

Answer 3

public class Gohkaku {
    public static void main(String[] args){
        
        int sum = 0;
        
        while(sum <= 100){
        
            int math = new java.util.Random().nextInt(51);
            int japanese = new java.util.Random().nextInt(51);
            int sience = new java.util.Random().nextInt(51);
            int social = new java.util.Random().nextInt(51);
            int[] score = { math,japanese,sience,social };

            sum = score[0] + score[1] + score[2] + score[3];

            if( sum >= 100 ){
                System.out.println(Integer.toString(sum) + "なので合格");
            } else if ( sum <= 100 ) {
                System.out.println(Integer.toString(sum) + "なので不合格"); 
            }
        }
    }
}

こんな感じでどうですかね

Comments

[atm_t]

ありがとうございます。
今確認したところ無事できました。
グローバル変数をwhile文の中で上書きしてしまえばよかったんですね。
勉強になりました。
また機会がありましたら宜しくお願い致します。

[sho_cs]

質問とはズレてしまいますがグローバル変数について誤解されているようなので参考リンクを書いておきます。
Javaには正確にはグローバル変数は存在しません。

http://www.kab-studio.biz/Programing/OOPinJava/11/02.html
http://www.openreference.org/articles/view/686#anchor3

[退会済みユーザー]

while(sum < 100)ですね。バグは本質とは関係ないと思いますが。すみません。

Answer 4

booleanを返すメソッドとして切り出して、whileの継続条件に使う手もあります。

import java.util.stream.IntStream;

public class Q23614 {

	public static void main(String[] args) {
		while(!test());
	}
	
	static boolean test(){
		int score = IntStream.generate(() -> (int)(Math.random() * 51)).limit(4).sum();
		boolean passed = score >= 100;
		System.out.println(score + (passed ? "なので合格" : "なので不合格"));
		return passed;
	}

}

Answer 5

入門中とのことですが、Java 8で導入されたStreamを使ったプログラム案です。

	public static void main(String[] args) {
		boolean passed = false;
		final Random r = new Random();
		while(passed == false){
			final int sum = r.ints(4, 0, 51).sum();
			passed = (sum >= 100);
			if(passed){
				System.out.println(sum + "なので合格");
			}else{
				System.out.println(sum + "なので不合格");
			}
		}
	}

Comments

[退会済みユーザー]

final int sum = r.ints(4, 0, 51).sum(); のfinalは必要ないと思います。（仮にsumをラムダ式の中から参照している場合でも、代入の後、sumを変更しなければ、コンパイラが暗黙finalとみなすので、明示的なfinalは必要ないのですが、sumはラムダ式で使われていないし...）finalにした意図はなんでしょうか？

[退会済みユーザー]

finalはなくても構いません。
個人的に再利用を防止する意味でfinalを積極的に使っているだけです。

[退会済みユーザー]

今、ラムダ式を勉強しているので、もう一つ質問させてください。ストリームの利用でpassed = (sum >= 100);は、filter()を使えますか。使ったら不合格の場合、nullが返されるのでしょうか。あるいは、"なので合格"、"なので不合格"を直接返すとしたら、どうすればよいでしょうか。本題とは異なる質問なので恐縮です。

[退会済みユーザー]

filter()は対象のデータをふるい分けて絞り込むときに使うものなので、
今回のケースでは適さないと考えています。

次のように三項演算子を使う方法はいかがでしょうか？
ーーーー
passed = (sum >= 100);
final String s = passed ? "なので合格" : "なので不合格";
System.out.println(sum + s);
ーーーー

次のようにOptionalを使う方法もありますが、非効率的です。
ーーーー
final String str = Optional.of(sum).map(s -> (s >= 100) ? "なので合格" : "なので不合格").get();
ーーーー

[退会済みユーザー]

ありがとうございました。ストリームAPIを使うと、たぶん外側のwhileも内側に入れられるのだろうと思います。内部でSystem.out.printlnするので、いわゆる純粋ではないけど。

Answer 6

処理を関数にして、その関数の呼び出しを while で繰り返すようにしてみました。

また スコアの生成や合計計算部分は、質問文のままでも動作しますが、すこし書き換えてみました。
（無駄な変数をなくした、科目の数が増えても書き開ける部分がすくなくなるようにした)
乱数の生成部分も他の方の回答の内容を反映させています。

import java.util.Arrays;
import java.util.stream.IntStream;
import java.util.Random;

public class Gohkaku {
  public static void main(String[] args) {
    Random rnd = new Random();
    while(!hantei(rnd)) {}
  }

  static boolean hantei(Random rnd) {
    int[] score = {
      rnd.nextInt(51),  // math
      rnd.nextInt(51),  // japanese
      rnd.nextInt(51),  // sicence
      rnd.nextInt(51),  // social
    };
    // int sum = score[0] + score[1] + score[2] + score[3];
    IntStream intStream = Arrays.stream(score);
    int sum = intStream.sum();

    boolean ans = sum >= 100;
    if (ans) {
      System.out.println(sum + "なので合格");
    } else {
      System.out.println(sum + "なので不合格");
    }
    return ans;
  }
}

実行例

$ javac Gohkaku.java 

$ java Gohkaku
113なので合格

$ java Gohkaku
93なので不合格
91なので不合格
88なので不合格
153なので合格

Answer 7

本題とは関係ないですが、いちいちRandomを生成しているのが非効率的です。
Math.random()を使うか、一つインスタンスを生成しておいて使うべきでしょう。

import java.util.Random;

public class Gohkaku { 
    public static void main(String[] args){ 
        int sum = 0;
        Random random = new Random();
        while(sum < 100)
        int math = random.nextInt(51);
        //あるいはこう書く 以下同じ
        //int math = (int)(Math.random() * 51);
        int japanese = random.nextInt(51); 
        int sience = random.nextInt(51); 
        int social = random.nextInt(51); 
        int[] score = { math,japanese,sience,social }; 
        int sum = score[0] + score[1] + score[2] + score[3]; 
        if( sum >= 100 ){ 
            System.out.println(sum + "なので合格"); 
        } else if ( sum <= 100 ) { 
            System.out.println(sum + "なので不合格");  
        } 
    } 
}