O incremento do contador no loop Bash não está funcionando

Eu tenho o seguinte script simples em que estou executando um loop e quero manter um COUNTER. Não consigo descobrir por que o contador não está atualizando. É devido ao subshell que está sendo criado? Como posso corrigir isso potencialmente?

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

Primeiro, você não está aumentando o contador. Mudar COUNTER=$((COUNTER))para COUNTER=$((COUNTER + 1))ou COUNTER=$[COUNTER + 1]aumentará.

Segundo, é mais difícil propagar novamente as variáveis ​​do subshell para o receptor conforme você supõe. Variáveis ​​em um subshell não estão disponíveis fora do subshell. Essas são variáveis ​​locais para o processo filho.

Uma maneira de resolvê-lo é usar um arquivo temporário para armazenar o valor intermediário:

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

FESTA TESTADA: Centos, SuSE, RH

COUNTER=$((COUNTER+1)) 

é uma construção bastante desajeitada na programação moderna.

(( COUNTER++ ))

parece mais "moderno". Você também pode usar

let COUNTER++

se você acha que melhora a legibilidade. Às vezes, Bash oferece muitas maneiras de fazer as coisas - filosofia Perl, suponho - quando talvez o Python "só existe uma maneira correta de fazer isso" possa ser mais apropriado. Essa é uma afirmação discutível, se é que houve alguma! De qualquer forma, eu sugeriria que o objetivo (neste caso) não é apenas incrementar uma variável, mas (regra geral) também escrever código que outra pessoa possa entender e apoiar. A conformidade contribui muito para conseguir isso.

HTH

Tente usar

COUNTER=$((COUNTER+1))

ao invés de

COUNTER=$((COUNTER))

Acho que essa chamada única do awk é equivalente ao seu grep|grep|awk|awkpipeline: teste-a. Seu último comando awk parece não mudar nada.

O problema com o COUNTER é que o loop while está sendo executado em um subshell, portanto, quaisquer alterações na variável desaparecem quando o subshell sai. Você precisa acessar o valor de COUNTER no mesmo subshell. Ou siga o conselho de @ DennisWilliamson, use uma substituição de processo e evite completamente o 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 ))

Em vez de usar um arquivo temporário, você pode evitar a criação de um subshell ao redor do whileloop usando a substituição do processo.

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

A propósito, você deve conseguir transformar tudo isso grep, grep, awk, awk, awkem um único awk.

Começando com o Bash 4.2, existe uma lastpipeopção que

executa o último comando de um pipeline no contexto atual do shell. A opção lastpipe não terá efeito se o controle do trabalho estiver ativado.

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

minimalista

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

É tudo o que você precisa fazer:

$((COUNTER++))

Aqui está um trecho de Learning the bash Shell , 3a edição, pp. 147, 148:

expressões aritméticas bash são equivalentes às suas contrapartes nas linguagens Java e C. [9] Precedência e associatividade são as mesmas que em C. A Tabela 6-2 mostra os operadores aritméticos suportados. Embora alguns desses sejam (ou contenham) caracteres especiais, não há necessidade de escapar com uma barra invertida, porque eles estão dentro da sintaxe $ ((...)).

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

Os operadores ++ e - são úteis quando você deseja aumentar ou diminuir um valor em um. [11] Eles funcionam da mesma forma que em Java e C, por exemplo, o valor ++ incrementa o valor em 1. Isso é chamado pós-incremento ; há também um pré-incremento : ++ valor . A diferença se torna evidente com um exemplo:

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

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

Este é um exemplo simples

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

Parece que você não atualizou countero script, usecounter++

Houve duas condições que causaram uma ((var++))falha na expressão :

1. Se eu definir o bash no modo estrito ( set -euo pipefail) e se eu iniciar o incremento em zero (0).

2. Iniciar um (1) é bom, mas zero faz com que o incremento retorne "1" ao avaliar "++", que é uma falha de código de retorno diferente de zero no modo estrito.

Eu posso usar ((var+=1))ou var=$((var+1))escapar desse comportamento

O script de origem tem algum problema com o subshell. Primeiro exemplo, você provavelmente não precisa do subshell. Mas não sabemos o que está oculto em "Um pouco mais de ação". A resposta mais popular tem bug oculto, que aumentará a E / S e não funcionará com subshell, porque restaura o loop interno do couter.

Não adicione o sinal '\', ele informará o intérprete do bash sobre a continuação da linha. Espero que ajude você ou qualquer um. Mas, na minha opinião, esse script deve ser totalmente convertido em script AWK, ou reescrito em python usando regexp ou perl, mas a popularidade do perl ao longo dos anos é degradada. Melhor fazê-lo com python.

Versão corrigida sem subcamada:

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

Versão com subshell se for realmente necessário

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