"If" "then" "else" 和 "return" 命令语法

Question

我是一名在焊接车间工作的管道安装工，并且需要音乐，因为商业广播非常重复。我去买了一个 RPI3 并使用 fm_transmitter 程序将 Raspian Stretch Lite 放在上面。我得到了足够长的 shell 脚本，收音机可以正常工作，但有时我会在短时间内再次播放一首歌。这只是偶然，因为 RANDOM_FILE 变量是从整个目录中选择的。我还获得了记录在文本文件中播放的每首歌曲的脚本。我想使用 tail 和 grep 在文本文件中搜索当前选择的 RANDOM_FILE 并检查它是否存在于日志中最后 X 首歌曲中。如果 RANDOM_FILE 存在于 grep 命令中，我希望脚本在定义新的 RANDOM_FILE 的顶部重新启动。如果它不存在，我希望脚本继续运行，允许 RANDOM_FILE 通过 fm_transmitter 程序播放。

我还希望有人检查定义 RANDOM_FILE 的字符串实际上是否真正随机且正确。

以下是当前形式的脚本。我相信我已经接近了，但是在某个地方出错了，因为我仍然会重复。我需要有关“if”“then”“else”和“return”命令的帮助。我希望这是一个快速简便的解决方案。

提前感谢所有花时间帮助我的人。

#!/bin/bash

while :
do
    files=(/home/pi/music/*.wav)
    RANDOM_FILE="${files[RANDOM % ${#files[@]}]}"

        if tail -n 25 /home/pi/transmit_log.txt | grep "$RANDOM_FILE" = true ; then
            echo -e "---SONG_SKIPPED---" >> /home/pi/transmit_log.txt ; return 7
        else

#            [ tail -n 25 /home/pi/transmit_log.txt | grep "$RANDOM_FILE" = false ] ; then
            return 22
        fi

    echo -e "$RANDOM_FILE" >> /home/pi/transmit_log.txt
    sox -v 3 "$RANDOM_FILE" -r 44100 -c 1 -b 16 -t wav - | sudo ./fm_transmitter -f 91.7 -
    sleep .5


done

Answer 1

我不相信你可以像这样在 if/else 语句中返回。 Return 在函数中用于返回值。 相反，您可以使用 continue ，它将跳过循环的下一次迭代（并像 codeforester 建议的那样修复测试）：

#!/bin/bash
while :
do
    files=(/home/pi/music/*.wav)
    RANDOM_FILE="${files[RANDOM % ${#files[@]}]}"

    if tail -n 25 /home/pi/transmit_log.txt | grep -q "$RANDOM_FILE" ; then
        echo "---SONG_SKIPPED---" >> /home/pi/transmit_log.txt
        continue
    fi

    echo "$RANDOM_FILE" >> /home/pi/transmit_log.txt
    sox -v 3 "$RANDOM_FILE" -r 44100 -c 1 -b 16 -t wav - | sudo ./fm_transmitter -f 91.7 -
    sleep .5
done

或者，最好将测试拆分如下所示，以使 if 条件更小且更具可读性：

already_played=$( tail -n 25 /home/pi/transmit_log.txt | grep "$RANDOM_FILE" )
if [ ! -z "$aldeady_played" ]; then
    ...
fi

Answer 2

问题出在此声明中：

if tail -n 25 /home/pi/transmit_log.txt | grep "$RANDOM_FILE" = true; then ...

正确的方法是：

if tail -n 25 /home/pi/transmit_log.txt | grep -q "$RANDOM_FILE"; then ...

在您的构造中，= true 将作为参数传递给grep。这不是你想要的，对吧？如果找到模式，grep -q 将返回 0（或 true），我相信这就是您想要的。