C语言中退出for循环后字符串尾部丢失问题求助

核心问题分析 #

你的代码里的sprintf(&name_final[i], "%d", name_final[i]);是导致数据丢失的直接原因：

• 执行name_final[i] = num_remainder % 10;时，赋值的是整数数值（比如37%10得到7，是整数7而非字符'7'）。
• 紧接着调用sprintf会把这个数值转成字符串，在name_final[i]位置写入对应字符后，自动在下一个索引位置添加字符串结束符'\0'。
• 循环从i=2到i=0执行时，每次sprintf都会覆盖后续位置的内容：比如处理i=1时，会把i=2的位置设为'\0'；处理i=0时，把i=1的位置设为'\0'。最终循环结束后，只有name_final[0]是有效字符，后续全是'\0'，自然无法显示尾部内容。

修复后的代码 #

直接将数字转换为对应ASCII字符，避免sprintf的自动结束符覆盖问题，同时规范内存操作：

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

char *get_out_file_name(int in_file_num);

int main(int argc, char *argv[])
{
    char* file_name = get_out_file_name(37);
    printf("name_final = %s \n", file_name);
    free (file_name);
    return 0;
}

char *get_out_file_name(int in_file_num) 
{ 
    const int max_name_length = 3; 
    const int file_type_length = 4; 
    const char *type_suffix = ".jpg"; 
    int num_remainder = in_file_num; 
    // 用calloc分配内存并自动初始化为0，避免随机值干扰
    char *name_final = calloc(max_name_length + file_type_length + 1, sizeof(char));
    if (!name_final) { // 检查内存分配是否成功
        return NULL;
    }

    for (int i = max_name_length - 1; i >= 0 ; i --) 
    { 
        printf("i = %d\n",i); 
        // 直接转换为ASCII字符：整数数值+字符'0'的ASCII值得到对应数字字符
        name_final[i] = '0' + (num_remainder % 10); 
        printf("name_final[i] = %c \n", name_final[i]);
        num_remainder /= 10; 
    } 
    printf("pre concat[1] = %c \n",name_final[1]);
    strcat(name_final, type_suffix); 
    return name_final; 
}

关键改动说明 #

1. 替换malloc为calloc：自动将内存初始化为0，省去手动添加字符串结束符的步骤，同时避免随机内存值的干扰。
2. 移除sprintf改用直接字符转换：'0' + (num_remainder %10)直接得到数字对应的ASCII字符，不会产生额外的'\0'覆盖之前的内容。
3. 添加内存分配失败检查：避免空指针操作导致程序崩溃。

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