1. Неконсистентный порядок модификаторов метода (паблик должен быть в начале) 2. Контекст не диспозится 3. Если это asp то можно брать контекст из DI вместо ручного создания 4. Если у тебя будет миллион методов WriteDb с разными параметрами, да ещё и для разных таблиц, ты очень быстро запутаешься и напишешь какую-то фигню (название класса и метода никак не передаёт, что и куда ты записываешь) 5. Да и вообще статические классы не рекомендую использовать для БД, уж очень руки чешутся с ним какое-то говно написать. Лучше уж репозиторий сделай.
6. Метод невозможно тестировать, зависимости невозможно замокать
я и не подумал как то про 4🤔 А как тогда найти правильный подход?
немає юзінга, і все статік
изначально была такая реализация, но мне не хочется всё пихать в Program.cs
ну є методи не статік
но проблема остается с 4
Каждый метод в названии должен отражать, что он делает А если у тебя много таблиц, то разбить класс на отдельные по группам
Это у тебя обычная консолька?
інтерфейс методу
коменты с маленькой буквы
1. private static readonly Regex.... = new(..., RegexOptions.Compiled) 2. method must be async with await db.SaveChangesAsync() and others
статичні класи а статичні методи для БД?
Обсуждают сегодня