Bedingungsoperatoren ? :

[siba]

Hallo!

Welchen Fehler begehe ich bei der Umsetzung von "? :" ?

public class IfErsatz {

	
	public static void main(String[] args) {
		int x = 5;
		(x == 5) ? System.out.println("Ja"); : System.out.println("Nein");

	}

}

 

[byte]

int x = 5;
System.out.println((x==5) ? "Ja" : "Nein");

 

[thE_29]

Tjo, jetzt wollte ich schon sagen, schau doch in die FAQ (ternärer Operator)

Aber erstens hat das irgendein schlaumod (beni :bae net dazugeinbaut, und 2tens wer weiß schon dass das so heißt

http://www.java-forum.org/de/viewtopic.php?t=11550

 

[me.toString]

Die Antwort von "byto" ist zwar richtig ... aber dennoch ein wenig dürftig ausgefallen?
Das dein ? : -Konstrukt nicht funktioniert hat (Fehlermeldung b eim Kompilieren ist auch recht eigenartig), ist ganz einfach.
Die beiden Möglichkeiten (also rechts und links vom Doppelpunkt) müssen einen Wert zurückliefern, der dann gleich weiterverwendet wird. System.out.println() gibt dir aber nix zurück.
Du musst also den Rückgabewert des Konstruktes auffangen und weiterverarbeiten:

String entscheidung = ( x == 5 ? "Ja" : "Nein" ); 
System.out.println( entscheidung );

Sieht, so finde ich zumindest, auch gleich viel aufgeräumter aus. ... natürlich könnte man auch beides in eine Zeile quetschen ... aber wir wollen ja keinen Wettbewerb im Kurzcodeschreiben gewinnen.

Michael

 

[byte]

... natürlich könnte man auch beides in eine Zeile quetschen ... aber wir wollen ja keinen Wettbewerb im Kurzcodeschreiben gewinnen.

Naja, das ist ja grad der Vorteil an dieser Notation, dass sie eben kurz ist. Wenns um Übersichtlichkeit geht, sollte man eh die normale If-Then-Else Notation verwenden.

 

[thE_29]

Jop

Aber diese ternären Dinger sind super

bestes Bsp.:

    for(int x = 0, y = 1; x != getRunningServicesCount(); y++)
      strRet[x] = isServiceRunning(y) ? ((++x == x) ? getServiceName(y) : "") : "";

 

[Murray]

 (++x == x)

Was bezweckst Du denn mit diesem Konstrukt?

 

[AlArenal]

Wollen wir jetzt auch in Java Wettbewerbe für unleserlichen Code einführen?

 

[thE_29]

x erhöhen ^^

Verstehst du das ganze


wenn isServiceRunning true zurückgibt, wird x erhöt und es soll aber ein string zurückgegeben werden...

das könnste könnte man so aufteiln

if(isServiceRunning(y)
{
 strRet[x] = getServiceName(y);
 x++;
}

es wird ja so durchgegangen

x = 0

strRet[0] = isServiceRunning(0) ? ((1 == 1) ? getServiceName(0) : "") : "";

vorne hatte x noch den niedirigern wert und hinten wirds erhöht ^^



@Al: Das habe ich eigentlich eingebaut um meine Mitarbeiter zu ärgern/verwirren

 

[norman]

Jop

Aber diese ternären Dinger sind super

bestes Bsp.:

    for(int x = 0, y = 1; x != getRunningServicesCount(); y++)
      strRet[x] = isServiceRunning(y) ? ((++x == x) ? getServiceName(y) : "") : "";

naja, ich finde du hast das da verkompliziert:

for(int x = 0, y = 1; x != getRunningServicesCount(); y++) {
	    	 
	    	 // deine Var.
	    	 strRet[x] = isServiceRunning(y) ? ((++x == x) ? getServiceName(y) : "") : "";
	    	 
	    	 // deine Var in langform
	    	 if (isServiceRunning(y)){
	    		 if (++x == x) {
	    			 strRet[x] = getServiceName(y);
	    		 } else {
	    			 strRet[x] = "";
	    		 }
	    	 } else {
	    		 strRet[x] = "";
	    	 }
	    	 
	    	 // verkürze langform
	    	 if ((isServiceRunning(y)) && (++x == x)) {
	    		 strRet[x] = getServiceName(y);
	    	 } else {
	    		 strRet[x] = "";
	    	 }
	    	 
	    	 // kurze kurzform
	    	 strRet[x] = ((isServiceRunning(y)) && (++x == x))? getServiceName(y) : ""; // <== !!!
	     }

so lässt sich das dann auch gleich besser verstehen

 

[Murray]

Der innere ternäre Operator

((++x == x) ? getServiceName(y) : "")

ist aber doch Quatsch; wie Du ja selbst erklärst, ist ++x == x ja immer true.
Insofern sollte sich das Ziel doch auch so erreichen lassen:

for(int x = 0, y = 1; x != getRunningServicesCount(); y++)
      if ( isServiceRunning(y)) strRet[x++] = getServiceName(y);

//EDIT: Typo

 

[thE_29]

Jo, nur habe ich dann eben wieder ein if

Es ging darum, kein if zu haben ^^

 

[norman]

wie gesagt,

strRet[x] = ((isServiceRunning(y)) && (++x == x))? getServiceName(y) : "";

 

[Murray]

Wenn schon kompliziert:

strRet[x++] = isServiceRunning( y) ? getServiceName(y) : strRet[--x];