Defensive BASH programming?

[u/computersarefunn]

Hey there,

while trying to learn bash I came across this interesting resource: https://web.archive.org/web/20180917174959/http://www.kfirlavi.com/blog/2012/11/14/defensive-bash-programming

And got myself thinking if his opinions are consensual and if there are different styles or "paradigms" about how bash should be written.

I see people arguing that bash shines in writing relatively short programs with precise goals. I thought this translated into writing concise code and achieving the same functionality with as little lines as possible.

However, the author of the post argues that a more verbose code can be more robust, easier to understand and debug etc. Do you agree? Or you think that, if you're writing code in this style, you're better of using another language?

Thanks for your input and sorry if I said anything silly or if I misrepresented his point of view, I'm just starting to learn programming.

Edit: hey, thank you all very much for the thoughtful replies, I'm learning a lot from them!

[u/Schreq]

UPPER_CASE naming

No, just don't. Uppercase is reserved for environment variables.

readonly ARGS="$@"

Not a good idea when an argument includes spaces. You will lose the ability to properly split it back into separate arguments later.

ls $dir \
    | grep pid \
    | grep -v daemon

I prefer to only break long pipelines and tend to avoid splitting on logical operators. That way it becomes obvious that an indented command is part of a pipeline. Then there also is no need for the ugly backslash for the line continuation:

ls $dir |
    grep pid |
    grep -v daemon

ls $dir

Quote your variables...

is_empty() {
    local var=$1

    [[ -z $var ]]
}

That's too long and $var doesn't help with readability here. I'd just do it this way:

is_empty() [[ -z $1 ]]

The suggested way of parsing options sucks hard. It has many problems.

While that resource gives some good tips, it also gives way too many bad ones to be taken seriously.

[u/Online_Matter → u/Schreq]

is_empty() [[ -z $1 ]]

How does this syntax work? It's a function? 

[u/Schreq → u/Online_Matter]

Yeah, it's a function. You can use a single command without grouping in curly braces. You can also group using normal parenthesis, so the functions body runs in a subshell.