ความผิดพลาดในฟังก์ชั่นเชลล์เพื่อนับจำนวนคู่

สำหรับการมอบหมายฉันต้องเขียนฟังก์ชั่นที่พิมพ์จำนวนคู่เมื่อจัดลำดับหมายเลข

ฉันใช้ชิ้นส่วนของรหัสที่ฉันใช้สำหรับการบ้านก่อนหน้า (เพื่อพิมพ์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. Shebang: ไม่มี/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

   (( ... ))เป็นบริบททางคณิตศาสตร์ มันมีประโยชน์มากกว่า[[ ... ]]การทำการตรวจสอบทางคณิตศาสตร์และนอกจากนี้คุณสามารถละเว้น$ตัวแปรก่อนหน้า (ซึ่งทำให้ง่ายต่อการอ่าน 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 builtins grepตัวอย่างเช่นหากคุณสามารถใช้ฟังก์ชันของคุณอาจง่ายกว่านี้มาก:

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

สิ่งนี้ทำให้แต่ละจำนวนเต็มเข้าในบรรทัดของตัวเองแล้วใช้grepในการนับบรรทัดที่ลงท้ายด้วยเลขคู่

อัปเดต - @PeterCordes ชี้ให้เห็นว่าเราสามารถทำได้โดยไม่ต้องใช้ grep - เพียงแค่ทุบตีบริสุทธิ์ตราบใดที่รายการอินพุตมีจำนวนเต็มที่เกิดขึ้นเป็นอย่างดี (โดยไม่มีจุดทศนิยม):

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

สิ่งนี้ทำงานได้โดยการสร้างรายการที่เรียกว่าevensโดยการกรองอัตราต่อรองทั้งหมดแล้วส่งคืนความยาวของรายการนั้น