Początki z programowaniem obiektowym

Question

Witam chciałbym się zapytać czy dobrze zrozumiałem to zadanie i czy o to w tym chodziło? Jeżeli powinienem zrobić to inaczej proszę o rady.Pytam ponieważ nie widzę sensu w tym zadaniu a że jestem w tym świeży to pewnie popełniłem sporo istotnych błędów, czekam na poprawki :)

---

/**
 * Created by Dominik on 01.05.2017.
 */

//4.Zaprojektuj rower. Niech ma obiekt kierownica, pedały, łańcuch i koła.
//Obiekt koło niech dziedziczy po klasie ogólnej,
//a w swoich szczególnych interpretacjach niech różni się tym,
//czym w rzeczywistości różni się koło przednie od tylnego.
//Zakładamy, że pedały są takie same.

public class Rower {
    public static void main (String[] args){
        Pedały SPD = new Pedały("Czarne", "platformowe");
        Łańcuch YBN = new Łańcuch(150,"xring");
        Kierownica ACO = new Kierownica(45);
        Koło Shoei = new Koło(28,29)
    }
}

---

/**
 * Created by Dominik on 01.05.2017.
 */
public class Pedały {
    private String name;
    private String color;
    private String type;

    public Pedały(String color, String type) {
        this.color = color;
        this.type = type;
    }
}

---

/**
 * Created by Dominik on 01.05.2017.
 */
public class Łańcuch {
        private String name;
        private int length;
        private String  oring;

        public Łańcuch(int length, String oring) {
            this.length = length;
            this.oring = oring;
        }
    }

---

/**
 * Created by Dominik on 01.05.2017.
 */
public class Kierownica {
    private String name;
    private int width;

    public Kierownica(int width){
        this.width = width;
    }
}

---

/**
 * Created by Dominik on 01.05.2017.
 */
public class Koło extends Rower {
    private int size1;
    private int size2;

    public Koło (int side1, int side2){
            this.size1 = side1;
            this.size2 = side2;
    }
}

 

Comments

w tak ogólnej klasie jak Rower pola nie powinny wzkazywać swoją nazwą na konkretny model. Dlatego też 
Łańcuch YBN = new Łańcuch(150,"xring");
zamienił bym na 
Łańcuch łańcuch = new Łańcuch(150,"xring");

O producencie powinnien nas informować obiekt Łańcucha a nie Roweru więc w Łańcuchu dodajemy.

public Łańcuch(int length, String oring) {
this.name = "XBN";
this.length = length;
this.oring = oring;
}

Z kołem chodziło chyba o to ze ma być klasa koło (abstrakcyjna najlepiej) Potem tworzysz 2 klasy KołoPrzednie i KołoTylnie które są rozszerzane o klase Koło. Rozszerzanie o klase Rower jest bez sensu.  Mam nadzieje ze pomogłem.

---

Hmmm, rozumiem dzieki wielkie w takim razie mam jeszcze pytanie moglbys mi wytlumaczyc na czym ten program wgl polega bo nie widze w nim calkowicie sensu poniewaz niewiem co ma za zadanie

---

Ma za zadanie zobrazowanie obiektowości. Że każdy obiekt jest odpowiedzialny za co innego i obiekty programu to takie części roweru mające swoje zadania.

Answer 1

1. To powinny być zmienne globalne klasy a nie zmienne metody.

2. Zmienne zaczyna się od małych liter ( zmienne stałe są nazywane z samych dużych liter)

3. Zrób dwie osobne zmienne dla kół, w końcu koła są osobne.

4. Pedały jw.