Question

I have written a shell script with a switch case and takes in options 1-5 and performs operations on the variables accordingly.

My code works fine though theres a small change I would like to make in my code.

When the code Enters the default case argument,after that it still prompts for “Enter two numbers: ” and then prints “Invalid Option “which is inappropriate.Ideally the code should stop after invalid option.

I’ve just learnt shell scripting so I’m not that good at it.

I have copy pasted my source-code below:

#! /bin/bash

echo "1. Add
        2.Substract
    3.Divide
    4.Multiply
    5.Quit"

read -p "Enter an option:" option
read -p "Enter two numbes: " first second

case $option in

1)income=$[ first + second ];;
2)income=$[ first - second ];;
3)income=$[ first / second ];;
4)income=$[ first * second ];;
*)echo "Invalid Option";
    exit 1;;

esac

echo "Income=$income"

exit 0

I tried using if,else …but that doesn’t serve the purpose.
*Alternative approach:*

#! /bin/bash                                                                                                                                                                                                                                                                       

echo "1. Add
        2.Substract
    3.Divide
    4.Multiply
    5.Quit"

read -p "Enter an option:" option

if [ option > 5 ]
then 

    echo "Invalid Option"
        exit 1

else 

read -p "Enter two numbes: " first second

fi

case $option in

1)income=$[ first + second ];;
2)income=$[ first - second ];;
3)income=$[ first / second ];;
4)income=$[ first * second ];;
*)echo "Invalid Option";
    exit 1;;

esac

echo "Income=$income"

exit 0

Answer 1

You should look at the select builtin, it will help reduce the complexity of what you want by quite a lot. You can replace the echo/case with it.

In this case I think your problem is that you have your cases as 1.) rather than 1). Your user would need to type the number with the period for the former to match.

Additionally, you need to have your shebang line as #!/bin/bash, it will not work with the space you have there.