Zählerinkrement in Bash-Schleife funktioniert nicht

Ich habe das folgende einfache Skript, in dem ich eine Schleife ausführe und eine pflegen möchte COUNTER. Ich kann nicht herausfinden, warum der Zähler nicht aktualisiert wird. Liegt es an der Subshell, die erstellt wird? Wie kann ich das möglicherweise beheben?

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

Erstens erhöhen Sie den Zähler nicht. Ändern COUNTER=$((COUNTER))in COUNTER=$((COUNTER + 1))oder COUNTER=$[COUNTER + 1]wird es erhöhen.

Zweitens ist es schwieriger, Subshell-Variablen an den Angerufenen zurückzugeben, wie Sie vermuten. Variablen in einer Unterschale sind außerhalb der Unterschale nicht verfügbar. Dies sind lokale Variablen für den untergeordneten Prozess.

Eine Möglichkeit, dies zu lösen, besteht darin, eine temporäre Datei zum Speichern des Zwischenwerts zu verwenden:

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

GEPRÜFTE BASH: Centos, SuSE, RH

COUNTER=$((COUNTER+1)) 

ist ein ziemlich ungeschicktes Konstrukt in der modernen Programmierung.

(( COUNTER++ ))

sieht "moderner" aus. Sie können auch verwenden

let COUNTER++

wenn Sie denken, dass dies die Lesbarkeit verbessert. Manchmal gibt Bash zu viele Möglichkeiten, Dinge zu tun - Perl-Philosophie, nehme ich an -, wenn der Python "es gibt nur einen richtigen Weg, dies zu tun" vielleicht angemessener ist. Das ist eine umstrittene Aussage, wenn es jemals eine gab! Wie auch immer, ich würde vorschlagen, dass das Ziel (in diesem Fall) nicht nur darin besteht, eine Variable zu erhöhen, sondern (allgemeine Regel) auch Code zu schreiben, den jemand anderes verstehen und unterstützen kann. Konformität trägt wesentlich dazu bei.

HTH

Versuchen zu benutzen

COUNTER=$((COUNTER+1))

anstatt

COUNTER=$((COUNTER))

Ich denke, dieser einzelne awk-Aufruf entspricht Ihrer grep|grep|awk|awkPipeline: Bitte testen Sie ihn. Ihr letzter awk-Befehl scheint überhaupt nichts zu ändern.

Das Problem mit COUNTER ist, dass die while-Schleife in einer Subshell ausgeführt wird, sodass alle Änderungen an der Variablen beim Beenden der Subshell verschwinden. Sie müssen auf den Wert von COUNTER in derselben Subshell zugreifen. Oder befolgen Sie den Rat von @ DennisWilliamson, verwenden Sie eine Prozessersetzung und vermeiden Sie die Subshell insgesamt.

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

Anstatt eine temporäre Datei zu verwenden, können Sie whilemithilfe der Prozessersetzung vermeiden, eine Unterschale um die Schleife zu erstellen .

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

Übrigens sollten Sie in der Lage sein, all das grep, grep, awk, awk, awkin ein einziges zu verwandeln awk.

Ab Bash 4.2 gibt es eine lastpipeOption, die

führt den letzten Befehl einer Pipeline im aktuellen Shell-Kontext aus. Die Lastpipe-Option hat keine Auswirkung, wenn die Jobsteuerung aktiviert ist.

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

minimalistisch

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

Das ist alles was Sie tun müssen:

$((COUNTER++))

Hier ist ein Auszug aus Learning the Bash Shell , 3. Auflage, S. 147, 148:

Bash- Arithmetik-Ausdrücke entsprechen ihren Gegenstücken in den Sprachen Java und C. [9] Vorrang und Assoziativität sind dieselben wie in C. Tabelle 6-2 zeigt die unterstützten arithmetischen Operatoren. Obwohl einige davon Sonderzeichen sind (oder enthalten), müssen sie nicht mit einem Backslash-Escapezeichen versehen werden, da sie innerhalb der Syntax $ ((...)) liegen.

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

Die Operatoren ++ und - sind nützlich, wenn Sie einen Wert um eins erhöhen oder verringern möchten. [11] Sie funktionieren auf die gleiche wie in Java und C, zB Wert ++ Schritte Wert von 1. Dieses genannt wird Nachinkrement ; Es gibt auch ein Pre-Inkrement : ++ Wert . Der Unterschied wird anhand eines Beispiels deutlich:

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

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

Dies ist ein einfaches Beispiel

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

Es scheint, dass Sie counterdas Skript nicht aktualisiert habencounter++

Es gab zwei Bedingungen, die dazu führten, dass der Ausdruck ((var++))für mich fehlschlug:

1. Wenn ich bash auf den strengen Modus ( set -euo pipefail) setze und mein Inkrement bei Null (0) beginne .

2. Das Beginnen bei eins (1) ist in Ordnung, aber Null bewirkt, dass das Inkrement "1" zurückgibt, wenn "++" ausgewertet wird, was im strengen Modus ein Fehlercode für den Rückkehrcode ungleich Null ist.

Ich kann dieses Verhalten entweder nutzen ((var+=1))oder ihm var=$((var+1))entkommen

Das Quellenskript hat ein Problem mit der Subshell. Erstes Beispiel: Sie benötigen wahrscheinlich keine Unterschale. Aber wir wissen nicht, was unter "Noch mehr Action" verborgen ist. Die beliebteste Antwort hat einen versteckten Fehler, der die E / A erhöht und nicht mit Subshell funktioniert, da der Couter innerhalb der Schleife wiederhergestellt wird.

Fügen Sie kein '\' - Zeichen hinzu, da dies den Bash-Interpreter über die Fortsetzung der Leitung informiert. Ich hoffe es wird dir oder irgendjemandem helfen. Aber meiner Meinung nach sollte dieses Skript vollständig in ein AWK-Skript konvertiert oder mit Regexp oder Perl in Python umgeschrieben werden, aber die Popularität von Perl über Jahre hinweg wird beeinträchtigt. Mach es besser mit Python.

Korrigierte Version ohne 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

Version mit Subshell, wenn es wirklich benötigt wird

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