bashスクリプトを使用したバインドされていない変数

Question

私のスクリプトをデバッグするときに必死になっているので、シニアの同僚から私に推奨される構造をいくつか使用しました。幹部から

#!/bin/bash -x 
set -ueo pipefail 
exec &>/tmp/dq.log 
source ${BASH_SOURCE%/*}/env-prd.sh 

times=${2:-1} 
sleep=${3:-1} 

name="all-dq_hourly" 


fs_lock_file="/tmp/mwa/jobs/prd-${name}.lock" 

(flock -n 200 
    log="/var/log/mwa/prd/$(date +%Y-%m-%d)__${name}.log" 
    for i in $(seq 1 $times); do 
     if [[ ! -f /tmp/stop ]]; then 
     couple commands 

     fi 
     sleep $sleep 
    done 

) 200>"$fs_lock_file" | tee -a $log 

rm $fs_lock_file 

、私は、tee -a $log一部を結合していない変数との問題があることがわかりますcouple commandsオールライト実行されます。私はログ経路でバッククエストを使用しようとしましたが、メリットはありません。私はfs_lock_fileと同じ問題を疑うが、私はまだロギングを修正していない。
誰かが私の目を開いて、何が見えないのか教えていただけますか？私は指定されたパスにスクリプトのログを記録できません。

Answer 1

あなたが) [...] サブシェル内の変数log(を割り当てています。その変数は、そのサブシェルの外側に束縛されていません。

この場合、logをサブシェルの外側に設定するのがベストです。つまり、サブセルブロックの前に変数の割り当てを移動してください。一般

似たようなケースでは、あなたは中括弧（groupコマンドの構文）{とサブシェルの括弧を交換しようとすることができ[...] }。

グループコマンドは、現在のシェルで実行されます。サブシェル構文とは対照的に、リストは改行またはセミコロンで終わらなければならないことに注意してください.bash（1）のマンページのリストセクションの複合コマンドを参照してください。

一般的なベストプラクティスでは、スクリプトや関数の先頭に変数名、特に定数を設定すると、この種のバグを回避できます。