C语言函数返回全局定义的字符串常量是否为合法可接受的编程风格？

首先明确结论：你这种返回全局只读字符串常量指针的思路本身是合规的，属于C/C++中可接受甚至是推荐的常见编程风格，但你当前的代码存在几处需要修正的问题。

现有代码的问题 #

• 返回值类型不匹配：你定义的SUN_NAME等常量都是const char* const类型（指针本身和指向的内容都不可修改），但函数返回值声明为char*，相当于主动丢弃了const限定符。如果调用者误修改返回的字符串内容，会触发未定义行为，必须把函数返回值类型改为const char*。
• default分支无返回值：当前如果传入的星期序号不在合法范围内，函数没有明确的返回值，会触发未定义行为。需要在default分支返回默认值（比如"Invalid"）或者空指针，建议提前做参数合法性校验。
• 全局常量冗余：如果这些星期名称常量只在这个转换函数中使用，没必要定义为全局变量，会污染全局命名空间，建议改为函数内的静态常量或者限定为当前翻译单元可见的static全局变量。

更简洁的优化实现 #

你完全可以不用单独定义7个全局常量，用静态数组映射的写法更易维护：

const char* dayOfWeekToChar(int day)
{
    // 静态字符串数组存储在只读数据段，生命周期和程序一致
    static const char* const week_names[] = {
        "Sun", "Mon", "Tue", "Wed", "Thu", "Fri", "Sat"
    };
    // 这里根据你实际的SUN_NR、SAT_NR定义调整判断逻辑
    if (day < SUN_NR || day > SAT_NR) {
        return "Invalid";
    }
    return week_names[day];
}

补充说明 #

这种实现方式返回的字符串指针指向程序只读数据段的常量，整个程序运行周期内都有效，不会出现野指针问题，也不需要调用者手动释放内存，比每次调用动态分配内存返回的方案效率更高、风险更低，只要保证返回值带const限定符禁止修改，就是完全合规的写法。

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