Çift sayıları saymak için kabuk işlevinde hata

Bir ödev için, bir dizi sayı sağlandığında çift sayıların sayısını yazdıran bir işlev yazmam gerekir.

Önceki bir ödev için kullandığım kod parçasını kullandım ( 1bir sayı çift 0olduğunda ve sayı tek olduğunda yazdırmak için)

Benim sorunum şimdi fonksiyonumun yazdırmaya devam etmesi 0. Neyi yanlış yapıyorum?

İşte benim senaryom:

#!/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
}

Sadece yerine unuttum $#(ile $) elementiçinde fordöngü:

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
}

Şimdi işlevi test etmek için:

$ 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

@ tatlı temel sorunu buldu, bazı kod incelemesi vereceğim:

1. Mesele: /usr/bin/bashUbuntu'da hiç yok . Öyle /bin/bash.

2. sum localİşlev adının dışında değişken ad alanını kirletmekten kaçınmanız ve kaçınmanız iyi olur . Ayrıca, şu -iseçeneği kullanarak bir tamsayı değişkeni bildirebilirsiniz :

   local -i sum=0

3. Her zaman değişkenlerinizi (ve parametrelerinizi) belirtin! Bu senaryoda gerekli değil, içine girmek için çok iyi bir alışkanlık:

   for element in "$@"
   do

   Bununla birlikte, in "$@"burada atlayabilirsiniz :

   for element
   do

   Verilmediğinde in <something>, fordöngü bağımsız değişkenlerin üzerinden dolaylı olarak döner. Bu, tırnak işaretlerini unutmak gibi hataları önleyebilir.

4. Sonucu hesaplamaya ve kontrol etmeye gerek yoktur. Hesaplamayı doğrudan şurada yapabilirsiniz if:

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

   (( ... ))olan aritmetik bağlam . [[ ... ]]Aritmetik kontroller yapmaktan daha kullanışlıdır ve ek olarak $önceki değişkenleri (okumayı kolaylaştırır, IMHO) atlayabilirsiniz .

5. Eşit denetleme parçasını ayrı bir işleve taşıdıysanız, okunabilirliği ve yeniden kullanılabilirliği artırabilir:

   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"
   }

Başka çözümlere açık olup olmadığınızdan emin değilim. Ayrıca harici yardımcı programları kullanıp kullanamayacağınızı veya tamamen bash yerleşikleriyle sınırlı olup olmadığınızı bilmiyorum. grepÖrneğin kullanabiliyorsanız , işleviniz çok daha basit olabilir:

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

Bu, her girdi tamsayısını kendi satırına yerleştirir ve ardından grepçift ​​basamakla biten satırları saymak için kullanır .

Güncelleme - @PeterCordes, giriş listesi sadece iyi biçimlendirilmiş tamsayılar (ondalık noktaları olmadan) içerdiği sürece, grep olmadan bile yapabileceğimize dikkat çekti - sadece saf bash:

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

Bu evens, tüm olasılıkları filtreleyerek ve ardından listenin uzunluğunu döndürerek çağrılan bir liste oluşturarak çalışır .