Sai lầm trong hàm shell để đếm số chẵn

Đối với một bài tập tôi phải viết một hàm in số lượng các số chẵn khi được cung cấp với một chuỗi các số.

Tôi đã sử dụng đoạn mã tôi đã sử dụng cho lần gán trước (để in 1khi số chẵn và 0khi số lẻ)

Vấn đề của tôi bây giờ là chức năng của tôi tiếp tục in 0. Tôi đang làm gì sai?

Đây là kịch bản của tôi:

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

Bạn chỉ quên thay thế $#bằng ( $) elementtrong forvòng lặp:

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
}

Bây giờ để kiểm tra chức năng:

$ 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

@dPlay đã tìm thấy vấn đề cốt lõi, tôi sẽ đưa ra một số đánh giá mã:

1. Các shebang: Không có /usr/bin/bashtrong Ubuntu. Đó là /bin/bash.

2. Thật tốt khi bạn đã khai báo sum localvà tránh làm ô nhiễm không gian tên biến ngoài hàm. Ngoài ra, bạn có thể khai báo nó một biến số nguyên bằng cách sử dụng -itùy chọn:

   local -i sum=0

3. Luôn trích dẫn các biến của bạn (và tham số)! Không cần thiết trong kịch bản này, nhưng một thói quen rất tốt để tham gia:

   for element in "$@"
   do

   Điều đó nói rằng, bạn có thể bỏ qua in "$@"ở đây:

   for element
   do

   Khi in <something>không được đưa ra, forvòng lặp ngầm lặp lại các đối số. Điều này có thể tránh những sai lầm như quên các trích dẫn.

4. Không cần phải tính toán và sau đó kiểm tra kết quả. Bạn có thể trực tiếp thực hiện phép tính trong if:

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

   (( ... ))là bối cảnh số học . Nó hữu ích hơn so [[ ... ]]với việc thực hiện kiểm tra số học và ngoài ra, bạn có thể bỏ qua các $biến trước (giúp dễ đọc hơn, IMHO).

5. Nếu bạn chuyển phần kiểm tra chẵn thành một chức năng riêng biệt, nó có thể cải thiện khả năng đọc và tái sử dụng:

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

Tôi không chắc chắn nếu bạn mở cho các giải pháp khác. Ngoài ra tôi không biết liệu bạn có thể sử dụng các tiện ích bên ngoài không, hoặc nếu bạn hoàn toàn bị giới hạn đối với các nội dung bash. Nếu bạn có thể sử dụng grep, ví dụ, chức năng của bạn có thể đơn giản hơn rất nhiều:

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

Điều này đặt mỗi số nguyên đầu vào trên một dòng riêng của nó, và sau đó sử dụng grepđể đếm các dòng kết thúc bằng một chữ số chẵn.

Cập nhật - @PeterCordes chỉ ra rằng chúng tôi thậm chí có thể thực hiện việc này mà không cần grep - chỉ cần bash thuần túy, miễn là danh sách đầu vào chỉ chứa các số nguyên được hình thành tốt (không có dấu thập phân):

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

Điều này hoạt động bằng cách tạo một danh sách được gọi evensbằng cách lọc ra tất cả các tỷ lệ cược, sau đó trả về độ dài của danh sách đó.