У меня есть 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.
Я заметил, что многие из нас здесь используютпроверка оболочкидля исправления наших 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"; }
решение1
Я предполагаю, что ваш скрипт имеет несколько экземпляров >> "$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"