From b0a1ada72b5d967c8c9617a2247cf3b5b56c2096 Mon Sep 17 00:00:00 2001 From: Eisuke Kawashima Date: Tue, 29 Jul 2025 10:32:25 +0900 Subject: [PATCH] Updated Optional (markdown) --- Optional.md | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/Optional.md b/Optional.md index 4995791..2ee28f6 100644 --- a/Optional.md +++ b/Optional.md @@ -41,6 +41,11 @@ desc: Suggest adding a default case in `case` statements example: case $? in 0) echo 'Success';; esac fix: case $? in 0) echo 'Success';; *) echo 'Fail' ;; esac +name: avoid-negated-conditions +desc: Suggest removing unnecessary comparison negations +example: [ ! "$var" -eq 1 ] +fix: [ "$var" -ne 1 ] + name: avoid-nullary-conditions desc: Suggest explicitly using -n in `[ $var ]` example: [ "$var" ] @@ -80,4 +85,10 @@ name: require-variable-braces desc: Suggest putting braces around all variable references example: var=hello; echo $var fix: var=hello; echo ${var} + +name: useless-use-of-cat +desc: Check for Useless Use Of Cat (UUOC) +example: cat foo | grep bar +fix: grep bar foo + ```