code review react rest api opinia krytyka

Question

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ć, 

Answer 1

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. 

 

Comments

dzięki za krytykę :), co znaczy?

wywal to do jakiegos pilku z serwisem

---

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