Created SC2160 (markdown)

koalaman
2015-07-18 11:47:15 -07:00
parent 64dda4c618
commit c6b3f473a6

33
SC2160.md Normal file

@@ -0,0 +1,33 @@
## Instead of '[ true ]', just use 'true'.
### Problematic code:
if [ true ]
then
echo "always triggers"
fi
### Correct code:
if true
then
echo "never triggers"
fi
### Rationale:
This is a stylistic suggestion to use `true` instead of `[ true ]`.
`[ true ]` seems to suggest that the value "true" is somehow relevant to the statement. This is not the case, it doesn't matter. You can replace it with `[ false ]` or `[ wombat ]`, and it will still always be true:
String | In brackets | Outside brackets
--------+--------------+-----------------
true | true | true
false | true | false
wombat | true | unknown command
It's therefore better to use it without brackets, so that the "true" actually matters.
### Exceptions:
None.