Do While Schleifenfrage

[Schrubber]

Hallo Leute,

Ich bin derweil an einem größeren Projekt dran und bin nun aber an einer kleinen Anweisung gescheitert

Und zwar ist mein Qellcode des Teils wie folgt:

		    int gegnerzug = (int) (Math.random()*9+1);
		  	
		    do 
		    {
		    	gegnerzug = (int) (Math.random()*9+1);
			    String feldzug = "feld"+gegnerzug;
		    }
		    while(feldzug != "_");
		    }

Was es eigentlich machen soll:
Es soll die variable gegnerzug einer Zufallszahl von 1-9 zuweisen. Diese soll dann an den Text "feld" angehängt werden und als String feldzug ausgegeben werden.
Zu guter letzt soll dann noch getestet werden, ob feldzug != "_" ist.

feldzug ist bei mir entweder "Q", "X" oder "_".

Alles funzt, bis auf dieses mit dem verknüpfen. Cannot be resolved, heißt es immer. Danke schonmal für die Hilfe


Schrubber

 

[Atze]

int gegnerzug = (int) (Math.random()*9+1);
		String feldzug;
        do 
        {
            gegnerzug = (int) (Math.random()*9+1);
            feldzug = "feld"+gegnerzug;
        }
        while(feldzug != "_");

versuchs mal so feldzug wäre sonst nur in der schleife sichtbar

 

[eRaaaa]

und du willst wahrscheinlich eher :

while(!"_".equals(feldzug));

???:L

 

[Atze]

lol, garnicht drauf geachtet

 

[Landei]

Aber wenn feldzug = "feld"+gegnerzug; dann wird die Variable doch niemals gleich "_" ?

 

[eRaaaa]

hahaha, stimmt

feldzug ist bei mir entweder "Q", "X" oder "_".

:toll:

 

[Atze]

ist zu spät um das zu sehen, sollte ins bett gehen

 

[Schrubber]

Ich meinte das so:

Er soll an das Wort feld die Zufallszahl anhängen.

Vordefiniert ist mein Spielfeld von Feld1-Feld9. Dies sind Strings, von daher brauch ich auch für die Feldzug genau diese variablen, die sich halt aus Feld + Zufallszahl zusammensetzen ^^


Bin da auch ratlos xD

 

[Schrubber]

Okay

Sorry für Doppelpost, wollt nur sagen, dass ichs ein wenig anders geschriebn hab, mit euren Tipps, nun funktioniert es

		  	int gegnerzug = (int) (Math.random()*9+1);
		  	int meinzug;
		  	String feldzug = null;
		  	String feldzug2 = null;
		  	        do 
		  	        {
		  	            gegnerzug = (int) (Math.random()*9+1);
		  	            if (gegnerzug == 1)
		  	            {
		  	            	feldzug = feld1;
		  	            }
		  	            if (gegnerzug == 2)
		  	            {
		  	            	feldzug = feld2;
		  	            }
		  	            if (gegnerzug == 3)
		  	            {
		  	            	feldzug = feld3;
		  	            }
		  	            if (gegnerzug == 4)
		  	            {
		  	            	feldzug = feld4;
		  	            }
		  	            if (gegnerzug == 5)
		  	            {
		  	            	feldzug = feld5;
		  	            }
		  	            if (gegnerzug == 6)
		  	            {
		  	            	feldzug = feld6;
		  	            }
		  	            if (gegnerzug == 7)
		  	            {
		  	            	feldzug = feld7;
		  	            }
		  	            if (gegnerzug == 8)
		  	            {
		  	            	feldzug = feld8;
		  	            }
		  	            if (gegnerzug == 9)
		  	            {
		  	            	feldzug = feld9;
		  	            }
		  	        }
		  	        while(!"_".equals(feldzug));

Danke Leute

Schrubber

 

[0x7F800000]

if (gegnerzug == 1)
                        {
                            feldzug = feld1;
                        }
                        if (gegnerzug == 2)
                        {
                            feldzug = feld2;
                        }
                        if (gegnerzug == 3)
                        {
                            feldzug = feld3;
                        }
                        if (gegnerzug == 4)
                        {
                            feldzug = feld4;
                        }
                        if (gegnerzug == 5)
                        {
                            feldzug = feld5;
                        }
                        if (gegnerzug == 6)
                        {
                            feldzug = feld6;
                        }
                        if (gegnerzug == 7)
                        {
                            feldzug = feld7;
                        }
                        if (gegnerzug == 8)
                        {
                            feldzug = feld8;
                        }
                        if (gegnerzug == 9)
                        {
                            feldzug = feld9;
                        }

vollkommen unabhängig davon, was es ist: es sieht sehr suboptimal aus. :autsch:

 

[Schrubber]

Aber es funktioniert einwandfrei

 

[eRaaaa]

Aber es funktioniert einwandfrei

in einer endlosschleife?

 

