Shell: Usando a função com parâmetros em if

Estou tentando executar o código abaixo, mas quando tento usar minha função na instrução if, recebo o -bash: [: too many argumentserro. Por que isso está acontecendo?

Agradeço antecipadamente!

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

Ao usar, if [ ... ]você está realmente usando o [utilitário (que é o mesmo que, testmas requer que o último argumento seja ]).

[não entende para executar sua função, espera seqüências de caracteres. Felizmente, você não precisa usar [nada aqui (pelo menos para a função):

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

Observe que também estou verificando o número inteiro primeiro, para que possamos evitar chamar a função se $dochecknão for 1.

Isso funciona porque ifrecebe um comando arbitrário e decide o que fazer a partir do status de saída desse comando. Aqui, usamos um [ ... ]teste juntamente com uma chamada para sua função &&, criando um comando composto. O status de saída do comando composto seria verdadeiro se o [ ... ]teste e a função retornassem zero como status de saída, sinalizando sucesso.

Como uma nota de estilo, eu não teria o teste de função se a matriz não não contêm o elemento, mas se se faz contêm o elemento e, em seguida,

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

Ter um teste de função uma bagunça vontade negativa a lógica em casos onde você fazer quer testar se a matriz contém o elemento ( if ! notContainsElement ...).

tentar

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

• -aopção não é um shell ou uma testopção

aqui você tem teste em duas partes

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

você liga então ifusando

if cmd1 && cmd2

como indicado -aé uma opção de teste, mas só pode ser usado com outra opção de teste, portanto, você pode usar

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

para testar que $aé menor que ambos $be $c, mas você não pode usar outro comando no escopo de teste.

Aqui está uma alternativa que algumas pessoas podem não gostar. Converta sua matriz da lista negra em uma sequência e veja se a sequência com uma fruta removida é a mesma. Editado para preencher as strings com espaços. Agradecemos a Scott por apontar o problema da maçã / abacaxi.

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

Eu acho que isso é mais simples, mas falta a lógica && que muitos preferem.