@juliion

Можно код-ревью проекта на C# с юнит тестами?

Начала изучать юнит тесты.

Чтобы попрактиковаться, сделала небольшой проект, суть которого реализация двухсвязного списка и покрытие кода юнит тестами. Использовала MSTests. Хотелось бы получить код-ревью.

Буду рада любому отзыву!

Ссылка на репозиторий: https://github.com/juliion/MyList
  • Вопрос задан
  • 128 просмотров
Решения вопроса 1
vabka
@vabka Куратор тега .NET
Токсичный шарпист
Сначала замечания по реализации:

1. Раз уж мы делаем свой аналог List - тогда есть смысл вынести его в библиотеку, а не с Exe.
TargetFramework стоит делать чуть более новым. Сейчас LTS - .NET 6 (но это не так уж и критично)

2. Почему List Только для char реализован? Почему не сделать его обобщённым?

3. Почему List не реализовывает стандартные интерфейсы? Тот же IEnumerable<T> хотябы.

4. PrintForward - явно какой-то лишний метод. Список не должен отвечать за вывод в консоль.

5. Публичный сеттер в Node выглядит как что-то опасное. Так можно изменить Next или Previous - лист изменится, вплоть до изменения количества элементов, но значение Count в самом листе не изменится, от чего всё поломается.

А вот тесты наоборот выглядят вполне неплохо, но надо бы покрытие посмотреть.
Ответ написан
Пригласить эксперта
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы