carte/pioche C# Collection

Version imprimable

Voir 40 message(s) de cette discussion en une page

Citation:

Envoyé par jopopmk

Alors déjà "tour" c'est pas folichon comme nom, perso ça me parle pas pour un sou.

enfait c'est un objet joueur intermédiaire, c'est une copie du joueur qui joue, je fait les modif et le remet dans ma liste....
comme currentPlayer de ton exemple,
je m'y colle bientot au nom j'essaye mais sur le moment je met ce qui me parle et je me rend compte que ça parle qu'a moi
Citation:
Voilà un exemple un peu plus complet et qui devrait t'aider à mieux comprendre :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 // ma liste de joueurs public List<Player> Players = new List<Player>(); // le joueur qui doit jouer public Player CurrentPlayer = null; // la fonction d'initialisation (en dur, pour toi ça se fera via ton interface) public void InitPlayers() { this.Players.Add(new Player("jean")); this.Players.Add(new Player("charles")); this.Players.Add(new Player("kevin")); this.Players.Add(new Player("andré")); this.Players.Add(new Player("pierre")); // le joueur courant est initialisé sur le premier this.CurrentPlayer = this.Players[0]; } // la fonction qui passe au joueur suivant public void ChangePlayer() { // Players.IndexOf(CurrentPlayer) te renvoie un chiffre entre 0 et 4 (dans mon exemple à 5 joueurs) // on fait donc +1 pour avoir le suivant // l'utilisation du % (modulo) va nous permettre de repasser vers 0 si jamais on obtient Players.Count (qui est égal à 5) CurrentPlayer = Players[(Players.IndexOf(CurrentPlayer) + 1) % Players.Count]; }
ooooh cool le modulo :king:

(Players.IndexOf(CurrentPlayer) + 1)
on est d'accord que CurrentPlayer n'est pas dans la liste, donc il cherche l'index de quelque chose qui est égal à CurrentPlayer ??

12/06/2015, 12h30
jopopmk

Citation:

Envoyé par Hemir

enfait c'est un objet joueur intermédiaire, c'est une copie du joueur qui joue, je fait les modif et le remet dans ma liste....
comme currentPlayer de ton exemple,

D'où mon "problème" avec le nom que tu as choisi (tour) et le choix dans mon exemple (CurrentPlayer).
NB : il ne s'agit pas d'une "copie" mais d'une "référence" (il n'y a pas de duplication en mémoire).

Citation:

Envoyé par Hemir

ooooh cool le modulo :king:

Dans sa définition c'est : le reste de la division euclidienne.

Citation:

Envoyé par Hemir

(Players.IndexOf(CurrentPlayer) + 1)
on est d'accord que CurrentPlayer n'est pas dans la liste, donc il cherche l'index de quelque chose qui est égal à CurrentPlayer ??

CurrentPlayer est dans la liste : il est initialisé sur le premier joueur dans la fonction InitPlayers.
12/06/2015, 13h22
Hemir

Citation:

Envoyé par jopopmk

D'où mon "problème" avec le nom que tu as choisi (tour) et le choix dans mon exemple (CurrentPlayer).
NB : il ne s'agit pas d'une "copie" mais d'une "référence" (il n'y a pas de duplication en mémoire).

abus de langage, c'est noté.

Citation:

CurrentPlayer est dans la liste : il est initialisé sur le premier joueur dans la fonction InitPlayers.

pour moi j'ai pas fait

Code:

this.Players.Add(CurrentPlayer);

donc il n'est pas dans la liste, ou bien il y a une notion qui m'echappe...
12/06/2015, 14h45
ebastien

Oui, manifestement il y a une notion qui t'échappe :

Code:

this.CurrentPlayer = this.Players[0];

CurrentPlayer pointe sur le premier élément de la liste. Donc il appartient bien à la liste. Ce n'est pas une copie mais un pointeur sur un objet.
Ce n'est pas comme en C++ où là tu aurais effectivement une copie en mémoire de l'objet.
12/06/2015, 15h01
Hemir

Citation:

NB : il ne s'agit pas d'une "copie" mais d'une "référence" (il n'y a pas de duplication en mémoire).

Citation:

Envoyé par ebastien

CurrentPlayer pointe sur le premier élément de la liste. Donc il appartient bien à la liste. Ce n'est pas une copie mais un pointeur sur un objet.

là j'ai bien compris :merci::ccool:

re,
après un petit week end je reviens dessus direct un soucis
je veux ajouter a mon joueur (CeluiQuiJoue = tour :D mais il me reste encore beaucoup a changé comme lstJoueur :oops:)
j'ai une propriété noJoueur mais elle commence a 1 donc je fais -1 pour avoir le bon indice
j'ai une propriété carteDetenu qui est une liste de carte vide

