求助：Bash脚本检测并删除7天前Aerospike备份文件遇问题

我看了你的脚本，发现几个语法和逻辑上的问题，接下来一步步帮你修正：

原脚本的核心问题 #

• 函数find_old语法不完整：else块没有对应的fi闭合，整个函数的条件判断结构缺失收尾
• 判断旧文件存在的方式有风险：[ -z $(find ...) ]在文件名包含空格时会触发语法错误，而且捕获find输出的方式效率很低
• 拼写错误：EMEIL应该是EMAIL
• 定义了LOGPATH但没实际使用，也没有调用find_old函数，相当于写了逻辑但没执行
• 没检查BACKUPDIR是否存在，万一目录不存在会直接报错

修正后的完整脚本 #

#!/bin/bash

# 配置参数
BACKUPDIR="/home/vagrant/script/aerospike_backups"
TIMESTAMP=$(date +%Y-%m-%d)
LOGPATH="/tmp/logs.txt"
ADMINACC="email@example.com"
EMAIL="rybka@gl.com"

# 先检查备份目录是否存在
check_dir() {
    if [ ! -d "$BACKUPDIR" ]; then
        echo "[$TIMESTAMP] Error: Backup directory $BACKUPDIR does not exist!" >> "$LOGPATH"
        exit 1
    fi
}

# 查找并删除7天以上的旧文件
find_old() {
    # 用find的-quit选项快速判断是否存在旧文件，避免输出内容带来的问题
    if find "$BACKUPDIR" -type f -mtime +7 -quit >/dev/null 2>&1; then
        # 删除旧文件并记录操作日志
        find "$BACKUPDIR" -type f -mtime +7 -delete
        echo "[$TIMESTAMP] Successfully deleted old backups in $BACKUPDIR" >> "$LOGPATH"
        return 0
    else
        echo "[$TIMESTAMP] No old backups found in $BACKUPDIR" >> "$LOGPATH"
        return 10
    fi
}

# 执行主逻辑
check_dir
find_old

# 可选：如果需要邮件通知，可以添加以下逻辑
# echo "Backup cleanup completed. Check log at $LOGPATH" | mail -s "Backup Cleanup Report" "$EMAIL"

关键改进说明 #

• 给所有变量加上双引号"$BACKUPDIR"，避免文件名带空格时触发语法错误
• 用find ... -quit高效判断旧文件存在性，找到第一个匹配项就停止，比捕获输出的方式更可靠高效
• 添加目录存在性检查，提前规避无效操作的报错
• 将所有操作日志写入指定的LOGPATH文件，方便后续排查问题
• 明确指定-type f只删除文件，如果需要删除旧目录可以改成-type d（不建议默认删除目录，避免误删）
• 补全了函数的语法结构，确保逻辑能正常执行

使用方式 #

1. 将脚本保存为cleanup_old_backups.sh
2. 赋予执行权限：chmod +x cleanup_old_backups.sh
3. 手动测试执行：./cleanup_old_backups.sh
4. 如需定时执行，可添加到crontab（比如每天凌晨2点执行）：

0 2 * * * /path/to/cleanup_old_backups.sh

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