Bash脚本Todo程序未生成done_file.txt问题求助

核心问题点 #

• 变量与文件名拼接错误：代码直接使用$pending_file.txt和$done_file.txt，若pending_file、done_file变量本身不带.txt后缀，会导致变量解析异常（比如变量未定义时会变成.txt），正确拼接应为${pending_file}.txt和${done_file}.txt。
• sed操作逻辑错误：原代码用sed -i "s/$finished_task//g"仅将匹配文本替换为空，而非删除整行；若任务包含/、&等特殊字符，会直接导致sed命令失效，且未判断任务是否真的存在于待办文件中。
• done文件写入逻辑缺失：原代码仅在done_file.txt不存在时才写入任务，若文件已存在，即使任务处理成功，也不会追加任务，这是done_file.txt始终不生成或无内容的关键原因。

修复后的代码 #

move_to_done(){
    read -p "Enter finished task description: " finished_task
    # 定义正确的文件路径
    pending_file_path="${pending_file}.txt"
    done_file_path="${done_file}.txt"

    if [ ! -f "$pending_file_path" ]; then
        echo "No pending tasks"
        return
    fi

    # 检查任务是否存在于待办列表
    if grep -qxF "$finished_task" "$pending_file_path"; then
        # 删除匹配的整行任务
        sed -i "/^$finished_task$/d" "$pending_file_path"
        # 直接追加任务到已完成文件（自动创建文件）
        echo "$finished_task" >> "$done_file_path"
        echo "Task marked as done"
    else
        echo "Task not found in pending list"
    fi
}

show_finished_tasks(){
    done_file_path="${done_file}.txt"
    if [ ! -f "$done_file_path" ]; then
        echo "No finished tasks"
    else
        cat "$done_file_path"
    fi
}

关键修复说明 #

• 文件名处理：单独定义文件路径变量，避免变量与后缀拼接时的解析错误。
• 任务存在性校验：用grep -qxF精确匹配整行任务，确保只处理真实存在的待办项。
• sed整行删除：使用/^$finished_task$/d删除完全匹配的整行，避免替换空内容导致的残留问题，同时给变量加引号防止特殊字符干扰。
• done文件写入优化：移除仅在文件不存在时写入的限制，直接用>>追加任务，确保任务一定会被记录，文件不存在时会自动创建。

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