Ошибка в функции оболочки для подсчета четных чисел

Для назначения я должен написать функцию, которая печатает количество четных чисел, когда предоставляется последовательность чисел.

Я использовал фрагмент кода, который использовал для предыдущего назначения (для печати, 1когда число было четным, а 0когда число было нечетным)

Моя проблема сейчас в том, что моя функция продолжает печатать 0. Что я делаю неправильно?

Вот мой сценарий:

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

Вы просто забыли заменить $#на ( $) elementв forцикле:

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
}

Теперь для проверки функции:

$ 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 обнаружил основную проблему, я дам небольшой обзор кода:

1. Шебанг: нет /usr/bin/bashв Ubuntu. Это /bin/bash.

2. Хорошо, что вы объявили sum local, и избежали загрязнения пространства имен переменной вне функции. Кроме того, вы можете объявить его целочисленной переменной, используя -iопцию:

   local -i sum=0

3. Всегда указывайте свои переменные (и параметры)! В этом сценарии нет необходимости, но есть очень хорошая привычка:

   for element in "$@"
   do

   Тем не менее, вы можете опустить in "$@"здесь:

   for element
   do

   Когда in <something>не задано, forцикл неявно зацикливается на аргументах. Это может избежать ошибок, таких как забывание цитат.

4. Там нет необходимости рассчитывать, а затем проверить результат. Вы можете напрямую сделать расчет в if:

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

   (( ... ))это арифметический контекст . Это более полезно, чем [[ ... ]]для выполнения арифметических проверок, и, кроме того, вы можете опустить $переменные before (что упрощает чтение, IMHO).

5. Если вы переместили четную часть в отдельную функцию, это может улучшить читаемость и возможность повторного использования:

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

Я не уверен, что вы открыты для других решений. Также я не знаю, можете ли вы использовать внешние утилиты или вы просто ограничены встроенными средствами bash. Если вы можете использовать grep, например, ваша функция может быть намного проще:

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

Это помещает каждое входное целое число в свою собственную строку, а затем использует grepдля подсчета строк, заканчивающихся четной цифрой.

Обновление - @PeterCordes указало, что мы можем даже сделать это без grep - просто чистый bash, при условии, что входной список содержит только правильно сформированные целые числа (без десятичных точек):

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

Это работает путем создания списка, который вызывается evensпутем фильтрации всех коэффициентов, а затем возвращается длина этого списка.