Справка по сценариям – Ubuntu 22.04.05 LTS

Вопрос или проблема

#!/bin/bash

if [ $# -ne 2 ]; then
    echo "Usage: $0 $image_url $desired_name" >&2
   exit 1
   fi
    
image_url=https://i.redd.it/od213chlczje1.jpeg
desired_name=my_background
image_url=$1
desired_name=$2

echo "Downloading the image from $image_url..."
wget -q "$image_url" -O original_image

Ответ или решение

Ваша проблема заключается в том, что скрипт останавливается на строке с echo "Usage: ", и это указывает на то, что ваш скрипт выполняет условия ветки if, предназначенной для случая, когда требуемые параметры не были переданы при вызове. Давайте разберем, в чем может быть дело и как можно исправить ваш скрипт.

Теория

Для начала разберем предназначение ключевых элементов вашего скрипта на bash и почему возникают определённые проблемы:

1. Проверка параметров: Ваш скрипт проверяет количество переданных аргументов с помощью конструкции if [ $# -ne 2 ];, что означает, что скрипт ожидает ровно два аргумента при его запуске. Если количество аргументов не равно двум, выполняются команды внутри этого блока if, и скрипт завершает свою работу с помощью exit 1.

2. Присваивание значений переменным: Вы задали переменные image_url и desired_name со значениями по умолчанию, но затем переназначаете их значениями из параметров командной строки $1 и $2.

3. Скачивание файла: Если параметры заданы правильно, скрипт использует wget для скачивания изображения по URL, переданному как первый параметр.

Пример

Рассмотрим пример правильного вызова скрипта, чтобы он функционировал как предполагается:

./your_script_name.sh https://example.com/path/to/image.jpeg my_image_name

При этом вызове:

• $1 будет равно https://example.com/path/to/image.jpeg
• $2 будет равно my_image_name

Если вызвать скрипт без этих аргументов, будет выполнена ветка с echo "Usage: ", после чего скрипт завершится.

Применение

Чтобы ваш скрипт работал корректно, рекомендуется учитывать следующее:

1. Убедитесь, что передаете необходимые параметры: При выполнении скрипта передавайте точно два аргумента: URL изображения и желаемое имя файла.

2. Обработка ошибок и улучшение вывода: Рекомендуется дополнительно улучшить скрипт для обработки общих ошибок и предоставления более информативных сообщений. Например, если скачивание завершилось неудачно, имеет смысл уведомить пользователя об этом.

3. Оптимизация порядка операций: Учитывая, что первые присваивания начальных значений переменным image_url и desired_name сразу перезаписываются, лучше убрать эти строки, чтобы избежать путаницы.

Доработанный скрипт

Вот улучшенная версия вашего скрипта с комментариями для лучшего понимания:

#!/bin/bash

# Проверка количества аргументов
if [ $# -ne 2 ]; then
    echo "Usage: $0 <image_url> <desired_name>" >&2
    exit 1
fi

# Присваивание значений переменным из аргументов скрипта
image_url=$1
desired_name=$2

# Скачивание изображения и проверка успеха
echo "Downloading the image from $image_url..."
if wget -q "$image_url" -O "$desired_name"; then
    echo "Image downloaded successfully as $desired_name."
else
    echo "Failed to download the image." >&2
    exit 2
fi

Дополнительные рекомендации

• Проверка существования файла: Перед скачиванием или после, вы можете добавить проверку на существование и совпадение имени файла, чтобы избежать перезаписывания уже существующих файлов.

• Логи и отчетность: Для сложных скриптов используйте логирование действий скрипта, чтобы при выполнении понимать, где конкретно произошла ошибка.

• Документация и комментарии: Добавьте комментарии, которые пояснят, что делает каждая часть скрипта — это поможет вам и другим пользователям понять логику.

Следуя этим советам, вы сможете избежать большинства типичных ошибок и улучшить стабильность и надежность bash-скрипта, что особенно важно для использования в продуктивных средах.