Discussion :

[PigKey6]

Bonjour à tous,

J'ai un problème dans mon application avec les passages par valeur d'objets de classe. À l'aide du déboggueur j'ai vu qu'en fait ma fonction modifiait le paramètre de l'appelant :

Voici mon code:

Code :

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

1
2
3
4
 
//jtir est une variable de type Scenario_joueurs[]
Fonction(jtir);
//jtir a pris les valeurs de jpen

Sachant que la fonction commence par :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
Fonction(Scenario_joueur[] jtir)
{
Scenario_joueur[] jpen = new Scenario_joueur[6];
                int y = 0;
                while (y < 6)
                {
                    jpen[y] = jtir[y];
                    y++;
                }
}

J'ai rajouté la variable jpen pour essayer d'éviter le problème mais ça ne change rien, ça doit etre un problème de c# que j'ai du zapper.
Donc dans ce cas la variable jpen est créé et je met dedans la valeur de jtir.

Dans ma fonction, jpen est modifié et se remplit avec des valeurs qui sont justes. cependant la portée de la variable jpen semble poser problème : à la fin de ma fonction tout s'est bien passé sauf que dans la fonction appelante la variable jtir contient désormais les memes valeurs que jpen alors que je ne l'ai jamais modifié.

En clair le code se comporte comme si j'avais fait un passage par référence pourtant je ne pense pas l'avoir fait.

Peut etre que qu'il faut faire une surcharge de l'opérateur '=' pour ma classe histoire qu'il copie chaque valeur et non l'adresse ?

J'espère avoir exposé clairement mon problème.

Si vous pouviez m'aider.

Merci d'avance,

[DonQuiche]

Bonsoir. En somme, tu passes un tableau de références à une fonction et tu aurais voulu pouvoir modifier ces références au sein de Fonction sans que ces changements ne soient répercutés en-dehors ?

Ceci n'est pas possible. Tu dois soit modifier le type de Scenario_joueur pour en faire un type valeur (struct) soit, d'une façon ou d'une autre, créer des clones de ces références et modifier ces clones uniquement (par exemple en surchargeant la méthode Clone() ou créant des constructeurs de copie).

Toutefois, l'obligation de faire tout cela peut indiquer un mauvais design (pas forcément). Il serait bon que tu prennes un peu de recul, que tu fasses le point sur les responsabilités et le rôle de la classe Scenario_Joueur et les traitements opérés par Fonction afin de t'assurer que le design est correct.

PS : En dupliquant le tableau, tu as constaté que les éléments contenus dans la copie sont toujours les mêmes que dans l'original. Le seul intérêt de dupliquer le tableau serait de pouvoir modifier cette copie (ordre des éléments, ajout ou retrait d'éléments) sans changer le tableau d'origine.
PPS : L'opérateur d'assignation n'est pas surchargeable.

[PigKey6]

Oui c'est exactement ça.

Et si dans Scenario_joueur je met un constructeur de copie qui prend en paramètres l'autre et assigne toutes les valeurs une par une ?

[DonQuiche]

Ça semble effectivement être la meilleure des solutions.
Il suffira simplement de bien explicitement appeler le constructeur de copie.

[plume13]

A noter que définir Scenario_joueur comme structure (struct) au lieu de classe (class) peut résoudre ce problème, mais l'utilisation de structures doit se limiter à quelques cas bien précis pour éviter de "nuire aux performances".
Plus d'infos ici : http://msdn.microsoft.com/fr-fr/library/ms229017.aspx