通过Bash脚本编辑Crontab：仅添加不存在任务的脚本异常问题

核心误区：逻辑运算符优先级踩坑 #

你写的这行判断逻辑有本质问题：

grep -q "$cron_cmd" cron || echo "$cron_cmd" >> cron && crontab cron && rm cron

Bash里&&的优先级比||高，这行代码会被解析成：

(grep -q "$cron_cmd" cron || echo "$cron_cmd" >> cron) && crontab cron && rm cron

这意味着：

• 不管grep有没有找到目标条目，只要前面的grep || echo执行成功（不管是grep命中还是echo追加成功），后面的crontab cron和rm cron都会执行。
• 更糟的是，如果grep因为匹配规则问题没命中已存在的条目（比如crontab里的条目有多余空格、或者cron_cmd里的特殊字符干扰），就会重复执行echo追加，导致重复添加。

另外还有两个隐藏问题：

1. 空crontab的错误处理：如果当前用户没有任何crontab条目，crontab -l会返回错误，导致cron文件内容异常，可能覆盖掉空crontab（虽然你的场景是添加条目，但风险依然存在）。
2. grep匹配不精准：默认grep是部分匹配，而且会把字符串当作正则表达式解析，一旦cron_cmd里有--这类特殊字符，或者crontab里的条目空格数量不同，就会匹配失败，误判为条目不存在。

修正后的脚本 #

改用清晰的if判断替代逻辑运算符组合，同时解决匹配和错误处理问题：

#!/bin/bash
set_cron() {
    cron_cmd="0 0 5 * * prosodyctl --root cert import /etc/letsencrypt/live/"
    # 安全获取当前crontab，无条目时生成空文件
    crontab -l 2>/dev/null > cron.tmp || true

    # 整行精准匹配，把命令当作固定字符串处理
    if ! grep -qxF "$cron_cmd" cron.tmp; then
        echo "$cron_cmd" >> cron.tmp
        crontab cron.tmp
    fi

    rm -f cron.tmp
}
set_cron

关键改进说明： #

• if ! grep ...结构：彻底避开运算符优先级陷阱，逻辑一目了然。
• grep -qxF参数：
  • -x：要求整行完全匹配，避免部分匹配导致的误判；
  • -F：把目标字符串当作固定文本处理，忽略其中的正则特殊字符，比如--。
• crontab -l 2>/dev/null || true：当没有crontab条目时，屏蔽错误输出，生成空的临时文件，保证后续操作正常。
• 临时文件命名cron.tmp：避免和系统命令cron重名，更规范。

额外优化方案 #

如果你的crontab条目允许字段间有多个空格（比如用户手动编辑时加了多余空格），可以用正则匹配替代固定字符串匹配：

#!/bin/bash
set_cron() {
    # 用\s+匹配任意数量的空格
    cron_regex="^0\s+0\s+5\s+\*\s+\*\s+prosodyctl --root cert import /etc/letsencrypt/live/$"
    crontab -l 2>/dev/null > cron.tmp || true

    if ! grep -qE "$cron_regex" cron.tmp; then
        echo "0 0 5 * * prosodyctl --root cert import /etc/letsencrypt/live/" >> cron.tmp
        crontab cron.tmp
    fi

    rm -f cron.tmp
}
set_cron

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