While oder if?

[Sunny81]

Hallo Leute - ich schon wieder.

Würde gerne bei der wahl von default haben dass man wieder waählen muss - sprich wieder in die Switch Abfrage gelangt. Ist das irgendwie möglich?

Danke



[JAVA=42]public class Bestellung


{

public static void main(String args[])
{



int select= IO.promptAndReadInt("Wählen Sie 1 um einen PC zu erstellen!"+
"\nWählen Sie 2 um einen Laptop zu erstellen!\nWählen Sie 3 um zu beenden!\n");

switch(select)

{




case 1:
String prozessor=null;
int ram=0;
int festplatte=0;




while(prozessor==null)
{
prozessor=IO.promptAndReadString("Bitte wählen Sie den Prozessor aus:");
if(!"INTEL".equals(prozessor) && !"AMD".equals(prozessor))
{
prozessor=null;
IO.writeln("Ungültig - Bitte erneut wählen!");
}

else
{
IO.writeln("Erfolgreich hinzugefügt!");
}
}




while(ram==0)
{
ram=IO.promptAndReadInt("Bitte wählen Sie den Arbeitsspeicher aus:");
if(ram<4 || ram>16)
{
ram=0;
IO.writeln("Ungültig - Bitte erneut wählen!");
}

else
{
IO.writeln("Erfolgreich hinzugefügt!");
}
}




while(festplatte==0)
{
festplatte=IO.promptAndReadInt("Bitte wählen Sie die Festplatte aus:");
if(festplatte<160 || festplatte>500)
{
festplatte=0;
IO.writeln("Ungültig - Bitte erneut wählen!");
}

else
{
IO.writeln("Erfolgreich hinzugefügt!");
}
}



Verwaltung o1 = new Verwaltung(prozessor, ram, festplatte);

o1.print();
break;


case 2:

String prozessor2=null;
int ram2=0;
int festplatte2=0;



while(prozessor2==null)
{
IO.promptAndReadString("Bitte wählen Sie den Prozessor aus:");

if(!"INTEL".equals(prozessor2) && !"AMD".equals(prozessor2))
{
prozessor2=null;
IO.writeln("Ungültig - Bitte erneut wählen!");
}

else
{
IO.writeln("Erfolgreich hinzugefügt!");
}
}




while(ram2==0)
{
ram2=IO.promptAndReadInt("Bitte wählen Sie den Arbeitsspeicher aus:");

if(ram2<4 || ram2>16)
{
ram2=0;
IO.writeln("Ungültig - Bitte erneut wählen!");
}

else
{
IO.writeln("Erfolgreich hinzugefügt!");
}
}


festplatte2=IO.promptAndReadInt("Bitte wählen Sie die Festplatte aus:");

while(festplatte2==0)
{
festplatte2=IO.promptAndReadInt("Bitte wählen Sie die Festplatte aus:");

if(festplatte2<160 || festplatte2>500)
{
festplatte2=0;
IO.writeln("Ungültig - Bitte erneut wählen!");
}

else
{
IO.writeln("Erfolgreich hinzugefügt!");
}
}



Verwaltung o2 = new Verwaltung(prozessor2, ram2, festplatte2);

o2.print2();
break;


case 3: IO.writeln("Vielen Dank für Ihren besuch - Auf Wiedersehen!");
break;

default: IO.writeln("Falsche Taste - Bitte erneut wählen.")
break;

}







}
}
}[/code]

 

[w0ddes]

Also normalerweise geht das ja, wenn man die Methode einfach neu aufruft.
-> Ich glaube (und garantiere nicht für Richtigkeit), dass man bei "default":

this.main();

einfügen muss.

 

[Spacerat]

Öhm... ja... nicht ist unmöglich, sag ich mal...
Bis auf deinen Code (sorry). Nicht das er nicht rel. gut formatiert wäre... aber diese kaum durchdachte Schachtelung von switch, if und while... da findet sich mit Sicherheit ein besseres Konstrukt. Z.B. das Auslagern der ein oder anderen if-Abfrage und / oder while-Schleife in eine separate Methode...
@Edit: [c]this.main()[/c]? ...[c]main()[/c] ist statisch?...!

 

[dngfng]

