C函数中无效参数的处理：最佳实践是什么？

针对你不想把函数从void改成int返回错误码的需求，这里有几个实用的方案：

1. 用断言抓调试阶段的错误 #

直接用assert宏在调试时强制检查参数，一旦无效就终止程序并输出错误信息，能快速定位调用方的问题：

#include <assert.h>

void set_dimensions(int width, int height, struct rectangle_t* rectangle){
    assert(width > 0 && height > 0 && "Width and height must be positive");
    rectangle->width = width;
    rectangle->height = height;
}

注意：发布版本中assert会被编译为空，不会做任何检查，所以这个方案只适合调试阶段排查问题，不能作为线上运行时的校验手段。

2. 用错误回调让调用方处理问题 #

提前定义一个错误回调函数，遇到无效参数时调用它，把错误信息抛给调用方自己处理，函数本身不用改返回值：

// 先定义错误回调的类型
typedef void (*error_handler_t)(const char* msg);
static error_handler_t g_error_handler = NULL;

// 给调用方提供设置回调的接口
void set_error_handler(error_handler_t handler){
    g_error_handler = handler;
}

void set_dimensions(int width, int height, struct rectangle_t* rectangle){
    if(width <= 0 || height <= 0){
        if(g_error_handler){
            g_error_handler("Invalid dimensions: width and height must be positive");
        }
        // 这里别乱改原有值，直接返回就行，避免意外
        return;
    }
    rectangle->width = width;
    rectangle->height = height;
}

调用方可以注册自己的处理逻辑，比如打日志、记录错误码，这样既不用改函数签名，又能让错误处理更灵活。

3. 保留原有值+打错误日志 #

如果项目有日志系统，遇到无效参数时直接打印警告日志，同时不修改rectangle的原有值，比强制重置为1更透明：

#include <stdio.h>

void set_dimensions(int width, int height, struct rectangle_t* rectangle){
    if(width <= 0 || height <= 0){
        fprintf(stderr, "Warning: Invalid dimensions (%d, %d) ignored. Values must be positive.\n", width, height);
        return; // 不修改原有尺寸，避免调用方不知情
    }
    rectangle->width = width;
    rectangle->height = height;
}

这样调用方会立刻收到警告，不会因为被偷偷改成1而导致后续逻辑莫名出错。

为什么不推荐强制重置为1？ #

强制把无效参数改成1会隐藏调用方的错误——调用方可能以为自己传的参数生效了，直到后续逻辑出问题才发现，反而增加调试难度。这种“静默修复”往往会埋下隐患，不如把问题暴露出来让调用方处理。

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