1. Если это апи, то я бы наследовался вместо Controller

Question

1. Если это апи, то я бы наследовался вместо Controller

от ControllerBase. Там нет ненужных тебе методов для MVC
2. Я бы сделал класс sealed
3. В контроллере желательно проверять входящие параметры и выкидывать исключение если его нет (пример: _logger = logger; => _logger = logger ?? throw new ArgumentNullException(nameof(logger));
4. В строчках типа "_db.Notes?.Skip(index).Take(count).ToList();". Желательно сделать везде ? чтобы в nullref не провалиться _db.Notes?.Skip(index?.Take(count)?.ToList();
5. Лучше логгировать всё исключения о чём говорил Вова
6. Делайте return внутри catch блока и вам не нужно будет создавать перед блоком обьявление коллекции
7. Вижу у вас то Notes?. то Notes!. Нужно придерживаться одного порядка, похоже на потенциальный nullref баг.
8. Возможно стоит использовать асинхронные перегрузки методов (здесь холивар вопрос, потому что помню кто-то говорил что это необязательно)

#dot #programming #russian

0

14.08.2022

9 ответов

21 просмотр

demn

унаследовал от ControllerBase и тут посыпало ошибками

0

14.08.2022

Ihor Volokhovych Автор вопроса

Vova Lantsov 🇺🇦
4. Глупости какие-то 3. Это настраивается и можно...

По 4 пункту попадался в неприятные ситуации, когда обращались к полю которого нет не через ?. Поэтому для себя извлёк правило если начал обмазываться ?, то до конца в цепочке По 3 — да. Но это стандартная практика

0

14.08.2022

Ihor Volokhovych Автор вопроса

demn

унаследовал от ControllerBase и тут посыпало ошибк...

Просто возвращайте range или Ok(range). Вам придёт жсон

0

14.08.2022

Vova Lantsov 🇺🇦

Ihor Volokhovych
По 4 пункту попадался в неприятные ситуации, когда...

.Notes гарантированно не null (это мы просто знаем) .Skip() ни в коем случае не вернёт null .Take() тоже И даже .ToList() вернёт пустой список вместо null

0

14.08.2022

Ihor Volokhovych Автор вопроса

Vova Lantsov 🇺🇦
.Notes гарантированно не null (это мы просто знаем...

Если гарантированно не null — согласен. В другом случае лично я бы обмазался во имя избежания непредвиденных nullref'ов

0

14.08.2022

Vova Lantsov 🇺🇦

Ihor Volokhovych
По 4 пункту попадался в неприятные ситуации, когда...

По 3 - стандартная у кого?) Я так не пишу, например

0

14.08.2022

Vova Lantsov 🇺🇦

Ihor Volokhovych
Если гарантированно не null — согласен. В другом с...

В любом случае нужно думать головой, главное не привыкнуть ставить ? везде где нужно и не нужно, это будет вредная привычка

0

14.08.2022

Ihor Volokhovych Автор вопроса

Vova Lantsov 🇺🇦
В любом случае нужно думать головой, главное не пр...

Про это согласен. Нужно сначала с головой подумать и расставить ? над коллекциями где нужно. Но если, к примеру, нет уверенности что всегда будет коллекция то тут стоит обмазаться во имя избежания ошибок. По 3 — (вот пример статьи) https://writesoft.wordpress.com/2017/04/08/cleaner-constructors-with-throw-expressions-in-csharp-7/. Как таковых советов от маек по этому поводу нет, но бывают случаи когда мы не может гарантировать что тот или иной сервис зарегистрирован или мы забыли зарегистрировать в контейнере. В таком случае валидация нам даст по голове и скажет что мы забыли зарегистрировать сервис

0

14.08.2022

Vova Lantsov 🇺🇦 · Accepted Answer

Vova Lantsov 🇺🇦

4. Глупости какие-то 3. Это настраивается и можно сделать чтоб кидал ошибку сам DI если не находит (не уверен что это поведение по умолчанию)

0

14.08.2022

169 похожих чатов

1. Если это апи, то я бы наследовался вместо Controller

9 ответов

Похожие вопросы