@krasnoludkolo
Lombok. Meh.

@Seveir: W klasie UnitConverter brakuje Tobie EOL (https://stackoverflow.com/questions/729692/why-should-text-files-end-with-a-newline#729795).
Widzę, że masz zdublowaną metodę writeProblemMessage -> stwórz nową klasę, której zadaniem będzie wypisywanie errorów.
W klasie UnitReader modyfikatory dostępów mają być defaultowe? Jak wsadzisz to do jakiegoś pakietu to nie będziesz miał dostępu do tych metod.
Staraj się nie zostawiać zakomentowanego kodu, zwłaszcza jeśli idzie to na mastera.

int index = 1; ?????? co oznacza ta zmienna? może powinno to byćPokaż całość

@Seveir:

nie rób takich rzeczy:

public class LengthList extends ArrayList

public class MainMenu

Pokaż całość

@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 precyzyjnych obliczeniach.
SpróbujPokaż 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żnaPokaż 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)

NoPokaż całość

@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ęPokaż całość

@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ć do bazki.Pokaż 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.