IF 语句多个 and or 条件

Question

我在 bash 中使用带有多个条件的 if 语句。

如何减少以下行语法。从你的设计角度来看，它看起来不错。

 if [ "$1" != "-l" ] && [ "$1" != "-a" ] && [ "$1" != "-h" ] && [ "$1" != "" ] && [ "$1" =  "-d" ] || [ "$1" = "-mv" ] || [ "$1" = "-dv" ] || [ "$1" = "-mr" ] || [ "$1" = "-dr" ];

谢谢

Answer 1

使用模式匹配。

if [[ $1 && $1 != -[lah] && $1 == -@(d|mv|dv|mr|dr) ]]; then

@(...) 是扩展模式的一个示例，在最新版本的bash 中默认情况下应在[[ ... ]] 内识别。如果您的版本不是最新版本，请将 shopt -s extglob 添加到脚本的开头。

---

事实上，你可以放弃$1 && $1 != -[lah]，因为$1 == -@(...) 的真实性暗示了它的真实性。

if [[ $1 == -@(d|mv|dv|mr|dr) ]]; then

---

您也可以只使用符合 POSIX 的 case 语句：

case $1 of
 -d|-mv|-dv|-mr|-dr) echo good option ;;
 *) echo bad option ;;
esac

Answer 2

您可以为匹配和不匹配的值创建 2 个数组，并检查元素 $1 是否匹配数组中的任何元素，如下所示。

nonmatch_array=( "-l" "-a" "-h" "" )
match_array=(  "-d" "-mv" "-dv" "-mr" "-dr" )

if [ `echo ${match_array[@]} | grep "$1"` ] || ! [ `echo ${nonmatch_array[@]} | grep "$1"` ] ; then 
    echo "is in array"
else 
    echo "is not in array"
fi

希望它对你有用。

Answer 3

首先尝试将代码的长度限制在 1 行。

if [  [ "$1" != "-l" ] 
   && [ "$1" != "-a" ]
   && [ "$1" != "-h" ] 
   && [ -n "$1" ] 
   && ( [ "$1" = "-d" ]
   || [ "$1" = "-mv" ]
   || [ "$1" = "-dv" ]
   || [ "$1" = "-mr" ]
   || [ "$1" = "-dr" ] ) ];

我添加了大括号，以明确您对 or 的含义。

现在您可以使用正则表达式组合所有匹配项：

   if [[ ! ("$a" =~ ^-(l|a|h|d|)$)
      &&    "$a" =~ ^-(mv|dv|mr|dr)$ ]]; then
      echo "Yes $a matches"
   fi

但请重新考虑您正在测试的内容。该测试仅在匹配-mv/-dv/-mr/-dr 时为真，因此您无需测试选项lah。

   if [[ "$a" =~ ^-(d|mv|dv|mr|dr)$ ]]; then
      echo "Yes $a matches"
   fi

您可以使用变量来提取选项：

   options="d|mv|dv|mr|dr"
   if [[ "$a" =~ ^-(${options})$ ]]; then
      echo "Yes $a matches"
   fi

每当代码变得难以阅读时（对于长代码或重复语句也是如此），您应该考虑使用函数。
下一个函数很短，但很难阅读：

options="d|mv|dv|mr|dr"
function checkoption1 {
   [[ "$a" =~ ^-(${options})$ ]]
}
checkoption1 "$a" &&
   echo "Yes $a matches"

我会选择稍微冗长的函数。我将包含您对 lah 的原始测试，以显示可能性。

# checkoption return 0 for match,
# returns 1 for forbidden option
# returns 2 for undefined option
function checkoption2 {
   case "$1" in
      -d|-mv|-dv|-mr|-dr) return 0 ;;
      -l|-a|-h|"")  return 1;;
      *) return 2;;
   esac
}
checkoption2 "$a" &&
   echo "Yes $a matches"

在接受您的代码之前，您应该进行一些测试。
我用一个小循环做了一些测试（现在所有答案都在一起）

function checkoption1 {
   [[ "$a" =~ ^-(${options})$ ]]
}

# checkoption return 0 for match,
# returns 1 for forbidden option
# returns 2 for undefined option
function checkoption2 {
   case "$1" in
      -d|-mv|-dv|-mr|-dr) return 0 ;;
      -l|-a|-h|"")  return 1;;
      *) return 2;;
   esac
}

for a in -mv mv -mvx -ms -mr -dr; do
       if [[ ! ("$a" =~ ^-(l|a|h|)$)
          &&    "$a" =~ ^-(d|mv|dv|mr|dr)$ ]]; then
          echo "Yes $a matches"
       fi
       if [[ "$a" =~ ^-(d|mv|dv|mr|dr)$ ]]; then
          echo "Yes $a matches"
       fi
       options="d|mv|dv|mr|dr"
       if [[ "$a" =~ ^-(${options})$ ]]; then
          echo "Yes $a matches"
       fi
       checkoption1 "$a" &&
          echo "Yes $a matches"
       checkoption2 "$a" &&
          echo "Yes $a matches 2"
 done