linux - “read”命令不在“while read line”循环中执行

Question

This question already has answers here:

Closed 2 years ago.

Read input in bash inside a while loop
（5个答案）
第一个帖子在这里！我真的需要帮助这一点，我在谷歌上看了这个问题，但找不到一个有用的答案。这就是问题所在。
我很喜欢在bash中编写一些类似框架的代码。每个人都可以创建自己的模块并将其添加到框架中。但是。为了知道脚本需要什么参数，我创建了一个“args.conf”文件，该文件必须位于每个模块中，如下所示：

LHOST;true;The IP the remote payload will connect to.
LPORT;true;The port the remote payload will connect to.

第一列是参数名，第二列定义是否需要，第三列是说明。总之，长话短说，框架应该逐行读取args.conf文件，以向用户询问每个参数的值。下面是一段代码：

info "Reading module $name argument list..."
while read line; do
    echo $line > line.tmp
    arg=`cut -d ";" -f 1 line.tmp`
    requ=`cut -d ";" -f 2 line.tmp`
    if [ $requ = "true" ]; then
        echo "[This argument is required]"
    else
        echo "[This argument isn't required, leave a blank space if you don't wan't to use it]"
    fi
    read -p " $arg=" answer
    echo $answer >> arglist.tmp
done < modules/$name/args.conf
tr '\n' ' ' < arglist.tmp > argline.tmp
argline=`cat argline.tmp`
info "Launching module $name..."
cd modules/$name
$interpreter $file $argline
cd ../..
rm arglist.tmp
rm argline.tmp
rm line.tmp
succes "Module $name execution completed."

如您所见，它应该为每个参数询问用户一个值。。。但是：
1）读取命令似乎没有执行。它只是跳过它，而参数没有任何价值
2）尽管args.conf文件包含3行，但循环似乎只执行一次我在屏幕上看到的只是一次“[这个参数是必需的]”，模块只是启动（并且由于没有必需的参数而崩溃…）。
真的不知道该怎么办，这里。。。我希望这里有人有答案。
提前谢谢！
（对最终的错误表示抱歉，我是法国人）
阿尔法。

最佳答案

正如@另一个人在评论中指出的，问题是循环中的所有read命令都是从args.conf文件读取的，而不是用户。我处理这个问题的方法是通过一个不同于stdin（fd#0）的文件描述符重定向conf文件；我喜欢使用fd#3来处理这个问题：

while read -u3 line; do
    ...
done 3< modules/$name/args.conf

（注意：如果shell的 read命令不理解 -u选项，请改用 read line <&3。）
在这个脚本中，我还有很多其他的建议：
不带双引号的变量引用，例如 echo $line而不是 echo "$line"，以及 < modules/$name/args.conf而不是 < "modules/$name/args.conf"。未被引用的变量引用被拆分为单词（如果它们包含空格），任何与文件名匹配的通配符都将被匹配文件的列表替换。这会导致非常奇怪和间歇性的错误。不幸的是，使用 $argline依赖于分词来分隔多个参数；如果使用 bash（不是一般的POSIX shell），则可以使用数组；我将讨论这个问题。
您在任何地方都使用相对文件路径，并且在脚本中 cding。这往往是脆弱和混乱的，因为文件路径在脚本的不同位置是不同的，并且用户传入的任何相对路径在脚本第一次在其他位置传入时都将变为无效。更糟糕的是，在 cd时没有检查错误，因此如果任何 cd由于任何原因失败，那么整个脚本的其余部分将在错误的位置运行，并奇怪地失败。你最好弄清楚系统的根目录在哪里（作为一个绝对路径），然后引用其中的所有内容（例如 cd）。
实际上，你没有检查任何地方的错误。在编写任何类型的程序时，通常都是一个好主意，试着考虑什么可能出错，以及程序应该如何响应（同时也希望没有想到的事情也会出错）。有些人喜欢使用 < "$module_root/modules/$name/args.conf"使他们的脚本退出，如果任何简单的命令失败，但 this doesn't always do what you'd expect。我喜欢在脚本中显式地测试命令的退出状态，类似于：

command1 || {
    echo 'command1 failed!' >&2
    exit 1
}
if command2; then
    echo 'command2 succeeded!' >&2
else
    echo 'command2 failed!' >&2
    exit 1
fi

您正在当前目录中创建临时文件，这可能会导致随机冲突（与脚本的其他运行同时，任何碰巧有您正在使用的名称的文件等）。最好在开始时创建一个临时目录，然后将所有内容存储在其中（同样，按绝对路径）：

module_tmp="$(mktemp -dt module-system)" || {
    echo "Error creating temp directory" >&2
    exit 1
}
...
echo "$answer" >> "$module_tmp/arglist.tmp"

（顺便说一下，请注意，我使用的是 set -e而不是backticks。它们更容易阅读，而且没有一些微妙的语法奇怪的反勾。我建议换一个。）
说到这里，您过度使用了temp文件；使用shell变量和内置的shell特性可以很好地完成许多工作。例如，与其从配置文件中读取行，然后将它们存储在临时文件中并使用 $()将它们拆分为字段，还不如简单地从 cut到 echo：

arg="$(echo "$line" | cut -d ";" -f 1)"

…或者更好的方法是，使用 cut的内置功能，根据 read设置为的值拆分字段：

while IFS=";" read -u3 arg requ description; do

