Shell: Funktion mit Parametern in if verwenden

Ich versuche, den folgenden Code auszuführen, aber wenn ich versuche, meine Funktion in der if-Anweisung zu verwenden, wird der -bash: [: too many argumentsFehler angezeigt. Warum passiert es?

Vielen Dank im Voraus!

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

Bei der Verwendung verwenden if [ ... ]Sie tatsächlich das [Dienstprogramm (das das gleiche ist, testaber erfordert, dass das letzte Argument lautet ]).

[versteht es nicht, Ihre Funktion auszuführen, es erwartet Zeichenfolgen. Zum Glück müssen Sie hier überhaupt nicht verwenden [(zumindest für die Funktion):

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

Beachten Sie, dass ich auch zuerst die Ganzzahl überprüfe, damit wir die Funktion möglicherweise überhaupt $dochecknicht aufrufen, wenn sie nicht 1 ist.

Dies funktioniert, weil ifein beliebiger Befehl verwendet wird und aus dem Beendigungsstatus dieses Befehls entschieden wird, was zu tun ist. Hier verwenden wir einen [ ... ]Test zusammen mit einem Aufruf Ihrer Funktion, wobei &&dazwischen ein zusammengesetzter Befehl erstellt wird. Der Exit-Status des zusammengesetzten Befehls wäre wahr, wenn sowohl der [ ... ]Test als auch die Funktion als Exit-Status Null zurückgeben würden, was den Erfolg signalisiert.

Als Stilnotiz würde ich den Funktionstest nicht durchführen lassen, ob das Array das Element nicht enthält, sondern ob es das Element enthält, und dann

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

Einen Funktionstest negativ Willen vermasseln Logik in den Fällen , in Erwägung , wo Sie tun testen wollen , ob das Array das Element enthält ( if ! notContainsElement ...).

Versuchen

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

• -aOption ist weder eine Shell noch eine testOption

Hier haben Sie einen zweiteiligen Test

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

Sie verknüpfen dann ifmit

if cmd1 && cmd2

Wie bereits erwähnt, -ahandelt es sich um eine Testoption, die jedoch nur mit anderen Testoptionen verwendet werden kann. Sie können sie daher verwenden

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

, der Test $aist niedriger als beide $bund $c, aber man kann nicht einen anderen Befehl innerhalb Prüfumfang verwendet.

Hier ist eine Alternative, die manche Leute vielleicht nicht mögen. Konvertieren Sie Ihr Blacklist-Array in eine Zeichenfolge und prüfen Sie, ob die Zeichenfolge mit der entfernten Frucht identisch ist. Bearbeitet, um die Zeichenfolgen mit Leerzeichen aufzufüllen. Vielen Dank an Scott für den Hinweis auf das Apfel- / Ananasproblem.

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

Ich denke, das ist einfacher, aber es fehlt die && Logik, die viele bevorzugen.