Shell Programming and Scripting

I see that you chose to ignore most of the suggestions I made last time and added exits outside your nested if statements statement (instead of as part of the else clauses that identify invalid input). Those exits will terminate your script whenever you select 1 or 2 in your main menu.

Please explain why you think it is necessary to test the value of $CHOSEN_KEY twice in cases 1 and 2:

Code:

case "$CHOSEN_KEY" in
(1) if [ "$CHOSEN_KEY" -eq 1 ]
    then process case 1
    fi;;
(2) if [ "$CHOSEN_KEY" -eq 2 ]
    then process case 2
    fi;;
esac

is an inefficient and confusing way to specify the same actions as:

Code:

case "$CHOSEN_KEY" in
(1) process case 1;;
(2) process case 2;;
esac

Hi Nawrajesh,

Try to remove the below lines from your code

Code:

 
exit ;
break;;

and replace it with

Code:

echo "End of Seclection" ;;

It worked fine for me while testing.