Discussion :

[Tonton Nico]

Bonjour tout le monde, ici tonton Nico (fraichement arrivé sur le site)

Je me suis mis depuis peu à Unity et j'ai une question d'ordre générale sur les bonnes pratiques (excuser moi si je ne suis pas dans la bonne section au passage, ou du moins pas la plus optimale).

Je fais un pti jeu 2D Idle (pour ceux qui connaissent) mais la question est bonne pour tous les projets par la suite; J'ai un "splash" screen avec un ControlleuJeu en mode Dontdestroyonload pour garder donc mes variables diverses en vie entre les menus de jeu/option/arbre de compétenses.
Sur ce Controlleurjeu j'ai mes variables dans un script en C# avec l'utilisation de public class pour ranger par catégorie mes variables, en vulgarisant/simplifiant actuellement je fais:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
[System.Serializable]
public class Level
{
    public int LevelSkill1;
    public int LevelSkill2;
 
}
[System.Serializable]
public class Cout
{
    public int CoutlSkill1;
    public int CoutlSkill1;
}
 
public class ControlleurJeu : MonoBehaviour {
    Code du jeu
}

Je trie par "catégorie" soit le level, le cout ... on peut imaginer par la suite du jeu forcément la VitesseDeDéplacement ou la DureeActivation, etc

La question bête est donc sur la bonne pratique (on y arrive enfin ) : vaut-il mieux faire dans ce sens la donc comme dans mon exemple/projet:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
Class Categorie 1                                    
     public int Skill1;                                      
     public int Skill2;    
 
Class Categorie 2
    public int Skill1;
    public int Skill2; 
 
etc
etc
etc

ou au contraire

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
Classe Skill1                                                     
     public int Categorie1;                            
     public int Categorie2;                            
 
Classe Skill2
   public int Categorie1;
   public int Categorie2;
 
etc
etc
etc

Comment faites vous en général? voir tout déclarer à la suite dans la class du ControlleurJeu sans utiliser de [System.Serializable]?
Il est vrai que si j'ai 30 skill et 2 catégorie, dans la première méthode cela fait que 2 class à déclarer et dans la deuxième 30 :/ ou c'est selon chaque programmeur et ses habitudes?


Remarque1: Désolé d'avance si c'est trivial car j'ai fais la prog en cours (il y a quelque années du coup) mais ce n'est pas du tout mon cursus principal
Question bête numéro 1 : J'utilise le [System.Serializable] (vu dans un tuto) pour une question de visualisation/ordre, c'est bien ou cela peut amener des soucis dans l'utilisation et il ne faut pas le faire? il y a mieux?
Question bête numéro 2 : le nombre de public class/variable par script est illimité j'imagine non? il vaut mieux quand même se limiter et faire plusieurs script quand on a trop de variable? ou ça sert a rien?

J'espère avoir été assez clair et merci pour les courageux d'avoir lus jusqu'ici

Merci d'avance pour ceux qui aurait envie de donner des conseils/avis
Tonton

[Bousk]

Salut,

pas de bonne ou mauvaise solution absolue selon moi.
Il faut que ça ait du sens dans ton cas.
Quelle est la hiérarchie entre skill et catégorie ? Est-ce que tu as une catégorie pour ranger les skills, ou chaque skill a juste une info de catégorie.
De même, si tu as 30 skills, tu ne créeras pas 30 class. Si tu es malin, tu as une class de base qui contient les infos pour paramétrer tous les skills. Eventuellement plusieurs mais tu devrais toujours avoir une class de base puisque ce genre de trucs se rangera généralement dans des collections.
Si tu as 3 skills boule de feu mineur, boule de feu majeur, boule de feu surpuissante, y'a fort à parier que les seules différences seront visuelles et un paramètre de puissance.

[Tonton Nico]

Merci pour ta réponse Bousk

Je vois, c'est surtout être cohérent dans le projet en cours ou selon ses habitudes de travail...et surtout être malin au début pour en faire le moins plus tard (mais je me rend compte que je n'ai pas assez réfléchis en amont mon algorithme/ce que je veux et que du coup je modifie mon architecture/code et (forcément) ce n'est bien du tout bouh le vilain tonton

Par contre pour les collections (et les 30 skills) j'avoue ne pas avoir suivis/comprendre; j'ai vu une utilisation de celles-ci rapidement quand on fait un foreach et c'est tout pour le moment, je dois rater quelque chose d'utile surement, je vais me renseigner dessus du coup

Si j'ai 3 skills de boules avec juste la puissances différentes je vois yes ou sinon il faut créer des variables spécifiques en effet, il faut bien que je fasse la distinction entre ce que je veux comme valeur générale ou spécifique, il y a surement des variables que je n'ai pas besoin de garder comme ça entre deux scènes en y réfléchissant mieux, ce qui optimisera la mémoire au passage (pour si peu ^^)


Tu m'as donné matière à réfléchir en tout cas, merci


edit: je passe en résolu