[Danger40]

Witam wykopowych #programista15k

Pisałem ostatnio zadanko, które polegało na napisaniu usługi z interfejsem REST/HTTP, która umożliwi aktualizowanie wartości czujników i zwracanie wadliwych silników.

Serwer miał pobrać dane wejściowe zawierające listę czujników do silników z pliku w formacie YAML na GitHubie.

HTTP GET
Miał zwracać jako JSON listę silników, gdzie ciśnienie na czujniku spadło poniżej podanego poziomu i jednocześnie temperatura na co najmniej jednym czujniku wzrosła powyżej podanej wartości

HTTP POST
Miał zaktualizować wartość dla czujnika uwzględniając wartości maksymalne i minimalne możliwe dla danego czujnika.

Czujniki były rozplanowane w ten sposób, że każdy silnik miał tylko jeden czujnik ciśnienia(posiadały one info o silniku do którego są przyczepione) i co najmniej jeden czujnik temperatury, który posiadał w sobie pole z "id" czujnika ciśnienia, ale nie
wiedział nic o silniku.

Fajnie by było, jakby ktoś mógł jakieś krótkie code review z uwagami uczynić (⌐ ͡■ ͜ʖ ͡■)

Kod do zadania tutaj:
https://github.com/d100495/recruitment_task_r
A w załączniku podrzucam metodę filtrującą.

Z góry dzięki

#programowanie #java #eclipse #javaee #maven #rest #webservice #api

[kochmap]

@Danger40: poza tym, że go zamknąłeś to jakieś rozwiązanie to jest. Można zrobić pull requesta z mastera do innego brancha, tylko ten branch musi pokazywać na jak najstarszy commit. Można też stworzyć nowe repo.

[Danger40]

@kochmap: https://github.com/d100495/recruitment_task_r/pull/3

Nie mam pojęcia jak zrobić pull requesta bez modyfikacji istniejącego kodu, zrobiłem 2 branche i nadpisałem kod (pewnie nie jest to jakieś super eleganckie rozwiązanie xD).

Edit.
No tego poprzedniego przez przypadek zamknąłem i dlatego komentarz też wywaliłem, sory.

[kochmap]

@Danger40: wieczorem siądę albo jutro do tego bo mam trochę jeszcze rzeczy do roboty. Odblokuj pw to napiszę Ci kiedy skończyłem.

[Danger40]

@kochmap: Super, dzięki za pomoc.