การเพิ่มตัวนับใน Bash loop ไม่ทำงาน

ฉันมีสคริปต์ง่ายๆต่อไปนี้ที่ฉันใช้งานลูปและต้องการรักษาไฟล์COUNTER. ฉันไม่สามารถเข้าใจได้ว่าเหตุใดตัวนับจึงไม่อัปเดต เป็นเพราะการสร้าง subshell หรือไม่? ฉันจะแก้ไขปัญหานี้ได้อย่างไร

#!/bin/bash

WFY_PATH=/var/log/nginx
WFY_FILE=error.log
COUNTER=0
grep 'GET /log_' $WFY_PATH/$WFY_FILE | grep 'upstream timed out' | awk -F ', ' '{print $2,$4,$0}' | awk '{print "http://domain.com"$5"&ip="$2"&date="$7"&time="$8"&end=1"}' | awk -F '&end=1' '{print $1"&end=1"}' |
(
while read WFY_URL
do
    echo $WFY_URL #Some more action
    COUNTER=$((COUNTER+1))
done
)

echo $COUNTER # output = 0

ขั้นแรกคุณไม่ได้เพิ่มเคาน์เตอร์ เปลี่ยนCOUNTER=$((COUNTER))เป็นCOUNTER=$((COUNTER + 1))หรือCOUNTER=$[COUNTER + 1]จะเพิ่มขึ้น

ประการที่สองมันยากกว่าที่จะเผยแพร่ตัวแปร subshell กลับไปยัง callee เมื่อคุณคาดเดา ตัวแปรใน subshell ไม่พร้อมใช้งานนอก subshell ตัวแปรเหล่านี้เป็นตัวแปรเฉพาะของกระบวนการย่อย

วิธีหนึ่งในการแก้ปัญหาคือการใช้ไฟล์ชั่วคราวเพื่อจัดเก็บค่ากลาง:

TEMPFILE=/tmp/$$.tmp
echo 0 > $TEMPFILE

# Loop goes here
  # Fetch the value and increase it
  COUNTER=$[$(cat $TEMPFILE) + 1]

  # Store the new value
  echo $COUNTER > $TEMPFILE

# Loop done, script done, delete the file
unlink $TEMPFILE

COUNTER=1
while [ Your != "done" ]
do
     echo " $COUNTER "
     COUNTER=$[$COUNTER +1]
done

TESTED BASH: Centos, SuSE, RH

COUNTER=$((COUNTER+1)) 

เป็นโครงสร้างที่ค่อนข้างเงอะงะในการเขียนโปรแกรมสมัยใหม่

(( COUNTER++ ))

ดู "ทันสมัย" มากขึ้น คุณยังสามารถใช้

let COUNTER++

หากคุณคิดว่าช่วยเพิ่มความสามารถในการอ่าน บางครั้ง Bash ให้วิธีการทำสิ่งต่างๆมากเกินไป - ปรัชญา Perl ที่ฉันคิดไว้ - เมื่อบางที Python "มีทางเดียวเท่านั้นที่จะทำได้" อาจจะเหมาะสมกว่า นั่นเป็นคำพูดที่ถกเถียงกันได้ถ้าเคยมี! อย่างไรก็ตามฉันขอแนะนำจุดมุ่งหมาย (ในกรณีนี้) ไม่ใช่แค่การเพิ่มตัวแปร แต่ (กฎทั่วไป) เพื่อเขียนโค้ดที่คนอื่นสามารถเข้าใจและสนับสนุนได้ ความสอดคล้องไปได้ไกลในการบรรลุเป้าหมายนั้น

HTH

ลองนำไปใช้

COUNTER=$((COUNTER+1))

แทน

COUNTER=$((COUNTER))

ฉันคิดว่าการโทร awk เดียวนี้เทียบเท่ากับgrep|grep|awk|awkไปป์ไลน์ของคุณ: โปรดทดสอบ คำสั่ง awk สุดท้ายของคุณดูเหมือนจะไม่เปลี่ยนแปลงอะไรเลย

ปัญหาเกี่ยวกับ COUNTER คือ while loop กำลังทำงานใน subshell ดังนั้นการเปลี่ยนแปลงใด ๆ ในตัวแปรจะหายไปเมื่อออกจาก subshell คุณต้องเข้าถึงค่าของ COUNTER ใน subshell เดียวกันนั้น หรือรับคำแนะนำของ @ DennisWilliamson ใช้การทดแทนกระบวนการและหลีกเลี่ยง subshell โดยสิ้นเชิง

awk '
  /GET \/log_/ && /upstream timed out/ {
    split($0, a, ", ")
    split(a[2] FS a[4] FS $0, b)
    print "http://example.com" b[5] "&ip=" b[2] "&date=" b[7] "&time=" b[8] "&end=1"
  }
' | {
    while read WFY_URL
    do
        echo $WFY_URL #Some more action
        (( COUNTER++ ))
    done
    echo $COUNTER
}

count=0   
base=1
(( count += base ))

แทนที่จะใช้ไฟล์ชั่วคราวคุณสามารถหลีกเลี่ยงการสร้าง subshell รอบ ๆwhileลูปได้โดยใช้การทดแทนกระบวนการ

while ...
do
   ...
done < <(grep ...)

ยังไงก็ตามคุณควรจะแปลงร่างทั้งหมดให้grep, grep, awk, awk, awkเป็นหนึ่งเดียวawkได้

เริ่มต้นด้วย Bash 4.2 มีlastpipeตัวเลือกที่

รันคำสั่งสุดท้ายของไปป์ไลน์ในบริบทเชลล์ปัจจุบัน ตัวเลือก lastpipe จะไม่มีผลหากเปิดใช้งานการควบคุมงาน

