The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



"Проект microG выпустил сборку LineageOS с альтернативной реа..."
Версия для распечатки Пред. тема | След. тема
Форум Разговоры, обсуждение новостей
Исходное сообщение [ Отслеживать ]
Отдельный RSS теперь доступен для каждого обсуждения в форуме и каждого минипортала.
. "Проект microG выпустил сборку LineageOS с альтернативной реа..." +4 +/
Сообщение от freehckemail (ok), 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. Короче и понятнее.

Ну и, наконец, отступы. Не пренебрегай ими. Форматируй код.


Ответить | Правка | Наверх | Cообщить модератору

Оглавление
Проект microG выпустил сборку LineageOS с альтернативной реа..., opennews, 04-Ноя-17, 23:42  [смотреть все]
Форумы | Темы | Пред. тема | След. тема



Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру