Fehlerhafte Eingabe holen

[Neuling123]

Hallo

Ich hab ein problem, und zwar nach Fehlerhafter Eingabe wird eine Fehlermeldung ausgepuckt und das Programm beendet. Ich würde aber gerne , das man die Eingabe Wiederholen kann. Ich weiss aber absolut nicht wie ich das hinbringen könnte. Hier der Code

	public static void main(String[] args) {
		double widerstandswert1 = 0;
		double widerstandswert2 = 0;
		double ergebnis = 0;
		String auswahl;
		
		Scanner sc;
		sc = new Scanner(System.in);
		
do {
	System.out.println("(P)arallel- oder (R)eihenschaltung?");


auswahl = sc.next();
} while (!auswahl.equals("r")&&!auswahl.equals("R")&&!auswahl.equals("p")&&!auswahl.equals("P"));
		
do
{
try
{
	System.out.println("Widerstandswert 1 eingeben");

	
	widerstandswert1 = sc.nextDouble();


	System.out.println("Widerstandswert 2 eingeben");

	widerstandswert2 = sc.nextDouble();
} catch (Exception e){
	System.out.println("Dies ist kein gültiger Eingabewert");
	System.exit(0);
}
while (sc.next() = "doubl")
	
if (auswahl.equals("P")||(auswahl.equals("p"))){
	ergebnis = Math.round((widerstandswert1*widerstandswert2)/(widerstandswert1+widerstandswert2));
	System.out.println("R1 in Ohm: "+widerstandswert1);
	System.out.println("R2 in Ohm: "+widerstandswert2);
	System.out.println("Der Gesamtwert der Parallelschaltung beträgt rund "+ergebnis);
}
if (auswahl.equals("R")||(auswahl.equals("r"))){
	ergebnis = Math.round(widerstandswert1+widerstandswert2);
	System.out.println("R1 in Ohm: "+widerstandswert1);
	System.out.println("R2 in Ohm: "+widerstandswert2);
	System.out.println("Der Gesamtwert der Reihenschaltung beträgt rund "+ergebnis);} 
}

	}
}

 

[Guest]

Sorry, falscher Code.. Der richtige:

import java.util.Scanner;
public class Widerstandsberechner {

	
	public static void main(String[] args) {
		double widerstandswert1 = 0;
		double widerstandswert2 = 0;
		double ergebnis = 0;
		String auswahl;
		
		Scanner sc;
		sc = new Scanner(System.in);
		
do {
	System.out.println("(P)arallel- oder (R)eihenschaltung?");


auswahl = sc.next();
} while (!auswahl.equals("r")&&!auswahl.equals("R")&&!auswahl.equals("p")&&!auswahl.equals("P"));
		

try
{
	System.out.println("Widerstandswert 1 eingeben");

	
	widerstandswert1 = sc.nextDouble();


	System.out.println("Widerstandswert 2 eingeben");

	widerstandswert2 = sc.nextDouble();
} catch (Exception e){
	System.out.println("Dies ist kein gültiger Eingabewert");
	System.exit(0);
}

	
if (auswahl.equals("P")||(auswahl.equals("p"))){
	ergebnis = Math.round((widerstandswert1*widerstandswert2)/(widerstandswert1+widerstandswert2));
	System.out.println("R1 in Ohm: "+widerstandswert1);
	System.out.println("R2 in Ohm: "+widerstandswert2);
	System.out.println("Der Gesamtwert der Parallelschaltung beträgt rund "+ergebnis);
}
if (auswahl.equals("R")||(auswahl.equals("r"))){
	ergebnis = Math.round(widerstandswert1+widerstandswert2);
	System.out.println("R1 in Ohm: "+widerstandswert1);
	System.out.println("R2 in Ohm: "+widerstandswert2);
	System.out.println("Der Gesamtwert der Reihenschaltung beträgt rund "+ergebnis);} 
}

	}

[/quote]

 

[SlaterB]

siehe auch
http://www.java-forum.org/de/viewtopic.php?p=361462

eine while-Schleife muss her

 

[Guest]