bash -c 'echo foo | while read -r s; do c=3; done; echo "$c"'

bash -c 'shopt -s lastpipe; echo foo | while read -r s; do c=3; done; echo "$c"'
3

ที่เรียบง่าย

counter=0
((counter++))
echo $counter

นี่คือสิ่งที่คุณต้องทำ:

$((COUNTER++))

นี่คือข้อความที่ตัดตอนมาจากLearning the bash Shell , 3rd Edition, pp. 147, 148:

bashนิพจน์ทางคณิตศาสตร์เทียบเท่ากับคู่ของมันในภาษา Java และ C [9] ลำดับความสำคัญและความสัมพันธ์จะเหมือนกับใน C ตารางที่ 6-2 แสดงตัวดำเนินการทางคณิตศาสตร์ที่ได้รับการสนับสนุน แม้ว่าอักขระเหล่านี้บางตัวจะเป็นอักขระพิเศษ (หรือมี) แต่ก็ไม่จำเป็นต้องแบ็กสแลช - Escape เพราะอยู่ในไวยากรณ์ $ ((... ))

..........................

ตัวดำเนินการ ++ และ - มีประโยชน์เมื่อคุณต้องการเพิ่มหรือลดค่าทีละค่า [11] มันทำงานเช่นเดียวกับใน Java และ C เช่นค่า ++ เพิ่มขึ้นทีละ 1 ซึ่งเรียกว่าpost-Increment ; นอกจากนี้ยังมีก่อนที่เพิ่มขึ้น : ++ ค่า ความแตกต่างจะเห็นได้ชัดจากตัวอย่าง:

$ i=0
$ echo $i
0
$ echo $((i++))
0
$ echo $i
1
$ echo $((++i))
2
$ echo $i
2

ดูhttp://www.safaribooksonline.com/a/learning-the-bash/7572399/

นี่คือตัวอย่างง่ายๆ

COUNTER=1
for i in {1..5}
do   
   echo $COUNTER;
   //echo "Welcome $i times"
   ((COUNTER++));    
done

ดูเหมือนว่าคุณไม่ได้อัปเดตcounterสคริปต์คือใช้counter++

มีสองเงื่อนไขที่ทำให้นิพจน์((var++))ล้มเหลวสำหรับฉัน:

1. ถ้าฉันตั้งค่า bash เป็นโหมดเข้มงวด ( set -euo pipefail) และถ้าฉันเริ่มการเพิ่มที่ศูนย์ (0)

2. การเริ่มต้นที่หนึ่ง (1) เป็นสิ่งที่ดี แต่ศูนย์ทำให้ส่วนเพิ่มคืนค่าเป็น "1" เมื่อประเมิน "++" ซึ่งเป็นความล้มเหลวของโค้ดส่งคืนที่ไม่ใช่ศูนย์ในโหมดเข้มงวด

ฉันสามารถใช้((var+=1))หรือvar=$((var+1))เพื่อหลีกเลี่ยงพฤติกรรมนี้ได้

ซอร์สสคริปต์มีปัญหากับ subshell ตัวอย่างแรกคุณอาจไม่ต้องการ subshell แต่เราไม่รู้ว่ามีอะไรซ่อนอยู่ภายใต้ "Some more action" คำตอบยอดนิยมมีบั๊กซ่อนอยู่ซึ่งจะเพิ่ม I / O และจะไม่ทำงานกับ subshell เพราะจะคืนค่า couter ภายในลูป

อย่าใส่เครื่องหมาย '\' เสริมเพราะจะแจ้งให้ล่าม bash ทราบเกี่ยวกับความต่อเนื่องของบรรทัด ฉันหวังว่ามันจะช่วยคุณหรือใครก็ได้ แต่ในความคิดของฉันสคริปต์นี้ควรถูกแปลงเป็นสคริปต์ AWK อย่างสมบูรณ์หรือเขียนใหม่เป็น python โดยใช้ regexp หรือ perl แต่ความนิยม perl ในช่วงหลายปีที่ผ่านมาลดลง ดีกว่าทำด้วย python

เวอร์ชันที่แก้ไขโดยไม่มี subshell:

#!/bin/bash
WFY_PATH=/var/log/nginx
WFY_FILE=error.log
COUNTER=0
grep 'GET /log_' $WFY_PATH/$WFY_FILE | grep 'upstream timed out' |\
awk -F ', ' '{print $2,$4,$0}' |\
awk '{print "http://example.com"$5"&ip="$2"&date="$7"&time="$8"&end=1"}' |\
awk -F '&end=1' '{print $1"&end=1"}' |\
#(  #unneeded bracket
while read WFY_URL
do
    echo $WFY_URL #Some more action
    COUNTER=$((COUNTER+1))
done
# ) unneeded bracket

echo $COUNTER # output = 0

เวอร์ชันที่มี subshell หากจำเป็นจริงๆ

#!/bin/bash

TEMPFILE=/tmp/$$.tmp  #I've got it from the most popular answer
WFY_PATH=/var/log/nginx
WFY_FILE=error.log
COUNTER=0
grep 'GET /log_' $WFY_PATH/$WFY_FILE | grep 'upstream timed out' |\
awk -F ', ' '{print $2,$4,$0}' |\
awk '{print "http://example.com"$5"&ip="$2"&date="$7"&time="$8"&end=1"}' |\
awk -F '&end=1' '{print $1"&end=1"}' |\
(
while read WFY_URL
do
    echo $WFY_URL #Some more action
    COUNTER=$((COUNTER+1))
done
echo $COUNTER > $TEMPFILE  #store counter only once, do it after loop, you will save I/O
)

COUNTER=$(cat $TEMPFILE)  #restore counter
unlink $TEMPFILE
echo $COUNTER # output = 0