• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

code review react rest api opinia krytyka

42 Warsaw Coding Academy
+1 głos
201 wizyt
pytanie zadane 20 lipca 2020 w JavaScript przez rob Bywalec (2,440 p.)

Proszę o code review ,

live

https://robkot500.github.io/Weather/build/

code

https://github.com/robkot500/Weather

każda opinia, rada, krytyka mile widziana

Co powinienem poprawić, zmienić, 

1 odpowiedź

0 głosów
odpowiedź 20 lipca 2020 przez Paweł Nąckiewicz Nałogowiec (48,990 p.)
wybrane 20 lipca 2020 przez rob
 
Najlepsza

1. Po pierwsze formatowanie kodu leży w całym projekcie.

2. zmienne w css -> header-wrapperA nic nie mowi. Poczytaj o BEM, możesz ustawić coś ala header-wrapper--red

3. w zagnieżdżaniu scss nie stosuj więcej niż 3 poziomów

4. Card.js 

    - dlaczego masz na starcie if'a którego powtarzasz 2 razy?
    - description = description.charAt(0).toUpperCase()... użyj zamiast tego templateString

    - data.list[index] - powtarzasz kilka razy = wywal to do zmiennej

    - ten switch case... cos okropnego - wywal to do jakiegos pilku z serwisem

    - srcI - zmien to na jakas sensowna nazwe srcImage lub imgPath

    - <br/> nie warto juz tego uzywac, zwlaszcza do robienia mrginow 

Footer.js - wywal go jak jest pusty

Header.js 
    - zrób plik .env i w nim trzymaj key
    - nie pisz ciała funkcji w JSX - w JSX odwoluj się tylko do funkcji po referencji.

<form onSubmit={e => {e.preventDefault();setChangePlace(() => e.timeStamp)}

Main.js 

   - po co ?

if(data == null)

New.js

   - wywal console.logi

   - to mozesz uprościć  

 if(width===false){
        // console.log("DESKTOP")
        // console.log(width)
        return (
                <>
            <h1>DESKTOP</h1>
</>

        )
    }
    if(width===true){
        // console.log("MOBILE")
        // console.log(width)
        return (
            <h1>MOBILE</h1>
        )
    }

Section.js - poczytaj o funkcji padStart

 

Generalnie masz spore braki w JavaScripcie, polecam najpierw jego się dobrze nauczyć a potem brać się za framework. 

 

komentarz 20 lipca 2020 przez rob Bywalec (2,440 p.)
dzięki za krytykę :), co znaczy?

wywal to do jakiegos pilku z serwisem
komentarz 20 lipca 2020 przez Paweł Nąckiewicz Nałogowiec (48,990 p.)
To review nie krytyka. Chciałeś rzeczy do poprawy więc wypisałem.

Zrób sobie jakiś plik .js, nazwij go WeaterImageSerive.js i niech on ma metode do zwracania odpowiedniego obrazka

Podobne pytania

0 głosów
1 odpowiedź 294 wizyt
pytanie zadane 26 lipca 2020 w JavaScript przez rob Bywalec (2,440 p.)
0 głosów
0 odpowiedzi 195 wizyt
0 głosów
0 odpowiedzi 655 wizyt
pytanie zadane 7 września 2021 w JavaScript przez rszczepanski02 Obywatel (1,180 p.)

93,383 zapytań

142,382 odpowiedzi

322,539 komentarzy

62,744 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj

VMware Cloud PRO - przenieś swoją infrastrukturę IT do chmury
...