Zpět na nástroje

Linter bash skriptů

Vložte shellový skript a najděte chyby, které se projeví až na serveru — neuvozené proměnné, skripty s #!/bin/sh používající syntaxi platnou jen v bashi, rm -rf $VAR, neuzavřené fi. Každé upozornění je vysvětleno srozumitelně, s přesným řádkem a sloupcem.

Vstup

Váš skript

Nic se nikam neodesílá — kontrola běží přímo v tomto okně prohlížeče. Vložte celý soubor včetně shebangu, aby fungovala kontrola dialektu.

Zkontrolováno řádků: 1.

Výsledek

Shrnutí

Kontrola založená na vzorech pro časté a závažné chyby v bashi — nejde o plnohodnotnou implementaci ShellCheck a nezachytí úplně vše. Pro důkladnou analýzu spusťte skript přes ShellCheck
Shebang
Shebang nebyl nalezen
Chyby
0
Varování
1
Poznámky
0

Vložte skript výše, nebo načtěte ukázku, a zkontrolujte ho.

Použito oprav: 1 (1 × shebang added)

Nálezy

Co se rozbije a proč

Pouze skutečné chyby — žádné stylistické ani přenositelnostní preference.

ZávažnostŘádek:sloupecPravidloZprávaOprava
warning1:1WX100

No shebang line

Without a #! line, what runs the script depends entirely on how it is invoked. "./script.sh" hands it to whatever the kernel falls back to, cron runs it under /bin/sh, and your interactive test under bash works fine — so it passes locally and fails as a cron job.

Add as the very first line: #!/usr/bin/env bash

Reference

Přehled častých chyb

Chyby, které tento linter hledá, a podoba jejich opravy. Jsou to ty, které přežijí lokální testování a selžou až v produkci.

Always quote expansions

cp $src $dst
cp "$src" "$dst"

Unquoted, the value is split on whitespace and then glob-expanded. One path with a space becomes two arguments.

"$@", never $@

mycmd $@
mycmd "$@"

"$@" is the only form that passes every argument through exactly as received, including empty ones.

[[ ]] is bash, [ ] is POSIX

#!/bin/sh … [[ -f $f ]]
#!/bin/sh … [ -f "$f" ]

On Debian and Ubuntu /bin/sh is dash. [[ does not exist there — "[[: not found" and the test is false.

Strings with =, numbers with -eq

[ "$n" = 10 ]
[ "$n" -eq 10 ]

= compares text, so "010" ≠ "10". -eq compares integers and errors loudly on non-numeric input.

cd can fail

cd /srv/app rm -rf ./cache
cd /srv/app || exit 1 rm -rf ./cache

If cd fails the script continues in the previous directory — and the next relative command hits the wrong tree.

Guard rm against an empty variable

rm -rf $DIR/
rm -rf "${DIR:?DIR is not set}"/

An unset $DIR makes this "rm -rf /". ${VAR:?msg} aborts instead of expanding to nothing.

Never read and write one file

sort f.txt > f.txt
sort f.txt > f.tmp && mv f.tmp f.txt

The redirect truncates the file before sort opens it, so you are left with an empty file.

Assignments take no spaces

NAME = value
NAME=value

With spaces the shell runs NAME as a command. The variable is never set and every later $NAME is empty.

Glob the directory, do not parse ls

for f in $(ls *.log)
for f in *.log

ls output is one string that gets split on whitespace, so filenames with spaces split into pieces.

read needs -r

while read line
while IFS= read -r line

Without -r, backslashes are eaten. Without IFS=, leading and trailing whitespace is stripped.

$? is only the last command

mycmd echo done [ $? -ne 0 ]
if ! mycmd; then …

The echo resets $?. Test the command directly rather than inspecting the status later.

Fail fast, fail loudly

#!/bin/bash
#!/bin/bash set -euo pipefail

-e stops on the first failure, -u catches typoed variable names, pipefail makes a broken pipeline fail.

Provozuješ vlastní herní server?

Herní servery Wespner s DDoS ochranou, NVMe disky a aktivací do pár minut.

Prohlédnout hosting