20、代码转换与重构实践

代码转换与重构实践

1. 代码异味处理

代码中常见的一种异味是注释掉的代码。对于这类代码，解决方法很简单：直接删除。因为可以从源代码仓库中恢复这些代码。

另一种常见问题是条件编译。代码中充斥着条件编译会使其难以理解。虽然有时条件编译不可避免，但在处理平台差异时，应将其作为最后的手段。例如，在查看代码时，发现大量代码被 #ifdef BOARD_V1 或 #ifdef BOARD_V2 包围， BOARD_V2 的出现次数多达数千次，代码规模从版本 1 到版本 2 几乎翻倍。这种暴力的方式可能不如改进设计，将不同板子的共性整合、差异分离的方法有效。建议使用链接器或函数指针来隔离平台依赖，如 TimeService 和 LightDriver 那样，将平台相关代码分组，使平台相关和无关代码分离。

2. 重构长函数示例

为了让大家感受重构，下面以一个长函数为例进行重构。

static void processEventsDueNow(Time * time, ScheduledLightEvent * event)
{
    Day today = time->dayOfWeek;
    int minuteOfDay = time->minuteOfDay;
    if (event->id != UNUSED)
    {
        Day day = event->day;
        if ( (