【Java】メソッドを作成し戻り値を使って条件分岐する【引数と戻り値】

Question

前提・実現したいこと

最近Javaを学び始めた初心者です。ご教授お願い致します。

【問】
変数 int num にシステムから入力値が与えられます。
下記のメソッドを作成してください
メソッドを作成したら、mainにてそのメソッドを呼出し、下記の条件分岐を実装してください。
なお、メソッドの呼び出し時の実引数は、入力値である変数 int num を設定してください。

calculationメソッド
0から数値型の引数までの数値の和を戻り値としてください。

mainメソッドでの条件分岐
戻り値が200以上の場合、「200以上です」と出力。
戻り値が100以上200未満の場合、「100以上200未満です」と出力。
戻り値が100未満の場合、「100未満です」と出力。
戻り値が0以下の場合、「数値が不正です」と出力。

下記のコードで実行して表示されるのは「100未満です　数値が不正です　数値が不正です　数値が不正です」
でした。与えられている数値は「１、１８、２０、－１」です。

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

正しい計算がmainメソッドに戻っていないか、
mainメソッド内の式が間違っているのか

該当のソースコード

import java.util.Scanner;
public class Main {
  public static void main(String[] args) {
    Scanner scan = new Scanner(System.in);
    String text = scan.next();
    int num = Integer.parseInt(text);

    int total = caluculation(num);
  //メソッドの呼び出し
  //caluclarionにtotalに代入

    if(total>=200){
      System.out.print("200以上です");
      //戻り値が200以上の場合、「200以上です」と出力
    }else if(100<=total && total>200){
      System.out.print("100以上200未満です");
      //戻り値が100以上200未満の場合、「100以上200未満です」と出力
    }else if(0<total && total<100){
      System.out.print("100未満です");
      //戻り値が100未満の場合、「100未満です」と出力
    }else if(total<0){
      System.out.print("数値が不正です");
      //戻り値が0以下の場合、「数値が不正です」と出力。
      }
   }
  static int caluculation(int num){
    for(int i=0;i<num;i++){
      num =num+i;
    }
    return num;
  }
}

試したこと

caliculationメソッド内の式をiを1にしてみたり、numを変えてみたりしましたが改善せず。
上の演算式に間違いがあるのか何度も見直しましたが自分自身では気が付きませんでした。
1時間以上も躓いてしまったのでご教授お願い致します。

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

Java、Windows10を使用しております。質問の意味が違っていたら申し訳ございません

Comments

[riko111]

> 正しい計算がmainメソッドに戻っていないか、
> mainメソッド内の式が間違っているのか
テキストエディタ＋コマンドプロンプトの開発環境で上記を確認したいのなら、if文の前に
System.out.println("totla=" + total);
を書いてみればよいのでは？

[退会済みユーザー]

jdbでいいじゃろ→コマンドでのデバッグ

[momon-ga]

> 0から数値型の引数までの数値の和を戻り値としてください。
ですが、引数が負数の場合、戻り値は負数になりますか？
たとえば、引数が-3の場合、０、-1、-2、-3の合計の-6でしょうか？
それとも、単純に0を返せばいいですか？それとも、合計せずに-3を返す？

Answer 1

    for(int i=0;i<num;i++){
      num =num+i;
    }

ここで比較対象(num)を加算しているので終わらない。

    int lim= num;
    for(int i=0;i<lim;i++){
      num =num+i;
    }

とかすれば？

Comments

[nitta_komachi]

回答ありがとうございます。確かにfor文の公式?として間違えておりました…
ご指摘された通り変数にnumをいれ式を作成したところクリア致しました。
本当にありがとうございました。

Answer 2

100<=total && total>200

100以上で200より大きい
になってます。

可読性の問題ですが、比較での条件式で
変数の位置は左辺なら全て左辺、右辺なら全て右辺に揃えた方が間違えにくくなると思います。

Comments

[m.ts10806]

0 の場合はなにもでない気が

[nitta_komachi]

