ABCDEFGHIJKLMNOPQRSTUVWXYZAA
1
ImięNazwiskoIndeksRazemZad 1Zad 2Zad 3Zad 4Zad 5Zad 6Zad 7Zad 8Zad 9Zad 10Zad 11
2
BibliotekaProsta JavaTabliceKlasaTramwaje/RobotyForumBezszachyBSTStos + wyjątki
Typy generyczne
Własna lista
3
Oddanie:8.03.201714.03.201721.03.201728.03.20174.04.201723.04.201711.04.20179.05.201716.05.201723.05.201730.05.2017
4
Poprawki:11.04.201719.04.201710.05.201730.05.201730.05.20176.06.2017
5
KonstantyCieśliński37075810OKOKOKOK (uwagi)OKOKOKOK--OKOK
7
ErykKijewski38573411OKOKOKOK (uwagi)OKOKOKOKOKOKOK
8
MateuszKobak3857605OKOKOK--OKOK----------
9
WiktoriaKośny38580411OKOKOK (uwagi)OK (uwagi)OKOKOKOKOKOKOK
10
AdrianKucharczuk38585511OKOKOKOK (uwagi)OKOKOKOKOKOKOK
11
AleksandraKwiecińska2937096OKOKOK--OK--1/21/2OK----
12
JakubPaliga3713508OKOKOKOK (uwagi)--OK--OKOK (uwagi)OK--
13
AliaksandrSarokin37252510OKOKOK (uwagi)OK (uwagi)OKOK--OKOKOKOK
14
MichałSarzyński3716789OKOKOKOKOKOKOK (uwagi)--OK (uwagi)--OK
15
WojciechSikorski3859528.5OKOK (uwagi)OKOKOKOK1/2----OKOK
16
OleksandraTumak38675611OKOKOKOKOKOKOKOKOKOKOK
17
MaciejTwardowski3864589OKOK (uwagi)OKOK (uwagi)OKOKOKOKOK----
18
KarolWaszczuk38648811OKOKOKOK (uwagi)OKOKOK (uwagi)OKOKOKOK
19
KamilaSledź7.5--OK--OK (uwagi)OKOK1/2OK--OKOK
20
21
Uwagi:
22
KonstantyCieśliński370758To Kierunek powinien wiedzieć, co to znaczy skręcić w prawo/skręcić w lewo, a nie Robot. Niepotrzebnie komplikuje Pan wypisywanie planszy: Robot powinie wypisywać dane zgodne z formatem w zadaniu. Nie powinien Pan parsować wyników własnego kodu.Stos miał być parametryzowany typem przechowywanych elementów. Napisał Pan zwykły stos liczb całkowitych.
24
ErykKijewski385734Zmienne plansza, roboty, kierunki powinny być zmiennymi lokalnymi w metodzie Main a nie polami statycznymi w klasie Main!
Bardziej idiomatyczne dla Javy jest używanie w konstruktorze przypisań w stylu this.imie = imie. W Javie nie używa się zmiennych o nazwach zaczynających się od podkreślenia.
25
MateuszKobak385760
26
WiktoriaKośny385804W nazwach metod nie używamy znaku podkreślenia, używamy konwencji camelCase.Obiekty Kierunek nie powinny się zmieniać. W "życiu" nie jest tak, że północ nagle staje się wschodem. To Roboty się obracają, a nie kierunki świata.
27
AdrianKucharczuk385855Metoda wyświetl jest niepotrzebna, to co robi jest niezgodne z nazwą.
28
AleksandraKwiecińska293709W zadaniu powinny być klasy WęzełPusty i WęzełPełny, które powinny zajmować się dodawaniem/szukaniem wartości. Klasa Drzewo powinna delegować operację do korzenia.
29
JakubPaliga371350Nazwy metod powinny zaczynać się małą literą.Konstruktory wyjątków powinny być publiczne.
30
AliaksandrSarokin372525Stringi trzeba porównywać za pomocą metody equals(). compareTo też działa, ale equals() jest bardziej idiomatyczne.Ułamki powinny być niezmienne. Metody dodaj(), odejmnij(), itp. powinny zwracać NOWY ułamek. Skracanie ułamków powinno być w konstruktorze - to konstruktor dba o poprawny stan obiektu.Hierarchia klas powinna być inna: Liść nie ma synów, więc nie potrzebuje pól lsyn i psyn. Trzeba odpowiednio zmienić hierarchię klas.Klasy wyjątków powinny być publiczne. Powinien być interfejs Stosu i oddzielnie implementacja. Obsługa wyrażeń w odwrotnej notacji polskiej powinna być w osobnej klasie.
31
MichałSarzyński371678Można było lepiej sparametryzować szukanie możliwych ruchów.Wyjątki zwykle mają w nazwie "exception", np. StackOverflowException
32
WojciechSikorski385952Nazwy metod powinny zaczynać się małą literąProgram się nie kompilujePrzykład użycia stosu powinien być w osobnej klasie. Stos powinien mieć wskaźnik na pierwszy element stosu, a nie na listę elementów.Brakuje sortowania listy
33
OleksandraTumak386756Metoda czynnikiPierwsze nie powinna być publiczna - nie jest częścią interfejsu Ułamka. Upraszczanie ułamków trzeba zrobić prościej: dzielimy przez największy wspólny dzielnik (nwd) licznika i mianownika.
34
MaciejTwardowski386458Nazwy klasy powinny zaczynać się wielką literąKierunek nie powinien się zmieniać. Zmienia się orientacja robota, ale sam obiekt kierunek się nie zmienia. To nie Plansza powinna obracać robota, robot sam powinien się obrócić.
35
KarolWaszczuk386488Nazwy metod powinny zaczynać się małą literą.Metoda possibleMoves() powinna być publiczna. Można ją podzielić na mniejsze metody.Rozwiązanie powinno używać klas WęzełPusty i WęzełPełny, które powinny zajmować się dodawaniem nowych wartości/szukaniem wartości
36
KamilaSledźBardziej eleganckie rozwiązanie: prywatny konstruktor Liczby z tablicy cyfrKlasy powinny zawierać metody zwracające opis
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102