Bashループのカウンターの増分が機能しない

ループを実行していて、を維持したい次の簡単なスクリプトがありCOUNTERます。カウンターが更新されない理由を理解できません。サブシェルが作成されるためですか？どうすればこれを修正できますか？

#!/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]それを増加します。

第2に、想定どおり、サブシェル変数を呼び出し先に逆伝播するのはより困難です。サブシェル内の変数は、サブシェルの外部では使用できません。これらは、子プロセスにローカルな変数です。

これを解決する1つの方法は、中間値を格納するために一時ファイルを使用することです。

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：Centos、SuSE、RH

COUNTER=$((COUNTER+1)) 

現代のプログラミングでは、非常に扱いにくい構成です。

(( COUNTER++ ))

より「モダン」に見えます。あなたも使うことができます

let COUNTER++

あなたがそれが読みやすさを改善すると思うなら。時々、Bashは物事を実行する方法が多すぎます-私が思うPerlの哲学-多分、Pythonが「それを行うにはたった1つの正しい方法しかない」がより適切かもしれないとき。それがあったとしても、それは議論の余地のある声明です！とにかく、（この場合）目的は変数をインクリメントすることだけではなく、（一般的なルール）他の誰かが理解してサポートできるコードを書くことでもあると思います。適合性はそれを達成するための長い道のりです。

HTH

使ってみる

COUNTER=$((COUNTER+1))

の代わりに

COUNTER=$((COUNTER))

この1つのawk呼び出しはgrep|grep|awk|awkパイプラインと同等だと思います。テストしてください。最後のawkコマンドは何も変更していないようです。

COUNTERの問題は、whileループがサブシェルで実行されているため、サブシェルが終了すると、変数への変更がすべて消えることです。同じサブシェルでCOUNTERの値にアクセスする必要があります。または、@ DennisWilliamsonのアドバイスに従って、プロセス置換を使用し、サブシェルを完全に回避します。

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

一時ファイルを使用する代わりに、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に、サポートされている算術演算子を示します。これらの一部は特殊文字（またはそれらを含む）ですが、$（（...））構文内にあるため、バックスラッシュでエスケープする必要はありません。

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

++および-演算子は、値を1つずつ増減する場合に便利です。[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++))が失敗する原因となった2つの条件がありました。

1. bashを厳格モード（set -euo pipefail）に設定し、増分をゼロ（0）から開始した場合。

2. 1から始めても問題ありませんが、厳密モードでゼロ以外のリターンコードエラーである "++"を評価するときに、ゼロはインクリメントに "1"を返します。

この動作を使用する((var+=1))かvar=$((var+1))、回避することができます

ソーススクリプトのサブシェルに問題があります。最初の例では、おそらくサブシェルは必要ありません。しかし、「Some more action」に何が隠されているのかはわかりません。最も人気のある答えはバグを隠しており、I / Oが増加し、ループ内のコンピューターが復元されるため、サブシェルでは動作しません。

'\'記号を追加するのを忘れないでください。行の継続についてbashインタープリターに通知されます。私はそれがあなたや誰かを助けることを願っています。しかし、私の意見では、このスクリプトは完全にAWKスクリプトに変換するか、正規表現またはperlを使用してpythonに書き直す必要がありますが、perlの人気は何年にもわたって低下しています。Pythonでそれを行う方が良いです。

サブシェルなしの修正バージョン：

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

本当に必要な場合はサブシェル付きのバージョン

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