Question

Co sądzicie o moim programie? Wiem że jest banalny, ale uczę się dziedziczenia i ogólnie klas i chciałem to jakoś zgarnąć w jedno. Dlatego nie piszcie mi komentarzy typu: "Idź lepiej ucz się C++".

 

#include <iostream>
#include <cstdlib>

using namespace std;

class NWD
{
protected:
    int lb1, lb2, maxx, nwd;

public:
    void funk()
    {
        cout<<"Podaj 2 liczby(calkowite): ";
        while(!(cin>>lb1>>lb2))
        {
            cout<<"Blad wczytywania liczby!\n";
            exit(0);
        }

        if(lb1>lb2)  {maxx = lb1;}

        else  {maxx = lb2;}

        for(int i=1; i<=maxx; i++)
        {
            if((lb1%i==0) && (lb2%i==0))
            {
                nwd = i;
            }
        }
        cout<<"\nNWD liczb "<<lb1<<","<<lb2<<" to: "<<nwd<<'\n';
    }
};

class NWW :public NWD
{
private:
    int nww;

public:
    void fun()
    {
        nww = (lb1*lb2)/nwd;
        cout<<"\nNWW liczb "<<lb1<<","<<lb2<<" to: "<<nww<<'\n';
    }
};

int main()
{
    NWW show;
    show.funk();
    show.fun();
}

 

Answer 1

1. O chłopie. NWW i NWD mogą co najwyżej razem dziedziczyć z klasy liczba czy coś takiego. Ale że jedno z drugiego? 
2. Czysty kod leży. Przeczytaj jakie są konwencje nazewnictwa. Funk znaczy tyle co wszystko.
3. Nie masz zdefiniowanych ani co gorsza zdeklarowanych konstruktorów ani, destruktorów. Nie wiesz zatem, że tworząc instancję NWW wywołujesz konstruktor NWD. Niejawnie ale jest.
4. I jeszcze raz konstruktora. Tam się nadaje wartości cechom klasy. 

Comments

Jeżeli nie chce to, przecież nie muszę mieć konstruktorów ani destruktorów. A klasa dziedziczy ze tej drugiej bo używam wzoru: NWW(a, b) = (a*b)/NWD(a,b).

---

To że wzór tak wygląda to nie znaczy masz tak dziedziczyć. To jest błąd projektowy. Kaczka dziedziczy z klasy Ptak, gdyż kaczka jest ptakiem. W tym przypadku NWW to jest jest NWD. Możesz zawrzeć instancje tych obiektów w 3 klasie, wrapperze całego programu.

Jeżeli nie chce to, przecież nie muszę mieć konstruktorów ani destruktorów.

Po to one są żeby z nich korzystać w prawidłowy sposób. Pytałeś co sądzimy o Twoim kodzie, więc mówię Ci, że ani to nie jest poprawne, ani estetyczne.  

---

To dlaczego działa w takim razie skoro program jest blędny? :)

---

Wiesz, przypisać liczbę można na wiele sposobów.

int x = 5;
int x = (int)round(5.1242);

Obydwa zapisy dają x równe 5, co nie znaczy, że drugi jest dobry.

Jeżeli masz podejście, że kompiluje się to jest dobre to zmień to podejście albo porzuć programowanie. Zrób to dla ludzi, którzy będą musieli z Tobą pracować.

---

Działa bo czemu miało by nie działać ale to są błędy projektowe.

1 Dziedziczy się (publicznie) gdy coś jest rodzajem czegoś np samochód jest rodzajem auta i dziedziczy np składową pojemność silnika która jest wspólna dla obydwu ,zawiera np 4 koła co jest przykładem relacji ma coś.Ale takie dziedziczenie jakie ty proponujesz nie jest poprawne bo dziedziczysz tylko po to żeby dziedziczyć (sztuka dla sztuki).

2 Nazywaj swoje zmienne,funkcje,klasy tak aby jak najlepiej oddawały znaczenie tego co znaczą ,robią itd.

3  I oczywiście konstruktory podzielam uwagi Ehlert

---

+Grzyboo Człowieku jeżeli ty widząc jakiś program który jest dla ciebie idiotyczny to nie pisz osobom słabszym w programowaniu,żeby przestały programować, bo są do dupy. Ja mam 14 lat i od niedawna uczę się programować.Nie jestem tak dobry jak ty. Moje podejście na razie jest jakie jest bo jeszcze się uczę. Jeżeli naprawde byłbyś dobrym programistą to wiedziałbys że początki bywają trudne i nie wszystko przychodzi od razu. Ale jak widzę już pozjadałeś wszystkie rozumy. Czy radą jest żebym porzucił programowanie? Bardzo to lubię i narazie nie ma widoków żebym to porzuciał. Także przed napisaniem czegoś radzę troche pomyśleć a nie pisać głupot. Wątpię czy 14-latki w ogóle pracują....

---

Ale ja jestem nowicjuszem. Jednak Twoje podejście "To dlaczego działa w takim razie skoro program jest blędny? :)" jest karygodne. Najpierw zaproponowałem inne rozwiązanie, zmianę podejścia. Jeżeli tego nie zrobisz, to tak, sądzę że powinieneś przestać.

Answer 2

1. NWW, NWD to materiał bardziej na funkcję a nie na aż 2 xD klasy, w najgorszym razie możesz użyć struktury. Lepiej skupić się na samym modelowaniu obiektów w stylu [obiekt->pojazd->samochód] etc...

2. Naucz się podstawowych konwencji nazewnictwa pól i metod.

3. Korzystaj z referencji języka i klas, najlepiej po angielsku.