Bash conditional based on exit code of command
Just remove the brackets:
#!/bin/bash
if ./success.sh; then
echo "First: success!"
else
echo "First: failure!"
fi
if ./failure.sh; then
echo "Second: success!"
else
echo "Second: failure!"
fi
Explanation: the thing that goes between if
and then
is a command (or series of commands), the exit status of which is used to determine whether to run the then
clause, or the else
clause. This is exactly what you want.
So why do people use brackets in if
statements? It's because normally you want to decide which branch of the if
to run based on some conditional expression (is "$a"
equal to "$b"
, does a certain file exist, etc). [
is actually a command which parses its arguments as a conditional expression (ignoring the final ]
), and then exits with either success or failure depending on whether the conditional is true or false. Essentially, [ ]
functions as an adapter that lets you use conditional expressions instead of command success/failure in your if
statements. In your case, you want success/failure not a conditional expression, so don't use the adapter.
BTW, you'll also sometimes see if [[ some expression ]]; then
and if (( some expression )); then
. [[ ]]
and (( ))
are conditional expressions built into bash syntax (unlike [
, which is a command). [[ ]]
is essentially a better version of [ ]
(with some syntax oddities cleaned up and some features added), and (( ))
is a somewhat similar construct that does arithmetic expressions.
BTW2 another thing you'll see in scripts is the exit status being tested by checking the special parameter $?
, which gives the exit status of the last command. It looks like this:
somecommand
if [ $? -eq 0 ]; then
echo "Somecommand: success!"
else
echo "Somecommand: failure!"
fi
I really consider this cargo cult programming. People are used to seeing [ ]
conditional expressions in if
statements, and this idiom puts the success test in the form of a conditional expression. But let me run through how it works: it takes the exit status of the command, puts it in a conditional expression, has [ ]
evaluate that and turn it right back into an exit status so if
can use it. That whole rigamarole is unnecessary; just put the command directly in the if
statement.
In addition to the accepted answer, note that you sometimes may wish to preserve the exit status for later use. In such cases, you can save it into a variable and then test that variable.
Here is an example using your commands and the Bash arithmetic (( ))
command for tests :
#!/bin/bash
./success.sh
es1=$?
echo "exit status = $es1"
if (( es1 == 0 )); then
echo "First: success! (es1 = $es1)"
else
echo "First: failure! (es1 = $es1)"
fi
./failure.sh
es2=$?
echo "exit status = $es2"
if (( es2 > 0 )); then
echo "Second: failure! ($es2 is greater than 0)"
else
echo "Second: success!"
fi