回答ありがとうございます。確かに見直してみると変な指揮を書いておりました…。
他の人にも見せられるような見やすいものを作るという事を念頭にコードを書くよう心掛けます！
あと、仰ったとおり
}else if(0<=total && total<100){
というコードを書いてみて、0の時も表示できるようになりました。ありがとうございます！

Answer 3

①以下と未満の使い分けが正しくない
以下：total <= 数字
未満：total < 数字

②最後に、引数の数字も足す必要がある
for(int i=0;i<num;i++){
ではなく
for(int i=0;i<=num;i++){
だと思います

③引数のnumに手を加えたくない
forの前に新しい変数作ってそこに足していって返してあげた方がいいです

【補足】
if文の最後はelseがベターだと思います
else ifが多い場合はswitch文を使うことを考えてもいいと思います
System.out.printlnだと最後に改行つけてくれます

Comments

[LouiS0616]

比較をswitch文で簡潔に書く方法を知らないのですが、どのようにすれば良いのでしょうか。

[oec]

人によって好みであったり、プロジェクトのルールによって使えない場合もありますが

switch(true){
case x<2:
.
.
.
}

という感じです。

[LouiS0616]

Java だと無理っぽいです。https://wandbox.org/permlink/uUFhTffSMH10buvJ

---
しかし言語に依ってはそのような書き方をする場合もあるのですね。興味深いです。

[maisumakun]

> else ifが多い場合はswitch文を使うことを考えてもいいと思います

Javaのswitchでは、caseには定数式しか与えられません。

https://docs.oracle.com/javase/specs/jls/se8/html/jls-14.html#jls-14.11

[oec]

わ、動作確認してなくてごめんなさい。
そして確認ありがとうございます。Javaではできないんですね。
Javascriptとx++では使ったことがあります。

ちょっと古いですが、こんな記事みつけました。面白いですね。
https://qiita.com/khsk/items/90f07f8bfeeb66cbaf32

[LouiS0616]

JavaScriptやPHPはほとんど触ったことが無いのですが、このようなイディオムが(好みが分かれるにせよ)あるのですね。

goto論争に近いものを感じます。非常に強力で、さらに明確な意味合いを持っている、あるいは持たせることができるけれど、使い方を一つ誤るとバグの温床になるということ。
switch文のフォールスルーを充分活かせる処理なのであれば個人的にはアリかなと思いました。

[nitta_komachi]

回答ありがとうございます。
おかげさまで100以上200未満ですを表示する式がおかしいことに気が付きました！
for文の前に変数を宣言し

int sum=num;
for(int i=0;i<sum;i++){
num =num+i;
}

このような処理を打ち込んだところ解決いたしました！
あと皆様の文を読んでなるほど…と思うことがあり、これからも必死に勉強していきたいと思います。
本当にありがとうございました！

Answer 4

calculationメソッド

0から数値型の引数までの数値の和を戻り値としてください。

  static int caluculation(int num){
    for(int i=0;i<num;i++){
      num =num+i;
    }
    return num;
  }

どう考えてもこれはおかしい。

Answer 5

なかなか難しい問題ですね、これでどうでしょう？

package testJava;

import java.util.Scanner;

public class Main {

	public static void main(String[] args) {
		Scanner scan = new Scanner(System.in);
		String text = scan.next();
		int num = 0;
		scan.close();
		try {
			num = Integer.parseInt(text);
		} catch (NumberFormatException e) {
			System.out.println("入力値が不正です。整数を入力してください。");
			System.exit(-1);
		}

		int total = caluculation(num);

		//System.out.println("デバック用ログ・トータル：" + total);

		//メソッドの呼び出し
		//caluclarionにtotalに代入

		if (total >= 200) {
			System.out.print("200以上です");
			//戻り値が200以上の場合、「200以上です」と出力
		} else if (total >= 100) {
			System.out.print("100以上200未満です");
			//戻り値が100以上200未満の場合、「100以上200未満です」と出力
		} else if (total >= 0) {
			System.out.print("100未満です");
			//戻り値が100未満の場合、「100未満です」と出力
		} else {
			System.out.print("数値が不正です");
			//戻り値が0以下の場合、「数値が不正です」と出力。
		}
	}

	static int caluculation(int num) {
		int returnNum = 0;
		for (int i = 0; i <= Math.abs(num); i++) {
			returnNum += i;
		}
		if (num < 0) {
			return -returnNum;
		}
		return returnNum;
	}

}

Comments

[momon-ga]

> 戻り値が0以下の場合、「数値が不正です」と出力。
おしいなぁ。0の場合は？

[junzi]

レビューありがとうございます、テストが足りていませんでした。。
total > 0で「数値が不正です」ですね！