ESP32/Arduino中client.readStringUntil与substring判断逻辑出错排查

你的代码存在两个核心问题，导致继电器控制逻辑失效：

1. 循环结束后line仅保留最后一行回复 #

你在while(client.available())循环里反复给line赋值，每次读取新行都会覆盖之前的内容。循环结束后，line存储的是服务器回复的最后一行，而不是包含"1"或"0"的目标行。如果服务器返回多行内容（比如HTTP响应头+状态行），你的判断逻辑根本看不到前面的指令行。

2. 字符串匹配的冗余与潜在格式问题 #

• line.substring(0)完全等价于直接使用line，这种写法没必要，直接用line == "1"更简洁。
• readStringUntil('\r')读取到回车符后停止，但服务器回复的行末尾通常还会跟着换行符\n，导致line实际内容可能是"1\n"而不是纯"1"，直接用等于判断会匹配失败。

修复后的代码示例 #

把判断逻辑移到循环内部，同时处理换行符和空白字符：

String line;
// 读取服务器所有回复，逐行处理
while(client.available()){
  line = client.readStringUntil('\r');
  // 移除末尾可能存在的换行符和空白字符
  line.trim();
  Serial.println(line);

  // 继电器1控制逻辑移到循环内，逐行检查
  if (line == "1") {
    Serial.println("Rele 1 ON");
    digitalWrite(Rele_1, LOW);
    myBr1 = 1;
  } else if (line == "0") {
    Serial.println("Rele 1 OFF");
    digitalWrite(Rele_1, HIGH);
    myBr1 = 0;
  }
  // 其他else逻辑可以在这里补充
}

额外优化建议 #

• 如果服务器回复的指令是固定格式（比如前缀+状态，比如"RELAY1:1"），可以用startsWith()来匹配，避免误判其他包含"1"/"0"的行：

  if (line.startsWith("RELAY1:1")) {
    // 执行继电器1开启逻辑
  }
  

• 避免使用readStringUntil()的阻塞特性（如果服务器超时未回复，会一直等待），可以考虑设置超时或者用readBytesUntil()配合缓冲区，更高效稳定。

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