@Seveir Tak na prawdę podział na brache itp jest umowny i najważniejsze żeby tobie było wygodnie

@Seveir:
trochę dziwne jest, że w liście z jednostkami dodajesz np. takie coś new Unit("BACK", -1)
Generalnie zarówno miary jak i odległości to pewien zamknięty zbiór, więc zamiast klasy Unit wykorzystałbym na Twoim miejscu tutaj enumy. Klasy miałyby sens, gdybyś dawał użytkownikowi możliwość definiowania jednostek.
Liczby reprezentujesz jako double, co nie jest jakimś wielkim błędem w takim małym programiku, ale od razu możesz sobie przećwiczyć BigDecimale jako prawidłowy wybór przy precyzyjnychPokaż całość

@dzeksondzekson: ja bym się nie brał za testy na tym poziomie, najpierw ogarnij porządnie język i programowanie obiektowe

@dzeksondzekson

no nie bardzo chyba bo u mnie dataUrodzenia to jest String

No i słabo. String to ciąg znaków. Nie nadaje się do reprezentacji daty.
Czytasz tę datę z pliku jako String. Wiadomo. Ale musisz to jak najszybciej zamienić na właściwy typ.

@masterix: Brak PHPDoc i type hintingu kuje w oczy. Wstrzykuj zależności do kontrolera, nie wyciągaj ich z kontenera.

@masterix: nie nazwałbym tego co robisz architekturą heksagonalną. Brak wstrzykiwania zależności by móc nimi żonglować (a nam tym polega ta architektura, wyciągasz bloczek "wolny MySQL" i wkładasz za niego "szybki Elasticsearch"), beznadziejnie zaimplementowany CQRS (to nawet nie jest CQRS bo nie ma literki Q, nie ma literki C, pewnie nie ma również R i S, chyba, że zmienna $commandBus jest tylko z nazwy), brak separacji zmiennych (modyfikator publiczny, kiedy można daćPokaż całość

@masterix: https://github.com/nachteil/domain-monitor/pull/1#pullrequestreview-40825762

@interface: Nie no, idąc tokiem rozumowania:
to mapa (jedna wartość do klucz)

to multimap (wiele wartości do klucza)

No to

> to już musi być w takim razie multimultimap

Pokaż spoiler

@Tylko_Japonki:

Nie stosuj wildcardów (*) przy importach.
Używaj camelCase do nazywania zmiennych.
Używaj klamer {} przy ifach i forach, nawet, jak instrukcje są jednolinijkowe.
Nie duplikuj kodu - jeśli kilka metod używa wspólnego kodu, postaraj się go wyabstrahować do osobnej metody.
Statyczne importy wrzuć na 'górę'.

@panczekolady: W robieniu sobie dobrych nawyków.

@fegwegw: jak wstrzykujesz zaleznosci do obiektu przez konstukrtor to nie musisz dawac nad nim @autowired. Spring sam zalapie

https://spring.io/blog/2016/03/04/core-container-refinements-in-spring-framework-4-3
@fegwegw:

@kornfan: Duplikacja jest problematyczna w momencie kiedy trzeba coś zmienić w logice, którą sobie kopiuj-wklejasz. W dwóch miejscach jeszcze ogarniesz, ale potem to już bardzo bug-genne jest.

@kornfan:
A obiecasz mi, że za dwa lata jak jakiś inny programista będzie zmieniał kod to będzie wiedział, że kod jest zduplikowany?

@fistasheq: Wrzucaj od razu plik, a nie się skradasz

@fistasheq: Jak dla mnie przesadzone. Jeśli uczysz się Observabli i chcesz z nich korzystać to rozumiem, ale ja bym je powyrzucał i zrobił logikę na normalnych funkcjach.

A jeśli chcesz, żeby inne komponenty reagowały na dane zdarzenia, to zrobiłbym jakiś event z określonym typem i rzucał do oddzielnego Service'u i niech te komponenty się subskrybują na konretne zdarzenia.
Coś w tym stylu:
http://abusanad.net/2016/08/20/angular-2-cross-modules-communication-eventaggregator/

@saquas: jednocześnie synchronized i volatile, do tego tylko zmieniasz tam ten licznik, no tak właściwe nie wiem po wątki skoro wszystko syncujesz.
AtomicInteger tak jak ktoś pisał wyjdzie tu znacznie lepiej jako shared counter.

@saquas: ja mam jedynie niewielkie uwagi:

- nawet, jak masz single-line statement po ifie, to używaj klamer {}, to pozwoli uniknąć błędów.
- enumy możesz formatować trochę inaczej:

JEDEN(),
DRUGI(),
OSTATNI(),
;

pozwoli to na mniej ingerencji w kod w przypadku chęci dodania kolejnej linijki

- często zostawiasz jakieś dziwne puste linie: https://github.com/marcinpanfil/hospital/blob/master/src/main/java/com/edgelab/hospital/Medicine.java (24,25,26)

- statyczne importy wrzuciłbym na 'góre' listy importów
- osobiście #!$%@?łbym te interfejsy z jednym implementorem (cośtamPrinter)

@Kain_to_Charlie_a_Delta_to_Kain:
Menu: o firmie, menu rozwijane po najechaniu nie klikaniu.
W karuzeli strzałki zamieniłbym na buttony. Teraz mogę kliknąć wysoko nad strzałką i DIV mi to łapie.
Zdjęcia całe klikalne, nie tylko napisy.
Ad. 3 Zrobiłbym to w programie graficznym, tak żeby krzesła miały miały podobny rozmiar.

@sokytsinolop: Tak, ja Andrzej :)

