подскажите по bash скрипту

Убери все знаки $ перед n должно быть read n

Можешь еще так сделать: read -p ’Enter the number: ’ n

вместо: echo ‘Enter the number:’ read n

спасибо большое! помогло)

А строку: echo ‘Congratiulations! You are right!’ поставь в конце цикла, цикл закончится, как только будет выполнено условие и эту строку не покажет.

Про $ вам уже сказали, но в принципе написано плохо. Во-первых, у вас не правильно посчитает количество файлов, если в именах будет перевод строки. Сообщение о правильности с первого раза не выведется, а потом, цикл никогда не завершится. Всё что одинаковое надо вносить внутрь цикла.

Короче, надо как-то так:

#!/usr/bin/env bash

declare -i m
for f in *.*; do
        m+=1
done

echo 'Hello! Guess the number of files in the current directory.'

while read -p 'Enter the number: ' n; do
    if [[ n -lt m ]]; then
        echo 'You are not right. Too low!'
    elif [[ n -gt m ]]; then
        echo 'You are not right. Too high!'
    else
        echo 'Congratiulations! You are right!'
        break
    fi
done

Я не согласен, цикл завершится когда количество посчитанных строк будет равно числу введенному. просто завершится он без вывода о том, что количество угадано.

цикл завершится

А, ну да. Беглое чтение оконфузило эти табуляции с <----> и прочее нагромождение read, что доскональный разбор сбойнул. :)

в вашем варианте тоже подсчет будет ошибочный, он будет считать файлы по патерну ., а если фалы будут не test.sh а test ? лучшим варантом будет просто *

<------>

«Alt»+«Shift»+"-" переключает отображение этого в mcedit.

в вашем варианте тоже подсчет будет ошибочный, он будет считать файлы по патерну ., а если фалы будут не test.sh а test ? лучшим варантом будет просто *

Это вопрос не ко мне. Такой паттерн выбрал ТС, может именно это ему и надо.

if [[ n -lt m ]]; then

Сравнение чисел лучше делать через круглые скобки:

if (( n < m )); then

Это где такое описано? Читал ман dash пару раз и не видел.

dash не bash. (( )), как и [[ ]] – это башизмы. [ ] (test, иными словами) работает везде.

Но у TS в шебанге написано bash, так что ему можно.

Upd. А нет, не у TS, а в подскажите по bash скрипту (комментарий). У TS непонятно, что за shell.

(( )), как и [[ ]] – это башизмы.

Ну тогда лучше я пусть буду в неведении, чем буду садиться на иглу.

Сравнение чисел лучше делать через круглые скобки:

Кто это вам сказал? Сегодня там целые, завтра добавится ещё и другая логика, со строками, файлами и пр. Скобочки хороши, когда надо сделать что-то типа ((a+b<c))

Бестолковое по своей ненужности. Когда такой ерундой занимаются обычно это делают «в темноте и под одеялом».

ls -1 .

что это за ересь?

Это «оптимизация» ради оптимизации. Так у вас будет выведено «You are right!» для EOF. Кавычки для «${n}» тут не нужны, в [[ «${n}» -lt «${m}» ]] не нужны даже ${}

не нужны даже ${}

Я следовал учебнику: https://www.opennet.ru/docs/RUS/bash_scripting_guide/x2565.html

Я следовал учебнику:

Этот учебник правильный для [ ], но для [[ ]] и (( )) там в принципе не показано, чем оно отличается и потому никуда не годится. Впрочем, это относится к выделенному жирным, в примерах без выделения таки проскакивает написание без кавычек.

Сравнение чисел лучше делать через круглые скобки

Чем лучше?

Чем лучше?

Вопрос эстетики: вместо `-lt` и `-gt` в коде будут более человекочитаемые `>` и `<`.