Errore nella funzione shell per contare i numeri pari

Per un compito devo scrivere una funzione che stampa il numero di numeri pari quando fornito con una sequenza di numeri.

Ho usato il pezzo di codice che ho usato per un compito precedente (per stampare 1quando un numero era pari e 0quando il numero era dispari)

Il mio problema ora è che la mia funzione continua a stampare 0. Che cosa sto facendo di sbagliato?

Ecco la mia sceneggiatura:

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

Hai appena dimenticato di sostituire $#con ( $) elementnel forciclo:

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
}

Ora per testare la funzione:

$ 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

@dessert ha riscontrato il problema principale, fornirò una recensione del codice:

1. Lo shebang: non c'è /usr/bin/bashin Ubuntu. Lo è /bin/bash.

2. È positivo che tu abbia dichiarato sum localed evitato di inquinare lo spazio dei nomi delle variabili all'esterno della funzione. Inoltre, puoi dichiararlo come una variabile intera usando l' -iopzione:

   local -i sum=0

3. Cita sempre le tue variabili (e parametri)! Non è necessario in questo script, ma è un'ottima abitudine entrare in:

   for element in "$@"
   do

   Detto questo, puoi omettere il in "$@"qui:

   for element
   do

   Quando in <something>non viene fornito, il forciclo si sovrappone implicitamente agli argomenti. Questo può evitare errori come dimenticare le virgolette.

4. Non è necessario calcolare e quindi controllare il risultato. Puoi eseguire direttamente il calcolo in if:

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

   (( ... ))è il contesto aritmetico . È più utile che [[ ... ]]per eseguire controlli aritmetici e inoltre è possibile omettere le $variabili precedenti (il che rende più facile la lettura, IMHO).

5. Se si sposta la parte di controllo uniforme in una funzione separata, è possibile migliorare la leggibilità e la riusabilità:

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

Non sono sicuro che tu sia aperto ad altre soluzioni. Inoltre non so se è possibile utilizzare utility esterne o se si è limitati ai soli built-in bash. Se puoi usare grep, ad esempio, la tua funzione potrebbe essere molto più semplice:

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

In questo modo ogni numero intero di input si trova sulla propria riga e quindi viene utilizzato grepper contare le righe che terminano in una cifra pari.

Aggiornamento - @PeterCordes ha sottolineato che possiamo farlo anche senza grep - solo bash puro, purché l'elenco di input contenga solo numeri interi ben formati (senza punti decimali):

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

Funziona creando un elenco chiamato evensfiltrando tutte le probabilità, quindi restituendo la lunghezza di tale elenco.