Które rozwiązanie jest lepsze i dlaczego?

Question

Chce stworzyć strukturę która jest parą uporządkowanych liczb. Które z tych rozwiązań powinno zostać użyte i dlaczego? A może jeszcze jakieś inne?

template<typename T>
struct Point
{
	T x, y;

        //pierwsze
	Point operator + (Point point)
	{
		return { x + point.x, y + point.y };
	}

        //drugie
	Point operator + (Point point) const
	{
		return { x + point.x, y + point.y };
	}

        //trzecie
	inline Point operator + (Point point)
	{
		return { x + point.x, y + point.y };
	}

        //czwarte
	inline Point operator + (Point point) const
	{
		return { x + point.x, y + point.y };
	}
};

Zauważyłem też że można "inline" dodać przed "Point" i po "Point", czym to się różni?

Jeszcze mogę w argumentach operatora wpisać (Point& point), powinienem to zrobić?

Comments

Jawne inline'owanie metod, które są zdefiniowane wewnątrz klasy, jest zbędne, ponieważ takie metody są domyślnie inline.

Answer 1

Podany operator + (tak jak i normalne, matematyczne) jest symetryczny względem argumentów, więc IMHO powinien być zaimplementowany jako funkcja (być może zaprzyjaźniona) a nie jako metoda klasy, bo w tym drugim przypadku oba argumenty traktowane są różnie. Metoda jest fajna dla np. operatora +=.

Przekazanie parametrów jako referencji jest całkiem OK - mniej pomocniczych konstruktorów się wykona.

Żaden z argumentów się nie zmienia więc przydałoby się parę const.

 

Comments

Czyli coś takiego?

template<typename T>
struct Point
{
    T x, y;
 
    const Point<T>add( Point<T>&point )
    {
        return { x + point.x, y + point.y };
    }
}

Czy to jest idealne rozwiązanie?

---

Raczej to:

template<typename T>
struct Point {
    T x, y;
    
    using this_type = Point<T>;

    this_type operator+(const this_type &p) const
    {
        return { x + p.x, y + p.y };
    }

// lub

    friend this_type operator+(const this_type &p1, const this_type &p2)
    {
        return { p1.x + p2.x, p1.y + p2.y };
    }

};

 

---

Fragment

using this_type = Point<T>;

Jest tylko dla czytelności kodu i nie ma wpływu na działanie programu?

I dlaczego zrobiłeś wewnątrz nawiasu 2 razy "const this_type"? Nie wystarczy zrobić jeden, a drugi w domyśle będzie przez program znajdowany?

---

Jest tylko dla czytelności kodu i nie ma wpływu na działanie programu?

Tak.

I dlaczego zrobiłeś wewnątrz nawiasu 2 razy "const this_type"?

Dałem dwie wersje: pierwsza operator jako metoda, druga - funkcja zaprzyjaźniona.

---

Czy te wersje się technicznie czymś różnią? Której używać? I dlaczego w tych miejsca dać const?

---

Technicznie różnią się tym, że jedna definiuje metodę, a druga funkcję. Której używać? Którą chcesz.

const w parametrach dajesz, by móc przekazywać przez referencję obiekty const.

Metoda jest const po to, by móc ją wywoływać na rzecz obiektu const.