シェル：ifでパラメーター付きの関数を使用する

以下のコードを実行しようとしていますが、ifステートメントで関数を使用しようとすると、-bash: [: too many argumentsエラーが発生します。なぜそれが起こっているのですか？

前もって感謝します！

notContainsElement () {
  local e match="$1"
  shift
  for e; do [[ "$e" == "$match" ]] && return 1; done
  return 0
}

list=( "pears" "apples" "bananas" "oranges" )
blacklist=( "oranges" "apples" )
docheck=1

for fruit in "${list[@]}"
do
    if [ notContainsElement "$fruit" "${blacklist[@]}" -a $docheck = 1 ]
    then
        echo $fruit
    fi
done

使用if [ ... ]するときは、実際に[ユーティリティを使用しています（これはと同じtestですが、最後の引数がである必要があります]）。

[関数の実行を理解できません。文字列が必要です。幸い、[ここでは（少なくとも関数に対して）を使用する必要はありません。

if [ "$docheck" -eq 1 ] && notContainsElement "$fruit" "${blacklist[@]}"; then
  ...
fi

また、最初に整数をチェックしているので、$docheck1でない場合は関数をまったく呼び出さないようにすることができます。

これifは、任意のコマンドを受け取り、そのコマンドの終了ステータスから何をするかを決定するため機能します。ここでは、[ ... ]テストと関数の呼び出しを併用し、&&その間に複合コマンドを作成します。[ ... ]テストと関数の両方が終了ステータスとしてゼロを返し、成功を通知した場合、複合コマンドの終了ステータスはtrueになります。

スタイルのメモとして、配列に要素が含まれていないかどうかをテストしませんが、要素が含まれているかどうかをテストします。

if [ "$docheck" -eq 1 ] && ! contains "$fruit" "${blacklist[@]}"; then ...

関数テストでネガティブを使用すると、配列に要素（）が含まれているかどうかをテストする必要がある場合、ロジックが台無しになりますif ! notContainsElement ...。

試す

if notContainsElement "$fruit" "${blacklist[@]}" && test "$docheck" = 1
then

• -aオプションは、シェルまたはでもないtest選択肢

ここで2つの部分のテストがあります

notContainsElement "$fruit" "${blacklist[@]}"
test $docheck = 1 ## or [ $docheck = 1 ]

あなたがif使用してリンクします

if cmd1 && cmd2

指摘したよう-aに、テストオプションですが、他のテストオプションでのみ使用できるため、

if [ "$a" -lt "$b" -a "$a" -lt "$c" ]

テストの$a両方よりも低い$bと$c、しかし、あなたがテスト範囲内で他のコマンドを使用することはできません。

これは一部の人々が好まないかもしれない代替案です。ブラックリスト配列を文字列に変換し、フルーツが削除された文字列が同じかどうかを確認します。文字列にスペースを埋め込むように編集されました。アップル/パイナップル問題を指摘してくれたスコットに感謝します。

badlist=" ${blacklist[@]} "
for f in "${list[@]}"
do
    if [[ "${badlist/" $f "/}" == "$badlist" ]]
    then
        echo "$f"
    fi
done

これはもっと簡単だと思いますが、多くの人が好む&&ロジックが欠けています。