1からaまでの積を求めるプログラムがうまく動かない

Question

Javaで1から入力した数までの積を求めるプログラムを作っています。
しかし、うまく動きません。
エラーは出ていないのですが、計算結果が異なってしまいます。
例えば3を入力すると2、5を入力すると24が出ます。
この結果からおそらく(a-1)までの積を表示しているのではと推測はしているのですが…

どの点のせいなのかご指摘お願いいたします。

import java.util.Scanner;

public class Chap2 {
	public static void main(String [] args) {
		Scanner stdIn = new Scanner(System.in);

		int a;

		do {
			System.out.println("正の整数を入力");
			a = stdIn.nextInt();

		if (a < 1) {
			System.out.println("もう一度");
			}
		} while (a < 1);

		int time = 1;
		int i = 1;

		do {
			time *= i;
			i++;

		}while (a != i);


		System.out.println("かけると" + time +"です");

	}
}

Answer 1

この結果からおそらく(a-1)までの積を表示しているのではと推測はしているのですが…

そこまで分かっているなら、プログラムの実行を追えばいいと思うのですが。

かけ算するループのなかで、iを増やしてaと等しくなったときにループを抜けるので、そのiはかけ算の対象になっていません。
修正の一例としては、

int i = 0;

do {
    i++;
    time *= i;
}while (a != i);

Comments

[Tazusa]

int i =0;　とするのは思いつきませんでした。
みなさんからご指摘いただいて、
}while (a != i);　ではなく、　}while (a != (i -1));　と修正していました。
回答ありがとうございます。

Answer 2

例えば3を入力した場合、
0. time *= iによりtimeは1
0. iをインクリメントして2
0. whileは2 != 3なので継続
0. time *= iによりtimeは2
0. iをインクリメントして3
0. whileは3 != 3が偽のため抜ける

このため、最後の3が掛け算できません。

Comments

[Tazusa]

たしかに順を追って考えればおっしゃる通りでした。わかりやすく示していただきありがとうございました。

Answer 3

time *= i;
i++;

順番のせい

Comments

[Tazusa]

ありがとうございます！

Answer 4

繰り返し処理をする方法はいろいろあります。
"java 繰り返し" で google 検索してみてください。

以下のコードは、for や range をつかって書いています。

import java.util.Scanner;
import java.util.stream.IntStream;

public class Chap2 {
    public static void main(String[] args) {
        Scanner stdIn = new Scanner(System.in);
        int a;
        do {
            System.out.println("正の整数を入力");
            a = stdIn.nextInt();
            if (a < 1) {
                System.out.println("もう一度");
            }
        } while (a < 1);

        int time = 1;
        for (int i = 1; i <= a; i++) {
            time *= i;
        }
        System.out.println("かけると" + time + "です");

        time = IntStream.rangeClosed(1, a).reduce(1, (value1, value2) -> {
            return value1 * value2;
        });
        System.out.println("かけると" + time + "です");
    }
}

実行例

参考

• 従来の for ループに代わる関数型の手法

https://www.ibm.com/developerworks/jp/java/library/j-java8idioms3/index.html

Comments

[Tazusa]

forやrangeを使う方法もあるのですね。参考にさせていただきます。詳しく説明してくださり、ありがとうございます。

Answer 5

数が大きい場合はオーバーフローしやすい計算です。

小さい数字のうちはうまくいくはずですが、どうでしょう。

Comments

[Tazusa]

すみません、具体例が抜けていました。例えば3を入力すると2、4を入力すると6、5を入力すると24が出ます。この結果からおそらく(a-1)までの積を表示しているのではと予想しています。
3のような小さい数でもうまく計算できていない原因がわからないです…