Исходное сообщение
"Представлена начальная реализация версии фреймворка node.js ..." Отправлено Crazy Alex, 21-Фев-12 14:23
Могу и сам перевести: Исходный код, имеющий сложную и запутаннуюю структуру, особенно использующий много GOTO, исключений, потоков или других "неструктурированных" конструкций ветвления. Он так назван потому что поток управления имеет склонность выглядеть как куча спагетти, т.е. переплетенным и иискривленным. напомню, речь идет вот об этом: https://github.com/isaacs/npm/blob/master/lib/init.js#L81 Окей, если не понимаете, что там не так - объясняю. Маоло того, что функцию надо бы разбить на куски просто из соображений размера, так еще и набор странно отформатированных коллбеков там, где хватило бы линейного кода, управляющего последовательностью запросов. Асинхронность (и, соотвественно, promice) там не нужна вообще - это утилита, в данной точке control flow в принципе не способная ничего сделать, пока не получит весь ввод от пользователя. А еще лучше было бы не просто заменить promiseChain на последовательность циклов ввода, а выделить каждую функцию проверки отдельно (можно внутри основной) и дать ей имя, сделать явный массив со структурами "приглашение + функция проверки", а в основной функции просто сделать цикл по нему. И вместо этого "кто на ком стоял" была бы абсолютно самодокументированная конструкция, понятная с первого взгляда на первой секунде. Вообще дурацкая джаваскриптовая традиция кучу всего объявлять in place (да еще неименованным) сильно мешает читабельности. Я еще (кое-как) понимаю, когда это делается для браузера с целью уменьшения размера кода, но на сервере такое применять - идиотизм.

Ваше сообщение
Имя*:
EMail:	Для отправки ответов на email укажите знак ! перед адресом, например, !user@host.ru (!! - не показывать email). Более тонкая настройка отправки ответов производится в профиле зарегистрированного участника форума.
Заголовок*:
Сообщение*:	> Могу и сам перевести: Исходный код, имеющий сложную и запутаннуюю структуру, особенно > использующий много GOTO, исключений, потоков или других "неструктурированных" конструкций > ветвления. Он так назван потому что поток управления имеет склонность выглядеть > как куча спагетти, т.е. переплетенным и иискривленным. > напомню, речь идет вот об этом: https://github.com/isaacs/npm/blob/master/lib/init.js#L81 > Окей, если не понимаете, что там не так - объясняю. > Маоло того, что функцию надо бы разбить на куски просто из соображений > размера, так еще и набор странно отформатированных коллбеков там, где хватило > бы линейного кода, управляющего последовательностью запросов. Асинхронность (и, соотвественно, > promice) там не нужна вообще - это утилита, в данной точке > control flow в принципе не способная ничего сделать, пока не получит > весь ввод от пользователя. > А еще лучше было бы не просто заменить promiseChain на последовательность циклов > ввода, а выделить каждую функцию проверки отдельно (можно внутри основной) и > дать ей имя, сделать явный массив со структурами "приглашение + функция > проверки", а в основной функции просто сделать цикл по нему. И > вместо этого "кто на ком стоял" была бы абсолютно самодокументированная конструкция, > понятная с первого взгляда на первой секунде. > Вообще дурацкая джаваскриптовая традиция кучу всего объявлять in place (да еще неименованным) > сильно мешает читабельности. Я еще (кое-как) понимаю, когда это делается для > браузера с целью уменьшения размера кода, но на сервере такое применять > - идиотизм.
	Введите код, изображенный на картинке: