[GrubyCzarny]

Siema mirki,prośba o pomoc.
Mam za zadanie przeanalizować ten kod,i wytknąć wszelkie uwagi.
Wymaganie jest proste,z podanego Stringa wypisujemy wszystkie słowa i liczbę ich wystąpień
https://pastebin.com/vJApbEVH

Na razie wymyśliłem że
- kod jest przekomplikowany bo można to zamknąć w jednej metodzie i zwrócić mapą
- no i brak Unit testów

dorzucicie coś ?

z góry dzięki

#java #programowanie #programista15k #codereview

[S.....s]

@GrubyCzarny: brak hashcode, niepoprawny zwracany typ, klasa util posiadająca stan, niekompletny javadoc, niewydajny algorytm

[GrubyCzarny]

@SuppressWarnings: Dzięki
A chciałoby ci się rozwinąć ?
co znaczy " klasa util posiadająca stan " ?
I gdzie masz niepoprawny zwracany typ ? ja tam same voidy widzę

[S.....s]

@GrubyCzarny bo w javadocu jest że zwraca listę, a nie zwraca, tylko zapisuje to z niewiadomych przyczyn do pola words

[i.....t]

@GrubyCzarny: Klasa Word ma skopany equals i wyszukiwanie elementów na liście pozwala na poprawne zliczanie tylko do 2 wystąpień. Przykład - dodaj do 'test' słowo "half" to w wyniku otrzymasz 2 wiersze z "half":

First occurred 1 times
half occurred 2 times
goals occurred 2 times
for occurred 2 times
Chelsea, occurred 1 times
Fulham occurred 1 times
and occurred 2 times
Leeds. occurred 1 times
Second occurred 1 times
half

[63274682374]

@GrubyCzarny: Zacząć przede wszystkim trzeba od tego, że algorytm jest błędny.
1. Przy podziale na słowa nie uwzględnia znaków interpunkcyjnych
2. W equals porównuje również count co samo w sobie błędem nie jest ale ten sam equals używany jest do wyszukiwania na liście i słowa występujące 2 i więcej razy znalezione już nie będą.

Dalej, jeśli ten algorytm miałby zostać, to:
1. Word word = new Word(values[i], 1); powinno być w

[63274682374]

@63274682374: ...no i w klasie Word pola powinny być prywatne

[GrubyCzarny]

dzięki wszystkim za pomoc,pozdrawiam

[ania-nowak1231]

@GrubyCzarny: https://stackoverflow.com/a/25441208 i string.split zeby podzielic na tokeny. łącznie 5 linijek