Bash循环中的计数器增量不起作用

Question

我有以下简单的脚本，我正在运行一个循环并希望维护一个COUNTER。我无法弄清楚为什么计数器没有更新。是因为创建了子shell吗？我该如何解决这个问题？

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

Answer 1

源脚本的子shell 有一些问题。 第一个例子，你可能不需要 subshel​​l。但我们不知道“Some more action”下隐藏了什么。 最流行的答案有隐藏的错误，这会增加 I/O，并且不适用于 subshel​​l，因为它会在循环中恢复 couter。

不要加 '\' 符号，它会通知 bash 解释器关于行继续。我希望它会帮助你或任何人。但在我看来，这个脚本应该完全转换为 AWK 脚本，或者使用 regexp 或 perl 重写为 python，但是 perl 多年来的流行度下降了。最好用 python 来做。

没有子shell的更正版本：

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

如果确实需要，则带有 subshel​​l 的版本

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

Answer 2

有两种情况导致表达式 ((var++)) 对我来说失败：

1. 如果我将 bash 设置为严格模式 (set -euo pipefail) 并且如果我从零开始增量 (0)。

   李>

2. 从一 (1) 开始很好，但在计算“++”时，零会导致增量返回“1”，这是严格模式下的非零返回代码故障。

我可以使用((var+=1)) 或var=$((var+1)) 来逃避这种行为

Answer 3

这是一个简单的例子

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

Answer 4

您可以通过使用进程替换来避免在while 循环周围创建子shell，而不是使用临时文件。

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

顺便说一句，您应该能够将所有grep, grep, awk, awk, awk 转换为单个awk。

从 Bash 4.2 开始，有一个 lastpipe 选项

运行最后一个命令 当前 shell 上下文中的管道。 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

Answer 5

这就是你需要做的所有事情：

$((COUNTER++))

这是Learning the bash Shell，第 3 版，第 147、148 页的摘录：

bash 算术表达式等价于它们在 Java 和 C 语言。[9]优先级和关联性相同 与 C 中一样。表 6-2 显示了支持的算术运算符。 虽然其中一些是（或包含）特殊字符，但有 无需反斜杠转义它们，因为它们在 $((...)) 内 语法。

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

++ 和 - 运算符在您想要递增或 将值减一。 [11]它们的工作方式与 Java 和 C 相同， 例如，value++ 将 value 增加 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/

Answer 6

count=0   
base=1
(( count += base ))

Answer 7

极简主义

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

Answer 8

COUNTER=$((COUNTER+1)) 

在现代编程中是一个相当笨拙的结构。

(( COUNTER++ ))

看起来更“现代”。你也可以使用

let COUNTER++

如果您认为这样可以提高可读性。有时，Bash 提供了太多的做事方式——我想是 Perl 哲学——而 Python “只有一种正确的方式去做”可能更合适。如果有的话，这是一个值得商榷的声明！无论如何，我建议目标（在这种情况下）不仅仅是增加一个变量，而是（一般规则）还要编写其他人可以理解和支持的代码。一致性对于实现这一目标大有帮助。

HTH

Answer 9

COUNTER=1
while [ Your != "done" ]
do
     echo " $COUNTER "
     COUNTER=$[$COUNTER +1]
done

经过测试的 BASH：Centos、SuSE、RH

Answer 10

我认为这个单一的 awk 调用相当于你的 grep|grep|awk|awk 管道：请测试它。您的最后一个 awk 命令似乎没有任何改变。

COUNTER 的问题在于 while 循环在子 shell 中运行，因此当子 shell 退出时，对变量的任何更改都会消失。您需要在同一个子 shell 中访问 COUNTER 的值。或者采纳@DennisWilliamson 的建议，使用进程替换，完全避免使用子shell。

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
}

Answer 11

首先，您没有增加计数器。将COUNTER=$((COUNTER)) 更改为COUNTER=$((COUNTER + 1)) 或COUNTER=$[COUNTER + 1] 会增加它。

其次，根据您的推测，将子shell 变量反向传播给被调用者会比较棘手。子shell 中的变量在子shell 之外不可用。这些是子进程的本地变量。

解决它的一种方法是使用临时文件来存储中间值：

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

Answer 12

看来你没有更新counter是脚本，使用counter++

Answer 13

尝试使用

COUNTER=$((COUNTER+1))

而不是

COUNTER=$((COUNTER))