
Eliajah
20.07.2017
09:29:49

Сергей
20.07.2017
09:29:56

Таймураз
20.07.2017
09:30:01

Сергей
20.07.2017
09:30:02
"синтетический"

Google

Сергей
20.07.2017
09:30:04
ахахахахах))0

Eliajah
20.07.2017
09:30:09
:)

Сергей
20.07.2017
09:30:13

Eliajah
20.07.2017
09:30:14
синтаксический

Сергей
20.07.2017
09:30:20
и стрелок

Ilgiz
20.07.2017
09:31:05

Mikhail
20.07.2017
09:31:14

Ilgiz
20.07.2017
09:32:46
понял немного асинхронность, пишу всю логику через callback либо promise а дальше че?

Сергей
20.07.2017
09:33:26

Aleksandr
20.07.2017
09:33:29

Таймураз
20.07.2017
09:33:54

Aleksandr
20.07.2017
09:35:55

Eliajah
20.07.2017
09:36:20

Google

Aleksandr
20.07.2017
09:37:24

Таймураз
20.07.2017
09:37:46

Misha
20.07.2017
09:40:21

Aleksandr
20.07.2017
09:42:23

Ilgiz
20.07.2017
09:43:24
а это нормально что пишеш ты скажем проект на callback и тут бах промисы вышли, хочеш промисы прям влюбился в них и начал на них забил на callback, часть проекта осталось на колбеках, и бах async/await вышел и тоже самое. Ну вроде логика остается таким же но ввид оформление кода разные, это нормально?
или надо срочно переписывать весь код если хочеш перейти на новый

Сергей
20.07.2017
09:44:13

Misha
20.07.2017
09:46:15

Ilgiz
20.07.2017
09:46:47

Таймураз
20.07.2017
09:51:06

Misha
20.07.2017
09:55:16

Dmitrii
20.07.2017
09:56:38
тут недавно handlebars хейтели, что вместо него посоветуете?

Ilgiz
20.07.2017
10:00:39
если я скину сюда ссылку на свой код, можете оценить что плохого что нет? код 200 строк
для джуна нормально или нет, в таком вроде

Dika
20.07.2017
10:02:44
Кидай

Ilgiz
20.07.2017
10:03:56
https://gist.github.com/ikasymov/42d63c6ed6e1c6efe5292b008b14001e
не обращяйте внимание на xpath исправлю их на динамичные

Dika
20.07.2017
10:05:37
Можно заменить request на request-promise/request-promise-native

Eliajah
20.07.2017
10:06:14

Ilgiz
20.07.2017
10:06:42
не знал что есть библиотека import

Google

Dika
20.07.2017
10:06:53

Eliajah
20.07.2017
10:06:59

Сергей
20.07.2017
10:07:06

Ilgiz
20.07.2017
10:07:10
а что ты имел ввиду?

Сергей
20.07.2017
10:07:27

Ilgiz
20.07.2017
10:07:44
ребят я думал наоборот это нормально
читаемость хорошая что где берется и после чего
название функции по мне дает понимаемость

Сергей
20.07.2017
10:08:11
это пиздец

Dika
20.07.2017
10:08:18
Переделывай все немедленно. Выноси в отдельные функции или юзай асинк авейт

Anton
20.07.2017
10:08:22
старая школа, когда поцаны угорали по колбекам, любили жейкваери. молодость и энергия

Сергей
20.07.2017
10:08:29
название функции callFunction прям понятно что делает

Ilgiz
20.07.2017
10:08:51
тут просто я все функции вызываю

Eliajah
20.07.2017
10:10:12
Ало, это кто? Это колбэкхел

Dika
20.07.2017
10:10:12
Раздели эту большую функции на маленькие.

arts
20.07.2017
10:10:22
1. Промисы в промисах в промисах — это адъ и израиль. Разверни в цепочку.
2. Пользуйся eslint.
3. Переходи на es6+.

Dika
20.07.2017
10:11:03

Ilgiz
20.07.2017
10:11:40

Eliajah
20.07.2017
10:12:47

Google

Ilgiz
20.07.2017
10:13:14

Dika
20.07.2017
10:13:37
Там промисы, а не коллбэки

Ilgiz
20.07.2017
10:14:23
мне наоборот на промисах код показался не красивым

Dika
20.07.2017
10:14:52
Смотри в сторону асинк авейт

Safort
20.07.2017
10:16:20

Ilgiz
20.07.2017
10:19:43

Misha
20.07.2017
10:20:57

Ilgiz
20.07.2017
10:21:42
Async/await
я думал он имел ввиду что не используя вообще колбеки писать проекты

Admin
ERROR: S client not available

Safort
20.07.2017
10:22:30
https://gist.github.com/ikasymov/42d63c6ed6e1c6efe5292b008b14001e
1. Если уж писать на ES6+, то везде, а не выборочно)
2. Прототипы встроенных объектов лучше не расширять.
3. Как уже сказали, у тебя большая вложенность, которую можно убрать заменив твои анонимные function expression на function declaration.
4. Ну и про линтер уже писали выше.

Ilgiz
20.07.2017
10:23:15

Safort
20.07.2017
10:23:50

Ilgiz
20.07.2017
10:23:59
а понял

Safort
20.07.2017
10:27:44
Но с промисами и async/await, конечно, лучше.

Вишневый чай
20.07.2017
10:30:29
решил с гос сайта спарсить расписание нужного автобуса. И угораздило меня заглянуть че там сервер присылает

Dika
20.07.2017
10:32:04
Это что? csv?

Google

Вишневый чай
20.07.2017
10:32:19
txt!

Misha
20.07.2017
10:32:20
Походу

Вишневый чай
20.07.2017
10:33:49
ну вообще да это csv, разделитель точка с запятой, формат почему-то txt. Кому вообще пришла мысль на фронт присылать такую жесть

Alex
20.07.2017
10:35:18
Почему это жесть?

Safort
20.07.2017
10:37:33

Alex
20.07.2017
10:38:01

Evgeny
20.07.2017
10:38:48
Зато меньше байт

Alexey
20.07.2017
10:38:53
Это не JSON. Значит не удобно. Не спорьте

Вишневый чай
20.07.2017
10:38:54
Почему это жесть?
потому что csv в формате txt? потому что для абменна данными надо использовать специально предназначеный язык json?

Safort
20.07.2017
10:39:06

Вишневый чай
20.07.2017
10:39:23
потому что csv пардает при парсинге каждый 10ый раз

Alex
20.07.2017
10:39:56

Alexey
20.07.2017
10:39:56
Во-первых, JSON нихрена нечитабельный если его много
Во-вторых, csv читабельней так-то (закатал в табличную прогу и ок)

Alex
20.07.2017
10:40:27

Вишневый чай
20.07.2017
10:40:29

Alexey
20.07.2017
10:40:43
Мы про читабельность или разбор данных на лету?

Вишневый чай
20.07.2017
10:40:51

Alex
20.07.2017
10:41:01

Safort
20.07.2017
10:41:42

Alex
20.07.2017
10:41:47
Есть ответ сервера. Ответ во вполне вменяемом формате, ответ можно спокойно распарсить.

Вишневый чай
20.07.2017
10:42:08
И что?
и то. Когда нибудь столкнетесь узнаете что такое (ой парсер упал из-за того что в ячейке была точка с запятой / прислали csv с другим разделителем / переносы строк формата linux и так далее

Alex
20.07.2017
10:42:42