C语言使用for循环为数字添加千位分隔符功能失效求助

问题排查 #

原程序无法运行的核心问题如下：

• 最致命的错误是for循环的终止条件写为赋值语句a=11，而非判断条件。C语言中赋值语句的返回值是被赋值的数，此处11为非0值，会导致循环永远为真，陷入死循环。
• 逗号插入逻辑是硬编码，仅适配当前12345678.23这一个输入，更换数字长度就会失效，没有通用适配性。
• void main()不符合C语言标准写法，标准主函数返回值应为int类型。
• 原代码中if(strlen(m)==j)的判断完全没必要，输入的字符串固定为11位，每次循环都做一次无意义判断浪费性能。

最小改动可运行版本（仅适配当前输入） #

#include<stdio.h>
#include<string.h>

int main()
{
    char m[20] = "12345678.23";
    int a, t = 1, r = 4;
    // 修正循环条件，遍历整个字符串
    for(a = 0; a < strlen(m); a++)
    {
        printf("%c", m[a]);
        // 整数部分对应位置插入逗号
        if(a == t || a == r)
        {
            printf(",");
        }
    }
    return 0;
}

运行输出为12,345,678.23，符合需求。

通用适配版本（支持任意长度带小数的正整数格式化） #

#include<stdio.h>
#include<string.h>

void format_num(char *num) {
    int len = strlen(num);
    int dot_pos = len;
    // 先找到小数点位置
    for(int i=0; i<len; i++) {
        if(num[i] == '.') {
            dot_pos = i;
            break;
        }
    }
    // 遍历输出并插入逗号
    for(int i=0; i<len; i++) {
        printf("%c", num[i]);
        // 仅在整数部分符合间隔要求的位置插入逗号
        if(i < dot_pos -1 && (dot_pos - i - 1) %3 ==0) {
            printf(",");
        }
    }
}

int main()
{
    char m[20] = "12345678.23";
    format_num(m);
    return 0;
}

该版本可适配任意长度的带小数正整数输入，不需要针对每个数字调整插入位置参数。

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