ГГ
канешно ты же не понимаешь сурдоперевод лол
Size: a a a
2021 October 30
ЯЧ
привет лучшему сишарп чату. Я написал своё первое приложение на asp net и очень бы хотелось начать свой путь веб разработчика с чистым опрятным кодом. Я ни в коем случае не наглею и не прошу код ревью но был бы рад если бы вы посмотрели этот небольшой проект, может бросятся в глаза какие-нибудь косяки которые следует исправить. Заранее спасибо! 🙏🏻
https://github.com/azaza12345/CsharpDapperExample
https://github.com/azaza12345/CsharpDapperExample
ЯЧ
буду рад любым замечаниям: как проёбам в архитектуре, так и плохим названиям методов / переменных 😞
m
У тебя контекст бд в репозитории, хз насколько это правильно/не правильно но я не видел чтоб так делали
m
Нету слоя данных
m
Поправьте если ошибаюсь
СУ
зависит от масштаба проекта и его предназанчения: для лабы в универ на коленках лишь бы работало - окей, для учебного проекта - ну хз-хз
СУ
за юнит тесты +респект
m
Т.е. паттерн репозиторий который на уровень выше по абстракции норм для учебного проекта а контекст ну можно запихнуть в репозиторий
A
Привет, подскажите почему может не обновляться combobox, UWP?
для проверки сделал кнопку которая делает Bindings.Update();
ItemsSource класса ObservableCollection
в коллекцию асинхронно добавляются значения
для проверки сделал кнопку которая делает Bindings.Update();
ItemsSource класса ObservableCollection
в коллекцию асинхронно добавляются значения
m
Некоторые для малых проектов вообще не вешают репозиторий в проект ограничиваясь контекстом
ЯЧ
кстати а по поводу юнит тестов что то кроме сервисного слоя тут можно или нужно протестировать или этого достаточно?
m
От проекта зависит иногда контроллеры тоже тестами покрывают
СУ
я прочитал твой коммент по диагонали, сорян
СУ
я подумал, что там из сервиса/контроллера идёт обращение
СУ
из репозитория контекст - это классика вообще, разве можно по другому
m
По поводу обращения я ниче не говорю, имею ввиду что контекст как отдельный класс вынести