Discussion :

[yyshtar]

Bonjour,

récemment je me suis remis au dev web, j'étais sur un jeu-vidéo en 2019, et j'ai essayé de faire un projet propre, le projet est assez court (principalement un moteur de blog), construit avec Angular 8 et express4.

Je manque cruellement de retour sur mon code et ça m'aiderai vraiment beaucoup de voir tout ce qui va pas afin de présenter un projet propre pour trouver une boite (déjà que je sors d'une formation courte). S'il vous plait si quelqu'un pouvait me dire un peu ce qu'il en est (j'accepte les critiques dures, c'est un peu le but).

voila la partie front: https://github.com/simplon-romainB/ome.
voila la partie back: https://github.com/simplon-romainB/homeserver.



les tests e2e ne sont pas encore terminés.

Cordialement et merci d'avance à la communauté.

[krakatoa]

j'ai regardé vite fait la partie front, le back avec nodejs je ne connais pas trop.


plusieurs points :

- utiliser des services pour alléger le code dans les composants.
à savoir, les composants doivent avoir un minimum de ligne.

donc, pas de code métier dans les composants mais des appels à des services(qui contiennent le code métier)
Le composant doit être facile à lire:
exemple:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
   this.currentArticle = this.sArticle.getCurrentArticle();

on comprends facilement ce que fait cette ligne, si on veut savoir comment.. on va voir dans le service.. de plus, on pourra utiliser getCurrentArticle dans un autre endroit si besoin..

de plus, il faut classer les services par fonctionnalités ou domaines..
un service qui gère les articles (et qui peut faire appel au service http)
un service qui gère les commandes
un service qui fait des requêtes http
....

- créer des fonctionnalités réutilisable et indépendante

une fonctionnalité = un module = un certain nombres de composants

exemple:
une fonctionnalité "panier" = composant container "panier" + composant liste "panier" + composant "une ligne panier" + composant "total" = le tout dans un module "panier"

il faut décomposer au maximum une fonctionnalité en plusieurs composants (qu'on regroupe donc, dans un module et j'insiste sur ce point)
ainsi, le composant "liste panier" aura son propre css et code. de plus, il passera une ligne au composant ligne
le composant "ligne panier" aura son propre css et code

ainsi, si il y a un bug ou que l'on doit faire une modif, c'est plus facile

ainsi dans un autre projet, on importe le module panier et c'est tout, c'est prêt à l'emploi

[yyshtar]

merci beaucoup je vais voir ça, ainsi que mon nombres de variables d'instance trop élevé^^