Erreur dans la fonction shell pour compter les nombres pairs

Pour une affectation, je dois écrire une fonction qui imprime le nombre de nombres pairs lorsqu'il est fourni avec une séquence de nombres.

J'ai utilisé le morceau de code que j'ai utilisé pour une affectation précédente (pour imprimer 1quand un nombre était pair et 0quand le nombre était impair)

Mon problème est maintenant que ma fonction continue d'imprimer 0. Qu'est-ce que je fais mal?

Voici mon script:

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

Vous avez juste oublié de remplacer $#par ( $) elementdans la forboucle:

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
}

Maintenant, pour tester la fonction:

$ 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 a trouvé le problème principal, je vais donner une révision du code:

1. Le shebang: Il n'y /usr/bin/bashen a pas dans Ubuntu. Ça l'est /bin/bash.

2. C'est bien que vous ayez déclaré sum localet évité de polluer l'espace de noms de variable en dehors de la fonction. De plus, vous pouvez le déclarer une variable entière en utilisant l' -ioption:

   local -i sum=0

3. Citez toujours vos variables (et paramètres)! Ce n'est pas nécessaire dans ce script, mais c'est une très bonne habitude à prendre:

   for element in "$@"
   do

   Cela dit, vous pouvez omettre in "$@"ici:

   for element
   do

   Quand in <something>n'est pas donné, la forboucle boucle implicitement sur les arguments. Cela peut éviter des erreurs comme oublier les devis.

4. Il n'est pas nécessaire de calculer puis de vérifier le résultat. Vous pouvez directement faire le calcul dans if:

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

   (( ... ))est le contexte arithmétique . C'est plus utile que [[ ... ]]pour effectuer des vérifications arithmétiques, et en outre, vous pouvez omettre les $variables avant (ce qui le rend plus facile à lire, à mon humble avis).

5. Si vous avez déplacé la partie de vérification uniforme dans une fonction distincte, cela pourrait améliorer la lisibilité et la réutilisabilité:

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

Je ne sais pas si vous êtes ouvert à d'autres solutions. De plus, je ne sais pas si vous pouvez utiliser des utilitaires externes, ou si vous êtes uniquement limité aux builds bash. Si vous pouvez utiliser grep, par exemple, votre fonction pourrait être beaucoup plus simple:

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

Cela place chaque entier d'entrée sur sa propre ligne, puis utilise greppour compter les lignes qui se terminent par un chiffre pair.

Mise à jour - @PeterCordes a souligné que nous pouvons même le faire sans grep - juste du bash pur, tant que la liste d'entrée ne contient que des entiers bien formés (sans virgule décimale):

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

Cela fonctionne en créant une liste appelée evensen filtrant toutes les chances, puis en renvoyant la longueur de cette liste.