Телеграмм чат группы codeblog

Size: a a a

CODE BLOG / C#

1466 membersпожаловаться на группу

2020 September 11

this. in CODE BLOG / C#

Стоп стоп стоп
А нахуя ты атрибуты на дто навесил?Вот это точно надо исправлять

источник

18:43пожаловаться #1

this. in CODE BLOG / C#

image_2020-09-11_18-33-41.png

(4.89 Кб)

В целом в проекте много проблем с названиями
Одна из них
лучший вариант
GetByCondition

источник

18:43пожаловаться #2

this. in CODE BLOG / C#

Почему не использовать метод Update?

источник

18:43пожаловаться #3

this. in CODE BLOG / C#

Ну это честно говоря не очень решения
если мне надо удалить по идентификатору я хочу просто его передать а не писать целую лямбду
Можно оставить то что есть и дописать перегрузку

источник

18:43пожаловаться #4

this. in CODE BLOG / C#

Теперь об этом
Я тебя еще раньше спрашивал
Для чего тебе целый отдельный слой под репозитории
А теперь как оказывается ты в нем и интерфейсы обьявляеш и реализуешь их
сомнительно

источник

18:43пожаловаться #5

this. in CODE BLOG / C#

Ну это шутка

источник

18:43пожаловаться #6

this. in CODE BLOG / C#

И под конец об архитектуре в целом
Очень странное построение если честно
Слои ощущаются незаконченными а разбиение хаотичным
Советую почитать об архитектуре
2-3 статьи
1 проект
А дальше все пойдет в разы проще

источник

18:43пожаловаться #7

this. in CODE BLOG / C#

Я не очень смотрел в сам сервис (хотя краем глаза там не все ок)
И в связи между слоями
Думаю еще парочку проблем найти можно было бы
Такая вот рецензия
Надеюсь будет полезно

источник

18:43пожаловаться #8

МЕ

Макс Епифанов... in CODE BLOG / C#

я в проект даже не залазил. пологаясь на скрины что сюда кидают

источник

18:44пожаловаться #9

МЕ

Макс Епифанов... in CODE BLOG / C#

к руд сервисы собственно где?

источник

18:44пожаловаться #10