У меня есть это shellcheck
предупреждение, которое я не могу понять:
In /mnt/e/bin/iconic line 540:
printf "FALSE|" >> "$IconsRaw" # Select field number 1
^-- SC2129: Consider using { cmd1; cmd2; } >> file instead of individual redirects.
Я заметил, что многие из нас здесь используют shellcheck , чтобы исправить наши скрипты bash / команды оболочки так что я надеюсь, что вопрос по теме.
Согласно комментариям, публикующим соответствующий раздел скрипта bash:
if [[ "$X" == "?" || "$Y" == "?" ]] ; then
: # Bad X or Y offset usually "Link to Name.ext~" (backup name)
else
let i++
printf "FALSE|" >> "$IconsRaw" # Select field number 1
printf "%s|" "$i" >> "$IconsRaw" # 2
printf "%s|" "${File##*/}" >> "$IconsRaw"
printf "%s|" "$Linkless" >> "$IconsRaw" # 4
printf "%s|" "$Date" >> "$IconsRaw" # 5
printf "%s|" "$X" >> "$IconsRaw" # 6
echo "$Y" >> "$IconsRaw" # 7
fi
Благодаря принятому ответу и комментариям я узнал, что shellcheck
нет только ловит ошибки в вашем коде, но также предлагает улучшения производительности. В этом случае имя файла $IconsRaw
открывалось и закрывалось много раз с каждым printf
и echo
.
Более эффективный код bash:
# X,Y screen coordinates invalid on backup files ending with "~"
! [[ "$X" == "?" || "$Y" == "?" ]] && { let i++; echo \
"FALSE|$i|${File##*/}|$Linkless|$Date|$X|$Y" >> "$IconsRaw"; }
Я предполагаю, что ваш сценарий имеет несколько экземпляров >> "$IconsRaw"
. В этом сообщении предлагается перенаправить вывод только один раз и сгруппировать команды в подоболочку. Предположительно, чтобы избежать накладных расходов на открытие и закрытие файла несколько раз.
Итак, вместо этого:
printf "FALSE|" >> "$IconsRaw" # Select field number 1
printf "%s|" "$i" >> "$IconsRaw" # 2
printf "%s|" "${File##*/}" >> "$IconsRaw"
printf "%s|" "$Linkless" >> "$IconsRaw" # 4
printf "%s|" "$Date" >> "$IconsRaw" # 5
printf "%s|" "$X" >> "$IconsRaw" # 6
echo "$Y" >> "$IconsRaw" # 7
Это:
{
printf "FALSE|" # Select field number 1
printf "%s|" "$i" # 2
printf "%s|" "${File##*/}"
printf "%s|" "$Linkless" # 4
printf "%s|" "$Date" # 5
printf "%s|" "$X" # 6
printf "%s\n" "$Y" # 7
} >> "$IconsRaw"
Бу, это также ненужное повторение printf
, и более эффективно просто :
printf '%s|%s|%s|%s|%s|%s|%s\n' \
'FALSE' "$i" "${File##*/}" "$Linkless" \
"$Date" "$X" "$Y" >> "$IconsRaw"