如何高效在字符串中间插入数字？C语言实现效率咨询

我原本的代码用来调用functionX并传入固定文件名：

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

int main(void){
    const char* pFilename = NULL;

    pFilename = "Something.png";

    functionX(pFilename, argX);
}

现在需要在循环里调用functionX，传入Something0.png、Something1.png这类带序号的文件名，我自己写了这段代码，但不确定能不能正常运行（因为functionX依赖特定外设没法测试），同时想知道就算能跑，这是不是时间效率最优的实现方式：

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

int main(void){
    const char* pFilename = NULL;
    char buffer[4];
    char nameStd[] = "Something";
    char namePng[] = ".png";
    char nameAll[17];

    pFilename = "Something.png";

    for (int i = 0; i < 100; i++) {
        snprintf(buffer, sizeof(buffer), "%d", i);
        strcat(nameAll, pFilename);
        strcat(nameAll, buffer);
        strcat(nameAll, namePng);
        functionX(nameAll, argX);
        memset(nameAll,0,strlen(nameAll));
    }
}

1. 现有代码的问题 #

你的代码无法正常运行，核心问题如下：

• nameAll未初始化，初始内存是随机垃圾数据，没有终止符'\0'，strcat会从内存中第一个'\0'的位置开始拼接，行为完全不可控。
• 逻辑错误：你需要的是Something+序号+.png，但代码拼接的是Something.png+序号+.png，最终会生成Something.png0.png这类错误文件名。
• memset(nameAll,0,strlen(nameAll))无效：如果nameAll没有正确的'\0'，strlen会返回错误长度；就算有，也应该用sizeof(nameAll)清空整个缓冲区，而非只清到'\0'的位置。

2. 正确且高效的实现方式 #

时间效率最优的方案是用snprintf一次性生成完整文件名，避免多次字符串拼接的开销，同时保证内存安全：

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

int main(void){
    // 缓冲区大小计算："Something"(9字符)+3位数字(100是3位)+".png"(4字符)+终止符'\0'=17，和你原有的nameAll大小一致
    char filename[17];
    const char* prefix = "Something";
    const char* suffix = ".png";

    for (int i = 0; i < 100; i++) {
        // 一次性格式化生成完整文件名，返回值为写入字符数（不含'\0'），可用来检查是否溢出
        snprintf(filename, sizeof(filename), "%s%d%s", prefix, i, suffix);
        functionX(filename, argX);
        // 无需额外清空，下一次snprintf会直接覆盖原有内容
    }
}

为什么这是高效的？ #

• 避免了strcat的多次遍历开销：每次strcat都要从字符串开头遍历到末尾找终止符，多次拼接会累积时间成本；而snprintf是一次性计算并写入，仅需一次操作。
• 减少了中间变量（比如你原有的buffer），内存使用更紧凑，也避免了中间缓冲区的错误。
• 逻辑简洁直观，不容易出错，同时保证了文件名的正确性。

补充：如果序号可能超过3位（比如循环到1000），要对应调整filename的缓冲区大小，避免溢出。比如序号最多4位时，缓冲区大小应为9+4+4+1=18。

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