Aktywne Wpisy
Zawiera treści 18+
Ta treść została oznaczona jako materiał kontrowersyjny lub dla dorosłych.
Wattson +101
Coraz niebezpieczniej robi się na ulicach miast, ludzie narodowości polskiej chodzą po ulicy i dźgają przypadkowych przechodniów. W przyszłych wyborach mam zamiar zagłosować na partię, która wyrzuci tych wszystkich Polaków z Polski.
#neuropa #bekazprawakow #bekazkonfederacji #bekazkuca
#neuropa #bekazprawakow #bekazkonfederacji #bekazkuca
Napisałem sobie pierwszą apke, nic skomplikowanego, prosty TODOList. Zobaczyłby może ktoś z was co robię źle, co można zrobić lepiej? ;)
Kod
1.
Task
: dawno nie kodziłem w Javie, alesuper()
w konstruktorze chyba zbędne – nie dziedziczysz po żadnej klasie.2.
Task
: w konstruktorze parametr nazywaszinfo
, dalej jużname
– do ujednolicenia.3.
TaskRepository
: nie traktuj repozytorium jak kolekcję (czyli nie powinna posiadać metodysort
) – ale raczej jako dostawcę kolekcji. Czyli ewentualnie możesz poprosić repozytorium o posortowaną kolekcję (zamiastrepo.sort(); repo.getAll()
raczej róbGrafiki na razie w wersji roboczej ;)
@krasnoludkolo: IMHO repo jest na tyle proste, że bym olał. Tak na prawdę to repozytorium nie ma za bardzo logiki, chyba, że ją potem dodasz i będziesz chciał wydzielić persystencję gdzie indziej, żeby mieć SOLID.
i co do 4. Jeśli te metody będą prywatne to jak uzyskać efekt z metod onResume onStop itp?
@krasnoludkolo: Repozytorium jest miejscem, skąd możesz czerpać obiekt(y). Służy Ci do tego, żeby mieć skąd pobrać jeden obiekt lub kolekcję obiektów.
To nie jest tak, że wykonujesz na nim jakieś działania. Nie sortujesz repozytorium. Możesz sortować kolekcję obiektów, ale nie
@krasnoludkolo: Jeszcze a propos tego. Wybraź sobie, że chcesz pobrać tylko taski, które mają priorytet powyżej określonego poziomu.
Nie będziesz robić metody typu
repo.setReturningOnlyWithPriorityHigherThan(priority);
Czyli powinienem zrobić tak, że przy każdorazowej zmianie danych zrobić coś takiego?
adapter.clear()
adapter.addAll(repo.getAllSortBy(sortedby))
Nie wiem więc jaka do końca jest rola tego adaptera, ale z tego co widzę
TaskListAdapter
dostaje repo w konstruktorze i je sobie trzyma w polu. Więc widzę dwie opcje:a.) dorób sobie
adapter.reload()
(w końcu ma repo, więc sam sobie może załadować noweTaskCollection
brzmi bardziej „pro” ;-)Tak czy siak, wtedy wywalasz wszystkie funkcje zapisywania, zamieniasz save na add, etc.
SingleResponsibilityPrinciple: albo jest kolekcją albo jej persystencją.
1. Podmioty, obiekty którymi zarządzasz (Task),
2. Zestawy, listy, kolekcje tych podmiotów (TaskCollection, ArrayCollection, Array, TaskArray),
3. Dostarczyciela/zarządcę powyższych: