Shell脚本删除函数异常：未删除记录反而生成重复记录如何修复？

你的Delete函数有几个关键问题导致删除失败反而生成重复记录，我来一步步帮你理清并修复：

原代码的核心问题 #

• 错误的行读取方式：for j in \cat Employee.txt` 会把文件内容按空格（而非换行）分割成多个片段，这意味着如果员工记录里有空格，或者文件每行是完整的逗号分隔记录，这个循环会把一行拆成多个j`值，导致重复处理、多次写入文件，最终生成重复记录。
• sed替换的风险：用sed s/$j/$line/g时，$j里的逗号、特殊字符会被sed当成正则表达式元字符，导致替换匹配错误；而且你在循环里每次都把整个文件重新写入，多次循环会让文件内容被重复覆盖或修改。
• 变量引用错误：if [ f = 0 ]里的f没有加$符号，永远不会触发"无记录"的提示，因为shell会把f当成字符串而非变量值。
• 逻辑偏差：你原本想删除记录，但代码逻辑是把匹配的行改成带,false的行，根本不是删除操作。

修复后的Delete函数 #

我重写了函数，采用正确的行读取方式和删除逻辑：

Delete() {
    clear
    f=0
    echo -n "Enter Employee NO: "
    read no

    # 创建临时文件存储需要保留的记录
    temp_file=$(mktemp)

    # 逐行读取原文件，筛选保留不匹配的记录
    while IFS= read -r line; do
        # 提取当前行的员工编号
        eno=$(echo "$line" | cut -d "," -f 1)
        # 匹配到目标编号时标记成功，跳过写入
        if [ "$no" -eq "$eno" ]; then
            f=1
            echo "Delete Successfully"
        else
            # 未匹配的行写入临时文件
            echo "$line" >> "$temp_file"
        fi
    done < Employee.txt

    # 用临时文件替换原文件，完成删除操作
    mv "$temp_file" Employee.txt

    # 检查是否找到目标记录
    if [ "$f" -eq 0 ]; then
        echo "No Record Found"
    fi
}

关键改进点 #

• 用while IFS= read -r line逐行读取：确保每行完整被读取，不会被空格拆分；IFS=防止行首/行尾的空格被截断，-r避免反斜杠被转义。
• 临时文件处理：避免在循环中反复写入原文件，而是把需要保留的行写入临时文件，最后替换原文件，这是处理文件修改的安全做法，防止中途出错导致文件损坏。
• 正确的变量引用：所有变量都用$引用，数字比较用-eq更严谨。
• 真正的删除逻辑：直接跳过匹配的行，不写入临时文件，最终原文件就只剩下未匹配的记录，实现删除效果。

额外建议 #

• 如果员工编号可能是字符串（比如带前缀），要把[ "$no" -eq "$eno" ]改成[ "$no" = "$eno" ]（字符串相等比较）。
• 处理文件时尽量用临时文件，避免直接在原文件上修改，防止脚本中途中断导致数据丢失。

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