如何在C语言中移除字符串中的所有换行符？代码问题求助

我来帮你捋捋这段代码里的几个关键问题，然后给出修复后的方案～

你的代码存在的核心问题 #

• 索引逻辑混乱：你直接修改循环变量i来跳过换行符，同时把an_name[i+1]赋值给new_name[i]，这会导致new_name的字符位置错位，而且for循环本身还会自动执行i++，很容易跳过后续正常字符。
• 缺少字符串终止符：C语言的字符串必须以\0结尾，你没有给new_name添加这个终止符，这会导致printf输出时出现乱码或者读取到内存里的垃圾数据。
• 代码块位置错误：你把printf("After removing...")、return 0这些收尾代码放在了循环内部，这会导致循环第一次执行就直接退出程序，根本没法处理完整的字符串。
• 重复计算字符串长度：每次循环都调用strlen(an_name)，会重复计算字符串长度，不仅效率低，还可能在字符串动态变化时出现意外问题。

修复后的代码 #

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

int main() {
    // 示例输入，你可以替换成实际的an_name变量
    char an_name[] = "Hello\nWorld\nThis is a test";
    // 注意：实际使用时要确保new_name的缓冲区足够大，避免溢出
    char new_name[256];
    int i = 0;
    // 单独维护new_name的写入索引，避免和遍历索引冲突
    int new_idx = 0;
    // 提前计算一次字符串长度，提升效率
    int str_len = strlen(an_name);

    printf("\nBefore removing: %s", an_name);
    printf("\nLength before removal: %d\n", str_len);

    // 遍历原字符串的每个字符
    while (i < str_len) {
        // 不是换行符就写入new_name，同时移动写入索引
        if (an_name[i] != '\n') {
            new_name[new_idx] = an_name[i];
            new_idx++;
        }
        // 不管是不是换行符，都移动遍历索引
        i++;
    }
    // 必须添加字符串终止符，否则不是合法的C字符串
    new_name[new_idx] = '\0';

    printf("After removing: %s\n", new_name);
    printf("Length after removal: %d\n", strlen(new_name));
    getchar();
    return 0;
}

改进说明 #

1. 分离索引：用new_idx单独管理new_name的写入位置，遍历原字符串的索引i只负责逐个检查字符，遇到换行符时直接跳过不写入，逻辑更清晰，不会出现错位。
2. 添加终止符：处理完所有字符后，给new_name加上\0，保证它是一个合法的C字符串，避免输出乱码。
3. 优化效率：提前计算一次原字符串的长度，避免循环中重复调用strlen。
4. 修正代码块位置：把输出、程序返回的代码移到循环外部，确保整个字符串都被处理完毕才执行收尾操作。
5. 兼容多换行符：不管原字符串里有多少个换行符，都能正确全部去除，同时不会访问超出字符串范围的内存。

另外提醒一下：如果an_name是动态分配的内存，记得new_name也要分配足够的内存（比如和an_name长度一致，因为去除换行符后长度只会更短），避免缓冲区溢出的问题。

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