Jak lepiej przypisywać wartości do właściwości w konstruktorze

Question

Hej, od jakiegoś czasu praktykuję C# i zaczynam dbać o jakość swojego kodu, stąd moje pytanie. Lepiej w konstruktorze przypisać wartości z metod zwracających obliczoną wartość, czy wywołać metodę i to w niej przypisać wartość. Dwa przykłady poniżej.

Wariant 1

public class Kwadrat
{
    public string Nazwa { get; set; }
    public double Pole { get; set; }

    public Kwadrat(string nazwa)
    {
        Nazwa = nazwa;
        Pole = ObliczPole();
    }

    private double ObliczPole()
    {
        ...
        return wartosc;
    }
}

Wariant 2

public class Kwadrat
{
    public string Nazwa { get; set; }
    public double Pole { get; set; }

    public Kwadrat(string nazwa)
    {
        Nazwa = nazwa;
        ObliczPole();
    }

    private void ObliczPole()
    {
        ...
        Pole = wartosc;
    }
}

 

Answer 1

Witaj. Generalnie to zależy od kontekstu i co tak naprawdę chcesz osiągnąć. Ja na przykład skłonny byłbym do dodania w konstruktorze długość boku, zwalidowal tę wartość i na jej podstawie automatycznie obliczyl pole. A czy użyjesz do tego jakiejś prywatnej metody czy nie to już zależy od ciebie. Wiadomo że trochę bardziej skomplikowane obliczenia warto byłoby opakowac w metodę... Jeśli chodzi o te warianty które przedstawiles, to obydwa są poprawne ale mają raczej sens tylko wtedy gdy ustawisz temu propsowi Pole, private albo protected set. Pozdrawiam

Answer 2

Pamiętaj że właściwość przez kompilator później i tak jest dzielony na dwie metody. Dlatego też walidacja może być umieszczana w 'set'.

Answer 3

Uważam, że taki void jest bez sensu, bo i tak przypisujesz wartość, tylko w innym miejscu.

Wolałbym rozwiązanie na podobieństwo tego:

using System;

namespace SomeNamespace
{
    public static class SquareAreaCalculator
    {
        public static double CalculateArea(double a)
        {
            return Math.Pow(a, 2);
        }
    }
}

namespace SomeNamespace
{
    public class Square
    {
        public string Name { get; set; }
        public double Area { get; set; }
    
        public Square(string name, double a)
        {
            Name = name;
            Area = SquareAreaCalculator.CalculateArea(a);
        }
    }
}

Ewentualnie – już przy tworzeniu obiektu po prostu obliczyć pole.