Как верно обработать результат функции которая возвращает boolean?

Question

[historydev]

checkLocationExists () {
  pathWithShieldingSlash=$(echo "$1" | sed 's/\//\\\//g')
  isPathExists=$(sed -n 's/.*\(location '"$pathWithShieldingSlash"'.*\).*{/\1/p' "$locations_path/$file_name")

  if [[ -z $isPathExists ]]; then
    return 0
  else
    return 1
  fi
}

test=$(checkLocationExists "$path")

if [[ "$test" = true ]]; then
  echo "FOUND"
else
  echo "NOT FOUND"
fi

Я всегда получаю NOT FOUND

А должен получать FOUND если найдено.

Answer 1

[Saboteur]

if checkLocationExists "$path"; then
  echo "FOUND"
else
  echo "NOT FOUND"
fi

да и функцию можно упростить

checkLocationExists () {
  pathWithShieldingSlash=$(echo "$1" | sed 's/\//\\\//g')
  isPathExists=$(sed -n 's/.*\(location '"$pathWithShieldingSlash"'.*\).*{/\1/p' "$locations_path/$file_name")
  return [[ -z $isPathExists ]]
}

тут в принципе даже return не обязателен - вернет результат последней команды, но с return лучше читается

Comments

[historydev]

Я добавил это условие, потому-что при попытке вернуть как в примере функции, ошибка:
line 45: return: [[: numeric argument required

Пример условия всё ещё не работает, его я уже пробовал:

location /projects/test/v2.2.2/socket.io/ 
location /projects/test/v2.2.2
location /projects/test/v2.2.2/socket.io/
location /projects/test/v2.2.2
line 45: return: [[: numeric argument required
NOT FOUND

[Saboteur]

historydev,
ну не очень очевидно что именно у вас в 45й строке.
Не очень похоже на то условие, что я указал.
Ошибка говорит, что требуется числовой аргумент. В том коде что я дал, нигде он не требуется.
Приведите кусок текста с 40 до 50 строки

[historydev]

Saboteur, мне врать незачем, вот весь скрипт, это не работает.

while getopts n:v:a:p:sd:mt flag
do
    case "${flag}" in
        n) name=${OPTARG};;
        v) version=${OPTARG};;
        a) address=${OPTARG};;
        p) port=${OPTARG};;
        s) socket=true;;
        m) modsecurity=true;;
        d) domain=${OPTARG};;
        t) test=true
    esac
done

locations_folder="/etc/nginx/locations"
path="/projects/$name/$version"
socketPath="$path/socket.io/"
file_name="$domain.$name.locations"
location=""

if [[ $test ]]; then
  locations_folder="./etc/nginx/locations"
fi







writeLocation () { printf "\n$location\n" >> "$locations_folder/$file_name"; }

checkLocationExists () {
  pathWithShieldingSlash=$(echo "$1" | sed 's/\//\\\//g')
  isPathExists=$(sed -n 's/.*\(location '"$pathWithShieldingSlash"'.*\).*{/\1/p' "$locations_folder/$file_name")

  echo "$isPathExists"

#  if [[ -z $isPathExists ]]; then
#    return 0
#  else
#    return 1
#  fi

  return [[ -z $isPathExists ]]
}

test=$(checkLocationExists "$path")

if checkLocationExists "$path"; then
  echo "FOUND"
else
  echo "NOT FOUND"
fi

[Saboteur]

А это что такое: "location /projects/test/v2.2.2" в вашем выводе?

у вас путь состоит из фразы "location /projects/test/v2.2.2" или просто "/projects/test/v2.2.2"

уточните

[Saboteur]

Попробуйте
return [[ -z "$isPathExists" ]]

[historydev]

Saboteur, location /projects/test/v2.2.2

Это найденные строки в файле, вывод здесь происходит:

isPathExists=$(sed -n 's/.*\(location '"$pathWithShieldingSlash"'.*\).*{/\1/p' "$locations_folder/$file_name")

  echo "$isPathExists"

[historydev]

Saboteur,

Попробуйте

Ошибка аналогичная, ide ругается тоже:

Если я идиот и это важно, скажите, я запускаю с винды, git bash.

[historydev]

Saboteur, и ругается по понятной причине:

Only one integer 0-255 can be returned. Use stdout for other data.
See SC2151.

Тоже самое написано в гугле, мол функция может возвращать только int 0-255, а для всего остального юзать stdout.

Вот я и решил добавить условие с 0 и 1, в гугле написали что это работает как и везде true/false, я пробовал так-же в условии == 1/0 - это не сработало.

[Saboteur]

А, не, это я идиот =)
return уберите, оставьте только [[ -z "$isPathExists" ]]

[Saboteur]

Я забыл, что самому return нужно дать именно параметр, а не выполнить команду
В данном случае можно без return, функция автоматом вернет результат выполнения последней команды

[historydev]

Saboteur, я уже пробовал, получаю "NOT FOUND" всё равно

[Saboteur]

Выполните отладку. Ваш echo мало что дает, сделайте детальнее:

checkLocationExists () {
  pathWithShieldingSlash=$(echo "$1" | sed 's/\//\\\//g')
  isPathExists=$(sed -n 's/.*\(location '"$pathWithShieldingSlash"'.*\).*{/\1/p' "$locations_path/$file_name")
  echo "isPathExist=[$isPathExists], string is empty or not: $([[ -z "$isPathExists" ]] && echo true || echo false)"
  [[ -z "$isPathExists" ]]
}

[historydev]

Saboteur,
Теперь я идиот. Не учёл что теперь true/false возвращается по правилам -z, поменял местами вывод условия и всё работает, спасибо!

Answer 2

[Вячеслав]

if checkLocationExists "$path"; then
   echo ok
fi
#или
checkLocationExists "$path" && echo ok

Comments

[historydev]

Второе не подходит, мне нужно условием внедрять в другую часть скрипта, а первое я уже пробовал, записав в переменную:

checkLocationExists () {
  pathWithShieldingSlash=$(echo "$1" | sed 's/\//\\\//g')
  isPathExists=$(sed -n 's/.*\(location '"$pathWithShieldingSlash"'.*\).*{/\1/p' "$locations_path/$file_name")

  echo "$isPathExists"

  if [[ -z $isPathExists ]]; then
    return 0
  else
    return 1
  fi
}

test=$(checkLocationExists "$path")

if checkLocationExists "$path"; then
  echo "FOUND"
else
  echo "NOT FOUND"
fi

location /projects/test/v2.2.2/socket.io/ 
location /projects/test/v2.2.2
location /projects/test/v2.2.2/socket.io/
location /projects/test/v2.2.2
NOT FOUND