Замечания по FlipByte

[alexey-malov]

bool IsInputDataCorrect(int inputNumber) 
{
    bool CheckInputDataCorrection = true;
    if ((inputNumber < 0) || (inputNumber > 255))
    {
        std::cout << "Invalid argument value. It must be beetwen 0 and 255. \n";
        CheckInputDataCorrection = false;
    }
    return CheckInputDataCorrection;
}

• [x] Имя функции должно быть более конкретным

  int inputNumber;
  ...
  uint8_t byte;
  if (!ConvertToByte(inputNumber, byte))
  {
  // сообщаем об ошибке
  }

• [x] Программа не соответствует принятому стилю кодирования. Ознакомьтесь с правилами именования переменных, функций, классов, расстановкой скобок, знаков препинания и отступов.
• [x] нет необходимости в переменной CheckInputDataCorrection , когда можно просто вызвать return

[alexey-malov]

    int outputNumber = FlipByte(inputNumber);

• [x] Тут лучше явно параметр FlipByte преобразовать при помощи static_cast к uint8_t

[alexey-malov]

int StringToInteger(char* input) 
{
    try
    {
        int inputNumber = std::stoi(input);
        return (inputNumber);
    }
    catch (const std::exception& e)
    {
        std::cout << e.what() << std::endl;
        return 1;
    }
}

bool ConvertToByte(int inputNumber)
{
    if ((inputNumber < 0) || (inputNumber > 255))
    {
        std::cout << "Invalid argument value. It must be beetwen 0 and 255. \n";
        return false;
    }
    return true;
}

    int inputNumber = StringToInteger(argv[1]);
    if (!ConvertToByte(inputNumber))
    {
        return 1;
    }

• [x] inputNumber внутри StringToInt - неудачное имя
• [x] Программа не отлавливает ошибки преобразования строки в число

[alexey-malov]

bool ConvertToByte(int inputNumber)
{
    if ((inputNumber < 0) || (inputNumber > 255))
    {
        std::cout << "Invalid argument value. It must be beetwen 0 and 255. \n";
        return false;
    }
    return true;
}

• [x] Функцию переименовать в FitsToByte

• [x] Вывод информации об ошибке лучше выполнять в main