JE Transforme TON Code React en Clean Code (Review)

Поделиться
HTML-код
  • Опубликовано: 7 янв 2025

Комментарии • 29

  • @M8V_lv
    @M8V_lv Год назад +4

    comme dab chapeau t le roi du code

  • @axelpo9390
    @axelpo9390 Год назад +3

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

  • @matthieudantes4129
    @matthieudantes4129 Год назад +3

    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
      @melvynxdev  Год назад

      oui faut que j'en faisse plus

  • @bastienv6233
    @bastienv6233 Год назад

    Super concept

  • @xXLeGrandStanXx
    @xXLeGrandStanXx Год назад

    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
      @melvynxdev  Год назад

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

    • @xXLeGrandStanXx
      @xXLeGrandStanXx Год назад +1

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

  • @rawz06
    @rawz06 Год назад +1

    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
      @renaud689 Год назад

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

    • @rawz06
      @rawz06 Год назад

      @@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
      @melvynxdev  Год назад +1

      ça dépends !

    • @smithjohn2367
      @smithjohn2367 Год назад

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

  • @renaud689
    @renaud689 Год назад

    Excellent ce format

  • @arnaudgutierrez4231
    @arnaudgutierrez4231 Год назад

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

    • @melvynxdev
      @melvynxdev  Год назад +1

      Hello, merci et non !

    • @arnaudgutierrez4231
      @arnaudgutierrez4231 Год назад

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

  • @ibraboo
    @ibraboo Год назад

    Il est bien ce format

  • @fromagetriste
    @fromagetriste Год назад

    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

  • @jeremymaquaire962
    @jeremymaquaire962 9 месяцев назад

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

    • @melvynxdev
      @melvynxdev  9 месяцев назад

      ah bah non sauf si tu es membre de mes formations

  • @itscooldawgdonteventrip
    @itscooldawgdonteventrip Год назад

    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.