Исходное сообщение
"Проект microG выпустил сборку LineageOS с альтернативной реа..." Отправлено freehck, 07-Ноя-17 15:31
> Ну вот из последнего: http://forum.motofan.ru/index.php?s=&showtopic=179276&view=f... Слушай, я понимаю, что ты старался, но... Это жесть. Это самая натуральная "портянка". Во-первых, вынеси переменные, которые будут конфигурируемы, в начало. Во-вторых, чем передавать циферки в качестве аргументов, лучше используй какие-либо флаги. Разбор производи getopt-ом: по моему опыту, эта утилита есть самый простой сбособ разбора аргументов. В-третьих, определять список файлов надо либо глобами (*), либо find-ом: ls с последующим sed-ом читать невозможно. Вот ты там хотел например sed-ом убрать директории и "."/".."... Но можно было бы, например, написать так: find . -mindepth 1 -maxdepth 1 -type f. Короче и понятнее. Ну и, наконец, отступы. Не пренебрегай ими. Форматируй код.

Ваше сообщение
Имя*:
EMail:	Для отправки ответов на email укажите знак ! перед адресом, например, !user@host.ru (!! - не показывать email). Более тонкая настройка отправки ответов производится в профиле зарегистрированного участника форума.
Заголовок*:
Сообщение*:	>> Ну вот из последнего: http://forum.motofan.ru/index.php?s=&showtopic=179276&view=findpost&p=1786215 > Слушай, я понимаю, что ты старался, но... Это жесть. Это самая натуральная > "портянка". > Во-первых, вынеси переменные, которые будут конфигурируемы, в начало. > Во-вторых, чем передавать циферки в качестве аргументов, лучше используй какие-либо флаги. > Разбор производи getopt-ом: по моему опыту, эта утилита есть самый простой > сбособ разбора аргументов. > В-третьих, определять список файлов надо либо глобами (*), либо find-ом: ls с > последующим sed-ом читать невозможно. Вот ты там хотел например sed-ом убрать > директории и "."/".."... Но можно было бы, например, написать так: find > . -mindepth 1 -maxdepth 1 -type f. Короче и понятнее. > Ну и, наконец, отступы. Не пренебрегай ими. Форматируй код.
	Введите код, изображенный на картинке: