Shell: Menggunakan fungsi dengan parameter jika

Saya mencoba untuk mengeksekusi kode di bawah ini tetapi ketika saya mencoba untuk menggunakan fungsi saya di pernyataan if saya mendapatkan -bash: [: too many argumentskesalahan. Mengapa ini terjadi?

Terima kasih sebelumnya!

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

Saat menggunakan if [ ... ]Anda sebenarnya menggunakan [utilitas (yang sama dengan testtetapi mengharuskan argumen terakhir ]).

[tidak mengerti untuk menjalankan fungsi Anda, ia mengharapkan string. Untungnya, Anda tidak perlu menggunakan [sama sekali di sini (setidaknya untuk fungsi):

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

Perhatikan bahwa saya juga memeriksa integer terlebih dahulu, sehingga kita dapat menghindari memanggil fungsi sama sekali jika $docheckbukan 1.

Ini berfungsi karena ifmengambil perintah sewenang-wenang dan memutuskan apa yang harus dilakukan dari status keluar dari perintah itu. Di sini kami menggunakan [ ... ]tes bersama dengan panggilan ke fungsi Anda, dengan &&di antaranya, membuat perintah majemuk. Status keluar perintah majemuk akan benar jika [ ... ]tes dan fungsi mengembalikan nol sebagai status keluarnya, menandakan keberhasilan.

Sebagai catatan gaya, saya tidak akan memiliki tes fungsi apakah array tidak mengandung elemen tetapi apakah jika memang mengandung elemen, dan kemudian

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

Memiliki tes fungsi akan mengacaukan negatif up logika dalam kasus di mana Anda lakukan ingin menguji apakah array berisi elemen ( if ! notContainsElement ...).

mencoba

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

• -aopsi bukanlah shell atau testopsi

di sini Anda memiliki dua bagian tes

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

Anda menautkan kemudian ifmenggunakan

if cmd1 && cmd2

seperti yang ditunjukkan -aadalah opsi tes, tetapi hanya dapat digunakan dengan opsi tes lainnya, sehingga Anda dapat menggunakannya

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

untuk menguji yang $alebih rendah dari keduanya $bdan $c, tetapi Anda tidak dapat menggunakan perintah lain dalam cakupan pengujian.

Berikut adalah alternatif yang mungkin tidak disukai sebagian orang. Konversikan daftar hitam Anda menjadi string, dan lihat apakah string dengan buah yang dihapus sama. Diedit untuk membuat string dengan spasi. Terima kasih kepada Scott karena telah menunjukkan masalah apel / nanas.

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

Saya pikir ini lebih sederhana, tetapi tidak memiliki logika && yang disukai banyak orang.