Java 二次方程式を繰り返し解く

Question

プログラム初心者です。（学習歴：1か月）

前提・実現したいこと

Java (Eclipse)にて
キーボード入力で"finish"と入力されるまで繰り返し二次方程式を解くJavaプログラムを作成したいです。
別で作成してあるプログラムをメソッドしてメゾット呼び出しを活用したプログラムにしたいです。また、その際に配列を活用したいと考えています。

###【具体的な実現したい流れ】
まず初めに（１）「Please Input:a,b,c」と表示させる。
（２）キーボードで数字を３つ入力する。例）1,-2,-3
※ここで「finish」と入力したら「Thanks you for using this system.」と表示させ、プログラムを終了させたいです。
※ここで数字以外を入力したら「Your Paramaters cannot understand.」と表示され、また（１）に戻るようにしたいです。
（３）そしたら入力した数字を表示させる。（例：「Line=1,-2,-3」と表示される。）
（ここで、1はa、-2はb、-3はcとし認識させたいです。）
（４）次に「x1=-1」「x2=3」と「Please Input:a,b,c」と表示させたいです。
（５）（２）～（４）を繰り返したいです。

###イメージしている表示されてほしい内容
Plese Input:a,b,c
1,-2,-3 (←自分がキーボードで入力した文字 1回目)
Line=1,-2,-3
x1=-1.00
x2=3.00
Plese Input:a,b,c
hello(←自分がキーボードで入力した文字　2回目)
Your Paramaters cannot understand.
Plese Input:a,b,c
3,5,9 (←自分がキーボードで入力した文字　3回目)
There is no solution because the discriminant is negative. D="+d
Plese Input:a,b,c
finish (←自分がキーボードで入力した文字　4回目)
Thanks you for using this system.

発生している問題

キーボード入力した数字をa,b,cとして認識させたいです。。
どこにどのプログラムや式を入れ込めばいいのか見当がつきません。

###今自分の中でわかっていること
・キーボードで入力した数字を順番にa,b,cと置きたいが、それの方法が分からない。
例）1,-2,-3とキーボードから入力したら、それぞれをa(=1),b(=-2),c(=3)として認識させたい。
・数字を入力した後、計算式のところに行ってくれるようにする方法が分からない。
・public static String getInput(){}に問題があることは認識しているのですが、全体的に意味不明になっており、どこから手を付ければいいかわからない状態です。

該当のソースコード

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

public class nijikannsuu {

	public static void main(String[] args) {
		String line = "";
		double a = 0, b = 0, c = 0;
		boolean terminate = false;
		while (true) {

			line = getInput();
			if (line.equals("finish")) {
				System.out.println("Thanks you for using this system.");
				return;
			} else {

			}
		}
	}

	public static String getInput() {
		BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
		double a = 0, b = 0, c = 0, d = 0, x1 = 0, x2 = 0;
		String line = null;
		Scanner sc = new Scanner(System.in);
		System.out.println("Please Input: a,b,c");
		String q = "finish";
		try {
			line = reader.readLine();
			String[] abc = line.split(",");
			if (line.equals("finish")) {
				return "finish";
			} else
				a = Double.parseDouble(abc[0]);
			b = Double.parseDouble(abc[1]);
			c = Double.parseDouble(abc[2]);
			System.out.println("line=" + a + ", " + b + ", " + c);
		} catch (NumberFormatException | IOException e) {//
			System.out.println("Could you enter the number?");
			System.out.println("Your Paramaters cannot understand.");
		}
		if (a == 0) {
			x1 = x2 = (-c) / b;
		} else {
			if (d < 0) {
				d = b * b - 4 * a * c;
				System.out.println("There is no solution because the discriminant is negative. D=" + d);
			} else {
				x1 = (-b + Math.sqrt(d)) / (2 * a);
				x2 = (-b - Math.sqrt(d)) / (2 * a);
				System.out.println("x1=" + x1);
				System.out.println("x2=" + x2);
			}
		}
		return line;
	}
}

試したこと

public static String getInput(){　にある
System.out.println("Plese Input : a,b,c");まではできたのですが、
そこから先がいかず、いじっているうちにまったくわからなくなりました。

初心者で各要素の仕組みをしっかりと把握していない状態です。
どこのコードがおかしいといったことをご指示してくれるだけでも構いません。
一部分だけでも構いませんので、ご教授してくださると幸いです。

Comments

[y_waiwai]

このままではコードが見づらいです。質問を編集し、<code>ボタンを押し、出てきた’’’の枠の中にコードを貼り付けてください

[Elpa]

大変失礼いたしました。こちらでよろしいでしょうか？

[swordone]

エラーメッセージはそれで全部ですか？

[Elpa]

エラーメッセージ　抜けていた部分がありました
solveEq(a,b,c);は必要あるのかも自分ではわからないです。

Answer 1

ublic static String getInput(){}に問題があることは認識しているのですが、全体的に意味不明になっており、どこから手を付ければいいかわからない状態です。

まずは、インデントをそろえるのが良いと思います。
文法的に波カッコが正しく一致しているか確認してみては？

try以降のforあたりから、おかしいですよね？

--
追記

eclipseに「}を入れて終了させてください」と言われたので、その通りにしました。

足りないからつけたすものでなく、自分の考えた通りの位置にカッコをつけてください。
なので、構造をわかりやすくするためにインデントが重要になります。

インデントはこちらでよろしいでしょうか？

インデントをそろえてるというのは波カッコの位置をそろえて、波カッコの中をタブ１個分（４とか８個の空白）ずらして、波カッコで囲われた範囲をすぐに判断できるようにします。

なので

    }        
    }

とか

}}

みたいなコードが出てきたらおかしいと思ってよいです。

eclipseを使っているなら、[ソース]-[フォーマット] (Ctrl+Shift+F)するとこんな感じになります。

public static String getInput() {
	BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
	double a = 0, b = 0, c = 0, d = 0, x1 = 0, x2 = 0;
	String line = null;
	Scanner sc = new Scanner(System.in);
	System.out.println("Please Input: a,b,c");
	try {
		for (int i = 0; i < 3; i++) {
			line = reader.readLine();//ここでキーボード入力を行えるようにする？
			System.out.println("line=" + a + ", " + b + ", " + c);//上で入力した数字をa,b,cとして認識させたい。
		}
	} catch (NumberFormatException | IOException e) {
		System.out.println("Could you enter the number?");
		System.out.println("Please try again!");
	}
	if (a == 0) {
		x1 = x2 = (-c) / b;
	} else {
		if (d < 0) {
			d = b * b - 4 * a * c;
			System.out.println("There is no solution because the discriminant is negative. D=" + d);
		} else {
			x1 = (-b + Math.sqrt(d)) / (2 * a);
			x2 = (-b - Math.sqrt(d)) / (2 * a);
			System.out.println("x1=" + x1);
			System.out.println("x2=" + x2);
		}
	}
	return line;
}

forループでNuberFormatExceptionが発生したときに再入力をさせたいのだろうと
インデントをずらしている？のかもしれませんが、実際にはループの外側に抜けちゃうのが、わかると思います。

想いでなく、文法に沿ってインデントを確認すると、構造がおかしいことに気づくはずです。
※繰り返しですが、eclipseにはフォーマット機能あるので、それ使う方が楽です。

try {
        for (int i = 0; i < 3; i++) {
            line = reader.readLine();//ここでキーボード入力を行えるようにする？
            System.out.println("line=" + a + ", " + b + ", " + c);//上で入力した数字をa,b,cとして認識させたい。
        }
    } catch (NumberFormatException | IOException e) {
// ここに来た場合、for文の外側なので try again と表示するだけで、次の処理へいく
        System.out.println("Could you enter the number?");
        System.out.println("Please try again!");
    }

Comments

[Elpa]

eclipseに「}を入れて終了させてください」と言われたので、その通りにしました。エラーコードは表示されなくなりました。インデントはこちらでよろしいでしょうか？　momon-ga様のおっしゃる通りforの中はめちゃくちゃです。しかし、一つやっていかないと頭が混乱してしまうので、一つずつご教授いただけますでしょうか？　次にしたいことはキーボード入力した数字をa,b,cと認識させることです。キーボード入力した数字を配列に入れ込むのでしょうか？　ご教授お願いいたします。

[Elpa]

momon-ga様　ご指示ありがとうございます。フォーマット機能を使いました。こちらで問題ないでしょうか？もう一度見返し、for文は必要ないと考えました。今度がfinishと入力したら、
「Could you enter the number?
Your Paramaters cannot understand.
Thanks you for using this system.」
と表示されました。これを、37行目のcatch文に行かせ、その後14~15行目に行かせるには
37行目のcatchの（）に何を入力すれば良いのでしょうか？
お忙しいところ恐れ入りますがご教授よろしくお願いいたします。

[momon-ga]

インデントは、いいと思います。
いろいろ修正する前に、getInputの動き(役割、いわゆる仕様)をきちんと決めた方がいいです。

で、例外をキャッチするのでなく、splitする前にif文で判定すれば、そもそも例外発生しません。
具体的には、
if (line.equals("finish")) {
return "finish"; // return line;
}
と、すればfinishを返せますが、実際にa,b,cを取得した場合って、13行目あたりの処理って
どうします？

[momon-ga]

仕様って、言葉が難しいとして簡単にいうと
13行目の
line = getInput();
で、受け取りたいStringの値って、どんなのが良いか？を決めてますか？

受け取りたい値のひとつは、"finish"だと思いますが・・・

[Elpa]

momon-ga様　ご返信が遅くなってしまい申し訳ございません。if文を用いればよかったのですね。ありがとうございます。
13行目の line=getInput();で受け取りたい値は
１、自然数
２、finish　です。

[momon-ga]

finishはStringなので"finish"になると思いますが、自然数は、Stringとして
どんな感じで受け取るのですか？
"1,-2,-3"みたいな感じですか？

[Elpa]

（早速のお返事感謝しております）おっしゃる通り、1,-2,3の数を受け取りたいと考えております。

Answer 2

メソッド solveEq(double, double, double) は型 nijikannsuu で未定義です

書いてあることそのままのエラーです。
この関数が定義されてません。

あと、コードのインデントが全くなされていません。
これではエラーチェックもしづらいですし、なによりコードが読みづらいです
適切にインデントするように書き直しましょう

Comments

[Elpa]

ひとまず関数を消去いたしました。そしたらエラーコードで
Exception in thread "main" java.lang.Error: Unresolved compilation problem:

at nijikannsuu.main(nijikannsuu.java:7)
と出ました。
public static void main(String[] args) の方を修正する必要があるということで間違いないでしょうか？