Skip to content

Code review script .sh #2

Description

@Dooc-Nuken

1 - Evite toujours le [ dans tes scripts sh, utilise test c'est plus lisible pas besoin de ] et tu remplaces ton [ par test tu mets ! avant test par exemple

  • if ! test -d "$dossier_temp" -

2- quand tu utilises une variable utilise la toujours avec des {} - exemple ${dossier_temp} ce qui permet de forcer la délimitation du nom de variable - exemple : ${filename}s là où $filenames ça te split pas à filename

3- Quand tu fais tes erreurs fais : echo "ton erreur" >&2
ça print dans stderr au lieu de stdout, ce qui fera que sur un pipe de ton script, l'erreur ira pas dans la sortie et s'affichera quand même

4- Sur un set variable mets toujours des guillemets loop_dev="$(losetup ...)" si tu refous des guillemets dans ta command substitution t'as pas besoin de les escape, les parenthèses font déjà ça genre qsd="$(echo "bonsoir")"

Metadata

Metadata

Assignees

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions