Marlin固件中M119指令的耗材传感器状态报告问题分析

问题背景

在Marlin固件的最新bugfix-2.1.x分支中，用户报告了一个关于M119指令(用于报告终端挡板状态)的问题。具体表现为耗材传感器的状态报告中缺少了"filament"标识文本，导致输出信息不完整。

问题表现

当用户执行M119指令时，输出结果中耗材传感器状态行缺少了应有的标识符：

Reporting endstop status
x_min: open
y_min: open
z_probe: TRIGGERED
: TRIGGERED  <-- 这里缺少了"filament"标识

技术分析

这个问题源于PR #27539对endstops.cpp文件的修改。原始代码中使用了明确的字符串标识：

print_es_state(READ(FIL_RUNOUT1_PIN) != FIL_RUNOUT1_STATE, F(STR_FILAMENT));

修改后的代码虽然简化了状态检测逻辑，但遗漏了字符串参数：

print_es_state(!FILAMENT_IS_OUT());  // 缺少了F(STR_FILAMENT)参数

正确的写法应该是：

print_es_state(!FILAMENT_IS_OUT(), F(STR_FILAMENT));

影响范围

这个问题影响了所有使用耗材传感器功能且更新到包含PR #27539代码版本的Marlin固件用户。虽然不影响传感器功能的实际工作，但会导致状态报告信息不完整，可能影响用户诊断和调试。

解决方案

修复方法很简单，只需在调用print_es_state函数时补充上字符串参数。开发团队应该：

1. 恢复字符串参数F(STR_FILAMENT)
2. 保持简化后的状态检测逻辑(!FILAMENT_IS_OUT())
3. 确保所有类似的传感器状态报告都包含适当的标识文本

预防措施

为避免类似问题，建议：

1. 在修改状态报告相关代码时，保持标识文本的完整性
2. 添加单元测试来验证M119指令的输出格式
3. 在代码审查时特别注意输出信息的完整性

总结

这个案例展示了即使是简单的代码优化也可能意外影响用户界面的一致性。在固件开发中，保持用户反馈信息的完整性和一致性同样重要，特别是在诊断和调试相关的功能上。开发者需要在优化代码逻辑的同时，确保不破坏已有的用户接口约定。