[0x7F800000]

Aber es funktioniert einwandfrei

Das spielt keine rolle^^ Es sieht trotzdem suboptimal aus.

 

[Schrubber]

Nö. Es ist niemals ne Endlosschleife

Er sagt ja, dass gegnerzug der zufallszahl zugeordnet wird. Daraufhin wird gegnerzug dem Feldzug zugeordnet, welcher am Ende prüft, ob das Feld den Wert "_" hat. Sonst kommt

	            if (feldzug != "_")
	            {
	            	System.out.println("Das Feld ist bereits vergeben. Bitte versuche es erneut!");
	            }

 

[razvan]

Würde dir raten eine Select/Case Anweisung zu benutzen.

 

[0x7F800000]

Würde dir raten eine Select/Case Anweisung zu benutzen.

So, jetzt bin ich aber gespannt, wieviel schlimmer man es noch machen könnte^^

 

[faetzminator]

:lol::lol: Ich finde die Ideen der Anfänger immer wieder amüsant. Ein Tip an Schrubber und razvan: Schaut euch Arrays, Enums, Lists etc. an Wenn z.B. feld1 - feld<n> als Array oder List definiert würde, könnte man [c]feldzug = feld[gegnerzug][/c] bzw. [c]feldzug = feld.get(gegnerzug)[/c] verwenden.

 

[Das Brot]

		  	            if (gegnerzug == 1)
		  	            {
		  	            	feldzug = feld1;
		  	            }
		  	            if (gegnerzug == 2)
		  	            {
		  	            	feldzug = feld2;
		  	            }
		  	            if (gegnerzug == 3)
		  	            {
		  	            	feldzug = feld3;
		  	            }
		  	            if (gegnerzug == 4)
		  	            {
		  	            	feldzug = feld4;
		  	            }
		  	            if (gegnerzug == 5)
		  	            {
		  	            	feldzug = feld5;
		  	            }
		  	            if (gegnerzug == 6)
		  	            {
		  	            	feldzug = feld6;
		  	            }
		  	            if (gegnerzug == 7)
		  	            {
		  	            	feldzug = feld7;
		  	            }
		  	            if (gegnerzug == 8)
		  	            {
		  	            	feldzug = feld8;
		  	            }
		  	            if (gegnerzug == 9)
		  	            {
		  	            	feldzug = feld9;
		  	            }

Das würde ich aber ändern in :

		  	            switch(gegnerzug){
                                    
                                        case 1: feldzug = feld1; break;
                                        case 2: feldzug = feld2; break;
                                        case 3: feldzug = feld3; break;
                                        case 4: feldzug = feld4; break;
                                         ....etc
                                    }

Das ist leserlicher.

Ich mein für was hat man den die Mehrfachauswahl

mfg Brot

 

[maki]

Abgesehen von den mehr als seltsamen If-Abfragen sollten for bzw. while Schleifen gegenüber do-while Schleifen bevorzugt werden.

 

[faetzminator]

@Das Brot: Na dann drück mal [c][ctrl]+[shift]+[f][/c] Wie gesagt, mit meiner Idee liesse sich das ohne Redundanzen programmieren - mit deiner hast du genau gleich viele wie jener des TO.

 

[eRaaaa]

@Das Brot: Na dann drück mal [c][ctrl]+[shift]+[f][/c] Wie gesagt, mit meiner Idee liesse sich das ohne Redundanzen programmieren - mit deiner hast du genau gleich viele wie jener des TO.

hätte aber zumindestens nen minimalen performancegewinn (ausser beim worst case ??), da der TO ja wirklich durch JEDE if-abfrage durchgeht (nicht mal else if !! )

 

[0x7F800000]

Wenn im code aus irgendeinem Grund zahlen größer als 1 vorkommen, sollte man an sie gedanklich 3 Nullen anhängen, und sich überlegen, ob der code dann immer noch genau so kurz sein würde. Sollte die Länge des codes irgendwo wie O(n) zur Länge irgendeiner eingabe wachsen, dann hat man den Sinn des Programmierens verfehlt.

Das ist leserlicher.

Gedankenexperiment: 9 durch 9000 ersetzen.

switch(stuff){
   case 1: ...
   case 2: ...
   case 3: ...
  ....//37 kilometer weiter
   case 9000: ...
}

"Leserlicher" hin oder her: jedem, der versucht diesen Mist zu lesen, fällt das Mausrad ab. :autsch:

 

[Leroy42]

	            if (feldzug != "_")
                       ...

Und du schreibst wieder nicht equals("_") obwohl
eRaaaa dich schon vorher darauf hinwies.

 

[0x7F800000]

wenn das eh nur ein haufen konstanter werte ist, die vorher alle festgelegt sind, dann sollte man es eigentlich doch lieber bei == lassen, aber statt strings dann irgendwelche sinnvolleren enums verwenden o.ä.