Späť na nástroje

Linter bash skriptov

Vložte shellový skript a nájdite chyby, ktoré sa prejavia až na serveri — neuvedené premenné, skripty s #!/bin/sh používajúce syntax platnú len v bashi, rm -rf $VAR, neuzavreté fi. Každé upozornenie je vysvetlené zrozumiteľne, s presným riadkom a stĺpcom.

Vstup

Váš skript

Nič sa nikam neodosiela — kontrola beží priamo v tomto okne prehliadača. Vložte celý súbor vrátane shebangu, aby fungovala kontrola dialektu.

Skontrolovaných riadkov: 1.

Výsledok

Zhrnutie

Kontrola založená na vzoroch pre časté a závažné chyby v bashi — nejde o plnohodnotnú implementáciu ShellCheck a nezachytí úplne všetko. Pre dôkladnú analýzu spustite skript cez ShellCheck
Shebang
Shebang nebol nájdený
Chyby
0
Varovania
1
Poznámky
0

Vložte skript vyššie, alebo načítajte ukážku, a skontrolujte ho.

Použitých opráv: 1 (1 × shebang added)

Nálezy

Čo sa rozbije a prečo

Iba skutočné chyby — žiadne štylistické ani prenositeľnostné preferencie.

ZávažnosťRiadok:stĺpecPravidloSprá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

Referencia

Prehľad častých chýb

Chyby, ktoré tento linter hľadá, a podoba ich opravy. Sú to tie, ktoré prežijú lokálne testovanie a zlyhajú až v produkcii.

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.

Prevádzkuješ vlastný herný server?

Herné servery Wespner s DDoS ochranou, NVMe diskami a aktiváciou do pár minút.

Prezrieť hosting