Kesalahan dalam fungsi shell untuk menghitung angka genap

Untuk tugas saya harus menulis fungsi yang mencetak jumlah angka genap ketika disediakan dengan urutan angka.

Saya menggunakan potongan kode yang saya gunakan untuk tugas sebelumnya (untuk mencetak 1ketika nomor genap dan 0ketika nomor itu ganjil)

Masalah saya sekarang adalah fungsi saya terus mencetak 0. Apa yang saya lakukan salah?

Ini skrip saya:

#!/usr/bin/bash
# File: nevens.sh

# Write a function called nevens which prints the number of even numbers when provided with a sequence of numbers.
# Check: input nevens 42 6 7 9 33 = output 2

function nevens {

        local sum=0

        for element in $@
        do
                let evencheck=$(( $# % 2 ))
                if [[ $evencheck -eq 0 ]]
                then
                        let sum=$sum+1
                fi
        done

        echo $sum
}

Anda lupa mengganti $#dengan ( $) elementdi forloop:

function nevens {
  local sum=0
  for element in $@; do
    let evencheck=$(( element % 2 ))
    if [[ $evencheck -eq 0 ]]; then
      let sum=sum+1
    fi
  done
  echo $sum
}

Sekarang untuk menguji fungsinya:

$ nevens 42 6 7 9 33
2
$ nevens 42 6 7 9 33 22
3
$ nevens {1..10..2} # 1 to 10 step 2 → odd numbers only
0
$ nevens {2..10..2} # 2 to 10 step 2 → five even numbers
5

@dabut telah menemukan masalah inti, saya akan memberikan beberapa tinjauan kode:

1. Shebang: Tidak ada /usr/bin/bashdi Ubuntu. Ini /bin/bash.

2. Ada baiknya Anda mendeklarasikan sum local, dan menghindari mencemari namespace variabel di luar fungsi. Selain itu, Anda dapat mendeklarasikannya sebagai variabel integer menggunakan -iopsi:

   local -i sum=0

3. Selalu kutip variabel Anda (dan parameter)! Tidak perlu dalam skrip ini, tetapi kebiasaan yang sangat baik untuk masuk:

   for element in "$@"
   do

   Yang mengatakan, Anda dapat menghilangkan di in "$@"sini:

   for element
   do

   Ketika in <something>tidak diberikan, forloop secara implisit loop atas argumen. Ini dapat menghindari kesalahan seperti melupakan kutipan.

4. Tidak perlu menghitung dan kemudian memeriksa hasilnya. Anda dapat langsung melakukan perhitungan di if:

   if (( (element % 2) == 0 ))
   then
       ((sum = sum + 1))
   fi

   (( ... ))adalah konteks aritmatika . Ini lebih berguna daripada [[ ... ]]untuk melakukan pemeriksaan aritmatika, dan selain itu Anda dapat menghilangkan $variabel sebelumnya (yang membuatnya lebih mudah dibaca, IMHO).

5. Jika Anda memindahkan bagian pemeriksa ke fungsi yang terpisah, itu mungkin meningkatkan keterbacaan dan penggunaan kembali:

   function evencheck
   {
       return $(( $1 % 2 ))
   }
   function nevens
   {
       local -i sum=0
       for element
       do
           # `if` implicitly checks that the returned value/exit status is 0
           if evencheck "$element"
           then
               (( sum++ ))
           fi
       done
       echo "$sum"
   }

Saya tidak yakin apakah Anda terbuka untuk solusi lain. Juga saya tidak tahu apakah Anda dapat menggunakan utilitas eksternal, atau apakah Anda murni terbatas pada bash builtin. Jika Anda dapat menggunakan grep, misalnya, fungsi Anda bisa menjadi jauh lebih sederhana:

function nevens {
    printf "%s\n" "$@" | grep -c '[02468]$'
}

Ini menempatkan setiap integer input pada barisnya sendiri, dan kemudian digunakan grepuntuk menghitung garis yang berakhir pada angka genap.

Pembaruan - @PeterCordes menunjukkan bahwa kita bahkan dapat melakukan ini tanpa grep - hanya bash murni, selama daftar input berisi bilangan bulat yang terbentuk dengan baik (tanpa titik desimal):

function nevens{
    evens=( ${@/%*[13579]/} )
    echo "${#evens[@]}"
}

Ini berfungsi dengan membuat daftar yang dipanggil evensdengan menyaring semua peluang, lalu mengembalikan panjang daftar itu.