Ja, eine while Schleife. Aber was soll bei while drinstehene?

while ( Betrag ist keine Zahl)

Sowas dacht ich mir. Aber ich weiss nicht wie man es richtig formuliert und ob es überhaupt geht. Hab auch schon versucht das bei Catch boolean x false gesetzt wird, und dann while ( b = false). Aber das geht auch nicht so richtig

 

[SlaterB]

mit 'weiß nicht' und ähnlichen blahblah wird das nicht gelöst
sondern mit konzentrierten Nachdenken,

> Aber ich weiss nicht [..] ob es überhaupt geht.

geht, nun weißt du es

> Aber ich weiss nicht wie man es richtig formuliert

wie man WAS formuliert?
überlege dir erstmal in deutscher Sprache, was passieren soll,
das ist die wichtigere Hälfte der Arbeit,

dann das Coden, wo genau liegt das Problem?

 

[Guest]

Ich hab mir sehr genau überlegt was passieren.

Das Problem ist, das ich als absoluter Neuling nicht weiss, wie ich das "Javagerecht" formuliere, so das es eben Java versteht. Das Problem ist nicht das ich mich nicht ausdrücken kann.
Ich glaube wir haben uns da etwas falsch verstanden nun.

Ich dachte mir eben, das es so abläuft:

Do

"Betrag eingeben!"
Betrag eingeben


While (Der Betrag eine Zahl ist)

Ich will eben das das Programm solange die Mitteilung "Betrag eingeben!" und den dazuhörigen Scanner benutzt bis der Betrag eine Zahl ist.


Und genau eben das "While -> Der Betrag eine Zahl ist", das ist mein Problem. Ich weiss nicht wie ich das dem Programm verständlich mitteile.

 

[Guest]

Ich meine natürlich: Ich hab mir sehr genau überlegt was passieren soll.

Pardon.

 

[SlaterB]

so klingt das sehr konstruktiv, dann kann man auch gut weiterhelfen:

double zahl = -1;
Do
"Betrag eingeben!"
Betrag eingeben (try catch Exception)
While (zahl == -1)

wenn eine falsche Zahl eingegeben wird, gibts eine Exception und die Zahl bleibt bei -1,
wenn sie nicht mehr -1 ist, dann war die Eingabe erfolgreich,

funktioniert natürlich nur, wenn -1 kein möglicher Eingabewert ist,
wenn alles erlaubt ist, dann musst du mit Objekten arbeiten:

Double zahl = null;
Do
"Betrag eingeben!"
Betrag eingeben (try catch Exception)
While (zahl == null)

 

[Guest]

Oh, mist. Wir haben uns schon wieder missverstanden!.. glaube ich

Ich möchte das der Benutzer solange den Betrag eingeben kann, BIS der Betrag irgendeine Zahl ist!

Nun bricht er - bei !!!Buchstaben!!eingabe - ab. Und das ist der springende Punkt, auf den ich hinaus will.

Also: Bei Buchstabeneingabe in einen Doublescanner soll der Benutzer solang eine Eingabe machen können, bis er eine Zahl eingibt



----

Oder du hast es so gemeint und ich interpretiere es falsch.


Gruß und Danke

 

[SlaterB]

ich antworte einfach mal 'sehe ich genauso',
und das Gespräch kommt nicht weiter

weil du keine konkrete Frage gestellt hast

 

[Guest]

Vielleicht kann mir ein (menschlich) kompetenterer User helfen.

 

[H3llGhost]

Oh, mist. Wir haben uns schon wieder missverstanden!.. glaube ich

Ich möchte das der Benutzer solange den Betrag eingeben kann, BIS der Betrag irgendeine Zahl ist!

Nun bricht er - bei !!!Buchstaben!!eingabe - ab. Und das ist der springende Punkt, auf den ich hinaus will.

Also: Bei Buchstabeneingabe in einen Doublescanner soll der Benutzer solang eine Eingabe machen können, bis er eine Zahl eingibt



----

Oder du hast es so gemeint und ich interpretiere es falsch.


Gruß und Danke

