Số lượt truy cập trong vòng lặp Bash không hoạt động

Tôi có đoạn script đơn giản sau đây khi tôi đang chạy một vòng lặp và muốn duy trì một COUNTER. Tôi không thể tìm ra lý do tại sao quầy không cập nhật. Có phải là do subshell mà được tạo ra? Làm thế nào tôi có thể sửa lỗi này?

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

Đầu tiên, bạn không tăng quầy. Thay đổi COUNTER=$((COUNTER))thành COUNTER=$((COUNTER + 1))hoặc COUNTER=$[COUNTER + 1]sẽ tăng nó.

Thứ hai, sẽ khó khăn hơn trong việc truyền bá các biến phụ cho callee khi bạn phỏng đoán. Các biến trong một lớp con không có sẵn bên ngoài lớp con. Đây là các biến cục bộ cho quá trình con.

Một cách để giải quyết nó là sử dụng tệp tạm thời để lưu trữ giá trị trung gian:

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

BASH KIỂM TRA: Centos, SuSE, rh

COUNTER=$((COUNTER+1)) 

là một cấu trúc khá vụng về trong lập trình hiện đại.

(( COUNTER++ ))

trông "hiện đại" hơn. Bạn cũng có thể dùng

let COUNTER++

nếu bạn nghĩ rằng cải thiện khả năng đọc. Đôi khi, Bash đưa ra quá nhiều cách để thực hiện - triết lý Perl tôi cho rằng - khi có lẽ Python "chỉ có một cách đúng để làm điều đó" có thể phù hợp hơn. Đó là một tuyên bố gây tranh cãi nếu có một! Dù sao, tôi muốn đề xuất mục tiêu (trong trường hợp này) không chỉ là tăng một biến mà là (quy tắc chung) để viết mã mà người khác có thể hiểu và hỗ trợ. Sự phù hợp đi một chặng đường dài để đạt được điều đó.

HTH

Thử sử dụng

COUNTER=$((COUNTER+1))

thay vì

COUNTER=$((COUNTER))

Tôi nghĩ rằng cuộc gọi awk duy nhất này tương đương với grep|grep|awk|awkđường ống của bạn : vui lòng kiểm tra nó. Lệnh awk cuối cùng của bạn dường như không thay đổi gì cả.

Vấn đề với COUNTER là vòng lặp while đang chạy trong một lớp con, do đó, mọi thay đổi đối với biến sẽ biến mất khi thoát khỏi lớp con. Bạn cần truy cập giá trị của COUNTER trong cùng một mạng con. Hoặc nghe lời khuyên của @ DennisWilliamson, sử dụng thay thế quy trình và tránh hoàn toàn phần phụ.

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 ))

Thay vì sử dụng một tệp tạm thời, bạn có thể tránh tạo một lớp con xung quanh whilevòng lặp bằng cách sử dụng thay thế quy trình.

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

Nhân tiện, bạn sẽ có thể biến đổi tất cả grep, grep, awk, awk, awkthành một awk.

Bắt đầu với Bash 4.2, có một lastpipetùy chọn là

chạy lệnh cuối cùng của một đường ống dẫn trong bối cảnh shell hiện tại. Tùy chọn Lastpipe không có hiệu lực nếu bật chức năng kiểm soát công việc.

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

tối giản

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

Đây là tất cả những gì bạn cần làm:

$((COUNTER++))

Đây là một đoạn trích từ Học bash Shell , Ấn bản thứ 3, trang 147, 148:

các biểu thức số học bash tương đương với các đối tác của chúng trong các ngôn ngữ Java và C. [9] Ưu tiên và kết hợp giống như trong C. Bảng 6-2 cho thấy các toán tử số học được hỗ trợ. Mặc dù một số trong số này là (hoặc chứa) các ký tự đặc biệt, không cần phải gạch chéo lại để thoát chúng, bởi vì chúng nằm trong cú pháp $ ((...)).

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

Các toán tử ++ và - rất hữu ích khi bạn muốn tăng hoặc giảm một giá trị theo một. [11] Chúng hoạt động giống như trong Java và C, ví dụ: value ++ tăng giá trị lên 1. Đây được gọi là tăng sau ; cũng có một giá trị gia tăng trước : ++ . Sự khác biệt trở nên rõ ràng với một ví dụ:

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

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

Đây là một ví dụ đơn giản

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

Có vẻ như bạn không cập nhật counterkịch bản, sử dụngcounter++

Có hai điều kiện khiến biểu thức ((var++))thất bại đối với tôi:

1. Nếu tôi đặt bash thành chế độ nghiêm ngặt ( set -euo pipefail) và nếu tôi bắt đầu gia tăng của mình ở mức 0 (0).

2. Bắt đầu từ một (1) là tốt nhưng bằng không làm cho số gia trả về "1" khi đánh giá "++" là lỗi mã trả về khác không trong chế độ nghiêm ngặt.

Tôi có thể sử dụng ((var+=1))hoặc var=$((var+1))để thoát khỏi hành vi này

Kịch bản nguồn có một số vấn đề với subshell. Ví dụ đầu tiên, bạn có thể không cần subshell. Nhưng chúng tôi không biết những gì được ẩn trong "Một số hành động nữa". Câu trả lời phổ biến nhất có lỗi ẩn, nó sẽ tăng I / O và sẽ không hoạt động với subshell, vì nó khôi phục couter bên trong vòng lặp.

Không fortot thêm dấu '\', nó sẽ thông báo cho trình thông dịch bash về việc tiếp tục dòng. Tôi hy vọng nó sẽ giúp bạn hoặc bất cứ ai. Nhưng theo tôi, tập lệnh này nên được chuyển đổi hoàn toàn thành tập lệnh AWK, hoặc nếu không thì viết lại thành python bằng cách sử dụng regrec, hoặc perl, nhưng mức độ phổ biến của perl trong nhiều năm bị suy giảm. Tốt hơn làm điều đó với python.

Phiên bản sửa lỗi không có 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

Phiên bản có subshell nếu thực sự cần thiết

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