Здравствуйте, elmal, Вы писали:

E>…Однако автор создает экземпляр определенного класса через new без параметров и далее выполняет функцию calculate, тоже без параметров. После чего созданный объект забывается и в цикле создается новый.
E>Я был только сторонний наблюдатель и на это посмотрел после того, как это было отправлено, просто как сторонний консультант по просьбе автора. Непосредственно тех, кто оценивал тестовое задание — все удовлетворило. Отсюда вопрос, я что — действительно слишком придираюсь, или по крайней мере в C# является бест практикой на каждый чих создавать класс с одним публичным методом и чтоб вызывающий код для простейшего случая не вызывал одну функцию, а вынужден сначала создавать объект оператором new, а затем вызывать один единственный метод чтоб что то посчитать?

Вопрос было бы правильнее всего адресовать автору кода: зачем он сделал так, а не как-то иначе. Возможно, он предвидел, что а) в реальной практике нужно будет поменять так и эдак б) при необходимости добавить тесты будет проще расширитьть этот класс в) да всё что угодно. ИМХО, оценивание тестового задания без беседы потом о том, как оно выполненно и почему скорее отвечаетнг а вопрос "очень ли оно плохо сделано", а чтобы разобраться с тем, на сколько оно сделано хорошо имеет смысл поговоритоь с автором и тогда вещи, которые вызывают недоумение при первом ревью могут проясниться.