Du willst also keine Buchstaben erlauben?

Wenn du ein richtiges Eingabeformular hast kann dir das glaube ich helfen:

import javax.swing.text.PlainDocument;
import javax.swing.text.AttributeSet;
import javax.swing.text.BadLocationException;
import javax.swing.JTextField;
import java.awt.Toolkit;

public class NegativeDoubleDocument
    extends PlainDocument
{
  private boolean bDot = false;
       /***************************************************************************
       * Funktion erlaubt ein minus oder + (nicht sichtbar) in den Zahlen
       * @param offset int
       * @param s string
       * @param attributeSet att
       * @throws BadLocationException
       **************************************************************************/
      public void insertString(int offset, String s, AttributeSet attributeSet) throws
          BadLocationException
      {
        try
            {
              if(s.equals(",")) //ersetzt den , durch einen Punkt
                s = ".";
              bDot = getText(0,getLength()).indexOf(".") == -1 ? false : true; //merkt sich ob ein Punkt oder nicht
              if (s.length() > 1)
              {
                //wenn der String größer als 1 ist, ist es eine ganze IP und somit eine Schleife
                for(int x = 0; x != s.length(); x++)
                  insertString(x,s.substring(x,x+1).replaceFirst(",","."),null); //ersetzt , durch Punkt
                //mach gar nix, weil die for schleife alles macht!
                  return;
              }
              else if(s.equals("-"))
              {
                String strTmp = getText(0,getLength()); //holt sich den Text
                System.out.println(strTmp);
                if(strTmp.indexOf("-")!=-1) //schaut nach ob ein minus vorhanden ist
                  strTmp = strTmp.replaceAll("-",""); //wenn ja weg
                else
                  strTmp = "-" + strTmp;
                replace(0,getLength(),null,attributeSet);
                super.insertString(0,strTmp,attributeSet);
                return;
              }
              else if(s.equals("+")) //bei einem + einfach ein weiteres - hinzufügen
              {
                if(getText(0,getLength()).indexOf("-")!=-1)
                  insertString(0,"-",attributeSet); //ruft sich selbst mit - auf und dann kommt obige Überprüfung
                return;
              }
              //OPTIONAL!!! STATT DEM ± kann man jedes Zeichen nehmen oder komplett weglassen
              //wenn man irgendwelche Zeichen zB noch haben will <!--- OPTIONAL ---!>
              else if(s.equals("±"))
              {
                String strTmp = getText(0,offset);
                if(strTmp.indexOf("±") != -1)
                  strTmp = strTmp.replaceAll("±","");
                else
                  strTmp = "±" + strTmp;
                replace(0,getLength(),null,attributeSet);
                super.insertString(0,strTmp,attributeSet);
                return; //damit aufgehört wird
              }
              //ENDE OPTIONAL!!!!!!!!!!!!!!!!!!!!
             //wenn das aktuelle Zeichen kein punkt ist oder wenn es ein Punkt ist und schon ein Punkt eingegegeben ist
              else if(!s.equals(".") || bDot)  // ==   if(!(s.equals(".") && !bDot))
                Double.parseDouble(s);

            }
            catch(Exception ex)
            {
              Toolkit.getDefaultToolkit().beep();
              return;
            }
            super.insertString(offset,s, attributeSet);
          }
    }

Ansonsten kann ich dir auch einen Code von mir posten:

    try
    {
     wahl=Double.parseDouble(eingabeTastatur.readLine());
    }
    catch(Exception e)
    {
     System.out.println("Ungültige Eingabe!");
    }

Verständnis:
Er probiert die Tastatureingabe als Double zu speichern.
Geht das ist das wirklich eine Zahl ansonsten kommt der Fehler, damit werden auch Sonderzeichen und so ein Gedöns abgefangen ...[/code]

 

[lukian]

Hi,

ich bin zwar neu hier, finde aber übrigens die Arroganz mancher Antworten hier ziemlich daneben. Diese Arroganz paart sich dann meistens auch mit dämlichen und irreführenden Hinweisen.

So sollte es gehen:

Scanner sc;
double betrag;
boolean isDouble;
do {
	sc = new Scanner(System.in);
	System.out.print("Betrag eingeben: ");
	try {
		if (sc.hasNext()){
			betrag = sc.nextDouble();
		}
		isDouble = true;
	} catch (InputMismatchException e){
		isDouble = false;
	}
	
} while(!isDouble);

Gruß,
lukian

 

[SlaterB]

kannst du die Arroganz näher beschreiben?

meine Philosophie ist jedenfalls nicht die, fertigen Code zu posten,
so wirst du nur dafür sorgen, dass Fragesteller x nix lernt sondern beim nächsten Problem wieder deinen Code erwartet..

und dabei meine ich natürlich nicht fertiges Wissen wie den Einsatz des Scanners,
viel interessanter ist das Gesamtkonstrukt aus while-Schleife + alles drum und dran,
darauf muss man selber kommen, nur dann ist man schlauer als vorher

 

[lukian]

Ich denke 12 Posts sind genug, so dass man auch mal Code anbieten kann. Man sollte helfen und nicht Leute erziehen. Die Aufgabe war - zumindest für mich - eindeutig, trotzdem wolltest Du unbedingt eine ausformulierte Frage auf dem Silbertablett serviert bekommen. Das fand ich arrogant und überhaupt nicht hilfreich.

Gruß,
lukian

 

[SlaterB]

an welcher Stelle siehst du das so?,
auch dich muss ich nun schon 2x bitten, doch mal genauer zu werden

an einer Stelle brachte ich Gast dazu, die Frage
> Und genau eben das "While -> Der Betrag eine Zahl ist", das ist mein Problem.

zu stellen, das muss nicht ausformuliert oder anders sein,
entscheidend ist, diese Frage überhaupt zu stellen (im Gegensatz zu 'ich weiß von nix, wie geht das alles?')

wenn man über das Problem nachdenkt, dann kann man nur an derart konkreten Teilen hängen bleiben,
und dafür dann Lösungen wie mein -1 oder dein while(!isDouble); anzubieten ist selbstverständlich, darauf kann man ja ohne Erfahrung nicht kommen

 

[Guest]

Hallo Lukian!

Danke für deine Hilfe. Genau dieses (!isDouble) war mein gesuchtes Problem, das ich nicht zu definieren wusste in Java. Ich danke dir und das hat mich schon schlauer gemacht, für andere (kleine) Sachen.

Ich finde es zwar auch gut das du, Slater nicht gleich den fertigen Code postet, jedoch ist die Art wie du es rüberbringst einfach blöd. Ich hab ja schon klar formuliert was ich suche und wo es hängt und nicht gesagt "Macht das mal fertig".

Gruß

 

[Guest]

do { 
      sc = new Scanner(System.in); 
   try { 
	   
	  System.out.print("Widerstand 1 eingeben: "); 
      widerstandswert1 = sc.nextDouble(); 
     
      System.out.print("Widerstand 2 eingeben: ");
      widerstandswert2 = sc.nextDouble();
      
      isDouble = true; 
   } catch 
   		(InputMismatchException e){ 
	   	 isDouble = false; 
   } 
    
} while(!isDouble);

So sieht mein fertiger Code aus. Nicht ersichtlich ist mir warum du " if (sc.hasNext()){ " verwendest

Es klappt da auch ohne?

Gruß

 

[SlaterB]

@vorvorheriges Posting:
ohne Argumente kann ich dies nicht nachvollziehen, falls es dich näher interessiert,

denn diese !isDouble-Variante habe ich ja vorher in ähnlicher Form auch gepostet und näher erläutert..

 

[H3llGhost]

[...]

So sieht mein fertiger Code aus. Nicht ersichtlich ist mir warum du " if (sc.hasNext()){ " verwendest

Es klappt da auch ohne?

Gruß

Ein kleiner Tipp am Rande ...
Besorg dir die Javadokumentation ...
Wenn du den JavaEditor benutzt kannst du das direkt einbinden ...

Weiterer Tipp ist die Java Insel und das Java Buch ...
Beides kostenlos ...