（请注意，由于 IFS的赋值是 IFS命令的前缀，因此它仅影响该命令；全局更改 read可能会产生奇怪的效果，应尽可能避免。）
类似地，将参数列表存储在文件中，将换行符转换为空格，然后读取该文件。。。您可以跳过任何或所有这些步骤。如果使用 IFS，请将arg列表存储在一个数组中：

arglist=()
while ...
    arglist+=("$answer") # or ("#arg=$answer")? Not sure of your syntax.
done ...

"$module_root/modules/$name/$interpreter" "$file" "${arglist[@]}"

（这种混乱的语法，加上双引号、大括号、方括号和at符号，通常是在 bash中展开数组的正确方法）。
如果不能依靠数组这样的扩展，那么至少可以用一个普通的变量来实现：

arglist=""
while ...
    arglist="$arglist $answer" # or "$arglist $arg=$answer"? Not sure of your syntax.
done ...

"$module_root/modules/$name/$interpreter" "$file" $arglist

... 但这可能会导致参数被word拆分和/或扩展到文件列表。

关于linux - “read”命令不在“while read line”循环中执行，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/43947458/

上一篇：linux - 如何在 LFS 系统上安装 wget

下一篇：linux - 命令在 CLI 中有效，但 1 在脚本中无效

Answer 1

正如@另一个人在评论中指出的，问题是循环中的所有read命令都是从args.conf文件读取的，而不是用户。我处理这个问题的方法是通过一个不同于stdin（fd#0）的文件描述符重定向conf文件；我喜欢使用fd#3来处理这个问题：

while read -u3 line; do
    ...
done 3< modules/$name/args.conf

（注意：如果shell的 read命令不理解 -u选项，请改用 read line <&3。）
在这个脚本中，我还有很多其他的建议：
不带双引号的变量引用，例如 echo $line而不是 echo "$line"，以及 < modules/$name/args.conf而不是 < "modules/$name/args.conf"。未被引用的变量引用被拆分为单词（如果它们包含空格），任何与文件名匹配的通配符都将被匹配文件的列表替换。这会导致非常奇怪和间歇性的错误。不幸的是，使用 $argline依赖于分词来分隔多个参数；如果使用 bash（不是一般的POSIX shell），则可以使用数组；我将讨论这个问题。
您在任何地方都使用相对文件路径，并且在脚本中 cding。这往往是脆弱和混乱的，因为文件路径在脚本的不同位置是不同的，并且用户传入的任何相对路径在脚本第一次在其他位置传入时都将变为无效。更糟糕的是，在 cd时没有检查错误，因此如果任何 cd由于任何原因失败，那么整个脚本的其余部分将在错误的位置运行，并奇怪地失败。你最好弄清楚系统的根目录在哪里（作为一个绝对路径），然后引用其中的所有内容（例如 cd）。
实际上，你没有检查任何地方的错误。在编写任何类型的程序时，通常都是一个好主意，试着考虑什么可能出错，以及程序应该如何响应（同时也希望没有想到的事情也会出错）。有些人喜欢使用 < "$module_root/modules/$name/args.conf"使他们的脚本退出，如果任何简单的命令失败，但 this doesn't always do what you'd expect。我喜欢在脚本中显式地测试命令的退出状态，类似于：

command1 || {
    echo 'command1 failed!' >&2
    exit 1
}
if command2; then
    echo 'command2 succeeded!' >&2
else
    echo 'command2 failed!' >&2
    exit 1
fi

您正在当前目录中创建临时文件，这可能会导致随机冲突（与脚本的其他运行同时，任何碰巧有您正在使用的名称的文件等）。最好在开始时创建一个临时目录，然后将所有内容存储在其中（同样，按绝对路径）：

module_tmp="$(mktemp -dt module-system)" || {
    echo "Error creating temp directory" >&2
    exit 1
}
...
echo "$answer" >> "$module_tmp/arglist.tmp"

（顺便说一下，请注意，我使用的是 set -e而不是backticks。它们更容易阅读，而且没有一些微妙的语法奇怪的反勾。我建议换一个。）
说到这里，您过度使用了temp文件；使用shell变量和内置的shell特性可以很好地完成许多工作。例如，与其从配置文件中读取行，然后将它们存储在临时文件中并使用 $()将它们拆分为字段，还不如简单地从 cut到 echo：

arg="$(echo "$line" | cut -d ";" -f 1)"

…或者更好的方法是，使用 cut的内置功能，根据 read设置为的值拆分字段：

while IFS=";" read -u3 arg requ description; do

（请注意，由于 IFS的赋值是 IFS命令的前缀，因此它仅影响该命令；全局更改 read可能会产生奇怪的效果，应尽可能避免。）
类似地，将参数列表存储在文件中，将换行符转换为空格，然后读取该文件。。。您可以跳过任何或所有这些步骤。如果使用 IFS，请将arg列表存储在一个数组中：

arglist=()
while ...
    arglist+=("$answer") # or ("#arg=$answer")? Not sure of your syntax.
done ...

"$module_root/modules/$name/$interpreter" "$file" "${arglist[@]}"

（这种混乱的语法，加上双引号、大括号、方括号和at符号，通常是在 bash中展开数组的正确方法）。
如果不能依靠数组这样的扩展，那么至少可以用一个普通的变量来实现：

arglist=""
while ...
    arglist="$arglist $answer" # or "$arglist $arg=$answer"? Not sure of your syntax.
done ...

"$module_root/modules/$name/$interpreter" "$file" $arglist

... 但这可能会导致参数被word拆分和/或扩展到文件列表。