Bash跨CentOS/Ubuntu脚本：仅未安装时下载安装包的逻辑优化咨询

原有代码的核心错误 #

• 发行版检测逻辑错误：检测到apt包管理器时错误将MANAGER设为dnf，应修正为apt
• 否定语法错误：Shell中逻辑否定用!而非NOT，且命令执行状态判断不需要包裹在[ ]/[[ ]]中
• 检测逻辑混淆：检测本地文件存在要使用-f参数，不能用检测命令是否存在的type
• 重复冗余判断：不需要分两行多次判断$MANAGER == "dnf"，合并到同一if块即可
• 需求描述中的dfc应为笔误，本次实现以代码中实际要安装的pydf作为检测目标

[ ] 与 [[ ]] 的区别 #

• [ ] 是POSIX标准的test命令，兼容性更强，适合需要兼容POSIX sh的脚本，使用时变量要加引号，逻辑与/或需要用-a/-o或者拆分多个[ ]用&&/||连接
• [[ ]] 是Bash等现代Shell的内置关键字，支持正则匹配、通配符匹配，不需要额外给变量加引号避免空格问题，逻辑运算符可以直接写在括号内，如果你脚本开头指定了#!/bin/bash，优先用[[ ]]写法更简洁不易出错

正确实现代码 #

#!/bin/bash

# 发行版与包管理器检测，修正原逻辑错误
type apt &> /dev/null && MANAGER=apt && DISTRO="Debian/Ubuntu"
type yum &> /dev/null && MANAGER=yum && DISTRO="RHEL/Fedora/CentOS"
# dnf优先级高于yum，存在时覆盖之前的yum设置
type dnf &> /dev/null && MANAGER=dnf && DISTRO="RHEL/Fedora/CentOS"

# 固定RPM路径与下载地址，避免重复书写
RPM_PATH="/tmp/pydf-12-11.fc35.noarch.rpm"
RPM_DOWNLOAD_URL="https://download-ib01.fedoraproject.org/pub/fedora/linux/development/rawhide/Everything/x86_64/os/Packages/p/pydf-12-11.fc35.noarch.rpm"

# 核心逻辑：包管理器为dnf且pydf未安装时执行下载安装
if [[ $MANAGER == "dnf" ]] && ! type pydf &> /dev/null; then
    # 下载RPM包，-q为静默下载参数，不需要可以去掉
    wget -q -P "$(dirname $RPM_PATH)" "$RPM_DOWNLOAD_URL"
    # 下载完成且文件存在时执行安装
    if [[ -f "$RPM_PATH" ]]; then
        rpm -i "$RPM_PATH"
        # 安装成功后删除安装包
        [[ $? -eq 0 ]] && rm -f "$RPM_PATH"
    fi
fi

如果需要兼容POSIX sh（无Bash环境），仅需调整判断写法即可：

if [ "$MANAGER" = "dnf" ] && ! type pydf >/dev/null 2>&1; then

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