Bash新手求助：基于外部文件的for循环中使用sed编辑行的代码问题排查

Hey，我帮你梳理下代码里的几个关键问题，这些就是导致效果不达预期的核心原因：

1. 变量编号的语法错误 #

你写的 Num=j +1 完全不符合bash的算术赋值逻辑，bash里要做算术运算得用$((...))结构，而且你没给Num设置初始值，第一次循环时Num是空值，自然没法生成正确的编号。

2. 文件读取方式的隐患 #

用for j in \cat /tmp/check.txt` 会把文件内容按**空格/制表符/换行符**拆分，虽然你的示例文件每行都是无空格的路径，但如果未来文件里出现带空格的行，代码直接就崩了。更稳妥的方式是用while read`逐行读取。

3. 临时文件的顺序颠倒 #

你的代码第一次循环时，file_destantion.txt$$还没创建就用sed修改，会直接报错；而且循环最后才复制原始文件，逻辑完全反了——应该先从原始文件复制到临时文件，再执行替换操作。

修正后的完整代码 #

#!/bin/bash

# 初始化编号变量，从1开始计数
Num=1

# 逐行读取check.txt内容，避免空格解析问题
while IFS= read -r j; do
    # 先复制原始文件到临时文件，确保每次替换都基于初始内容
    cp -rp file_destantion.txt file_destantion.txt$$
    
    # 执行替换（你用+作为分隔符的思路很对，避开了路径里的/）
    sed -i "s+/tmp/old_name/+/${j}/+gi" file_destantion.txt$$
    
    # 重命名为带编号的log文件
    mv file_destantion.txt$$ "file_destantion.log${Num}"
    
    # 编号自增
    Num=$((Num + 1))
done < /tmp/check.txt

# 可选：清理可能残留的临时文件
rm -f file_destantion.txt$$

关键说明 #

• IFS= read -r j：IFS=保证读取行时不会忽略开头/结尾的空格，-r避免转义字符被错误解析，确保每行内容完整读取。
• 把cp放到循环开头：这样每个log文件都是基于原始file_destantion.txt替换的结果，不会出现上一次替换影响下一次的情况。
• 编号自增的正确写法：Num=$((Num + 1))是bash标准的算术运算格式。
• 简化文件名：把file_destantion.txt$$.log$Num改成file_destantion.log${Num}更简洁，如果你需要保留PID可以加回去。

跑这个修正后的代码，就能得到你想要的效果：file_destantion.log1对应/tmp/check70的替换结果，log2对应/tmp/check70_1，log3对应/tmp/_check7007。

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