Question

我使用Arduino编程，我的程序包含很多while循环。当Arduino收到一个角色时，它必须进行一些计算并从它收到角色时所处的循环中断开。我将给你一个更简单的例子（假设i和j设置为0）：

while (i < 256)
{
    // some calculations #1
    i++;

    if (Serial.available() > 0)
    {
        setStringOne = "string one"
        setStringTwo = "string two"
        setStringThree = "string three"
        setStringFour = "string four"

        break;
    }
}

while (j < 256)
{
    // some calculations #2
    j++;

    if (Serial.available() > 0)
    {
        setStringOne = "string one"
        setStringTwo = "string two"
        setStringThree = "string three"
        setStringFour = "string four"

        break;
    }
}

你可以看到，在这两种情况下，我在if句子中使用了相同的代码。我希望它能够写出这样的东西。

while (i < 256)
{
    // some calculations #1
    i++;

    if (Serial.available() > 0)
        checkAndBreak();
}

while (j < 256)
{
    // some calculations #2
    j++;

    if (Serial.available() > 0)
        checkAndBreak();
}

void checkAndBreak()
{
    if (Serial.available() > 0)
    {
        setStringOne = "string one"
        setStringTwo = "string two"
        setStringThree = "string three"
        setStringFour = "string four"

        break;
    }
}

使用外部方法中断循环。

它给了我一个错误＆＃34; break语句不在循环内或切换＆＃34;，这是预期的，因为它不知道要从哪个循环中断，但我只是想知道是否可以制作这些方面的东西。

提前致谢！

Answer 1

你不能那样打破，所以没办法。只需平衡每种方法的作用：

while (i < 256)
{
      if (Serial.available() > 0)
      {
             setThoseStrings();
             break;
      }
      i++;
}

可选地

while (i < 256)
{
        if (checkSerialAndSetStrings())
        {
               break;
        }
        i++;
}

这看起来更短但是如果您需要在其他情况下设置字符串（例如，在计时器用完时设置它们），您只是浪费时间从checkSerialAndSetStrings删除序列检查并更新您的代码。我和＃1一起去。

Answer 2

你可以用return做类似的技巧，虽然对于一些老派程序员来说可能看起来不那么明显。假设Serial.avaiable()返回无符号整数，希望在计算之前检查它，并且做实际上需要更新当您更新字符串时i,j计数作为后效应，我会这样做：

// Obviously this function somehow has // access to the strings being modified // And possibly to some other state affected by calculations #1 and #2 // I'm leaving it similar to how you've written it // for the sake of clarity bool checkAndBreak() { if (!Serial.available()) return false; setStringOne = "string one"; setStringTwo = "string two"; setStringThree = "string three"; setStringFour = "string four"; return true; } size_t i = 0, j = 0; do { // calculations #1 } while (++i, !checkAndBreak() && i < 256); do { // calculations #2 } while (++j, !checkAndBreak() && j < 256);

请注意，最后i,j会产生与示例中相同的值。另一个有趣的方式是：

// define `i,j` outside of the loops if you need their final values // !i and !j checks are to make sure checkAndBreak() // won't execute before the loop body for (size_t i = 0; (!i || !checkAndBreak()) && i < 256; ++i) // calculations #1 for (size_t j = 0; (!j || !checkAndBreak()) && j < 256; ++j) // calculations #2

您还可以将计算提取到函数中：

void calc1(size_t i) { ... } void calc2(size_t j) { ... } for (size_t i = 0; i < 256 && (calc1(i++), !checkAndBreak()); ); for (size_t j = 0; j < 256 && (calc2(j++), !checkAndBreak()); );

这样您先进行< 256检查，然后使用旧calc1/calc2值执行i/j，之后i/j值会增加，然后检查是执行。如果检查返回true，则整个for循环终止。

请注意这些做法与KISS原则不一致，因此您可以调整代码，前提条件和后期效果来简单地编写

for (size_t i = 0; i < 256; ++i) { // do the calculations #1 // now check the side effects of those calculations if (serialCheckedAndStateChanged()) break; } // The same for #2

Answer 3

这是一个很好的例子，为什么你不应该在方法/过程中使用“break”，“continue”或多个return语句。您无法提取减少冗余的方法。我的建议是重新编写代码，使其无需使用break-statement。

我会尝试以下方法：

do {
    // some calculations #1
    i++;
} while (i < 256 && Serial.available() == 0)

if (Serial.available() > 0)
{
    setStringOne = "string one"
    setStringTwo = "string two"
    setStringThree = "string three"
    setStringFour = "string four"
}

do {
    // some calculations #2
    j++;

} while (j < 256 && Serial.available() == 0)

if (Serial.available() > 0)
{
    setStringOne = "string one"
    setStringTwo = "string two"
    setStringThree = "string three"
    setStringFour = "string four"
}

现在您可以改进代码：

提取方法并将模板模式应用于计算代码（尽可能在Arduino中）以减少冗余。

提取方法的示例：

void determineString()
{
    if (Serial.available() > 0)
    {
        setStringOne = "string one"
        setStringTwo = "string two"
        setStringThree = "string three"
        setStringFour = "string four"
    }
}

使用外部方法从循环中断

3 个答案: