Marlin固件中温度报告冗余问题的分析与解决

问题背景

在Marlin固件（一个广泛应用于3D打印机的开源固件）中，用户发现当使用M109和M190命令等待加热时，温度报告会出现冗余现象。具体表现为：当启用M155自动温度报告功能后，加热等待过程中会同时触发两种温度报告机制，导致串口输出中出现重复的温度数据。

问题现象

正常情况下，温度报告应该如下简洁：

T:45.79 /0.00 B:59.99 /60.00 @:0 B@:20
T:45.64 /0.00 B:60.01 /60.00 @:0 B@:18
...

但实际观察到的却是每行温度数据都被重复输出：

T:44.95 /70.00 B:59.99 /60.00 @:127 B@:20 W:?
T:44.95 /70.00 B:59.99 /60.00 @:127 B@:20
T:45.21 /70.00 B:60.00 /60.00 @:127 B@:19 W:?

技术分析

这个问题源于固件中温度报告机制的设计缺陷：

1. 双重报告机制：M155命令设置的自动温度报告和M109/M190命令内部的温度报告同时工作，没有相互协调。

2. 代码冗余：在temperature.cpp文件中，加热等待循环中包含了独立的温度报告逻辑，这部分功能与M155的功能重复。

3. 设计不一致：温度报告的格式处理分散在多个地方，特别是"W:x"（等待时间）的附加信息处理不够统一。

解决方案

经过开发者讨论，提出了以下改进方案：

1. 移除冗余代码：删除M109/M190命令中独立的温度报告逻辑，完全依赖M155的自动报告机制。

2. 统一报告格式：将"W:x"等待时间信息整合到统一的温度报告函数中，确保格式一致性。

3. 保持功能完整：确保在禁用M155自动报告时，M109/M190仍能提供必要的温度反馈。

实现细节

关键修改包括：

1. 从加热等待循环中移除独立的温度报告代码块
2. 将等待时间显示逻辑整合到print_heater_states()函数
3. 确保所有加热等待命令（M109、M190等）使用统一的报告机制
4. 优化串口输出效率，避免不必要的重复数据

影响评估

这一改进将带来以下好处：

1. 减少串口数据量：避免重复报告可以显著降低串口通信负担。
2. 提高代码可维护性：统一温度报告机制使代码更清晰、更易于维护。
3. 保持功能兼容性：虽然改变了内部实现，但用户可见的功能和行为保持不变。

总结

Marlin固件中的温度报告冗余问题是一个典型的代码功能重叠案例。通过分析问题根源并实施统一的解决方案，不仅解决了当前问题，还提高了代码的整体质量。这一改进展示了开源社区如何通过协作来优化和完善复杂系统的设计。

对于3D打印机用户和开发者而言，理解这类底层机制有助于更好地诊断问题并参与固件改进。这也体现了良好代码架构设计的重要性，特别是在实时控制系统中。