Fehler in der Shell-Funktion, um gerade Zahlen zu zählen

Für eine Aufgabe muss ich eine Funktion schreiben, die die Anzahl der geraden Zahlen druckt, wenn sie mit einer Folge von Zahlen versehen ist.

Ich habe den Code verwendet, den ich für eine vorherige Aufgabe verwendet habe (um zu drucken, 1wann eine Zahl gerade und 0wann die Zahl ungerade war)

Mein Problem ist jetzt, dass meine Funktion weiter druckt 0. Was mache ich falsch?

Hier ist mein Drehbuch:

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

Sie haben nur vergessen, in der Schleife durch $#( $) zu ersetzen :elementfor

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
}

Testen Sie nun die Funktion:

$ 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 hat das Kernproblem gefunden, ich gebe eine Codeüberprüfung :

1. The shebang: /usr/bin/bashIn Ubuntu gibt es keine . Es ist /bin/bash.

2. Es ist gut, dass Sie sum localden Variablennamensraum außerhalb der Funktion deklariert und nicht verschmutzt haben. Darüber hinaus können Sie es mit der folgenden -iOption als Ganzzahlvariable deklarieren :

   local -i sum=0

3. Zitieren Sie immer Ihre Variablen (und Parameter)! In diesem Skript ist es nicht erforderlich, aber es ist eine sehr gute Angewohnheit, sich darauf einzulassen:

   for element in "$@"
   do

   Das heißt, Sie können das in "$@"hier weglassen :

   for element
   do

   Wenn dies in <something>nicht angegeben ist, fordurchläuft die Schleife implizit die Argumente. Dies kann Fehler wie das Vergessen der Anführungszeichen vermeiden.

4. Es ist nicht erforderlich, das Ergebnis zu berechnen und anschließend zu überprüfen. Sie können die Berechnung direkt in den folgenden Schritten durchführen if:

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

   (( ... ))ist der arithmetische Kontext . Dies ist nützlicher als die [[ ... ]]Durchführung von arithmetischen Prüfungen. Außerdem können Sie die $Vor-Variablen weglassen (was das Lesen erleichtert, IMHO).

5. Wenn Sie den Teil mit der geraden Prüfung in eine separate Funktion verschoben haben, kann dies die Lesbarkeit und Wiederverwendbarkeit verbessern:

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

Ich bin mir nicht sicher, ob Sie für andere Lösungen offen sind. Ich weiß auch nicht, ob Sie externe Dienstprogramme verwenden können oder ob Sie sich nur auf Bash-Builtins beschränken. Wenn Sie grepzum Beispiel verwenden können, könnte Ihre Funktion viel einfacher sein:

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

Dadurch wird jede Eingabe-Ganzzahl in eine eigene Zeile gesetzt und anschließend grepdie Zeilen gezählt, die mit einer geraden Ziffer enden.

Update - @PeterCordes wies darauf hin, dass wir dies sogar ohne grep tun können - nur reine Bash, solange die Eingabeliste nur gut geformte Ganzzahlen enthält (ohne Dezimalstellen):

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

Dies funktioniert, indem Sie eine Liste erstellen, die aufgerufen wird, evensindem Sie alle Quoten herausfiltern und dann die Länge dieser Liste zurückgeben.