Jutro mam batalie na mid jave + angular. Beda pytania i szmuglowanie ale tez bedzie wyswietlony kawalek kodu i trzeba bedzie wylapac bledy. W jakis sposob chcialbym sie przygotowac i tutaj pytanie do starszyzny. Na co moglbym zwrocic uwage przy takim kawalku? Tak od razu zebym mogl gadac i gadac, jakies porady? Tipy? Z gory dzieki:)
- Wykorzystać wiedzę nabytą do tej pory
- Zwrócić uwagę czy składnia jest poprawna
- Zwrócić uwagę czy kod się wykona poprawnie
- Zwrócić uwagę czy są miejsca gdzie kod nie wykona się poprawnie i rzuci np wyjątek
- Zwrócić uwagę na to czy kod można napisać lepiej i czy warto go optymalizować
- Być w stanie stwierdzić co ten kod robi / w jakiej warstwie aplikacji będzie się znajdować.
- W teorii można się uczepić code-style, ale to kwestia preferencji autora/reviewera/konwencji w projekcie/firmie
Czy sam byś nie poradził sobie z tym, gdybym Ci nie napisał tej listy? Jakich tip'ów oczekujesz?
Zbesztaj od razu cały kod i powiedz że (nie ma opcji że nie będzie następujących przesłanek):
- nie jest thread-safe
- nie jest wydajny
- nie ma logów / audytów
- nie obsługuje edge-casów
- prawdopodobnie nie jest ładny i czytelny
stanley123 napisał(a):
Jutro mam batalie na mid jave + angular. Beda pytania i szmuglowanie ale tez bedzie wyswietlony kawalek kodu i trzeba bedzie wylapac bledy. W jakis sposob chcialbym sie przygotowac i tutaj pytanie do starszyzny. Na co moglbym zwrocic uwage przy takim kawalku? Tak od razu zebym mogl gadac i gadac, jakies porady? Tipy? Z gory dzieki:)
Możesz głośno myśleć o tym, co widzisz, nawet jeśli wydaje ci się to błahe. Przynajmniej ja tak mam, że jak coś widzę i nie wiem, czy to ważne, to mnie to rozprasza. A jak powiem, to już mam poczucie, że zrobiłem acknowledgement i że mogę iść dalej.
Poza tym czasami faktycznie nie wiesz, czego ktoś oczekuje i gadanie czegokolwiek pozwala wybadać reakcję drugiej strony i ten ktoś drugi może cię naprowadzić na to, co powinieneś powiedzieć, żeby tamten był zadowolony.
Możesz trochę poćwiczyć przed spotkaniem z Chat GPT. Całkiem dobre robi CRki. Jest też fajny plugin do Intellij, do statycznej analizy kodu - SonarLint. Powodzenia!
I jak poszło? Co było?