Discussion :

[guitz]

Bonjour,



Mon projet est un city builder sur Unreal Engine 5 (c++ et blueprints)

Les rectangles rouges sont des classes c++. Les blancs sont des Blueprints. Les flèches rouges sont #include . Les flèches bleues sont l'héritage (classe c++ -> Blueprint). Les flèches noires sont des références de classes de Blueprint.

MyNodeClass, MySegmentClass et InfrastructureGraphManagerClass sont utilisés pour définir et mettre à jour le graphe du réseau routier.

Par exemple :

Code :

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

AMySegmentClass* AInfrastructureGraphManager::ConnectNodes(AMyNodeClass * Nod1, AMyNodeClass * Nod2, SegmentType SegType, bool IsCurv)

Code :

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

FSubdividedSegment AInfrastructureGraphManager::SubdivideSegment(AMySegmentClass * Segment, FVector NewNodeLocation)

DrawPreviewInfrastructure_BP permet de prévisualiser le réseau routier avant de le valider.
Par exemple :

Code :

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

1
2
3
4
DrawStraightSegment()
DrawTangentSegment()
DrawTangentSegmentsAndCurve()
SnapMouseOnExistingNode()

DrawInfrastructure_BP permet de dessiner le réseau routier après validation de l'aperçu.
Par exemple :

Code :

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

1
2
3
4
DrawSegmentAndTwoNodes()
DrawSegmentAndTwoNodesFromColinear()
SplitStraightSegmentForCrossroads()
ConvertTurnNodeToCrossroads()

Spline_BP contient des composants spline et un procedural mesh (qui est utilisé pour les jonctions procédurales). Chaque instance de Spline_BP est stockée dans le tableau SplinesArray à l'intérieur de DrawInfrastructure_BP. Chaque index de ce tableau est stocké dans la variable MySegmentClass->SplinesArrayIndex.

Par exemple :

Code :

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

1
2
AttachMeshes()
UpdateSplineComponents()

ZonesBlock_BP est utilisé pour stocker les données des zones du bâtiment. L'une de ses variables est un tableau 2D de structures Zones.

Par exemple :

Code :

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

1
2
3
4
Update()
Draw()
Erase()
SetConvexHull()

ZonesManager_BP permet de gérer les ZonesBlock_BP en fonction du réseau routier.

Par exemple :

Code :

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

1
2
3
4
SpawnZBsAroundSubSpline()
SetZonesPoints()
UpdateZBsConvexHull()
SpawnZBsAroundSegment()

Comme vous le savez peut-être, CityBuilderPlayerController_BP est utilisé pour appeler les fonctions que nous voulons en fonction des événements de souris/clavier.

Est-il nécessaire s'il vous plaît d'optimiser ce diagramme? Et si oui, comment ?

[LittleWhite]

Bonjour,

À première vue, je dirai que le graphe à trop d'interdépendance (trop de flèche). Mais, je ne suis pas certain que ce soit une bonne façon de procéder pour donner un jugement.
En développement, on donne souvent la règle de responsabilité unique. Si chaque classe/blueprint, fait une chose, alors c'est bon signe. Et on pourra regarder du côté de SOLID. Mais...
revenons sur la réalité. Vous ne faites pas de la programmation "classique", vous faites de la programmation de jeux vidéo, dans Unreal Engine, avec les blueprints. Par conséquent, je partirai sur une autre réflexion :

• remarquez vous des problèmes de performances si vous affichez 100, 1000 ou 10000 objets utilisant votre code ;
• avez vous des difficultés à avancer dans votre jeu, car le code n'est pas facile à utiliser, ou pas facile à mettre à jour.

C'est en répondant à ces deux questions que vous pourrez savoir si cela est nécessaire d'améliorer le code, ou si vous pouvez le laisser tel quel et continuez à programmer votre jeu.

Car en réalité, si on passe son temps à faire un code tip-top, en respectant tous les merveilleux principes que l'on peut trouver sur Internet, alors le jeu ne se fait jamais.

[guitz]

Merci LittleWhite,

En effet il faut trouver un compromis entre le respect des bonnes pratiques (conventions de nommage, diagramme de classes etc) et productivité