yermochenko / aqa-dev Goto Github PK

Классы в классе (UserGenerator, EmployeeGenerator and CandidateGenerator), тем более те, которые могут быть переиспользованы, это не очень хорошо. Лучше всего делать один функциональный класс - один файл (касается всего проекта).

Для чего предназначена эта константа?

Посмотрите пожалуйста: в задании указано "каждый второй человек", а в реализации начинается с первого.

Неоднозначное имя интерфейса и метода, которые не отражают суть интерфейса и метода. И посмотрите пожалуйста нейминг конвеншн в C# - почти все интерфейсы оканчиваются на -able. Мои варианты имени интерфейса и метода IDisplayable and DisplayInfo(). Необязательно к исполнению, но обязательно принять к сведению.

Подсчёт производится неверно. Могу предложить следующее решение - поиск индексов экстремумов и подсчёт суммы по индексам.

Имя интерфейса IReportGenerator, а метод OutputReport. Имя метода не соответствует выполняемой функции. Могу предложить Generate или GenerateReport. Output крайне редко используется как глагол. Также интерфейсы желатeльно вынести в отдельную папку, например Interfaces.

Пожалуйста, вынесите чтение файла в отдельный класс.

Можно вынести работу в отдельный класс Job or UserJob. Необязательно.

У Вас реализовано 4 сортировки из блока А, а в задании сказано "Реализовать на выбор один из алгоритмов сортировки из блока A и один из алгоритмов сортировки из блока B. ". Добавьте, пожалуйста, одну сортировку из блока В.