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

Question

[juliion]

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

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

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

Ссылка на репозиторий: https://github.com/juliion/MyList

Answer 1

[Василий Банников]

Сначала замечания по реализации:

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

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

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

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

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

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

Comments

[juliion]

Спасибо

[Василий Банников]

juliion, если помог, то отмечай решением

[oleg_ods]

juliion, в дополнение к замечаниям автора ответа, можно заменить
throw new Exception("Wrong index");

на

throw new IndexOutOfRangeException();

Он как раз для этого переназначен=)