Shell: utilizzo della funzione con parametri in if

Sto cercando di eseguire il codice seguente, ma quando provo a utilizzare la mia funzione nell'istruzione if ottengo l' -bash: [: too many argumentserrore. Perché sta succedendo?

Grazie in anticipo!

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

Quando si utilizza if [ ... ]si utilizza effettivamente l' [utilità (che è la stessa testma richiede che sia l'ultimo argomento ]).

[non capisce di eseguire la tua funzione, si aspetta stringhe. Fortunatamente, non è necessario utilizzare [affatto qui (almeno per la funzione):

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

Nota che sto anche controllando prima l'intero, quindi possiamo evitare di chiamare la funzione se $dochecknon è 1.

Questo funziona perché ifaccetta un comando arbitrario e decide cosa fare dallo stato di uscita di quel comando. Qui usiamo un [ ... ]test insieme a una chiamata alla tua funzione, con &&in mezzo, creando un comando composto. Lo stato di uscita del comando composto sarebbe vero se sia il [ ... ]test che la funzione restituissero zero come stati di uscita, segnalando il successo.

Come nota lo stile, non avrei il test funzionale se la matrice non non contiene l'elemento, ma se se lo fa contenere l'elemento, e poi

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

Avere un test di funzionamento un pasticcio volontà negativo fino logica nei casi in cui si fare desidera verificare se l'array contiene l'elemento ( if ! notContainsElement ...).

provare

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

• -aL'opzione non è né una shell né testun'opzione

qui hai un test in due parti

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

si collega quindi in ifuso

if cmd1 && cmd2

come sottolineato -aè un'opzione di test, ma può essere utilizzata solo con altre opzioni di test, quindi è possibile utilizzare

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

per verificare che $asia inferiore a entrambi $be $c, ma non è possibile utilizzare altri comandi nell'ambito del test.

Ecco un'alternativa che ad alcune persone potrebbe non piacere. Converti l'array della blacklist in una stringa e vedi se la stringa con un frutto rimosso è la stessa. Modificato per riempire le stringhe di spazi. Grazie a Scott per aver segnalato il problema mela / ananas.

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

Penso che sia più semplice, ma manca la logica di && che molti preferiscono.