你的思路方向是对的，但几个细节问题导致删除操作没生效，咱们一步步来修复：

原脚本的核心问题 #

1. 用ls解析文件名不可靠：如果备份文件名包含空格、换行或特殊字符，ls的输出会被错误拆分，导致xargs执行失败
2. 月份比较未处理跨年场景：比如当前是1月（MONTH=01），上月是12月，$2 < m的判断会直接失效（12不小于1）
3. 年份处理函数未完成：你只写了函数开头，没实现往年数据的删除逻辑
4. xargs的换行分隔符兼容性差：-d "\n"不是所有系统都支持（比如macOS的BSD xargs就没有这个参数）

修正后的完整脚本 #

我重新写了更可靠的版本，用find代替ls处理文件，同时解决跨年和年份判断的问题：

#!/bin/bash

# 备份目录绝对路径
BACKUP_DIR="/root/copy/dbbackup/smpp_credits"

# 获取当前年份和月份（格式：YYYY, MM）
CURRENT_YEAR=$(date +'%Y')
CURRENT_MONTH=$(date +'%m')

# 函数：删除上月及更早的备份（兼容跨年场景）
deleteOldBackups() {
    # 计算上月的年份和月份
    if [ "$CURRENT_MONTH" = "01" ]; then
        LAST_YEAR=$((CURRENT_YEAR - 1))
        LAST_MONTH="12"
    else
        LAST_YEAR=$CURRENT_YEAR
        LAST_MONTH=$(printf "%02d" $((CURRENT_MONTH - 1)))
    fi

    # 遍历备份目录下的所有文件/目录
    find "$BACKUP_DIR" -maxdepth 1 -mindepth 1 | while read -r item; do
        # 提取文件名中的年份和月份（假设备份名格式为：xxx-YYYY-MM-xxx）
        ITEM_NAME=$(basename "$item")
        ITEM_YEAR=$(echo "$ITEM_NAME" | awk -F "-" '{print $2}')
        ITEM_MONTH=$(echo "$ITEM_NAME" | awk -F "-" '{print $3}')

        # 判断是否为需要删除的旧备份：年份早于上年，或年份等于上年但月份早于等于上月
        if [ "$ITEM_YEAR" -lt "$LAST_YEAR" ] || ([ "$ITEM_YEAR" -eq "$LAST_YEAR" ] && [ "$ITEM_MONTH" -le "$LAST_MONTH" ]); then
            echo "准备删除旧备份：$item"
            rm -rf "$item"
        fi
    done
}

# 执行删除操作
deleteOldBackups

关键优化点说明 #

• 用find遍历文件：彻底避免ls的文件名解析问题，同时支持文件和目录的统一处理
• 兼容跨年逻辑：1月时自动切换到上一年的12月，保证月份判断准确
• 逐行安全处理：用while read -r代替xargs，避免特殊字符导致的命令执行错误
• 先验证再删除：测试阶段可以保留echo输出，确认要删除的文件正确后，再去掉echo行执行真正的删除

测试与Cron配置注意事项 #

1. 先做安全测试：在终端运行脚本时，先注释掉rm -rf "$item"，只保留echo输出，确认待删除的文件是预期的旧备份，避免误删重要数据
2. Cron环境适配：Cron的环境变量比终端少很多，建议在脚本里使用命令的绝对路径（比如/usr/bin/find、/usr/bin/rm），或者在Cron任务中指定环境变量
3. Cron任务示例：比如每天凌晨2点执行脚本，编辑Cron任务（crontab -e）：

   0 2 * * * /bin/bash /path/to/your/delete_old_backups.sh >> /var/log/backup_cleanup.log 2>&1
   

   加上日志输出方便后续排查问题

内容的提问来源于stack exchange，提问作者The Nerdy Geek