Code Review projektów początkujących! Szybsza nauka programowania
Комедия
👉 Zapisz się na newsletter: zaczynamyprogramowac.pl 👈
🎤 Discord: / discord 🎤
🔥 Zapisz się do grupy na FB: / zaczynamy.programowac 🔥
Liczby zmiennoprzecinkowe - www.samouczekprogramisty.pl/l....
Film od delgacji zdarzeń / event delegation:
• Event Delegation - pod...
Jakiś czas temu założyliśmy kanał na KZread i kilka osób też odzywało się do nas z podobnymi prośbami, dlatego chcieliśmy połączyć jedno z drugim i nagrać Code Review w formie video. Dzięki temu możemy lepiej przekazać ewentualne uwagi, osoby które się zgłosiły więcej się uczą, a inne osoby początkujący, które oglądają, również mają okazję dowiedzieć się czegoś ciekawego z takiego review.
Inicjatywa jest raczej świeża, także kilka rzeczy wymaga doszlifowania, ale mam nadzieję, że mimo tego review w takiej formie niesie ze sobą dodatnią wartość :)
Пікірлер: 27
Cześć. Dzięki za przedstawienie i omówienie mojego projektu (BookIn) w CR. Pozdrawiam serdecznie.
Mega są te odcinki! Prosimy o więcej! Dziękuje za to co robicie 😊
@JZP
Жыл бұрын
Będzie!
Code Review to świetne narzędzie! Wsparte jeszcze odpowiednimi zasadami w narzędziu do pisania kodu + automaty do analizy kodu robi świetną robotę!
Brawo :-)
Cześć, dzięki za wybór mojego projektu i CR
Co do konwencji niużywania camel case w CSS - sam nie fanem takiej notacji jeżeli chodzi o nazwy klas, ale odkąd używam css modules, to ją stosuję, bo inaczej trzeba klasy obudowywać dodatkowo w nawias i cudzysłów, co jest strasznie niewygodne.
10:57 zabezpiecza przed dzieleniem przez zero ale blokuje dzielenie przez np. 0.5, 0.7 itd.
24:12 w tym przypadku na scrolla lepiej by było użyć throttle, bo jakbym scrollował cały czas bez przerwy to ten kolor się nie zmieni w nawigacji
Dobrze jest pisać komentarze lecz w przypadku komentarzy możemy zmienić kod a zapomnieć o zmianie komentarza, ale jeszcze lepiej jest dawać nazwy zmiennych funkcji itp tak żeby były jasno opisane co robi dana funkcja to będzie jeszcze lepsza praktyka ;)
@00unn
Жыл бұрын
słuszna uwaga, aby zagłębić temat to warto np. przeczytać książkę "Czysty Kod" R. Martina
@JZP
Жыл бұрын
Zdecydowanie! Polecam każdemu
Cześć! Cieszę się, że gra saper przypadła Ci do gustu i dziękuje za CR. Wspomniałeś o tym żebym rozdzielił kod na dwa "poziomy abstrakcji". Miałem taki pomysł ale nie zrobiłem tego z premedytacją. Kod napisałem w taki sposób aby po każdym kliknięciu w jakieś pole plansza nie renderowała się na nowo tylko aby zmiany zachodziły tylko i wyłącznie w obrębie jednego pola. Według mnie sprawia to, że kod działa wydajniej - chociaż pewnie przy tak małej aplikacji nie byłby widoczny w jakikolwiek sposób spadek tej wydajności.
@JZP
Жыл бұрын
To dobre podejście, niekoniecznie ze względu na wydajność (faktycznie nie ma to znaczenia w przypadku sapera) ale na prostotę rozwiązania. Nie zmienia to faktu, że da się wydzielić poziomy abstrakcji i zachować Twój sposób renderowania ;) Zachęcam Cię do spróbowania
Hej, czy użycie evala w kalkulatorze nie jest przypadkiem niebezpieczną praktyką?
@JZP
Жыл бұрын
Zależy. Użycie evala jest niebezpieczne gdy nie kontrolujesz danych wejściowych i mógłbyś wykonywać jakis kod trwałe
Co do projektu z 35:00 w react. ApiKey nie trzymamy nigdy na githubie nawet jako env. Jak wejdziemy na Live i przykładowo chcemy się zalogować niestety wycieka twój ApiKey do firebase. W kodzie również jest go łatwo znaleźć.
nie ma linka w opisie do delegacji zdarzen
@JZP
Жыл бұрын
Dodane, dzięki!
52:05 Stefan Batory myslalem ze ten król polski xDD
Ten kalkulator wygląda identycznie do tego: kzread.info/dash/bejne/f5WKm7iqdrDgcpc.html
@birdobird_
Жыл бұрын
Faktycznie, aczkolwiek szczerze powiedziawszy nawet nie widziałem tego wideo i się nim nie sugerowałem, ale sądzę, że jest parę podobnych :)
@fuukowatty9817
Жыл бұрын
@@birdobird_ nie jest identyczny wiec jezeli sie ktos wzorowal to moze podobnie wygladac, tak mysle.
Kiedy Code Review?
muzyczka jak u @Dave2d zawsze pod koniec leci 😃
sporo tych projektów pachnie copy-paste z tutoriali