Powrót do narzędzi

Linter skryptów Bash

Wklej skrypt powłoki i znajdź błędy, które ujawniają się dopiero na serwerze — niecytowane zmienne, skrypty #!/bin/sh używające składni tylko dla basha, rm -rf $VAR, niezamknięte fi. Każde ostrzeżenie jest wyjaśnione prostym językiem, z dokładną linią i kolumną.

Dane wejściowe

Twój skrypt

Nic nie jest przesyłane na serwer — sprawdzanie odbywa się w tej karcie. Wklej cały plik, wraz z shebangiem, aby sprawdzanie dialektu działało poprawnie.

Sprawdzono 1 linii.

Wynik

Podsumowanie

Sprawdzanie oparte na wzorcach dla typowych, istotnych błędów basha — to nie jest pełna implementacja ShellCheck i nie wychwyci wszystkiego. Uruchom przez ShellCheck dla pełnej analizy
Shebang
Nie wykryto shebangu
Błędy
0
Ostrzeżenia
1
Uwagi
0

Wklej skrypt powyżej lub wczytaj przykład, aby go sprawdzić.

Zastosowano 1 poprawek: 1 × shebang added

Wyniki

Co się zepsuje i dlaczego

Tylko błędy poprawności — bez preferencji stylistycznych czy przenośności.

WażnośćLinia:KolumnaRegułaKomunikatPoprawka
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

Materiały

Ściągawka typowych błędów

Błędy, których szuka ten linter, i kształt poprawki. To te, które przechodzą testy lokalne, a zawodzą na produkcji.

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.

Prowadzisz własny serwer gier?

Serwery gier Wespner z ochroną DDoS, dyskami NVMe i aktywacją w kilka minut.

Zobacz hosting