偶数を数えるシェル関数の間違い

割り当ての場合、一連の数字が提供されたときに偶数の数を出力する関数を作成する必要があります。

以前の割り当てで使用したコードを使用しました（数字が偶数の場合と数字が奇数の場合に印刷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/bashUbuntuにはありません。です/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なしでもこれを実行できることを指摘しました。

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

これは、evensすべてのオッズを除外して呼び出されるリストを作成し、そのリストの長さを返すことで機能します。