Prośba o CodeReviwe

Question

Witam,

chciałbym poprosić o przegląd kodu i uwagi/ wskazówki co można zrobić lepiej/inaczej.
Poniżej link do Git'a:
https://github.com/krzysztof83/Quiz

Z góry dziękuję za poświęcony czas.
Pozdrawiam,
Krzysztof

Comments

Nie wnikam w merytoryczą zawartość. A tak od strony techniki programowania to rzuca się w oczy brak użycia pętli w oczywistych fragmentach kodu , np. w pliku user

Zacznij używać pętli. :)

---

Jeremus - dzięki za odpowiedz natomiast chyba nie wiem gdzie tam wcisnąć petlę, poniżej link do klasy User:

https://github.com/krzysztof83/Quiz/blob/master/src/main/java/model/User.java

Chyba że chodziło Ci o klase UserService, ale tez nie wiem gdzie miałbym tam dodać pętle:

https://github.com/krzysztof83/Quiz/blob/master/src/main/java/service/UserService.java

Jeśli wiec zechciałbyś rozwinąć swoja wypowiedz to będę wdzięczny.

---

sory chodziło mi o :

DBStart.java

wszystkie incjalizacje : wypadałoby użyć tabel i pętli.

chyba ze to tylko dane wprowadzone do testowania wprowadzone w dowolny sposób :)

---

hehe 2 razy skasowałem sobie dane w bazie poprzez odkomentowanie linii:

<!--<property name="hibernate.hbm2ddl.auto" value="create"/>-->

w persistence.xml

Wiec stworzylem sobie klase gdzie szybko te dane dodam ponownie, ale fakt lepiej to poprawić.

Dzięki za odpowiedz.

---

Tylko tak rzuciłem oko i nazwałeś klasę StartInterface, która nie jest interfejsem. To jest bardzo mylące. Możesz trochę uprościć dodawanie do listy zamiast:

 String answer41 = "ani razu";
        String answer42 = "nieskonczenie wiele razy";
        String answer43 = "6";
        String answer44 = "7";
        answers4.add(answer41);
        answers4.add(answer42);
        answers4.add(answer43);
answers4.add(answer44);

// to

Collections.addAll(answers,answer42,answer43,answer44 )