[Nofenak]

Ostatnio robiłem zadanie rekrutacyjne na stanowisko Juniora (nie wymagali nawet żadnego doświadczenia) z takmi wymaganiami:

As an api consumer, given username and header “Accept: application/json”, I would like to list all his github repositories, which are not forks. Information, which I require in the response, is:

Repository Name

Owner Login

For each branch it’s name and last commit sha

As an api consumer, given not existing github user, I would like to receive 404 response in such a format:

{
    “status”: ${responseCode}

    “Message”: ${whyHasItHappened}
}

As an api consumer, given header “Accept: application/xml”, I would like to receive 406 response in such a format:

    {

    “status”: ${responseCode}

    “Message”: ${whyHasItHappened}
   }
Notes:
Please full-fill the given acceptance criteria, delivering us your best code compliant with industry standards.

Please use [https://developer.github.com/v3](https://developer.github.com/v3) as a backing API

Application should have a proper README.md file

Tutaj zrobione to zadnie przeze mnie: https://github.com/Sampeteq/github-task
Dzisiaj rano dzwoni do mnie rekruter i dosłownie zjechał mnie ostro za takie rzeczy jak:
1.Paginacja, której nie było w wymaganiach a ją dodałem. Myślałem, że to standard w każdym API, tym bardziej, że to API GItHuba zwracało czasami kilkadziesiąt wyników.
2.Mappowanie odpowiedzi z WebClienta na Stringa zamiast na jakąś klasę mimo, że później jest to robione.
3.Użycie ObjectMappera do mapowania jsona z WebClient na obiekt Javowy. Powiedział, że to mnie definitywnie skreśla
4.Utworzenie WebClienta za pomocą create() zamiast utworzenia beana. Powiedział, że widać, że nie znam Springa
4. Użycie MockMVC w testach integracyjnych controllera zamiast WireMocka czy czegoś podobnego, nazwał to zbrodnią

Jak to oceniacie? Czy rekruter miał racje i takie błędy dyskwalifikują kandydata czy może moje rozwiązanie jest w miarę ok i rekruter przesadził? Nie ukrywam, że byłem w sporym szoku. Nie bardzo mogłem wejść z nim w polemikę, bo telefon zerwał mnie z łóżka i byłem zaspany
#programowanie #programista15k #java #pracbaza

[ElMatadore]

@Nofenak: ja Javy nie znam zbytnio, ale jak ten typ serio w takim tonie do Ciebie gadal to niech #!$%@? xd

[dawittt]

@Nofenak już #!$%@?ąc od zasadności uwag, to jeśli tak to faktycznie Ci przedstawił to trafiłeś na frustrata, który lubi się dowartościować z rana. Nie chciałbyś z nim pracować i tak

[kernik]

@Nofenak: za samo programowanie w Javie należy się miejsce w dziewiątym kręgu ( ͡° ͜ʖ ͡°) myślę że rekruter tym razem był łaskawy

[E.....o]

@Nofenak:

#!$%@? od twojego rozwiazania.. to niesamowite ze zeby napisac w tej technologii taki trywialny program potrzeba tyle boilerplatu i abstrakcji i że da się przy tym zrobic tyle błędów XD

[kwas_rybonukleinowy]

@Nofenak: Powiem to tak, od dłuższego czasu przebranżawiam się na programistkę z montażystki video. Przerobiłam JetBrains Academy od Hyperskill, mam tytuł zawodowy inżyniera oraz naukowy (xd) magistra oraz realizuje swoje poboczne projekty.

1. Zacznijmy od: GET /api/repos/{username}/fork=false?pageSize=&pageNumber=

Coś takiego jest mocno nie efektywne nie powinno mieszać się @PathVariable oraz @RequestParam

2.Mappowanie odpowiedzi z WebClienta na Stringa zamiast na jakąś klasę mimo, że później jest to robione.

Jest to niepotrzebna operacja wystarczy

[j.....2]

@Nofenak: Przykra sprawa. Nie lekceważę jej. W poniedziałek o 15 odniosę się do niej publicznie na mikroblogu.

[bart1234]

@kwas_rybonukleinowy:

Coś takiego jest mocno nie efektywne nie powinno mieszać się @PathVariable oraz @RequestParam

pierwsze słyszę, że niby czemu?

[k.....e]

@Nofenak:
1. Nie wypowiem się
2,3,4,5 - ma rację

Ale czy to eliminuje kandydata na juniora to nie wiem

[JanPawelDrugiLechWalesaPierwszy]

@bart1234: Pierwsze słyszysz bo to nieprawda.

[azzQ]

@Nofenak: Zerknąłem pobieżnie, ale moim zdaniem jak na juniora to zdecydowanie jest ok, a nawet jak mu nie pasowało to i tak powinien się lepiej zachować, szczególnie że np. tworzenie beana dla WebClienta też nie jest takie oczywiste i czasami lepiej wstrzyknąć WebClient.Builder i z niego stworzyć konkretnego WebClienta.

[Waffenek]

@Nofenak: Trafił ci się niepoważny frustrat. O ile faktycznie masz dość niestandardową strukurę pakietów z dość arbitralnym podziałem na application i infrastructure(chyba próbowałeś się porwać na hexagonal architecture, jednak coś nie wyszło bo masz cykl - w takim przypadku powinieneś między innymi przenieść interfejs GitHubClient do application) to ostra ocena jest nie na miejscu.

1.Paginacja, której nie było w wymaganiach a ją dodałem. Myślałem, że to standard w każdym API, tym

[rockip]

Coś takiego jest mocno nie efektywne nie powinno mieszać się @PathVariable oraz @RequestParam

@kwas_rybonukleinowy: jestem z .NETowego świata, ale nie sądzę żeby to było prawdą w jakimkolwiek frameworku do tworzenia web API

[rtp_diov]

@Nofenak - nie masz się czym przejmować. Ta aplikacja napisana jest w mojej skali 9/10 . Jakiś junior Cię oceniał - ciesz się, że nie trafiłeś pracować pod nim bo tylko byś #!$%@?ł pod nosem na jego pomysły i byś się cofał w rozwoju. Na szybko:

Na minus:
- lombok zamiast użyć nowszej javy albo kotlina
- mało czytelny i legacy maven zamiast gradle
- mapowanie branch model <-> branch dto i

[nad__czlowiek]

Powiem to tak, od dłuższego czasu przebranżawiam się na programistkę z montażystki video. Przerobiłam JetBrains Academy od Hyperskill, mam tytuł zawodowy inżyniera oraz naukowy (xd) magistra oraz realizuje swoje poboczne projekty.

No i fajnie, ale moja córka skończyła prawo, a ja mam doświadczenie komercyjne ( ͡°( ͡° ͜ʖ( ͡° ͜ʖ ͡°)ʖ ͡°) ͡°):

1. Zacznijmy od: GET /api/repos/{username}/fork=false?pageSize=&pageNumber=

[rtp_diov]

Coś takiego jest mocno nie efektywne nie powinno mieszać się @PathVariable oraz @RequestParam

@kwas_rybonukleinowy: Dlaczego?

Należy pamiętać, że ObjectMapper może prowadzić do niepożądanych zachowań, takich jak ignorowanie nieznanych właściwości, co może prowadzić do utraty danych. Jednak to są skrajne sytuacje, alternatywą jest standard JSON-B (pozdro dinozaury) lub Gson / Moshi

@kwas_rybonukleinowy: wiesz, że używając automatów (po prostu klasę) Spring aby to zmapować pod spodem domyślnie używa object mappera? To co

[NaChuiMiTaCHata]

naukowy (xd) magistra

@kwas_rybonukleinowy: Zejdź na ziemię, magister to również tytuł zawodowy.

[pielgrzym_wladca_egiptu]

@Nofenak: Dlatego najlepsze sa rekrutacje do big tech, klepiesz 3 algosy i gitara, nigdzie nie spotkalem takich kozakow jak w big techu mimo tego ze nikt nie kazal im klepac jakies gowno appki na rozmowie. Za to najwiekszych frustratow spotykalem wlasnie w firmach gdzie na rozmowie kaza ci znac framework na pamiec czy komendy wiersza polecen xd, albo wymieniac pattern designy z pamieci

[coll]

@Nofenak: Co do punktu 1 to dorzucę swoje trzy grosze, bo często sam spotykałem się z nadinterpretacją zadania. Jeśli jest podany jakiś kontrakt, czy sformułowane oczekiwane zapytanie i odpowiedź, to jeśli zmienisz któreś z nich, to może to być uznane jako trochę nieumiejętność zrozumienia zadania. Inną rzeczą byłoby, gdyby przykładowo nie było nic wspomniane o logach, a Ty byś takie logowanie dodał. Warto podczas robienia zadań rekrutacyjnych patrzeć też na tą

[M.....i]

Dzieki ci Panie ze nie pracuje w polskiej gowno firmie z takimi typami jak ten „rekruter” xd

[seniorwykopek]

@Nofenak na juniora to powinni się zapytać co to github i jak się do kompa zalogować a nie pretensje... szkoda czasu na takich ludzi