Czy jak na początkującego to dobry kod? (kalkulator, java)

Question

Otóz jestem dość początkującym programistą, uczę się głównie poprzez ksiazke "Java, Rusz głową!" i trochę z tutoriali z youtuba. I tak zrobiłem prosty kalkulator, możecie sprawdzić czy ten kod jest dobry jak na początkującego: http://pastebin.com/PYxA1CWt

 

Możecie napisać czy jest dobry ten kod czy coś byście zmienli ;)

Answer 1

Jak dla mnie, to wydaje się całkiem czytelnie, ale od siebie dodałabym obliczenia na liczbach rzeczywistych zamiast int, a w następnej kolejności działania zapisała w osobnych funkcjach - wszystko oczywiście w ramach praktyki. I Head First jest jednak już dość przestarzałą książką. Radzę uzupełniać wiedzę o nowsze materiały.

Comments

Rusz głową to drugie wydanie (nowsza część ) książki Head First jakby co ;) a tak to skąd uzupełniać wiedzę? Bo wiadomo, że nikt nie kupuje książek co miesiąc z nowszą wersją, ani nikt nie nagrywa tutoriali na yt co miesiąc. Niby można z dokumentacji, ale mój angielski nie jest jeszcze perfect (wiesz, 1 gimnazjum dopiero)...

---

Tak, sama mam drugą edycję i dotyczy ona Javy 5. Aktualnie wersją używaną jest Java 8, gdzie każda wersja zmienia podejście. Co do źródeł, to przede wszystkim Oracle oni mają całkiem sporo poradników i tutoriali. Z polskich też trochę jest np. Java Start - do podstaw wystarczy.

I spokojnie, to nie jest tak, że w Javie co miesiąc wychodzi nowy standard, ale warto mieć pojęcie o nowych wersjach.

Answer 2

    public void cos() {}

"cos" niewiele komuś powie, zmień tą nazwę na inną, bardziej pasującą.
Do tego nazwy funkcji powinny być zapisywane systemem notacji Pascal Case.

 int wyborNum = Integer.parseInt(wybor);   //argument był zmienną String, więc zmieniamy ją
                                             //na zmienną typu int

Nie musisz rzutować zmiennej, z tego co widzę potrzebna ci jest tylko w switch case, możesz więc do tego użyć po prostu przysłanego stringa.

Comments

A ten zapis: wyborNum nie nazywa się przypadkiem Camel Case?

---

Nazywa, zmienne powinny być tak zapisywane, a funkcje/właściwości/klasy zapisem Pascal Case. np: DodajLiczbe, Sortuj, ZnajdzNajwiekszyElement

---

Z tym Pascal Case to trosze nie prawda. W javie konwencja jest taka że wszystko z wyjatkiem nazwy klas zaczynamy z malej. Czyli metody, funkcjie, atrybuty, zmienne, pakiety piszemy z małej i używamy Camel Case(z wyjatkiem pakietów). Klasy z dużej. A stałe wielkimi literami i zamiast spacji podkreślenie.

---

"(...) pakiety piszemy z małej i używamy Camel Case(z wyjatkiem pakietów)." :)

---

No chciałem podkreślić że nazwy pakietów to jedyne miejsce w którym wielkie litery są "zakazane". A wyszło no jak wszyło.

Answer 3

Nazwy metod przerobić na angielskie w całości.