Do/While Problem (Exception in thread "main" java.

[alco]

Hallo, ich bin blutiger Anfänger in Java und mir relativ sicher das ihr meinen Fehler schnell finden werdet.

Zweck des Programms: Primzahlen berechnen und ausgeben ob Primzahl oder nicht. Möchte das Programm auf Wunsch wiederholen.

Hier der Code:

import java.io.*;
public class Prime 
{

	public static void main (String argv[]) throws IOException
	{
		
		char wdh;
		double x;
		boolean isprim=false;
		String str;
		do	{
		BufferedReader eingabe=new BufferedReader(new InputStreamReader(System.in));
		System.out.println("Zahl eingeben: ");
		str=eingabe.readLine();
		x = Double.parseDouble(str);
		for(int i=2;i<x;i++)
				{if (x%i==0)
					{isprim=true;		
					}
				}
					if (!isprim)
							{System.out.println("Die Zahl " +x+ " ist eine Primzahl");
							}
					else 
							{System.out.println("Die Zahl " +x+ " ist keine Primzahl");
							}
						System.out.println("Wiederholen? (j/n) :");
								wdh=(char)System.in.read();
				
			}
		while(wdh=='J'||wdh=='j');
		}
}

Wenn ich das Programm laufen lasse, und J oder j eingebe zum wiederholen, bekomme ich in der Console folgende Fehlermeldung:

Exception in thread "main" java.lang.NumberFormatException: empty String
	at sun.misc.FloatingDecimal.readJavaFormatString(Unknown Source)
	at java.lang.Double.parseDouble(Unknown Source)
	at Prime.main(Prime.java:16)

Ich hab mich schon dumm und dmälich danach gegoogelt, die while Schleife auf etliche verschiedene Weisen versucht zum laufen zu bewegen, aber es scheint ja oben in der Eingabe das Problem. Muss ich dort vielleicht noch irgendetwas einfügen? Die console sagt ja : empty string

Wobei der String ja auch wenn ich das Programm zum ersten mal durchlaufe empty ist.

Zudem ist es irgendwie seltsam, das wenn ich frage ob die zahl !isprim ist, also ob es keine Primzahl ist, ausgegeben wird , das es eine ist, und andersherum (Daher auch die vertauschte Ausgabe).

Ich habe das Programm vor ein paar Jahren auch in C++ geschrieben, und von der Logik her ist ja eigentlich kein Fehler im Programm?

Ich kann mir denken ,dass das Problem warscheinlich total simpel ist und irgendwo in den Grundlagen erklärt steht, doch habe ich mein Script schon danach durchsucht und wie gesagt stundenlang google zu rate gezogen.

 

[Ariol]

import java.io.*;

public class Prime
{

	public static void main(String argv[]) throws IOException
	{

		double x;
		boolean isprim = true;
		String str;
		String wdh;

		boolean run = true;

		while (run)
		{
			BufferedReader eingabe = new BufferedReader(new InputStreamReader(System.in));
			System.out.println("Zahl eingeben: ");
			str = eingabe.readLine();

			try
			{
				x = Double.parseDouble(str);

				for (int i = 2; i < x; i++)
				{
					if (x % i == 0) //<-- ist teilbar, also keine Primzahl
					{
						isprim = false;
					}
				}

				if (isprim)
				{
					System.out.println("Die Zahl " + x + " ist eine Primzahl");
				}
				else
				{
					System.out.println("Die Zahl " + x + " ist keine Primzahl");
				}

				boolean redo = true;

				while (redo)
				{
					System.out.println("Wiederholen? (j/n) :");
					wdh = eingabe.readLine().trim().toLowerCase();
					if (wdh.startsWith("n"))
					{
						run = false;
						redo = false;
					}
					else if (wdh.startsWith("j"))
					{
						run = true;
						redo = false;
					}
				}
			}
			catch (NumberFormatException nfe)
			{
				System.err.println("\"" + str + "\" ist keine Zahl\n");
			}
		}

	}
}

Die Numberformatexception kommt, wenn du statt einer Zahl nix eingibst. Aus nix kann man keine Zahl machen ^^

 

[SlaterB]

> Wobei der String ja auch wenn ich das Programm zum ersten mal durchlaufe empty ist.

hmm, du bekommst aber doch schon zuerst eine Eingabeaufforderung?
warum ist diese erste Eingabe dann 'empty'?
in jedem Fall musst du empty mit einem if/ else abfangen, Double.parseDouble() kann damit nicht umgehen

> Zudem ist es irgendwie seltsam, das wenn ich frage ob die zahl !isprim ist, also ob es keine Primzahl ist, ausgegeben wird

weil du
isprim=true
setzt genau dann wenn x%i==0 also x keine Primzahl ist,
setzte isprim vor der Schleife auf true und in der Schleife auf false wenn x%i==0 ist oder nenne die Variable isnotprim
oder oder oder

du kannst alles programmieren

 

[alco]

ach so funzt das.. okay.. also ist es in java nicht üblich , sowas einfach mit nem char zu machen?

 

[Pappenheimer++]

Bei deiner ursprünglichen Version hätte man statt "j" etwas wie "j42" eingeben müssen, dann hätte er die 42 überprüft