Shell: использование функции с параметрами в 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

Обратите внимание, что я также сначала проверяю целое число, чтобы мы могли вообще не вызывать функцию, если $docheckне равен 1.

Это работает, потому что ifпринимает произвольную команду и решает, что делать из состояния выхода этой команды. Здесь мы используем [ ... ]тест вместе с вызовом вашей функции с &&промежуточным созданием составной команды. Состояние выхода составной команды будет истинным, если и [ ... ]тест, и функция вернули ноль в качестве своих состояний выхода, сигнализируя об успехе.

Как примечание стиля, у меня не было бы функционального теста, если массив не содержит элемент, но если ли содержит элемент, и затем

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

Имея тестовую функцию отрицательный испортит логику в тех случаях , когда вы делаете хотите проверить , содержит ли массив элемента ( if ! notContainsElement ...).

пытаться

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

• -aопция не является ни оболочкой, ни testопцией

здесь у вас есть две части теста

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

Я думаю, что это проще, но не хватает логики &&, которую многие предпочитают.