Rozpoczęcie przygody jako junior

0

Cześć właśnie kończę staż i zastanawiam się nad zmianą firmy moje pytanie brzmi czy w ogóle z takim githubem i umiejętnościami tam zaprezentowanymi jest to realne?
https://github.com/Ganit20

4

fajnie by bylo jakbys dopisal README do projektow. Dajmy na to MailClient - sam zaimplementowales protokoly? uzyles gotowych bibliotek? nie ma README, nie chce mi sie w kodzie doczytywac jak to dokladnie dziala

1

Idź na rozmowę to się dowiesz.

1

Dlaczego chcesz zmienić firmę?
Na juniora po tym kodzie bym Cię nie przyjął

1

Tak na szybko. Za dużo logiki w kontrolerach

1

Po stażu powinieneś wiedzieć na jakie umiejętności jest zapotrzebowanie w prawdziwej pracy.

5

Gdybym jako rekruter zobaczył ten kod to raczej poszło to by na Twoją niekorzyść, ponieważ jest to po prostu niechlujne. Nazwy klas raz dużą, raz małą literą, to samo właściwości. Nazwy commitów typu 'ok', metody na 200 linii, gdzie można to było ładnie podzielić. Poza tym brak spójności w formatowaniu kodu. Niby szczegóły, ale jeżeli traktujesz to jako wizytówkę to bym uważał i nie chwalił się się tym aż tak bardzo :) Chyba, że realia teraz takie, że bez kodu nie zaproszą Cię na rozmowę na juniora.

0

Czysty kod Robert C. Martin
Cracking the coding interview Gayle Laakmann McDowell

Przeczytaj to, poćwicz algorytmy i na wakacje wysyłaj CV na juniora.

Jak piszesz kod to staraj się robić komentarze/dokumentację.

0
shitlord napisał(a):

Cracking the coding interview Gayle Laakmann McDowell

Cracking the coding interview? Niby po co?
Kolega, który pyta czy nadaje się na Juniora raczej nie zamierza uderzać na razie do FAANG.

0
boska_cebula napisał(a):

Dlaczego chcesz zmienić firmę?
Na juniora po tym kodzie bym Cię nie przyjął

Chyba trochę przesadzasz, na juniora wystarczy w zupełności. Chyba że Twoja firma szuka seniorów za minimalną :D

0
boska_cebula napisał(a):

Na juniora po tym kodzie bym Cię nie przyjął

Original Poster przedstawił się z imienia i nazwiska, więc można sobie zobaczyć kto, gdzie, jakie wymagania, kogo szukają (w Krakowie).
Może zdaniem niektórych OP nie jest wymiataczem-juniorem, może "oni w ich firmie" ;) by go nie przyjęli, ale...
OP ma dobre jak wygląda rynek. W Krakowie student po stażu może szukać nowej firmy i zakładam, że spokojnie znajdzie. ;) Powodzenia.

0

Wszyscy by tylko przygody rozpoczynali, a uczyć się nie ma komu.

4
  • commity to mają świetne nazwy: "ok", "UWP-Client", "bugs", "end"; ale już w SpaceInvaders jest lepiej,
  • mieszasz też odpowiedzialność - klasa Login odpowiada za rzeczy związane z połączeniem, jak również wyświetla pewne komunikaty u użytkownika,
  • nazewnictwo metod, zmiennych, klas i parametrów jest takie sobie i czasami niezgodne z konwencjami C# (wielkie litery/małe litery); do tego jest mnóstwo literówek w wielu miejscach,
  • kod jest też niechlujny - jakieś nadmiarowe spacje, brak formatowania - są automaty do tego, patrz np. CodeMaid,
  • co to w ogóle jest? https://github.com/Ganit20/CarCostNotepad/blob/master/ViewModel/SaveSystem.cs#L65,
  • w InternetBrowser w katalogu ViewModels jest sobie UrlOperations.cs... które w ogóle nie jest ViewModelem, a namespace ma podany jako Player.View.Card. Jaki Player? Jaki Card?
  • a nawet więcej - masz bardzo wiele ViewModeli, które absolutnie nimi nie są.

1 użytkowników online, w tym zalogowanych: 0, gości: 1