JE Transforme TON Code React en Clean Code (Review)

Рет қаралды 7,499

Melvynx • Apprendre à coder

Күн бұрын

Пікірлер: 31

@M8V_lv 11 ай бұрын

comme dab chapeau t le roi du code

@melvynxdev 11 ай бұрын

merci !

@matthieudantes4129 11 ай бұрын

vraiment top ce format, pas mal de youtubeurs US le font, si tu peux répéter l'opération ça serait appréciable!! Avec un exemple sur pk utiliser un customHook serait optimal. Merci

@melvynxdev 11 ай бұрын

oui faut que j'en faisse plus

@axelpo9390 11 ай бұрын

Pas mal le nouveau concept, sa permet de voir où on peut faire des erreurs et comment les corriger

@melvynxdev 11 ай бұрын

merci !

@jeremymaquaire962 6 ай бұрын

Salut, j'aimerais une review. Est ce possible? Mais pas sur KZbin

@melvynxdev 6 ай бұрын

ah bah non sauf si tu es membre de mes formations

@fromagetriste 11 ай бұрын

tu codes depuis combien de temps sur react ? t'as l'air à l'aise. Moi je viens de Python Django, je pète des câbles avec JavaScript et le front end, j'en suis au stade où ça me gave mais on s'accroche

@melvynxdev 11 ай бұрын

ça fait 4-5 ans

@TheBlackManMythLegend 11 ай бұрын

on pige beaucoup mieux par l'erreur parce que en presentant le truc bien parfois ne comprends pas POURQUOI on fait comme ceci et pas comme cela.

@melvynxdev 11 ай бұрын

yeah trop bien !

@xXLeGrandStanXx 11 ай бұрын

0:40 Je pense surtout que ça devrait être sort côté serveur et pas côté client, histoire que ce soit généré statiquement (et regénéré toutes les heures, par exemple) avec les datas déjà triées ? Tout dépend de si ce sont des données qui bougent beaucoup ou non et de s'il les met en cache ou pas. 4:10 À mon sens, ce n'est pas un composant mais un générateur qui sert à renvoyer ou bien un ReactNode (ou bien un ReactNode[]). Personnellement ça ne me choque pas, et je trouve ça bien plus déroutant de le convertir en composant. Si tu regardes la définition de ce qu'est ComponentType, ce n'est pas tout à fait la même chose sémantiquement. De plus, je ne vois absolument pas l'intérêt, si ce n'est que ça ajoute de la lourdeur. 5:50 Alors, oui, mais ça implique aussi de devoir manipuler intelligemment prevState. (Pour en savoir plus : "All 12 useState & useEffect Mistakes Junior React Developers Still Make in 2023") 6:35 Alors à vrai dire, c'est une question qui est propre aux standards web. Une pression de la touche espace ou enter sur un provoquera aussi un clic. Mais sur un , ce sera seulement la touche enter qui provoquera aussi un clic. Pour le reste, en revanche, je suis plutôt d'accord dans l'ensemble. Mais j'ai pas trouvé ça super agréable, honnêtement... Mais bon, une prochaine fois, peut-être.

@melvynxdev 11 ай бұрын

Tu as pas trouvé ça agréable ???

@xXLeGrandStanXx 11 ай бұрын

@@melvynxdev Oé, déso, je t'ai trouvé un peu trop dur et un peu trop catégorique.

@arnaudgutierrez4231 11 ай бұрын

Salut Sympa le concept. Tu fais du terraform et du ansible aussi ?

@melvynxdev 11 ай бұрын

Hello, merci et non !

@arnaudgutierrez4231 11 ай бұрын

@@melvynxdev ok. Dommage ☺️ En même temps tu peux pas tout couvrir ☺️

@bastienv6233 11 ай бұрын

Super concept

@melvynxdev 11 ай бұрын

merci !

@rawz06 11 ай бұрын

Très bon code review, allez j'en rajoute deux 3:39 Quand vous écrivez ça, vous devez avoir un WARNING ALERTE Lorsque vous avez énormément de propriété comme ça - On créé un type ProductCardProps ou on met tout ça - On ne fait pas de destructuration, on fait des props.name, props.category. Tu vas te retrouver avec des tonnes et des tonnes de variables. Petite amélioration également, évite les imgSrc, imgAlt, imgWidth mais non non non. Mets moi ça dans un type ProductCardImageProps. export type ProductCardImageProps { src: string; alt: string; width: number; height: number; } export type ProductCardProps { name: string; category: string; price: string; image: ProductCardImageProps; href: string; //etc } export function ProductCard(props: ProductCardProps) { //code here } 8:52 Alors c'est pas une review mais Disabled sans le d j'ai l'impression de lire « j'ai manger » 😆

@renaud689 11 ай бұрын

d'accord avec toi, sauf sur la destructuration (que tu apelle destruction 😂) , car ecrire props.quelquechose a chaque fois c'est chiant

@rawz06 11 ай бұрын

@@renaud689 Merci je corrige parfois je m'emmêle les pinceaux. Pour les props on destructure oui mais quand il y a un, deux trois variables max, quand il y en a 15, on le fait pas sinon ça devient un foutoir.

@melvynxdev 11 ай бұрын

ça dépends !

@smithjohn2367 10 ай бұрын

Personnellement j'aurai utiliser le pattern Compound pour ce genre de composant et comme ça plus aucunes props :=)