Discussion :

[diki38]

Bonjour, je débute en JAVA, et j'ai un projet de cours qui consiste à developper un jeu de cartes.
Ma classe ci dessous consiste uniquement à gerer la distribution des cartes.

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
19
20
21
22
23
24
public class JeuxdeCartes {
 
        // creation du jeu des 52 cartes
        String[] couleur = { "Trèfle", "Carreaux", "Coeur", "Pique" };
        String[] figure = { "2", "3", "4", "5", "6", "7", "8", "9", "10","Valet", "Dame", "Roi", "As"};
	String COULEUR= "";
	String FIGURE= "";
 
	public static void DistributionCarte(int NbJeu){
		for (int i = 0; i < NbJeu; i++){
		int clr = (int)Math.random()*5;
		COULEUR=couleur[clr];
		int fgr = (int)Math.random()*14;
		FIGURE=figure[fgr];
		System.out.println("Le joueur a un(e)" + FIGURE + "de" + COULEUR);
		}
	}
 
	public static void main(final String args[]){
	int NbJeu = Integer.parseInt(args[0]); // nb joueurs ==> nombres de fois que l'on distribue les cartes
	DistributionCarte(NbJeu);
	}
 
}

Mon souci c'est que j'ai toujours au niveau de la fonction Random.Math des erreurs de compilations :

non-static variable COULEUR cannot be referenced from a static context

Merci d'avance pour votre aide.

[ngpub]

Si le grand compilateur a dit "non-static variable COULEUR cannot be referenced from a static context ", alors de la doc sur "static" tu chercheras et ton erreur tu corrigeras.

[GérardMartinelli]

Tu n'as pas besoin de déclarer ta méthode DistributionCarte en static .
Par contre pour exécuter ton programme, la méthode main doit être modifiée comme ceci :

public static void main(final String args[])
{
int NbJeu = Integer.parseInt(args[0]);
new JeuxdeCartes() . DistributionCarte(NbJeu);
}

Par contre j'ai essayé de voir ce que donnait ton ^programme, et il donne toujours le même résultat. Pourquoi ?
Parce que ton casting (int) Math.random()*5
s'applique à random et non au produit par 5. Or (int) random) renvoie toujours 0. Donc 0*5 = 0.
Il faut écrire (int) (Math.random()*5)
dans ce cas le casting s'applique au produit.

[diki38]

merci bcp mgpub, reste plus qu'a reprendre mon programme qui ne marche pas du tout, j'ai employer des boucles ou il ne fallait pas, la fonctionne math.random ne fonctionne pas non plus, mais en tout cas une nouvelle fois merci !

[GérardMartinelli]

Regarde la modif à mon précédent post. Tu verras la raison. J'étais en train de le modifier pendant ton post.

Je voudrais aussi ajouter quelque chose à propos des multiplicateurs (5 ou 14)

Tu oublies qu'en Java, l'index des tables commence à 0. Donc une table qui a 13 éléments a son index qui va de 0 à 12. Lorsque tu multiplies un nombre x tel que 0<x<1 par n, tu peux obtenir n-1 , soit dans ton cas 4 ou 13 qui sont des index invalides.

Donc change ces instructions par :

(int) (Math.random()*figure.length);
et
(int) (Math.random()*couleur.length);

ou mieux, crée une methode random pour simplifier. Ton code deviendrait :

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
19
20
21
22
23
24
25
26
27
28
29
 
public class JeuxdeCartes 
{ 
     // creation du jeu des 52 cartes
     final static  String[] couleur = { "Trèfle", "Carreaux", "Coeur", "Pique" };
      final static   String[] figure = { "2", "3", "4", "5", "6", "7", "8", "9", "10","Valet", "Dame", "Roi", "As"};
 
 public static void DistributionCarte(int NbJeu)
 {
   for (int i = 0; i < NbJeu; i++)
   {
      String COULEUR= random(couleur);
      String FIGURE=random(figure);
      System.out.println("Le joueur a un(e)" + FIGURE + "de" + COULEUR);
   }
 }
 
 private static String random(String[] table)
 {
   return table[(int)(Math.random()*table.length)];
 }
 
 public static void main(final String args[])
 {
  int NbJeu = Integer.parseInt(args[0]); // nb joueurs ==> nombres de fois  que l'on distribue les cartes
  DistributionCarte(NbJeu);
 }
 
}