В чем ошибка в коде?

Question

[Андрей]

Если введи y или Y, то вывод YES
Если ввели n или N, то вывод NO
Всегда выводит YES

#!/bin/bash
read str
if [ $str == "y*" -o $str == "Y*" ]
then
	echo "YES"
else
	if [ $str == "n*" -o $str == "N*" ]
	then
		echo "NO"
	fi
fi

Answer 1

[xotkot]

Если кто-то знает как можно еще минимизировать и ускорить код, прошу показать)

read -e -n 1 -p "ответ ? [Y/n] " y
if [[ "$y" =~ ^[y,Y] ]]
  then echo "OK"
  else echo "NO"
fi

даже еще чуть проще можно так как мы и так считываем один символ:

read -e -n 1 -p "ответ ? [Y/n] " y
if [[ $y =~ [yY] ]]
  then echo "OK"
  else echo "NO"
fi

Comments

[Андрей]

read -e -n 1 -p y
Можешь пожалуйста рассказать про каждый параметр команды read?
Я прочитал в интернете, но не понял:(
Как-то по проще пожалуйста.
И что будет если я не напишу тот или инной параметр, что будет?
Спасибо огромное)

[Андрей]

-n 1 - ну это я понял, считывать только один символ :)

[Андрей]

=~ - это что значит?

[xotkot]

Андрей,
read -e -n 1 -p "ответ ? [Y/n] " y

-e для интерактивной оболочки: использует Bash-интерфейс readline для того что бы читать данные
-p строка приглашения . Без оконечного перевода строки, Обычно в ней выводят подсказку, перед тем как команда read будет считывать данные.

The read builtin
использовал -e так как помню что когда ввод был в интерактивной оболочке zsh то наблюдались проблемы

=~ - это что значит?

я просто напрямую использовал регулярное выражение в объявлении условия if

p/s
еще можно добавить положительное решение "ОК" при нажатии на Enter, то есть по умолчанию будет подразумеваться согласие, это можно сделать добавив проверку на условие пустой строки ^$

read -e -n 1 -p "ответ ? [Y/n] " y
if [[ $y =~ ^$|[Yy] ]]
  then echo "OK"
  else echo "NO"
fi

Answer 2

[Андрей]

Вот переписал так:

#!/bin/bash
read str 
if [ $str == "y" ] || [ $str == "Y" ] 
then echo "YES" 
else echo "NO" 
fi

Все работает!
Если кто-то знает как можно еще минимизировать и ускорить код, прошу показать)

Comments

[Vitsliputsli]

Слишком не увлекайтесь минимизацией, вполне нормально, единственное, защитите код от ввода пустой строки:
if [ "$str" == "y" ] || [ "$str" == "Y" ]
А с учетом, что обрабатываете только 1 символ, лучше и читать только 1.