Operatory C++ / konstruktor kupiujący

pytanie zadane 9 grudnia 2015 w C i C++ przez krolkoreipolnocnej Nowicjusz (240 p.)
edycja 9 grudnia 2015 przez krolkoreipolnocnej

Cześć, dopiero uczę się programowania, a więc być może mam bardzo łatwe pytanie, ale jednak nie mogę go sobie wytłumaczyć za pomocą materiałów w internecie i będę wdzięczny za pomoc :)

Mam za zadanie stworzyc klase, ktora bedzie robiła dodawanie, odejmowanie etc. obiektow klasy. Z tym ze musze w tym przypadku skorzystac z konstruktora kopiujacego (pierwszy raz go tutaj używam) oraz operatorow typu (A operator + (&A);) Nie do końca mogę powiązać te wszystkie pojęcia ze sobą i jakby ktoś mógł mnie naprowadzić na właściwą drogę, będę bardzo wdzięczny! Poniżej moje próby..

class MojaLiczba {
public:
    MojaLiczba(int liczba);
    ~MojaLiczba();
    int L1;
   MojaLiczba (MojaLiczba &ML);    //konstruktor kopiujacy

    void podziel(MojaLiczba A, MojaLiczba B) {
        value = A.L1 / B.L1;
        cout << "Wynik: " << value << endl;
        }
    
    void dodaj(MojaLiczba A, MojaLiczba B) {
        value = A.L1 + B.L1;
        cout << "Wynik: " << value << endl;
    }
    void pomnoz(MojaLiczba A, MojaLiczba B) {
        value = A.L1 * B.L1;
        cout << "Wynik: " << value << endl;
    }
    void odejmij(MojaLiczba A, MojaLiczba B) {
        value = A.L1 - B.L1;
        cout << "Wynik: " << value << endl;
    }
private:
    int value;
};

2 odpowiedzi

odpowiedź 9 grudnia 2015 przez Szykem2 Nałogowiec (29,510 p.)
wybrane 9 grudnia 2015 przez krolkoreipolnocnej

Najlepsza

Po pierwsze jeżeli w argumencie przesyłasz jakiś obiekt to należy przesyłać go przez stałą referencję, chyba że masz uzasadniony przypadek, że musi być przez niestałą,

MojaKlasa(const MojaKlasa& obj);

Następnie nie ma sensu robić metod podziel, odejmij itd. bo wywołanie by musiało wyglądać:

obj = obj1.dodaj(obj1, obj2);

zamień te metody na np:

MojaKlasa& operator+(const MojaKlasa& obj){                            
    L1 += obj.L1;                                                //operacja dodawaia
    cout << "dodaje" << endl;
    return *this;                                                //zwracasz obiekt, w ktorym zmieniona jest wartosc L1
}

//teraz wywołanie
obj = obj1 + obj2;

Skoro zmienna value nie jes używana to nie powinno jej być a L1 powina być prywatna. Dobrym zwycajem jest nazywanie zmiennych prywatnych nazwami zaczynającymi się od '_' (chodzi o to, że wiadomo co jest zmienną prywatą a co publiczną).

komentarz 9 grudnia 2015 przez krolkoreipolnocnej Nowicjusz (240 p.)

Bardzo dziękuję!
Teraz mi się w głowie to bardziej poukładało :)

komentarz 9 grudnia 2015 przez Patrycjerz Mędrzec (192,320 p.)

Ciekawi mnie, po co mu ten konstruktor kopiujący?

komentarz 9 grudnia 2015 przez krolkoreipolnocnej Nowicjusz (240 p.)

wiem, że o wiele prościej dałoby się zrobić bez niego, ale jest tutaj tylko i wyłącznie po to aby go zrozumieć i przećwiczyć i okazało się to nie takie łatwe dla mnie)