@sokytsinolop: Moznaby to zrobic obiektowo. Az sie prosi.
Zamiast tablic lepiej chyba byloby uzyc jakiegos sensownego ceplusplusowego kontenera (std::vector, std::array).
Pomijajac wszystko... w generation() chyba lepiej zebrac wszystkie pola do uaktualnienia a potem je uaktualnic zamiast kopiowac tablice tam i spowrotem.

@Blade:

1. Testy automatyczne. Skąd mam wiedzieć czy Twój kod w ogóle działa?
2. Też irytuje mnie mieszanie camelCase ze snakecase
3. https://en.wikipedia.org/wiki/Single_responsibility_principle – masz widgeta, który w tej chwili wie jak robić dwie rzeczy na raz: zarządzać taskami i zapisywać swój stan. Jak rozpoznać, że się narusza SRP? Pomyśl jakie zmiany mógłbyś chcieć wprowadzić. Np mógłbyś chcieć zmienić format zapisu z JSONa. Albo w ogóle pisać doPokaż całość

zmienne w camelCase chyba

@Blade: no niestety:
https://www.python.org/dev/peps/pep-0008/#method-names-and-instance-variables
I to też należy odnieść do zmiennych wewnątrz "normalnych" funkcji.

•generalnie to zrobiłem ten program bo nie lubię playera twitcha, jest okropny i lubi się zacinać mimo że mam jeszcze połowę wolnych zasobów.

@jaggi: Z tych samych powodów odpalam skryptem stream'a via livestreamer + WMP ( ͡° ͜ʖ ͡°)

@qwelukasz: kilka lat temu to by było niezłe, teraz chyba mało kto korzysta z noip.

#codereview

@qwelukasz:

1. Composer i PSR-4 do autoloada klas i zaciągania zależności
2. Przestrzenie nazw
3. Nie CURLuj „ręcznie” tylko weź pośrednika w stylu Guzzle
4. Wcięcia Ci się pokiereszowały
5. Ścieżkę do ciacha wstrzykuj w konstruktorze, zamiast hardkodować
6. Nie rób ciężkiej logiki w konstruktorze (logowanie się) – w dobie kontenerków DI możesz chcieć zainicjalizować klasę, ale jeszcze nie użyć. Lepiej odpalić to na początku metod publicznych albo pozostawić odpaleniePokaż całość

@krasnoludkolo:
Tak z grubsza na szybko:
Nie nazywaj klasy aktywności MainApp, bo to sugeruje rozszerzanie klasy Application, nazywaj je po prostu z sufixem Activity. Poza tym poczytaj o appcompacie i rozszerzaj AppCompatActivity. Jako kontekst w wielu miejscach możesz po prostu podawać kontekst aktywności, nie ma potrzeby przekazywania kontekstu aplikacji. Unikaj jednoliterowych nazw zmiennych typu "i". Postaraj się też bardziej precyzować nazwy metod, configure() nic nie mówi o tym co konfigurujesz. JeśliPokaż całość

@szyna55 Zawsze kiedy moj 1,5 roczny syn ucina sobie popołudniową drzemkę na moim łóżku to mnie zachciewa się srać. I nie mogę iść, bo oczywiście jak tylko wyjdę to mały skula się, #!$%@? o podłogę i będzie ałka. Więc zaciskam poślady, stękam, chrzakam, ale nie idę. Z Twoją aplikacja koniec z tym! Będę mógł iść na tron, patrzeć czy dzieciak leży spokojnie a w razie czego lecieć na ratunek! Dzięki!