c - DRY 如何与 "Separation of concern"和 "One function, one task"组合？

Question

<分区>

Answer 1

对于 C 代码，经常遇到 NULL 指针检查是完全正常的，尤其是涉及到函数参数时。就个人而言，我更愿意让调用者解决问题，如:

if (p == NULL) {
    /* maybe do some cleanup and then: */
    return errcode;
}

公共(public)函数，即作为 API 一部分的函数，应始终检查 NULL 指针。指定为 static 的函数可能会被 IMO 放弃这些检查。最后，总是有 assert()。编译器标志 -NDEBUG 可以抑制这些检查。我在 static 函数中使用 assert() 而不是 if 语句，并在“公共(public)”函数中用于显示调用者没有进行测试的测试实际上从整体上理解 API，例如在链表库中:

void list_print(list **l)
{
    assert(l != NULL);    /* no valid list passed by reference can ever be NULL */

    if (*l == NULL)       /* but it can be empty */
        return;

    /* print list */
}

至于你的第二个问题，我可以看到三个选项:

1) 让一切保持原样 - 毕竟，它正在运行。

2) 引入新功能:

int function_1_2a_3();
int function_1_2b_3();

3) 引入新的参数化函数:

int function_1_2_3(int type);

就个人而言，我更喜欢后一种方法，但这实际上只是一种风格问题。