在使用函数前将数据加载到其静态内存中是否属于不良实践？

先看示例代码：

void myFunc (char * str) {
    static char * str_inside;

    // Load data into the static variable
    if (str != NULL) {
        str_inside = str;
        return;
    }

    /* do stuff with str_inside in consecutive calls */
}

void doStuff () {
    // Calls the function, which has data already loaded
    myFunc (NULL);
}

int main () {
    char str[] = "Hello!";

    // Load data into the function
    myFunc (str);

    doStuff ();
}

这种写法的初衷是解决doStuff无法直接访问原始字符串的问题，比如在GUI异步事件处理这类场景中，调用链没法直接传递数据。但这种做法确实属于不良实践，存在诸多明确风险：

• 线程安全隐患：静态变量是进程全局共享的，多线程环境下多个线程调用myFunc时，会直接覆盖str_inside的值，或者引发数据竞争，导致程序行为完全不可控。比如线程A刚设置好字符串，线程B立刻传入新值覆盖，后续线程A的处理逻辑就会用到错误的数据。

• 内存访问风险：示例里的str是栈上的局部变量，当main函数执行完毕或者当前作用域结束后，这块栈内存会被系统回收，但str_inside仍然指向这块已释放的内存。后续调用myFunc(NULL)时，就会访问无效内存，触发未定义行为——轻则程序崩溃，重则出现难以排查的内存乱码问题。

• 代码可读性与可维护性极差：这种隐式的状态传递完全依赖严格的调用顺序，新开发者很难快速理解myFunc的逻辑，更无法预知调用顺序错误会引发的问题。如果不小心在未传入有效字符串的情况下调用myFunc(NULL)，程序直接出错，排查难度极高。

• 丧失可重入性：因为依赖静态变量保存状态，myFunc无法被重入。比如你刚设置了一个字符串准备处理，中途又调用myFunc传入另一个字符串，之前的状态会直接丢失，无法同时处理多组独立的数据。

替代方案 #

可以用更清晰、安全的方式传递状态：

• 直接修改doStuff的接口，让它接收需要处理的数据作为参数，比如void doStuff(char* str)，逻辑直观且无隐式依赖。
• 异步场景下，使用带上下文参数的回调函数，把需要的数据和回调逻辑绑定在一起，这也是GUI框架常用的做法。
• 若确实需要全局状态，建议使用显式的全局变量，而非藏在函数内部的静态变量，至少能让开发者一眼看到状态的存在，提前规避风险。

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