LINUX.ORG.RU
Форум — General  

посмотрите мой bash-скрипт


0

1

ребята, я изучаю bash, code.google.com и практикуюсь в написании скриптов, посмотрите пожалуйста мой код, может можно его как улучшить? https://code.google.com/p/video-sibnet-ru-downloader/source/browse/trunk/sibn...

я думаю что взгляд опытного человека сразу сказал бы - вот тут ты делаешь по нубски, может вылезти ошибка или вообще вместо этих 3 строчек можно написать одну, или логика запутанная или не знаю ещё что там



Последнее исправление: kuzyara (всего исправлений: 1)
← есть какая простая чертилка планировок квартир?
Дистрибутив на базе Debian GNU/Linux 6.0.5 Squeeze Xfce → 
if ! [ $_i ]; then    # читается: если нет опции -i (то парсим страницу)

на 

if [ -z "$_i" ]; then
А в остальном всё не так плохо

derlafff ★★★★★
()

Оберните все переменные в двойные кавычки для безопасности.

AITap ★★★★★
()
Ответ на: комментарий от Homura_Akemi

Раз это bash, то нет смысла дёргать /usr/bin/test, т.е. нужно заменить «[ list ]» на «[[ list ]]».

Ты бы лучше жевало, чем пердело в лужу

anonymous
()
Ответ на: комментарий от zolden

иди забухай лучше

(голосом Булдакова) Ну! За твоё здоровье!;)

Hertz ★★★★★
()

может можно его как улучшить?

Переписать на нормальном яп.

anonymous
()

Главная фича любого баш-скрипта - чтобы он правильно работал.
Это не си и не плюсы, чтобы на красоту кода дрочить.

pekmop1024 ★★★★★
()
Ответ на: комментарий от anonymous

Раньше вроде было иначе. Однако [ и [[ всё равно различаются: первое buitin, второе синтаксическая конструкция.

Homura_Akemi ★
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.
← есть какая простая чертилка планировок квартир?
General
Дистрибутив на базе Debian GNU/Linux 6.0.5 Squeeze Xfce →