C# Foreach循环每次循环条件检查异常问题求助

我来帮你梳理下这个问题的核心和解决思路～

首先，你遇到的问题本质是：第一次循环后comm变成空字符串，导致后续循环无法正确触发向Arduino发送数据的逻辑，甚至让程序卡在循环里。大概率是因为你没有在每次循环迭代时重新读取Arduino的最新串口数据，或者在处理逻辑中不小心清空了comm却没有重新获取。

核心解决思路：每次循环都重新读取串口数据 #

要让程序每次都能检查到Arduino发送的最新comm指令，关键是不要只在循环外读取一次comm，而是把串口读取的逻辑放到循环内部，确保每次迭代都获取最新的串口输入。

示例代码调整 #

假设你的串口对象是serialPort1，SortedList的结构是<string, double>，可以参考下面的代码逻辑：

// 初始化你的数据列表
SortedList<string, double> sensorData = new SortedList<string, double>();
// 假设已经向sensorData添加了数据项

// 外层循环保持程序持续运行（根据你的实际场景调整，比如放在后台线程里）
while (true)
{
    string comm = string.Empty;
    try
    {
        // 每次循环都读取最新的串口数据，Trim去掉换行、空格等干扰符
        comm = serialPort1.ReadLine().Trim();
    }
    catch (Exception ex)
    {
        // 处理串口读取异常，比如串口断开的情况
        Console.WriteLine($"串口读取错误: {ex.Message}");
        System.Threading.Thread.Sleep(1000);
        continue;
    }

    // 检查是否收到Arduino的"OK"指令
    if (comm == "OK")
    {
        Console.WriteLine("收到OK指令，开始发送数据...");
        // 遍历SortedList发送数据
        foreach (var item in sensorData)
        {
            // 按约定格式向Arduino发送键值对数据
            serialPort1.WriteLine($"{item.Key}:{item.Value}");
            // 加短延迟确保Arduino能处理完当前数据
            System.Threading.Thread.Sleep(80);
        }
        // 发送完成后可以给Arduino发送确认信号
        serialPort1.WriteLine("DATA_SENT");
    }

    // 避免CPU占用过高，每次循环后短暂休眠
    System.Threading.Thread.Sleep(50);
}

额外注意事项 #

• 串口读取方式：如果ReadLine()会导致程序阻塞（比如Arduino没发送数据时），可以改用ReadExisting()读取缓冲区所有数据，或者设置串口的ReadTimeout属性；
• 线程安全：如果你的SortedList会被其他线程修改，遍历的时候要加锁（lock(sensorData) { ... }），避免并发访问异常；
• Arduino端配合：要确保Arduino在发送"OK"后，能正确接收你发送的数据，并且不会因为处理不及时导致串口缓冲区溢出；
• comm重置：不要在循环中随意清空comm而不重新读取，确保每次循环的comm都是最新的串口输入。

内容的提问来源于stack exchange，提问作者Tomas Michalek