Сценарий Bash для перезаписывания / добавления файла назначения с помощью меню опций [duplicate]

Как вы заявили, проблема, которую вы пытаетесь решить, - это (выделено мной):

Любопытно, что код в вашем скрипте, который проверяет, существует ли файл, не использует if, хотя он, вероятно, должен. Вместо этого он использует while:

Это цикл. Если исходный файл существует, все внутри этого цикла выполняется до соответствия

в последней строке вашего скрипта. Это цикл while, поэтому после его запуска первый раз снова выполняется тест. Если файл все еще существует, тело цикла запускается снова и т. Д. И т. Д. И т. Д.

Но ничто в теле цикла никогда не останавливает существующий файл. Четыре возможности:

select - фактически цикл. Ваши команды break ломают select, а не внешние while. Учитывая описанное описание проблемы, вы почти наверняка не имеете никаких оснований использовать внешний цикл. Поэтому одним хорошим решением было бы изменить внешний while на if. Для этого вам также понадобится:

Если вы выберете это решение, вы также захотите переместить некоторые из команд, которые у вас есть за пределами while (который должен быть if), чтобы они находились внутри него. Прямо сейчас пользователю предлагается выбрать вариант, даже если никакое действие не будет принято. Эта дополнительная ошибка не будет исправлена, просто изменив значение while на if (и сделав два других необходимых изменения для этого).

Альтернативное решение - сделать ваш скрипт более ранним, если [ -e "$sourceFile" ] неверно. Один из способов сделать это:

Другой, если вы предпочитаете использовать if, это:

Если вы используете любой из них, то остальные кода - код, который вы собираетесь запускать только в том случае, если существует исходный файл, - не нужно будет вставлять в вне структуры управления . Обратите внимание, что вы по-прежнему хотите, чтобы это произошло раньше, а не после, команд, которые запрашивают у пользователя какие действия предпринять.

Обратите внимание, что вы можете «исправить» ваш код, заменив команды break с exit. Вы также можете «исправить» его, передав числовой аргумент break, чтобы рассказать ему, сколько уровней гнездования вырваться из (команды большинства языков break не поддерживают это, но Bash's does). Тем не менее, я рекомендую не использовать любой из этих подходов - за исключением, возможно, просто попробовать это, потому что вы добавляете больше сложности в код, который вместо этого должен быть исправлен, упрощая его. Итак, одним хорошим решением было бы изменить внешний while на if. , поэтому, тем не менее, вы решаете проблему, это должно быть таким образом, чтобы упростить код, делая это.

Если, с другой стороны, вы действительно намереваетесь, чтобы все это повторялось снова и снова, тогда вы должны рассмотреть, какое условие значимо нарушит ваш внешний цикл. Например, возможно, вы хотели запросить исходные и целевые файлы в каждой итерации внешнего цикла. Если это так, вам нужно будет изменить свой код, чтобы это сделать.

Наконец, я рекомендую отступывать свой код таким образом, что, когда структура управления охватывает несколько строк, строка, которая открывает ее и строка, которая закрывает его, имеет отступы на том же уровне, и строки внутри него (за исключением случайных ситуаций, когда это невозможно сделать) больше отступают. Таким образом, вы узнаете, какой fi соответствует if, который done соответствует while / until / for / select, и который esac соответствует тому, что case. Благодаря тому, что легче распознать это, вы также сможете лучше идентифицировать, когда нужное ключевое слово может , и путем отступов содержимого в согласованном виде вы сможете лучше идентифицировать когда логика, если ваш сценарий отличается от того, что вы намереваетесь.

Как вы заявили, проблема, которую вы пытаетесь решить, - это (выделено мной):

Любопытно, что код в вашем скрипте, который проверяет, существует ли файл, не использует if, хотя он, вероятно, должен. Вместо этого он использует while:

Это цикл. Если исходный файл существует, все внутри этого цикла выполняется до соответствия

в последней строке вашего скрипта. Это цикл while, поэтому после его запуска первый раз снова выполняется тест. Если файл все еще существует, тело цикла запускается снова и т. Д. И т. Д. И т. Д.

Но ничто в теле цикла никогда не останавливает существующий файл. Четыре возможности:

select - фактически цикл. Ваши команды break ломают select, а не внешние while. Учитывая описанное описание проблемы, вы почти наверняка не имеете никаких оснований использовать внешний цикл. Поэтому одним хорошим решением было бы изменить внешний while на if. Для этого вам также понадобится:

Если вы выберете это решение, вы также захотите переместить некоторые из команд, которые у вас есть за пределами while (который должен быть if), чтобы они находились внутри него. Прямо сейчас пользователю предлагается выбрать вариант, даже если никакое действие не будет принято. Эта дополнительная ошибка не будет исправлена, просто изменив значение while на if (и сделав два других необходимых изменения для этого).

Альтернативное решение - сделать ваш скрипт более ранним, если [ -e "$sourceFile" ] неверно. Один из способов сделать это:

Другой, если вы предпочитаете использовать if, это:

Если вы используете любой из них, то остальные кода - код, который вы собираетесь запускать только в том случае, если существует исходный файл, - не нужно будет вставлять в вне структуры управления . Обратите внимание, что вы по-прежнему хотите, чтобы это произошло раньше, а не после, команд, которые запрашивают у пользователя какие действия предпринять.

Обратите внимание, что вы можете «исправить» ваш код, заменив команды break с exit. Вы также можете «исправить» его, передав числовой аргумент break, чтобы рассказать ему, сколько уровней гнездования вырваться из (команды большинства языков break не поддерживают это, но Bash's does). Тем не менее, я рекомендую не использовать любой из этих подходов - за исключением, возможно, просто попробовать это, потому что вы добавляете больше сложности в код, который вместо этого должен быть исправлен, упрощая его. Итак, одним хорошим решением было бы изменить внешний while на if. , поэтому, тем не менее, вы решаете проблему, это должно быть таким образом, чтобы упростить код, делая это.

Если, с другой стороны, вы действительно намереваетесь, чтобы все это повторялось снова и снова, тогда вы должны рассмотреть, какое условие значимо нарушит ваш внешний цикл. Например, возможно, вы хотели запросить исходные и целевые файлы в каждой итерации внешнего цикла. Если это так, вам нужно будет изменить свой код, чтобы это сделать.

Наконец, я рекомендую отступывать свой код таким образом, что, когда структура управления охватывает несколько строк, строка, которая открывает ее и строка, которая закрывает его, имеет отступы на том же уровне, и строки внутри него (за исключением случайных ситуаций, когда это невозможно сделать) больше отступают. Таким образом, вы узнаете, какой fi соответствует if, который done соответствует while / until / for / select, и который esac соответствует тому, что case. Благодаря тому, что легче распознать это, вы также сможете лучше идентифицировать, когда нужное ключевое слово может , и путем отступов содержимого в согласованном виде вы сможете лучше идентифицировать когда логика, если ваш сценарий отличается от того, что вы намереваетесь.