et la je veux prendre une carte de ma CarteList , qui seras bleu , avec find je prend le 1er qui rempli la condition mais je ne sais pas comment écrire ma condition et l'ajouter a ma liste carteDetenu
(remove je peux avec l'index ça va mais c'est la condition le soucis)

Code:

Jeu.Instance.lstJoueur[CeluiQuiJoue.NoJoueur - 1].carteDetenu.Add(Pioche.Instance.CarteList.Find(Carte.couleur.bleu)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 public class Pioche
    {
        public List<Carte> CarteList = new List<Carte>();
        private static Pioche listCarte = null;
        private static readonly object padlock = new object();
 
        Carte bleu = new Carte(Carte.typeCarte.couleur, Carte.couleur.bleu);
        Carte jaune = new Carte(Carte.typeCarte.couleur,Carte.couleur.jaune);
        Carte marron = new Carte(Carte.typeCarte.couleur, Carte.couleur.marron);
        Carte orange = new Carte(Carte.typeCarte.couleur,Carte.couleur.orange);
        Carte rouge = new Carte(Carte.typeCarte.couleur,Carte.couleur.rouge);
        Carte vert = new Carte(Carte.typeCarte.couleur,Carte.couleur.vert);
        Carte violet = new Carte(Carte.typeCarte.couleur,Carte.couleur.violet);
        Carte joker = new Carte(Carte.typeCarte.joker, Carte.couleur.aucun);
        Carte plus2 = new Carte(Carte.typeCarte.plus2, Carte.couleur.aucun);        
        Carte dernManche = new Carte(Carte.typeCarte.dernManche,Carte.couleur.aucun);
 
 
        public void initialiserPioche()
        {
            Pioche.Instance.AjouteNeuf(bleu);
            Pioche.Instance.AjouteNeuf(jaune);
            Pioche.Instance.AjouteNeuf(marron);
            Pioche.Instance.AjouteNeuf(orange);
            Pioche.Instance.AjouteNeuf(rouge);
            Pioche.Instance.AjouteNeuf(vert);
            Pioche.Instance.AjouteNeuf(violet);
            Pioche.Instance.Ajoutetrois(joker);
            Pioche.Instance.AjouteNeuf(plus2); // il faut 3 cartes +2 , donc j'utilise ma methode pour en add 9
            Pioche.Instance.CarteList.Add(plus2);  // j'en ajoute 1 de plus ça fait 10
            Pioche.Instance.CarteList.Insert(63, dernManche);
        }
        public void AjouteNeuf(Carte c)
        {
            Pioche.Instance.CarteList.Add(c); 
            Pioche.Instance.CarteList.Add(c); 
            Pioche.Instance.CarteList.Add(c); 
            Pioche.Instance.CarteList.Add(c); 
            Pioche.Instance.CarteList.Add(c); 
            Pioche.Instance.CarteList.Add(c); 
            Pioche.Instance.CarteList.Add(c); 
            Pioche.Instance.CarteList.Add(c); 
            Pioche.Instance.CarteList.Add(c); 
        }
        public void Ajoutetrois(Carte c)
        {
            Pioche.Instance.CarteList.Add(c); 
            Pioche.Instance.CarteList.Add(c); 
            Pioche.Instance.CarteList.Add(c); 
        }

15/06/2015, 16h43
egautier18

Salut,

N'hésite pas à regarder sur MSDN, tu trouveras beaucoup d'exemple et d'explication de méthode.

Code:

Pioche.Instance.CarteList.Find(c => c.Property == "bleu")

Si c'était bien la question ?

A+

Citation:

Envoyé par egautier18

Salut,

N'hésite pas à regarder sur MSDN, tu trouveras beaucoup d'exemple et d'explication de méthode.

Code:

Pioche.Instance.CarteList.Find(c => c.Property == "bleu")

Si c'était bien la question ?

c'est le bon but , mais je n'ai pas de méthode .Property & bleu n'est pas un string
je sais pas comment l'adapter
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 public class Carte { public enum typeCarte {couleur, joker, plus2, dernManche} public enum couleur {bleu, jaune, marron, orange, rouge, vert, violet, aucun} private typeCarte type; private couleur color; public Carte(typeCarte t, couleur c) { this.type = t; this.color = c; } }

16/06/2015, 11h28
Remsd91

mdr hemir bien Delvaux ?
16/06/2015, 11h43
ebastien

Bonjour,

Ta classe Carte devrait comporter des propriétés publiques du genre CouleurCarte et TypeCarte (avec un getter et un setter). Tu pourras ensuite adapter le code de egautier18 :

Code:

Pioche.Instance.CarteList.Find(c => c.CouleurCarte == Couleur.Bleu)

Citation:

Envoyé par ebastien

Bonjour,

Ta classe Carte devrait comporter des propriétés publiques du genre CouleurCarte et TypeCarte (avec un getter et un setter). Tu pourras ensuite adapter le code de egautier18 :

Code:

Pioche.Instance.CarteList.Find(c => c.CouleurCarte == Couleur.Bleu)

arf enfait j'ai 2 propriété mais j'avais pas fait les get & set ....
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 public class Carte { public enum typeCarte {couleur, joker, plus2, dernManche} public enum couleur {bleu, jaune, marron, orange, rouge, vert, violet, aucun} private typeCarte type; private couleur color; public Carte(typeCarte t, couleur c) { this.type = t; this.color = c; } }
je peux réutiliser celle là :oops:

16/06/2015, 13h44
Guigui156

Le code de ebastien laisse à désirer...
16/06/2015, 13h57
Hemir

Citation:

Envoyé par Guigui156

Le code de ebastien laisse à désirer...

c'est gentil de dire ça s'en explication...
moi ça fait ce que je veux, si ça ne le fait pas bien faut expliquer pourquoi... :moinsser:
16/06/2015, 14h00
ebastien

Citation:

Envoyé par Guigui156

Le code de ebastien laisse à désirer...

C'est ce que j'appelle un commentaire constructif :lol:
Pourquoi ne pas étayer un peu votre remarque ?
17/06/2015, 17h57
StringBuilder

Bonjour tout le monde...

Je prends le train en route (un peu en retard je dirais même) et j'en reviens à la question initiale.

J'ai lu "List<Carte>" comme réponse.

Permettez-moi de m'insurger :o

Une "pioche", c'est quoi ? Une pile de carte.
Et une pile, c'est quoi en C# ? Une Stack<Carte> ou une Queue<Carte> selon si elle est de type FIFO ou FILO (si on remet des éléments dedans, si on les remet sous la pioche, ou par dessus).

Si tu ne rajoute jamais rien, peut importe.

Donc sur une Stack ou Queue, on fait des "Pop()" pour récupérer l'élément du dessus. Et des "Push()" pour ajouter des éléments (au dessus ou en dessous de la pile).
C'est bien plus propre qu'un List<> où on sait très bien faire des Add() et Remove(), sans vraiment savoir où chercher.

A noter que tu peux parfaitement créer une Stack<> ou Queue<> à partir d'une List<> (plus simple pour mélanger les carte en début de partie).
18/06/2015, 10h04
Hemir

Citation:

Bonjour tout le monde...
Je prends le train en route (un peu en retard je dirais même) et j'en reviens à la question initiale.

J'ai lu "List<Carte>" comme réponse.

Permettez-moi de m'insurger :o

il n'est jamais trop tard comme on dit :roll:
faut taper sur egautier18 :whistle:

Citation:

Envoyé par StringBuilder

Une "pioche", c'est quoi ? Une pile de carte.
Et une pile, c'est quoi en C# ? Une Stack<Carte> ou une Queue<Carte> selon si elle est de type FIFO ou FILO (si on remet des éléments dedans, si on les remet sous la pioche, ou par dessus).

Si tu ne rajoute jamais rien, peut importe.

Donc sur une Stack ou Queue, on fait des "Pop()" pour récupérer l'élément du dessus. Et des "Push()" pour ajouter des éléments (au dessus ou en dessous de la pile).
C'est bien plus propre qu'un List<> où on sait très bien faire des Add() et Remove(), sans vraiment savoir où chercher.

exactement le principe d'une pioche :)
pour mes rangé et les carte détenu par les joueur je fais une pile aussi ?
sachant que les rangé il ne peut pas y avoir plus de 3 carte, avec la liste j'etait obligé de faire un if( .count <= 3)

Citation:

A noter que tu peux parfaitement créer une Stack<> ou Queue<> à partir d'une List<> (plus simple pour mélanger les carte en début de partie).

je garde ma liste et je fais des insert dans la pile ou je change ma list<> en stack<>
comment melanger ?? puisqu'avec la liste moi je faisait un random qui me servait d'indice pour tirée les carte :oops:
18/06/2015, 10h18
StringBuilder
Une Stack, comme une List, a une propriété, qu'on peut définir à la construction "Capacity".
Donc au lieu de faire des "if count > 3", met le Capacity à 3, et gère proprement les exceptions levées par l'objet. Je trouve ça plus propre.
(bon, après, y'a deux écoles quand au Early Error Management et le Lazy Error Management).

Moi je suis partisan de la fénéantise, donc du Lazy.
C'est d'ailleurs l'approche du langage ADA, dont la stabilité n'a jamais pu être égalée :
=> Un programme ne peut compiler que si l'ensemble des exceptions qu'il pourrait lever sont correctement gérées.

Sinon, tu peux créer une Stack à partir d'une List. C'est d'ailleurs ce que je t'ai proposé pour la création de la pioche. C'est plus facile d'alimenter une liste et de la mélanger que de le faire avec une Stack.

Donc :
Code:

1 2 3 4 List<Carte> cartes = new List<Cartes>(3); // capacité de 3 [...] // Alimentation + mélange Stack<Carte> Pioche = new Stack<Carte>(cartes);
Sinon, pour le mélange, je t'invite à visiter ce lien :
http://stackoverflow.com/questions/2...stt-in-c-sharp

fainéantise*

sinon je pense avoir compris le "GetProducts();" c'est mon alimentation d'ailleurs je la trouve pas très joli , je vous met le code voir si vous voyez moyen de l'optimiser

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
 
public List<Carte> CarteList = new List<Carte>();
 
Carte bleu = new Carte(Carte.typeCarte.couleur, Carte.couleur.bleu);
Carte jaune = new Carte(Carte.typeCarte.couleur,Carte.couleur.jaune);
Carte marron = new Carte(Carte.typeCarte.couleur, Carte.couleur.marron);
Carte orange = new Carte(Carte.typeCarte.couleur,Carte.couleur.orange);
Carte rouge = new Carte(Carte.typeCarte.couleur,Carte.couleur.rouge);
Carte vert = new Carte(Carte.typeCarte.couleur,Carte.couleur.vert);
Carte violet = new Carte(Carte.typeCarte.couleur,Carte.couleur.violet);
Carte joker = new Carte(Carte.typeCarte.joker, Carte.couleur.aucun);
Carte plus2 = new Carte(Carte.typeCarte.plus2, Carte.couleur.aucun);        
Carte dernManche = new Carte(Carte.typeCarte.dernManche,Carte.couleur.aucun);
 
 
public void initialiserPioche()
{
    Pioche.Instance.AjouteNeuf(bleu);
    Pioche.Instance.AjouteNeuf(jaune);
    Pioche.Instance.AjouteNeuf(marron);
    Pioche.Instance.AjouteNeuf(orange);
    Pioche.Instance.AjouteNeuf(rouge);
    Pioche.Instance.AjouteNeuf(vert);
    Pioche.Instance.AjouteNeuf(violet);
    Pioche.Instance.Ajoutetrois(joker);
    Pioche.Instance.AjouteNeuf(plus2); // il faut 3 cartes +2 , donc j'utilise ma methode pour en add 9
    Pioche.Instance.CarteList.Add(plus2);  // j'en ajoute 1 de plus ça fait 10
    Pioche.Instance.CarteList.Insert(63, dernManche);
}
public void AjouteNeuf(Carte c)
{
    Pioche.Instance.CarteList.Add(c); 
    Pioche.Instance.CarteList.Add(c); 
    Pioche.Instance.CarteList.Add(c); 
    Pioche.Instance.CarteList.Add(c); 
    Pioche.Instance.CarteList.Add(c); 
    Pioche.Instance.CarteList.Add(c); 
    Pioche.Instance.CarteList.Add(c); 
    Pioche.Instance.CarteList.Add(c); 
    Pioche.Instance.CarteList.Add(c); 
}
public void Ajoutetrois(Carte c)
{
    Pioche.Instance.CarteList.Add(c); 
    Pioche.Instance.CarteList.Add(c); 
    Pioche.Instance.CarteList.Add(c); 
}

EDIT: plus important

Citation:

Erreur 1 La méthode d'extension doit être définie dans une classe statique non générique

18/06/2015, 11h10
jopopmk

Re,

oui pour les méthodes d'extension (fonction statique avec un paramètre this) il faut les placer dans une classe statique.
Perso dans mes projets de test j'ai une classe statique appelées Extensions où je les mets toutes mais c'est pas le plus propre.
18/06/2015, 11h32
ebastien

Bonjour,

Je n'ai pas lu les règles de ton jeu et je ne sais pas comment tu vas utiliser tes cartes présentes dans la pioche. Toutefois, il faut bien avoir conscience que tu as une seule instance de carte par couleur et que tu ajoutes cette instance 9 fois dans ta pioche. En gros, tu as une seule carte que tu ajoutes 9 fois dans une pioche... Cela ne va-t-il pas poser problème ?

Voir 40 message(s) de cette discussion en une page