Error en la función de shell para contar números pares

Para una tarea, tengo que escribir una función que imprima el número de números pares cuando se proporciona una secuencia de números.

Usé el código que usé para una tarea anterior (para imprimir 1cuando un número era par y 0cuando el número era impar)

Mi problema ahora es que mi función sigue imprimiendo 0. ¿Qué estoy haciendo mal?

Aquí está mi guión:

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

Simplemente olvidó reemplazar $#con ( $) elementen el forbucle:

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
}

Ahora para probar la función:

$ 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 encontrado el problema central, daré una revisión de código:

1. El shebang: No hay /usr/bin/bashen Ubuntu. Es /bin/bash.

2. Es bueno que hayas declarado sum localy evitado contaminar el espacio de nombres variable fuera de la función. Además, puede declararlo como una variable entera utilizando la -iopción:

   local -i sum=0

3. ¡Siempre cita tus variables (y parámetros)! No es necesario en este script, pero es un muy buen hábito para entrar:

   for element in "$@"
   do

   Dicho esto, puedes omitir el in "$@"aquí:

   for element
   do

   Cuando in <something>no se proporciona, el forbucle recorre implícitamente los argumentos. Esto puede evitar errores como olvidar las comillas.

4. No hay necesidad de calcular y luego verificar el resultado. Puedes hacer el cálculo directamente en if:

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

   (( ... ))Es el contexto aritmético . Es más útil que [[ ... ]]para realizar comprobaciones aritméticas, y además puede omitir las $variables anteriores (lo que hace que sea más fácil de leer, en mi humilde opinión).

5. Si movió la parte de verificación uniforme a una función separada, podría mejorar la legibilidad y la reutilización:

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

No estoy seguro si estás abierto a otras soluciones. Además, no sé si puede utilizar utilidades externas, o si está limitado exclusivamente a bash builtins. Si puede usar grep, por ejemplo, su función podría ser mucho más simple:

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

Esto coloca cada número entero de entrada en su propia línea y luego usa greppara contar las líneas que terminan en un dígito par.

Actualización: @PeterCordes señaló que incluso podemos hacer esto sin grep, solo bash puro, siempre que la lista de entrada contenga enteros bien formados (sin puntos decimales):

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

Esto funciona creando una lista llamada evensal filtrar todas las probabilidades y luego devolver la longitud de esa lista.