Соответствующий формат Ternary Operator

Question

Это то, что у меня есть. Я не уверен, как правильно его написать. Я попробовал Googling, но безрезультатно. Пожалуйста, не съеживаться слишком много:

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" 
     : ""; 

Что я хотел это сделать, это:

// 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; 

Любая помощь будет оценена. Я просто хочу узнать, как я могу писать свои программы, чтобы быть более эффективными и меньшими. Поблагодарили бы за отзыв.

Answer 1

Помещенный некоторые скобки вокруг всего выражения, в противном случае вы будете в конечном итоге печать булевы значения:

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 

Правильный код:

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

более эффективным

не связаны с тем, что вы там делаете (если под «эффективным» вы имеете в виду лучшие характеристики во время выполнения).

меньше

Скромная цель, но не если читаемость дотягивает из-за этого (и даже меньше, так что, если из-за синтаксической сложности ... недостающего круглые скобки ... конечный результат неправильный).

Помните: Код написан для людей, чтобы читать.

Вы должны, вероятно, придерживаться подхода if и else, который вы также показываете в вопросе. Тем не менее, ИМХО «хороший» подход (если вам действительно нужно абстрагироваться над этим) будет упаковать его в какую-нибудь функцию:

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(); 
} 

Используется как

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

ли это на самом деле помогает с читаемости это выбор, который я оставляю читателю.