SZ
First Last
Ну малая жертва ради благой цели.
ну такое... дешевле профессионалок с профильных сайтов вызвонить, если ты об этом
Size: a a a
2021 February 06
FL
ну такое... дешевле профессионалок с профильных сайтов вызвонить, если ты об этом
Согласен. Еще не факт что не болеет ни чем. А подцепить можно много интересного.
SZ
First Last
Согласен. Еще не факт что не болеет ни чем. А подцепить можно много интересного.
ой, обычно от своих горячо любимых цепляют
FL
ой, обычно от своих горячо любимых цепляют
Как то даже романтично прозвучало 😭
SZ
First Last
Как то даже романтично прозвучало 😭
ну если это романтика для тебя, будь в курсе: от своей самки гораздо вероятнее заразиться
FL
Я уже говорил, что ты душный ?
SZ
First Last
Я уже говорил, что ты душный ?
я вообще мерзота та ещё)
_
Всем привет. Написал маленький анализатор. Хотел спросить ваше мнение на счет кода, архитектуры приложения. Может дадите пару советов плиз. Кому несложно, посмотрите одним глазом. Большое спасибо
https://github.com/Skyuzii/HarabaAnalyzer
https://github.com/Skyuzii/HarabaAnalyzer
Выглядит на мой скромный взгляд не плохо.
Разве что меня смутил момент, где у тебя 4 вложенных foreach что говорит о сложности алгоритма О(n^4)
Разве что меня смутил момент, где у тебя 4 вложенных foreach что говорит о сложности алгоритма О(n^4)
_
Ещё у тебя попытка логировать в консоль идёт прямо в классе который «делает работу»
То есть появляется прямая зависимость от console. Лучше убрать эту зависимость и внедрить работу с логом через DI
То есть появляется прямая зависимость от console. Лучше убрать эту зависимость и внедрить работу с логом через DI
_
Всем привет. Написал маленький анализатор. Хотел спросить ваше мнение на счет кода, архитектуры приложения. Может дадите пару советов плиз. Кому несложно, посмотрите одним глазом. Большое спасибо
https://github.com/Skyuzii/HarabaAnalyzer
https://github.com/Skyuzii/HarabaAnalyzer
Да, класс воркер делает определено слишком много работы. И запускается анализатор, и логирует, и читает откуда то данные. Т.е. нарушение S, O из solid.
Но повторюсь, в целом мне понравилось, ты молодец 👍
Но повторюсь, в целом мне понравилось, ты молодец 👍
P
Здрасьте
P
Я опять бухой
FL
Выглядит на мой скромный взгляд не плохо.
Разве что меня смутил момент, где у тебя 4 вложенных foreach что говорит о сложности алгоритма О(n^4)
Разве что меня смутил момент, где у тебя 4 вложенных foreach что говорит о сложности алгоритма О(n^4)
На счет этого. Мой внутренний детектор говнокода, говорит, что это было бы не плохо вынести в переменную отдельную, а так же LINQ оформлять по условию, новое обращение с новой строки. Вроде случая 2. А так же переменная бы говорила, что именно там хранится и что мы хотели достать.
FL
Выглядит на мой скромный взгляд не плохо.
Разве что меня смутил момент, где у тебя 4 вложенных foreach что говорит о сложности алгоритма О(n^4)
Разве что меня смутил момент, где у тебя 4 вложенных foreach что говорит о сложности алгоритма О(n^4)
FL
Я опять бухой
P
Спасибо за внимание
FL
Не ты один. Я сам пол стаканчика вина навернул
P
First Last
Не ты один. Я сам пол стаканчика вина навернул
Лол
SZ
Я опять бухой
ты чо, мудак?
P
Я виски с колой