重写bash命令以符合ShellCheck

Question

如何重写以下bash命令以使其符合ShellCheck ？

memory=$(cat /proc/meminfo | grep 'MemTotal:' | awk {'print $2}' 2> /dev/null)

目前，它抱怨：

Answer 1

Shellcheck投诉是

使用cat filename | grep ' pattern ' cat filename | grep ' pattern '而不是grep ' pattern ' filename
awk命令中的第一个大括号位于单引号的外部，因此为文字； 应该是awk '{ command }'

因此，满足shellcheck的版本看起来像

memory=$(grep 'MemTotal:' /proc/meminfo | awk '{print $2}')

我不确定为什么您重定向了标准错误，所以我删除了它。

但是，将grep输出管道传输到awk很少是最佳解决方案。 awk几乎可以执行grep可以做的所有事情，因此您可以进一步简化为

memory=$(awk '/MemTotal:/ { print $2 }' /proc/meminfo)

没有管道！

Answer 2

只是猜测，您可能想要：

memory=$(grep 'MemTotal:' /proc/meminfo | awk '{print $2}' 2> /dev/null)

但是，那是shellcheck输出的直接解释，所以我不确定您要问什么。 就目前而言，您本质上是在问：“为什么这行不通？”。