P
Найди анимешницу уже
Хмммм
Size: a a a
2021 February 06
FL
А вот не факт! Эти 4 цикла сами по себе смущают меня. Упаковка их в linq не решает проблему алгоритмической сложности и добавляет сложность в понимании linq
Проблема номер два. Как я понял, идет логирование Но где связь верхнего цикла с внутренним ? мы выводим @class с верхнего цикла, и parameter с нижнего. Но они не связаны по идее. Может какое то внутреннее устройство, на на цикле не вижу
_
Ну, я думаю алгоритмическая сложность оправдана тем, что я бегаю по синтаксическому дереву. Хотя может можно и упростить. Это мой первый анализатор на roslyn
И да, в соседней ветке тебе указали на то что ты написал экстеншн для ienumerable. Подумай как сделать иначе
С
Всем привет. Написал маленький анализатор. Хотел спросить ваше мнение на счет кода, архитектуры приложения. Может дадите пару советов плиз. Кому несложно, посмотрите одним глазом. Большое спасибо
https://github.com/Skyuzii/HarabaAnalyzer
https://github.com/Skyuzii/HarabaAnalyzer
С
Всем привет. Написал маленький анализатор. Хотел спросить ваше мнение на счет кода, архитектуры приложения. Может дадите пару советов плиз. Кому несложно, посмотрите одним глазом. Большое спасибо
https://github.com/Skyuzii/HarabaAnalyzer
https://github.com/Skyuzii/HarabaAnalyzer
SZ
да полно
ну это по паспорту
VZ
ну это по паспорту
што
С
Всем привет. Написал маленький анализатор. Хотел спросить ваше мнение на счет кода, архитектуры приложения. Может дадите пару советов плиз. Кому несложно, посмотрите одним глазом. Большое спасибо
https://github.com/Skyuzii/HarabaAnalyzer
https://github.com/Skyuzii/HarabaAnalyzer
проверять нужно аттрибут, а не имя класса, на любой аттрибут не из списка реагирует неадекватно)
SZ
ну как.... бап полно, общаться не с кем
VZ
ну как.... бап полно, общаться не с кем
Да как не с кем, есть
VZ
У тебя наверняка тестировщицы есть на работе
VZ
Программистки
VZ
Бухгалтерши
С
Всем привет. Написал маленький анализатор. Хотел спросить ваше мнение на счет кода, архитектуры приложения. Может дадите пару советов плиз. Кому несложно, посмотрите одним глазом. Большое спасибо
https://github.com/Skyuzii/HarabaAnalyzer
https://github.com/Skyuzii/HarabaAnalyzer
вывод ужасен, пустые строки не нужны, читать трудно)
SZ
Бухгалтерши
лучше вискаря
SZ
ну нах
FL
И у меня самооценка такая низкая, что шлюха еще меньше сделает, пушо я буду думать, что со мной только ради денег этим занимаютсч
Это ты до процесса говоришь. А вот по будет с тобой, наговорит красивых слов, сделает приятно, и ты уже другой человек.
SZ
β🎀
Всем привет. Написал маленький анализатор. Хотел спросить ваше мнение на счет кода, архитектуры приложения. Может дадите пару советов плиз. Кому несложно, посмотрите одним глазом. Большое спасибо
https://github.com/Skyuzii/HarabaAnalyzer
https://github.com/Skyuzii/HarabaAnalyzer
Тыкни меня завтра, если ревью все еще будет интересно. Несколько моментов я отмечу.
S
проверять нужно аттрибут, а не имя класса, на любой аттрибут не из списка реагирует неадекватно)
Ну, все верно. В методах контроллера проверяет параметры на наличие атрибутов источника привязки. Это по идее нужно для WEB Api. Единственное, ты обошел создав имя "NotAController". Не хотел подключать нугетосвкий пакет webapi, чтобы проверять на тип controllerBase