Proper Ternary Operator Format

Question

Dies ist, was ich habe. Ich bin mir nicht sicher, wie ich es richtig schreiben soll. Ich habe versucht zu googlen, aber ohne Erfolg. Bitte erschaudern nicht zu viel:

cout << (guess > randomNumber) ? "\nWhoops! Try again!\n You guessed higher than the random number!\n\n" 
     : (guess < randomNumber) ? "\nWhoops! Try again!\n You guessed lower than the random number!\n\n" 
     : ""; 

Was es ich bin zu wollen dies tun:

// Gives hint that inputted number is higher or lower 
    // than inputted number 
    if (guess > randomNumber) 
     cout << "\nWhoops! Try again!" 
     << " You guessed higher than the random number!\n" 
     << endl; 
    else if (guess < randomNumber) 
     cout << "\nWhoops! Try again!" 
     << " You guessed lower than the random number!\n" 
     << endl; 

Jede mögliche Hilfe würde geschätzt. Ich möchte nur lernen, wie ich meine Programme effizienter und kleiner schreiben kann. Würde mich über Feedback freuen.

Answer 1

einige Klammern um den gesamten Ausdruck, sonst Boolesche Werte Druck am Ende Sie gehen:

int guess = 10; 
    int randomNumber = 9; 

    cout << (guess > randomNumber) ? "\nWhoops! Try again!\n You guessed higher than the random number!\n\n" 
      : (guess < randomNumber) ? "\nWhoops! Try again!\n You guessed lower than the random number!\n\n" 
      : "" ; 

// Output: 1 

Proper Code:

int guess = 10; 
    int randomNumber = 9; 

    cout << ((guess > randomNumber) ? "\nWhoops! Try again!\n You guessed higher than the random number!\n\n" 
      : (guess < randomNumber) ? "\nWhoops! Try again!\n You guessed lower than the random number!\n\n" 
      : ""); // Notice the brackets! 

/*Output: 
Whoops! Try again! 
You guessed higher than the random number!*/ 

Answer 2

effiziente

zu nicht verwendet, was Sie da tun (wenn von „effizient“ Sie bessere Laufzeitmerkmale bedeuten).

kleine

Ein bescheidenes Ziel, aber nicht wenn Lesbarkeit kurz, weil es fällt (und noch weniger, wenn aufgrund der syntaktischen Komplexität ... fehlenden Klammern ... das Endergebnis ist falsch).

Erinnern: Der Code ist für Menschen geschrieben zu lesen.

Sie sollten wahrscheinlich mit der if und else Ansatz bleiben, die Sie auch in der Frage zeigen. Das heißt, ein IMHO „gut“ Ansatz (wenn Sie wirklich abstrakt über diese benötigen) wäre es in eine Funktion zu packen:

template<class T, class X, class Y, class Z> 
void comp_if(T value, T reference, X less, Y equal, Z greater) { 
    if (value < reference) less(); 
    else if (value > reference) greater(); 
    else equal(); 
} 

Gebraucht wie

// missing real macros a lot 
comp_if(foo, bar, 
    []() {cout << "less"; }, 
    []() {cout << "equal";}, 
    []() {cout << "greater"}); 

Ob dies wirklich mit Lesbarkeit hilft ist eine Entscheidung, die ich dem Leser überlassen muss.