How to check read input for validity in bash script

Question

I need to check user input of 5 characters against a predetermined list of servers in an if then statement to ensure the then statement only acts upon a correct input.

Here is my code

    printf "please select a system (serv1, serv2, serv3 or serv4):"
    read -e -n 5 input
      if [[ $input == "serv1" || "serv2" || "serv3" || "serv4" ]]
        then
          execute some code with $input value
        else
          echo "$input is an invalid selection"
      fi

This issue I'm having is regardless of user input it acts as if it's a valid entry.

I think this question may provide some insight: stackoverflow.com/questions/21157435/… — j_b
– j_b, Commented Mar 29, 2022 at 14:49

Zlemini · Accepted Answer · 2022-03-29 14:59:12Z

2

The conditional requires repetition of the $input variable like so:

if [[ $input == "serv1" || $input == "serv2" || $input == "serv3" || $input == "serv4" ]]
  then
      execute some code with $input value
    else
      echo "$input is an invalid selection"
  fi

answered Mar 29, 2022 at 14:59

Zlemini

5,0132 gold badges24 silver badges25 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

VeryDogeWow · Accepted Answer · 2022-03-29 15:04:38Z

1

As @Zlemini already stated, you need to fix the syntax of your if statement, because a string without a comparison/operator will always return to true.

What you could alternatively do is the following:

VALID_SERVERS="serv1 serv2 serv3 serv4"

if [[ "${VALID_SERVERS}" == *"$input"* ]]; then
  echo "execute some code with $input value"
else
  echo "$input is an invalid selection"
fi

The code above will check whether or not the provided value is a substring of VALID_SERVERS.

answered Mar 29, 2022 at 15:04

VeryDogeWow

17112 bronze badges

Comments

fpmurphy · Accepted Answer · 2022-03-29 16:09:16Z

1

Another way of doing this is to use a case statement:

#!/bin/bash

printf "please select a system (serv1, serv2, serv3 or serv4): "
read -e -n 5 input

case $input in
    serv1 | serv2 | serv3 | serv4) echo "execute some code with $input value";;
                                *) echo "$input is an invalid selection";;
esac

answered Mar 29, 2022 at 16:09

fpmurphy

2,5771 gold badge19 silver badges24 bronze badges

Collectives™ on Stack Overflow

How to check read input for validity in bash script

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related