Это мой код, написанный на этой неделе. Несколько обфусцирован дабы не палить продакшн. Добавлю пастбин https://pastebin.com/7drRnpPJ Что тут можно увидеть: 1. Очень многословный код даже по меркам шарпа. 2. Написано прямо и грубо. Не видно ни оптимизаций, ни попыток как-то обобщить похожие места. 3. Неконсистентный нейминг. Похоже на то, что куски были утащены с других мест и не переделаны до конца. 4. Много повторений и похожих кусков.
Это работающий код, написанный за один раз без возвратов к работающим частям. У задачи был жесткий дедлайн, поэтому писалось хоть и без спешки, но некрасиво.
Когда я это писал, я в первую очередь думал о том, чтобы это работало и относительно просто читалось. В плане читабельности получилось неплохо. Хоть и много, но разобраться труда не составит. Теперь, когда эта функциональность ушла на прод - я могу немного отрефакторить, чтобы успокоить свою душу.
· 09.06
А что это за язык?
ответить
коммент удалён
· 09.06
Во-во, на него и погрешил сначала.
ответить
ответ удалён
· 09.06
Оооо, олдовый js с обилием var. Я такого больше не хочу
ответить
ответ удалён
· 09.06
Я сначала подумал, что это JavaScript, но, чем больше вчитывался, тем больше подозревал, что нет.🙂
ответить
ответ удалён
· 09.06
Это c#. Можно перепутать с джавой)
ответить
ответ удалён