**Ja bym jeszcze zrobił zmiany w CV. **
Po pierwsze zmień interlinię na coś mniejszego. Dwie strony do pierwszej pracy nie są akceptowalne.
Po drugie non-commercial experience wygląda jakby miało spację przed nim - nie powinno czegoś takiego mieć.
Wypieprz Professionality and responsibility, to jest "praca w młodym i dynamicznym zespole" w wersji CV.
Stylistycznie słowo "satisfactory" jest mocno subiektywne, moim zdaniem lepiej jest dać "some experience" albo coś tego typu.
Wyciep to o uczeniu się Javy od dwóch lat. Nie ma to absolutnie żadnego wpływu na cokolwiek. To samo z czasami tworzenia tych projektów - jak ktoś się spyta to im powiesz.
Skonfiguruj sobie https korzystając z let's encrypta, z bardzo głupich powodów bardziej ufam stronom z zieloną kłódeczką.
Software Engineering
-> software engineering principles
Masz wpisany TypeScript ale nie masz wpisanych żadnych narzędzi typu transkompilator które powinieneś wykorzystywać jeśli go stosujesz.
To basics w toolach odnosi się tylko do mavena, czy też do gita? Trudno to jednoznacznie sparsować. Zresztą to basics w odniesieniu do mavena można wywalić bo więcej raczej po juniorze nie oczekują.
Wywal informację o braku certyfikatów w języku. Nie ma to żadnego znaczenia, i tak nikt ci nie każe na rozmowę certyfikatu przynieść - swoją drogą są ludzie którzy mają CAE a nie są w stanie jednego zdania od początku do końca przeburknąć, więc sprawdza się to i tak ustnie.
W sekcji other pierwszy punkt zaczyna się z małej litery a reszta z wielkich - wygląda nieładnie.
To heavily w projektach wywal. Proszę.
Oszczędź miejsce wywalając www z linków (ciepnij najwyżej 301kę z nie-www).
Jeśli jakiś projekt jest mniej udany niż inne to go po prostu wypieprz ale zostaw na githubie.
Tak jak powiedziano wszędzie - dawaj wszędzie wersje dziadostwa którego użyłeś.
**
Co do samych projektów:**
Jeśli twierdzisz że znasz mavena ale nie wykorzystujesz go w absolutnie żadnym z projektów to raczej jesteś chyba niepoważny. Do tego nie sądzisz mam nadzieję że sobie sam wymyślę jakie springowe jarki mam sobie zaciągnąć jak będę chciał sobie testowo odpalić projekt?
Struktury katalogów w tych projektach są zrąbane. Zmaveń je.
W jakimś pliku znalazłem takie coś:
/*
* To change this license header, choose License Headers in Project Properties.
* To change this template file, choose Tools | Templates
* and open the template in the editor.
*/
W tym samym pliku są nieużywane importy. Nie jestem robotem, ale zauważyłem to od razu. Przejedź ten kod jakimś linterem przynajmniej (np jakiś nie-nazistowski checkstyle)
Wywal informacje o autorze w plikach. Przy normalnych projektach tego się nie daje bo i tak sporo osób dziubie po kodzie.
Użycie XMLowej konfiguracji springa w nowym projekcie w 2017 roku może być trochę kontrowersyjne.
Masz autogenerowane metody w testach które są nieużywane. Zaśmiecają tylko mózg w trakcie przeglądania, usuń je.
Pomijając "no shit sherlock" w autogenerowanym komentarzu to wygląda jakby było robione na odczep:
/**
* Test of getResponse method, of class CrawlersManager.
*/
@Test
public void testGetResponse() {
}
Testy powinny mieć opisaną zamierzoną funkcjonalność - na przykład test testRunCrawler
sprawdza w zasadzie tylko czy coś się nie wypieprzyło, czy parę pól zostało ustawionych i czy znalazło jakiś obrazek. Nie sprawdza jaki to obrazek. Może warto by było na potrzeby testów postawić jakiś lokalny serwer http i tam wrzucić coś i na tej podstawie sprawdzać czy ci nie ściągnął obrazków których nie oczekiwałeś?
https://github.com/Fantazjatyk/CrawlerWebApp/blob/master/test/java/pl/michalszymanski/web/crawler/services/EmailDelivererTest.java
:(
https://github.com/Fantazjatyk/CrawlerWebApp/blob/master/test/java/pl/michalszymanski/web/crawler/services/KeyValidatorTest.java
tutaj nie ma opisu tego co oznacza "valid" czy "invalid". Gdzieś powinieneś opisać (w testach, nie w kodzie) jak wygląda poprawny kod.
https://github.com/Fantazjatyk/CrawlerWebApp/blob/master/test/java/pl/michalszymanski/web/crawler/services/TestClass.java
rozjechane formatowanie, w dodatku może warto to było zrobić jako klasę wewnętrzną testu który tego używa?
https://github.com/Fantazjatyk/CrawlerWebApp/blob/master/test/java/pl/michalszymanski/web/crawler/Conf.java
Nie płacisz za ilość użytych literków. Nazwij tę klasę jakoś lepiej bo później w dużym projekcie trudno do takiego pliku będzie nawigować po nazwie.
CSSy crawlera mają rozwalone formatowanie.
W niektórych formatkach thymeleafa piszesz po polsku, w innych po angielsku. Zdecyduj się albo podepnij i18n.
Zrobię sobie przerwę i będę cisnąć dalej.