mach doch einfach eine Whileschleife wie folgt um deine switch Statement.

	    public static void main(String args[] {
	 
	    boolean redo = true;
	    while(redo) {

            redo=false;

            int select= <input>;
	    
	    switch(select) {
                  ....
                  ....
                default:    System.out.println("Falsche Taste - Bitte erneut wählen.");
	        redo = true;
	        break;
            }
            }

Vor dem while einfach ein boolean auf wahr setzten.
Sofort inerhalb der schleife auf falsch und nur falls der nutzer auf default landen sollte wird der boolean wieder auf wahr gesetzt und die schleife nochmals ausgeführt.

 

[Sunny81]

Danke schon mal an alle.

@ Spacerat
Hättest du mir ein paar Tips diesbzgl?
Bin noch sehr unerfahren was Java angeht.
merci

 

[dngfng]

Besser konsturkte gibt es zu hauf.
z.b.

int input = 0;
boolean invalidEntry = true;

while(invalidEntry) {
    input = <getInput>
    if(inputValid==true) {
      invalidEntry==false;
    }
}

switch
....

 

[Sunny81]

Oh da versteh ich grad nur Bahnhof... :shock:

 

[dngfng]

Danke schon mal an alle.

@ Spacerat
Hättest du mir ein paar Tips diesbzgl?
Bin noch sehr unerfahren was Java angeht.
merci

Als erstes erst mal all den Code den du im Switch Statement hast in einen separate Methode auslagern.

z.b.

	    private void createPC() {
	    	
            String prozessor=null;
            int ram=0;
            int festplatte=0;

            while(prozessor==null)
            {
                prozessor=IO.promptAndReadString("Bitte wählen Sie den Prozessor aus:");
                if(!"INTEL".equals(prozessor) && !"AMD".equals(prozessor))
                {
                prozessor=null;
                IO.writeln("Ungültig - Bitte erneut wählen!");
                }
            
                else
                {
                IO.writeln("Erfolgreich hinzugefügt!");
                }
            }
	    	return;
	    }

Und diese dann aus dem Switch Statement aufrufen.

 

[Spacerat]

...na klar... aber wirlich nur ein Ansatz

public static void main(String[] args)
{
  Bestellung best = null;
  do {
    int select = ... wie es oben steht...

    switch(select) {
    case 1:
      best = createDesktopComputer();
      break;
    case 2:
      best = createLaptopComputer();
      break;
    case 3:
      System.exit(0);
      break;
    default:
      // empty block
    }
  } while(best == null);
}

private static Bestellung createDesktopComputer()
{
  ... usw
}


private static Bestellung createLaptopComputer()
{
  ... usw
}

 

[dngfng]

Oh da versteh ich grad nur Bahnhof... :shock:

Dann mal ganz von vorne.

Bevor du überhaupt in den Switch reingehst überprüfst du ob die Eingabe korrekt ist.

Also machst du eine while schleife in der du denn User um eingabe bittest, und bendest diesen nur wenn eine richtige Eingabe gemacht wurde.

Und erst mit dieser korrekten eingabe gehst du überhaup in den Switch.

z.b.

	    	boolean korrekteEingabe = false;
	    	
	    	int eingabe = 0;
	    	
	    	while(!korrekteEingabe)  {
	    		eingabe = <abfrage der Eingabe>;
	    		
	    		if(eingabe=1||eingabe==2) korrekteEingabe = true;
	    		
	    	}

 

[dngfng]

@Spacerat schicker Code.

Sunny - sag nur wenn du davon etwas nicht verstehen solltest.

 

[Sunny81]

Also diese 2 Sachen wäre mir nicht ganz klar - Ansonsten verstehe ich es so dass halt anstatt alles in den Switch reinzuschreiben dort nur die Methoden aufgerufen werden. Oder?

do {


System.exit(0);

 

[dngfng]

do {
} while(<condition>)


ist eine Schleife - die immer mindestens einmal ausgeführt wird.

Bei

while(<condition>) {}

Muss die Bedingungen ja schon für die erstmalige Ausführung erfüllt sein.

Daher sollte man wenn ein Stück Code mindestens einmal ausgeführt werden soll eine do-while anstatt einer while schleife verwenden.

System.exit
Bei Erreichen des Endes der main-Methode wird das Programm automatisch beendet (außer wenn noch weitere Threads laufen, siehe die Kapitel über Graphische User-Interfaces und über Threads).

Mit System.exit(0); oder System.exit(n); kann man die Applikation vorzeitig beenden.

Bei erfolgreicher Beendigung gibt man als Exit-Code 0 an, bei einer nicht erfolgreichen Beendigung eine positive ganze Zahl als Fehler-Code, im einfachsten Fall den Wert 1.

 

[Spacerat]

Fußgesteuerte while-Schleife nennt man das glaub' ich:

do {
  // irgendwelche Anweisungen
} while(ereignisNochNichtEingetroffen);

Hier wird die Schleife nicht eher verlassen, bis eine gültige Bestellung getätigt wurde ([c]best != null[/c]) oder das Programm verlassen werden soll (3 -> [c]System.exit(0)[/c]). Die Methoden zum Erstellen der Computer müssen also eine Instanz der Klasse Bestellung zurückgeben.

 

[dngfng]

Kopfgesteuerte while-Schleife nennt man das glaub' ich

Nicht ganz das ist eine Fuß gesteuerte schleife, da die Kondition ab ende sitzt.

Kopfgesteuert while(...) {}
Bauchgesteuert while(true) { if(...) break; }
Fußgesteuert do {} while(...)

Das ist jetzt wirklich nur klugscheiserei - der code ist auf jeden Fall gut.

 

[Spacerat]

...Is ja jut... hab's längst verbessert...

 

[Sunny81]

Könnte mir einer den Code nochmal etwas näher bringen.

private static Bestellung createDesktopComputer();

ist doch eineMethode?
Wo wird die dann ausgeschrieben?
In die gleiche Klasse?

was ist dieses best?

 

[Spacerat]

...also... Deine Klasse mit der Main-Methode heisst in deinem Beispiel ja Bestellung. Dieses [c]best[/c] soll eine Instanz von Bestellung werden. Die ccreate-Methoden sind weitere statische Methoden der Klasse Bestellung (deswegen ist der Code auch nicht all zu toll ). In diesen werden nun die jeweiligen PCs erstellt (was vorher in deinen verschachtelten while-Konstrukten passiert ist). Wurde der PC erfolgreich erstellt, verlässt man die Methoden jeweils mit [c]return new Bestellung();[/c]. Soll die Bestellung abgebrochen werden, verlässt man die Methode halt mit [c]return null;[/c].