Kenaikan penghitung di loop Bash tidak berfungsi

Saya memiliki skrip sederhana berikut di mana saya menjalankan loop dan ingin mempertahankan file COUNTER. Saya tidak dapat mengetahui mengapa penghitung tidak memperbarui. Apakah karena subkulit yang sedang dibuat? Bagaimana cara memperbaikinya?

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

Pertama, Anda tidak meningkatkan penghitung. Mengubah COUNTER=$((COUNTER))menjadi COUNTER=$((COUNTER + 1))atau COUNTER=$[COUNTER + 1]akan meningkatkannya.

Kedua, lebih sulit untuk memperbanyak variabel subkulit ke callee seperti yang Anda duga. Variabel dalam subkulit tidak tersedia di luar subkulit. Ini adalah variabel lokal untuk proses anak.

Salah satu cara untuk mengatasinya adalah dengan menggunakan file temp untuk menyimpan nilai antara:

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 DIUJI: Centos, SuSE, RH

COUNTER=$((COUNTER+1)) 

adalah konstruksi yang cukup kikuk dalam pemrograman modern.

(( COUNTER++ ))

terlihat lebih "modern". Anda juga bisa menggunakan

let COUNTER++

jika menurut Anda itu meningkatkan keterbacaan. Kadang-kadang, Bash memberikan terlalu banyak cara untuk melakukan sesuatu - filosofi Perl saya kira - ketika mungkin Python "hanya ada satu cara yang benar untuk melakukannya" mungkin lebih tepat. Itu adalah pernyataan yang bisa diperdebatkan jika memang ada! Bagaimanapun, saya akan menyarankan tujuan (dalam kasus ini) tidak hanya untuk menaikkan variabel tetapi (aturan umum) untuk juga menulis kode yang dapat dipahami dan didukung orang lain. Kesesuaian sangat membantu untuk mencapai itu.

HTH

Coba gunakan

COUNTER=$((COUNTER+1))

dari pada

COUNTER=$((COUNTER))

Saya rasa satu panggilan awk ini setara dengan grep|grep|awk|awkpipeline Anda : silakan coba. Perintah awk terakhir Anda tampaknya tidak mengubah apa pun.

Masalah dengan COUNTER adalah perulangan while berjalan dalam subkulit, jadi setiap perubahan pada variabel menghilang ketika subkulit keluar. Anda perlu mengakses nilai COUNTER di subkulit yang sama. Atau ikuti saran @ DennisWilliamson, gunakan substitusi proses, dan hindari subkulit sama sekali.

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

Daripada menggunakan file sementara, Anda dapat menghindari pembuatan subkulit di sekitar whileloop dengan menggunakan substitusi proses.

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

Ngomong-ngomong, Anda harus bisa mengubah semua itu grep, grep, awk, awk, awk menjadi satu awk.

Dimulai dengan Bash 4.2, ada file lastpipe opsi itu

menjalankan perintah terakhir dari pipeline dalam konteks shell saat ini. Opsi pipa terakhir tidak berpengaruh jika kontrol pekerjaan diaktifkan.

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

minimalis

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

Ini semua yang perlu Anda lakukan:

$((COUNTER++))

Berikut kutipan dari Learning the bash Shell , Edisi ke-3, hlm. 147, 148:

ekspresi aritmatika bash setara dengan rekan-rekan mereka dalam bahasa Java dan C. [9] Presedensi dan asosiativitas sama seperti pada C. Tabel 6-2 menunjukkan operator aritmatika yang didukung. Meskipun beberapa di antaranya adalah (atau berisi) karakter khusus, tidak perlu meloloskan diri dari garis miring terbalik, karena mereka berada di dalam sintaks $ ((...)).

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

Operator ++ dan - berguna saat Anda ingin menambah atau mengurangi nilai satu per satu. [11] Mereka bekerja sama seperti di Java dan C, misalnya, nilai ++ menambah nilai sebesar 1. Ini disebut kenaikan pasca ; ada juga pra-increment : ++ nilai . Perbedaannya menjadi jelas dengan contoh:

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

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

Ini adalah contoh sederhana

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

Tampaknya Anda tidak memperbarui counterskrip is, gunakancounter++

Ada dua kondisi yang menyebabkan ekspresi ((var++))gagal untuk saya:

1. Jika saya menyetel bash ke mode ketat ( set -euo pipefail) dan jika saya memulai kenaikan saya dari nol (0).

2. Memulai dari satu (1) tidak masalah, tetapi nol menyebabkan kenaikan menghasilkan "1" saat mengevaluasi "++" yang merupakan kegagalan kode pengembalian bukan nol dalam mode ketat.

Saya bisa menggunakan ((var+=1))atau var=$((var+1))menghindari perilaku ini

Skrip sumber memiliki masalah dengan subkulit. Contoh pertama, Anda mungkin tidak membutuhkan subkulit. Tapi Kami tidak tahu apa yang tersembunyi di bawah "Beberapa tindakan lagi". Jawaban paling populer memiliki bug tersembunyi, yang akan meningkatkan I / O, dan tidak akan berfungsi dengan subkulit, karena memulihkan couter di dalam loop.

Jangan menambahkan tanda '\', ini akan menginformasikan juru bahasa bash tentang kelanjutan baris. Saya harap ini akan membantu Anda atau siapa pun. Tapi menurut saya skrip ini harus sepenuhnya dikonversi ke skrip AWK, atau ditulis ulang ke python menggunakan regexp, atau perl, tetapi popularitas perl selama bertahun-tahun menurun. Lebih baik lakukan dengan python.

Versi yang Dikoreksi tanpa subkulit:

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

Versi dengan subkulit jika benar-benar dibutuhkan

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