“如果”条件未正确评估

Question

要从 shell 接收包含在 (0, 8) 区间内的整数值，我将 cin 转换为数组的 uint8_t 元素，执行以下操作：

    char answer;

    do
    {
        // Instructions
        std::cout << "Linear actuator resolution is:\n"
                  << "\u0394x = \u03B1r/2^i, with \u03B1 = " << std::to_string(ALPHA)
                  << " degrees, r = " << std::to_string(pulleyR) << " m and i in [0 : 8]\n";

        // Parameter selection
        std::cout << "Please enter a valid value for param 'i': ";
        std::cin >> Engines_uSteppingLevel[RAIL];
        if(Engines_uSteppingLevel[RAIL] > (RES_LEVELS - 1))
            // Wrong selection, repeat question
            std::cout << '\r';
        else
        {
            // Print out the selected resolution and ask user to confirm or restart selection
            std::cout << "Selected linear resolution: " << std::fixed << std::setprecision(4)
                      << ALPHA*pulleyR/(1<<Engines_uSteppingLevel[RAIL])
                      << "m, enter 'y' to confirm, any other key to change selection ";
            std::cin >> answer;
            if(answer == 'y')
                break;
        }
    }while(true);

即使我输入了正确的值，循环也不会中断。

输入：

std::cout << Engines_uSteppingLevel[RAIL] << ' ' << (RES_LEVELS - 1) << '\n';

代替：

   std::cout << '\r';

shell 输出是：

Please enter a valid value for param 'i': 0
0 8
Please enter a valid value for param 'i': 3
3 8

这没有意义。

Answer 1

您在这里遇到的问题在于 C++ 处理 uint8_t 类型的方式：它认为它是一个字符。如果您将调试输出语句更改为：

std::cout << static_cast<int>(Engines_uSteppingLevel[RAIL]) << ' ' << (RES_LEVELS - 1) << '\n';

您会看到，当您输入“0”时，ASCII“0”存储在您的 uint8_t 变量中，其值为 48。类似地，当您输入“3”时，您存储的是 51，以此类推。

对您有用的最简单的解决方案是使用字符转换，如下所示：

Engines_uSteppingLevel[RAIL] -= '0';

请注意，此解决方案仅适用于单个数字值。更健壮、可扩展的解决方案将涉及 std::string 或 char 缓冲区，然后调用 atoi、strtoul、std::stoul 或类似的东西。