짝수를 세는 쉘 함수의 실수

과제를 위해 일련의 숫자가 제공 될 때 짝수의 숫자를 인쇄하는 함수를 작성해야합니다.

나는 (인쇄 나는 이전 할당에 사용되는 코드의 조각을 사용하는 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
}

루프 에서 $#( $) 로 바꾸는 것을 잊었습니다 .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
}

이제 기능을 테스트하십시오.

$ 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. shebang : /usr/bin/bash우분투 에는 없습니다 . 그것은이다 /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

   (( ... ))인 산술 콘텍스트 . [[ ... ]]산술 검사를 수행하는 것보다 유용하며 , 추가로 $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모든 배당률을 걸러 내고 목록의 길이를 반환하여 호출 되는 목록을 작성하여 작동 합니다.