简化空指针检查的Do While循环实现方案咨询

我太懂你的困扰了——本来想用do-while循环避免“先处理第一个节点，再循环处理后续节点”的重复代码，结果反而因为遍历中修改节点地址的操作，还是出现了重复逻辑，完全违背了用这个循环的初衷。别着急，我有两个实用方案帮你搞定：

方案1：把节点移动操作整合到循环条件中 #

这种写法能让循环体只保留核心处理逻辑（比如打印节点），把移动指针的操作直接放到do-while的条件里，彻底消除重复代码。

假设你的节点定义是这样的：

typedef struct Node {
    int key;
    int value;
    struct Node* next;
} Node;

优化后的遍历代码可以写成：

void printBucket(Node* bucketHead) {
    if (bucketHead == NULL) {
        printf("当前桶为空\n");
        return;
    }

    Node* current = bucketHead;
    do {
        // 这里只写核心的节点处理逻辑，比如打印
        printf("键：%d，值：%d\n", current->key, current->value);
        // 把节点移动操作放到循环条件中，不用在循环体重复写
    } while ((current = current->next) != NULL);
}

这种写法既保证了至少执行一次（完全符合你用do-while的初衷），又没有任何重复代码。循环条件里的赋值会先执行，再判断是否为NULL，完美衔接下一轮循环，逻辑清晰又简洁。

方案2：封装遍历逻辑为通用函数 #

如果你的代码里需要多次遍历哈希表的链表（比如打印整个哈希表、释放哈希表内存等），把遍历逻辑封装成独立函数，能避免重复编写do-while代码块。

比如封装一个打印整个哈希表的函数：

#define TABLE_SIZE 10

// 先封装单个桶的遍历打印
void printBucket(Node* bucketHead) {
    if (bucketHead == NULL) {
        printf("空桶\n");
        return;
    }

    Node* current = bucketHead;
    do {
        printf("键：%d，值：%d\n", current->key, current->value);
    } while ((current = current->next) != NULL);
}

// 再封装整个哈希表的打印
void printHashTable(Node* hashTable[]) {
    for (int i = 0; i < TABLE_SIZE; i++) {
        printf("=== 桶%d ===\n", i);
        printBucket(hashTable[i]);
    }
}

之后不管你需要打印多少次哈希表，只需要调用printHashTable(hashTable)就行，不用再重复写遍历的循环代码。如果需要处理其他逻辑（比如释放节点），只需要再封装一个类似的freeBucket函数，复用同样的do-while遍历逻辑即可。

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