Изменение переменной в функции в зависимости от флага getopts bash

Я изменил пару вещей, потому что понял, что объяснять все мелкие детали в комментариях будет слишком долго.

Вот код:

#!/bin/bash

#script to capture camera serial port logs via usb or UART.

file_name='DashcamLog'
#Date e.g: 20210204T120159 (ISO 8601)
current_date=$(date +%Y%m%dT%H%M%S)
BAUDRATE='115200'
#change to reflect udev rules e.g /dev/h1*
BOARDS=(/dev/ttyUSB* /dev/ttyACM*)

usage()
{
    echo "Usage:"
    echo "Use $0 -a to use GNU screen to automatically capture logs on USB plug in"
    echo "Use $0 -s to use GNU screen to capture logs WITHOUT automatic capture on USB plug in."
    echo "Use $0 -m to use minicom to capture logs WITHOUT automatic capture on USB plug in."
    exit 1
}

#detects current devices + starts logging using screen(-s)/minicom(-m)
startLog()
{
    for i in "${BOARDS[@]}"
    do
        port_name="${i#/dev/}"
        serial_no=$(udevadm info --attribute-walk "$i" | grep -m 1 "ATTRS{serial}")
        new_file_name="${file_name}_${current_date}_${HOSTNAME}_${port_name}.log"
        case "$1" in
            "screen")
                echo "PUT THE SCREEN COMMAND HERE"
                ;;
            "minicom")
                echo "PUT THE MINICOM COMMAND HERE"
                ;;
            \?)
                echo "Invalid Option: -$OPTARG" 1>&2
                usage
                ;;
        esac
        echo "$port_name serial_no:  $serial_no $new_file_name"
    done
}

while getopts ":hms" opt
do
    case "$opt" in
        h)
            usage
            ;;
        s)
            startLog screen
            ;;
        m)
            startLog minicom
            ;;
        \?)
            echo "Invalid Option: -$OPTARG" 1>&2
            usage
            ;;
    esac
done
shift $((OPTIND -1))

Подробности:

• когда вы определяете переменные screen и minicom, у вас было "$( ... )". Команда внутри $() выполняется именно тогда, а не позже, когда вы поставите $1 позже. Вот почему я удалил эту часть и поместил команды прямо в функцию.
• в startLog() я повторно использовал case. Также можно использовать if.
• BOARDS - это массив. Поэтому, когда вы выполняете $BOARDS без указания индекса, вы получаете только первый элемент. Вы должны сделать ${BOARDS[@]}, чтобы обработать все элементы.
• если вы не знали, BOARDS=(/dev/ttyUSB* /dev/ttyACM*) расширит *, поэтому BOARDS будут определены как все файлы /dev/ttyUSB* и все файлы /dev/ttyACM*. Он не останется с *.
• когда вы определяете переменную из другой команды (var=$(command)), вам не нужно помещать вокруг нее " ". Это также сбивает с толку, поскольку вы должны заключать переменные в двойные кавычки внутри $(), поэтому у вас будут двойные кавычки внутри других двойных кавычек.

Я также помещаю его в тот стиль кода, который предпочитаю, но он гибкий и зависит от личных предпочтений.

Вы можете поместить свой код на https://www.shellcheck.net/ для проверки синтаксиса.