Использование структур или классов в качестве параметров

Question

Я довольно новичок в C# и программировании в целом, поэтому почему я задаю этот вопрос.

Недавно я получил мой код, рассмотренный коллегой, и он заявил, что для следующего метода я должен использовать класс для некоторых параметров.

public List<Items> DoSomething(List<OtherItems> someItems, int amountOfItems, 
    DbGeopgrahpy Location) 
{ //More stuff, irrelevant to the question. 

Он указал, что я должен поставить amountOfItems и Location в классе, который будет содержать эти значения вместо этого.

Этот метод вызывается из контроллера, который возвращает эти данные клиенту.

Теперь мой вопрос: почему? Я понимаю, что это улучшит читаемость (возможно, imo), но есть ли преимущества использования классов в качестве параметров? Я думаю, что могу предположить, что это повлияло бы на производительность отрицательно, когда вы должны объявлять класс каждый раз, когда хотите вызвать этот метод.

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

Answer 1

Одним из преимуществ использования класса DTO как аргумента метода вместо сбора параметров является предотвращение модификации сигнатуры в будущем. Это может быть полезно для общедоступных методов API, например, так как вы можете изменять API без нарушения изменений/версий версий.

Предположим, вам необходимо обновить/изменить метод, поэтому вам нужно передать дополнительный аргумент. Если ваш метод ожидает только одного аргумента, то есть class/struct, вам не нужно менять подпись метода. Вместо этого вы просто добавляете дополнительное свойство в свой класс DTO.

Answer 2

Лично я считаю, что если у меня есть более 3 или 4 параметра, которые логически сгруппированы (принадлежат к одному и тому же виду объекта или информации), то я буду использовать класс.

Это очень произвольно, как вы находите. Каждый человек имеет свой собственный набор правил относительно того, когда они хотят использовать класс/структуру.

Answer 3

Это скорее понимание дизайна, я думаю. Если метод ожидает слишком много параметров, которые могут быть инкапсулированы в другой объект, то это означает, что вы не могли бы сделать хорошее определение объекта. Кроме того, по мере того, как ваше программное обеспечение начнет расти, вы обнаружите, что гораздо лучше попытаться сгруппировать все в классах, чтобы вы и люди, которые с вами работали, будут намного легче выполнять этот способ.

//avoid 
public void Checkout(string shippingName, string shippingCity, 
     string shippingSate, string shippingZip, string billingName, 
     string billingCity, string billingSate, string billingZip) 
{ 

} 

//DO 
public void Checkout(ShippingAddress shippingAddress,BillingAddress billingAddress) 
{ 
} 

Как вы видите, общие свойства были сгруппированы в общий класс, который группирует их. Вы должны применить свои критерии и сделать хорошую группировку.

Answer 4

Check this => Choosing Between Class and Struct