Что означает предупреждение: не все пути управления возвращают значение? (С++)

Ваш компилятор недостаточно умен, чтобы принять во внимание, что единственными двумя вариантами для side являются left и right, поэтому он считает возможным выполнение ни одного оператора return. Когда side не является ни left, ни right, ваша функция не говорит, какое значение возвращать.

Ошибка означает, что если сторона не является ни левой, ни правой, ваша функция не вернет значение - либо сторона объявлена ​​неправильно, либо ваше перечисление. Перечисление должно быть определено как

enum orientation {left, right};

Поэтому попробуйте изменить свою структуру ориентации на это.

Если side не является left или right, то возвращаемое значение не определено.

Несмотря на то, что orientation является перечислением только с двумя значениями (прямо сейчас), оно может иметь другое значение по любой из следующих причин:

• В будущем вы можете изменить заголовок, чтобы включить другие значения в перечисление, поэтому защитное программирование предполагает, что это произойдет (и ваш компилятор сейчас мил и предупреждает вас).
• side может быть неинициализирован, поэтому это не может быть ни left, ни right
• side могло быть присвоено другое значение посредством приведения типов, например. *((int*)&side) = 2

Возможные решения включают в себя:

• Замените второй if на else, как предложено sth.

• Измените его на:

  if(side == left) {
      return ...;
  } else if(side == right) {
      return ...;
  } else {
      ...handle error...
  }
  

Предупреждение означает, что ваш метод можно пройти без возврата какого-либо явного значения. С вашим кодом:

std::ostream& hand::show(std::ostream& os) const
{
    if(side == left)
    {
        return os<<display[0]<<display[1]<<display[2]<<display[3]<<display[4];
    }
    if(side == right)
    {
        return os<<display[4]<<display[3]<<display[2]<<display[1]<<display[0];
    }
}

если side != left и side != right, то ничего не возвращаете. Обычный способ решения этой проблемы — предположить, например, что если не «слева», то всегда считать «справа»:

std::ostream& hand::show(std::ostream& os) const
{
    if(side == left)
    {
        return os<<display[0]<<display[1]<<display[2]<<display[3]<<display[4];
    }
    return os<<display[4]<<display[3]<<display[2]<<display[1]<<display[0];
}

Вы можете сделать то, что сказано, или, поскольку в этом случае вы действительно возвращаете одно и то же в любом случае...

std::ostream& hand::show(std::ostream& os) const
{
    if(side == left)
    {
        os<<display[0]<<display[1]<<display[2]<<display[3]<<display[4];
    }
    else
    {
        os<<display[4]<<display[3]<<display[2]<<display[1]<<display[0];
    }
    return os;
}

Чтобы избавиться от предупреждения, замените второй if на else:

std::ostream& hand::show(std::ostream& os) const
{
    if(side == left)
    {
        return os<<display[0]<<display[1]<<display[2]<<display[3]<<display[4];
    }
    else
    {
        return os<<display[4]<<display[3]<<display[2]<<display[1]<<display[0];
    }
}

Как предполагают другие, проблема в том, что ваш side может быть ни left, ни right.
Вы можете изменить свою функцию, чтобы она выполняла одно из следующих действий:

1. замените второй оператор if на else или удалите условие целиком, так как если сторона не левая, она должна быть правой.
2. следуйте предложению Натаниэля Флата и измените тип ориентации на перечисление.
3. вызвать исключение в качестве последнего оператора функции.