Comments (2)
Mmmm, it kind of goes both ways. I like the simplicity of not using -n
because for people that forget all the operators it's actually more readable. Esp if they know they can not use -n
for the same thing they'll wonder "why is -n being used? what does it mean again?"
On top of that, I'm trying to focus on Bash and double-brace tests. So if your plan is to be portable to non-Bash shells, this is important, but not what my guide is optimizing for.
from bashstyle.
There's [[ ! -z "whatever" ]]
if you're trying to be minimal with operators.
I think that, on the scale of "making people remember hard things", [[ ! -z
is the best way to represent "string not null", but [[ -n
is less of a burden to comprehend than [[ "$string" ]]
. -n
involves remembering one more test flag (which, when not clear from context/comments, are always one quick Google search away): [[ "$variable" ]]
requires understanding that, unlike commands (including the single-brace [
test), expanding the first argument to [[
is not equivalent to substituting the content of the string, and double-brace has this whole special syntax that applies to how it evaluates arguments that doesn't apply for anything else.
And, to go back to my initial point, a forgotten operand is easier to type by accident.
from bashstyle.
Related Issues (20)
- Why to always use "if [[ ... ]]; then; fi" instead of a conditional statement HOT 7
- Tabs are not evil! HOT 11
- Use curly braces for all variable references HOT 6
- Use `command -v` instead of `which` &c HOT 5
- Locale independence HOT 1
- Throwing `set -a` and `set +a` around a variable pair list HOT 2
- Using defaults and "${VAR:?}" HOT 3
- Ownership and modes HOT 1
- Tracing and Debugging HOT 2
- Mention shellcheck HOT 3
- support for differences when considering zsh/sh compatibility? HOT 2
- Guidance on documenting functions HOT 7
- set -u HOT 1
- Example of "This can be queried/extracted with a simple function using reflection." HOT 2
- Google's Bash styleguide link is broken HOT 2
- [discuss] Suffix all bash scripts with .sh
- Article reference link appears broken.
- Add license information
- Coverage
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from bashstyle.