文字列から数値を返す方法を教えてください

Question

前提・実現したいこと

文字列から数値に返す

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

コンパイルエラーはなし

該当のソースコード

//BufferedReaderを使って標準入力から文字列を受け取り、
//int型数値を返す inputIntNumberメソッドを作成してください。
//例外処理は inputIntNumberメソッド内で行い、
//メインメソッドは inputIntNumberメソッドを呼び出して、正の整数を表示するだけの機能を持ちます。

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

public class InputCheck{
	public static void main(String[] age){
		int num = 0;
		inputIntNumber();
		System.out.println("正の整数は" + num);
	}
	public static void inputIntNumber(){
		InputStreamReader isr = new InputStreamReader(System.in);
		BufferedReader reader = new BufferedReader(isr);
		System.out.println("文字列を入力してください");
			try{
				String line = reader.readLine();
				int num = Integer.parseInt(line);
//				if(0 < num)
					
			}catch(IOException e){
				System.out.println("例外は" + e + "です");
			}catch(NumberFormatException e){
			System.out.println("数字ではありません");
		}

}
}

試したこと

ここに問題に対して試したことを記載してください。

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

ここにより詳細な情報を記載してください。

Comments

[m.ts10806]

何かの課題のようですが、teratailは課題・宿題代行のサイトではありません。
単に「教えてください」では丸投げです。
現在のコードにおける問題点、起きていること、調べたこと・試したことを具体的に記載してください。

Answer 1

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

//BufferedReaderを使って標準入力から文字列を受け取り、
//int型数値を返す inputIntNumberメソッドを作成してください。
//例外処理は inputIntNumberメソッド内で行い、
//メインメソッドは inputIntNumberメソッドを呼び出して、正の整数を表示するだけの機能を持ちます。

public class InputCheck {
    public static void main(String[] age) {
        System.out.println("正の整数は" + inputIntNumber());
    }

    public static int inputIntNumber() {
        InputStreamReader isr = new InputStreamReader(System.in);
        BufferedReader reader = new BufferedReader(isr);
        while (true) {
            System.out.println("文字列を入力してください");
            try {
                int num = Integer.parseInt(reader.readLine());
                if (0 < num) {
                    return num;
                }
                System.out.println("正の数ではありません");
            } catch (IOException e) {
                System.out.println("例外は" + e + "です");
            } catch (NumberFormatException e) {
                System.out.println("数字ではありません");
            }
        }
    }
}

Answer 2

int 型整数を返すなら、

public static void inputIntNumber(){

ではなく、
public static int inputIntNumber(){
でしょう

そして、返り値はreturn文に記述しましょう

Comments

[syunn]

ご指摘ありがとうございます。調べて解決しました

Answer 3

int型数値を返す inputIntNumberメソッドを作成してください。

このような指示に対して、

public static void inputIntNumber(){

このようなシグネチャを書くのがまずおかしいと気付けるようにして下さい。

考え方

何か値を返すメソッドを書きたかったら、まずひな型を書いてやれば良いんです。

public static int inputIntNumber() {
    return 42;
}

まずこの時点でコンパイルが通るか、思ったとおりの実行結果になるか確認してやります。
42という数値に意味はありません。int型なら別に0でも1でも666でも構いません。

これができたら、後は入力処理を放り込んでやります。
ユーザが必ず、正の整数として正しく解釈できるような入力をしてくれると仮定します。

public static int inputIntNumber() {
    いろんな処理

    int num = Integer.parseInt(input);
    return num;
}

そしたら後は例外処理のフローを書き、動作を確認すれば完成です。
答えを書くのは避けておきます。

---

一気に完成形を作ろうとしないで下さい。
バグが分散し、特定が面倒になります。特に慣れていない内は。

Comments

[syunn]

いつもアドバイスありがとうございます。

Answer 4

このままでは「丸投げ」です。
丸投げは別名、「他人依存」もしくは「他力本願」です。

丸投げしていると実力になりません。

まずは「どのように調べた(が、どの部分がわからない)」とか「こういう風に組んだ(だけど、結果がこれになる。こういう風になってほしいが...)」とかみたいに書きましょう。
それだけでかなり印象が変わります。

---

そもそも基礎すらできていませんね。

        int num = 0;
        inputIntNumber();
        System.out.println("正の整数は" + num);

とやっても、numは宣言(および初期化)しただけで、取得すらしていません。

あとは返すだけでは?