Информация об изменениях

Сообщение Re[3]: Это сеньорский код? от 21.12.2021 9:13

Изменено 21.12.2021 9:18 bnk

Re[3]: Это сеньорский код?
Здравствуйте, CreatorCray, Вы писали:

DP>>Именно этот код — да, крутого сеньора или даже выше.


CC>Люди с опытом стараются писать просто, тут же местами налеплено в кучу.

CC>Если приходится писать сложно то пишут коммент, что за нафиг и почему так.
CC>По коммитам кстати видно что написано было изначально более просто и наглядно, это потом уже автор "улучшил".
CC>Впрочем да, на фоне другого js что я видел этот довольно таки чистый.

Здесь просто печать индустрии жаваскрипта как мне кажется. Трудное детство, деревянные игрушки.
Чувак сделал работу за компилятор.

DP>>Просто тут контекст другой: числодробительный код, для поддержки которого надо глубже понимать тематику.

CC>Это всё лирика.
CC>Но вот для куска, который привёл ТС, было бы неплохо хотя бы написать комментарий WTF там и зачем.
CC>Ну или разбить это всё на несколько строк, ибо много сущностей в одной строке ухудшают читаемость кода.

Да чего тушеваться, код конкретно нечитаемый, невзирая на лица.
Если автор просидел на этой либе последние 10 лет, это его все равно не оправдывает, IMHO можно было более человечный код написать.
Re[3]: Это сеньорский код?
Здравствуйте, CreatorCray, Вы писали:

DP>>Именно этот код — да, крутого сеньора или даже выше.


CC>Люди с опытом стараются писать просто, тут же местами налеплено в кучу.

CC>Если приходится писать сложно то пишут коммент, что за нафиг и почему так.
CC>По коммитам кстати видно что написано было изначально более просто и наглядно, это потом уже автор "улучшил".
CC>Впрочем да, на фоне другого js что я видел этот довольно таки чистый.

Здесь просто печать индустрии жаваскрипта как мне кажется. Трудное детство, деревянные игрушки.
Чувак сделал работу за компилятор.

DP>>Просто тут контекст другой: числодробительный код, для поддержки которого надо глубже понимать тематику.

CC>Это всё лирика.
CC>Но вот для куска, который привёл ТС, было бы неплохо хотя бы написать комментарий WTF там и зачем.
CC>Ну или разбить это всё на несколько строк, ибо много сущностей в одной строке ухудшают читаемость кода.

Да чего тушеваться, код конкретно нечитаемый, невзирая на лица.
Если автор просидел на этой либе последние 10 лет, это его все равно не оправдывает, IMHO можно было более человечный код написать.

Впрочем, вся D3 в стиле юниксовых команд 80-х, с дурацкими сокращениями типа x.classed() ака "class edit".
Такой уж склад мышления у некоторых.