Erreur de compilation avec Java

Version imprimable

Bonjour,
J'ai un petit souci lors de la compilation d'un algo pour générer des valeurs aléatoires dans une matrice. Je ne vois pas d'où vient le problème.

Voici le code :

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
// Génération aléatoire de valeurs.
public class generaleatoire{
    public static final int MAX= 20 ;
 
 
      public static void creation(int[][]mat){
      int i,j;
      int N=mat.length;
      int M=mat.length;
 
      for(i=0;i<N;i++){
          for(j=0;j<M;j++){
       System.out.print(mat[i][j]+"\t");
          }
            System.out.println();
        }
      }
 
      public static void remplissagemat(int[][]mat){
      int i,j,cpt;      
	cpt=0;
	while(cpt!=5){
		i=(int) (Math.random()*(MAX+1));
		j=(int) (Math.random()*(MAX+1));
 
	if (mat[i][j]=0){
		mat[i][j]=(int) (Math.random()*(MAX+1));
		cpt=cpt+1;
 
        }
      }
    }
 
 
//Programme principal.
  public static void main(String [] args){
  System.out.println("Donnez la taille du plateau en commencant par le nombre de lignes puis en second le nombre de colonnes");
  int N=Clavier.readInt();
  int M=Clavier.readInt();
  int[][] mat1= new int[N][M];
  System.out.println("Le plateau est le suivant :");
  remplissagemat(mat1);
  creation(mat1);
  }
}

Merci d'avance pour votre aide. Bon dimanche.;)

13/04/2008, 12h41
Clorish

Tu peux preciser ou est l'erreur et quel est elle ? ma boulle de cristal est au SAV :)
13/04/2008, 18h40
Javabad

Voici l'erreur :
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 16
at generaleatoire.remplissagemat(generaleatoire.java:22)
at generaleatoire.main(generaleatoire.java:37)

Merci d'avance.;)
13/04/2008, 19h04
Nuro
Cette erreure t'indique que tu cherche a acceder un élément de ton tableau qui n'existe pas car tu es est allé trop loin dans l'incrémentation de tes indices.
Code:

1 2 3 4 5 6 7 8 9 while(cpt!=5){ i=(int) (Math.random()*(MAX+1)); j=(int) (Math.random()*(MAX+1)); if (mat[i][j]=0){ mat[i][j]=(int) (Math.random()*(MAX+1)); cpt=cpt+1; }
L'erreur vient vraisemblablement du choix de ton i et de ton j.

Tules choisis entre 1 et 20 alors que ta matrice peut etre bien differente.

tu devrait avoir :
Code:

1 2 3 4 5 6 7 8 9 while(cpt!=5){ i=(int) (Math.random()*(N+1)); j=(int) (Math.random()*(M+1)); if (mat[i][j]=0){ mat[i][j]=(int) (Math.random()*(MAX+1)); cpt=cpt+1; }
Ca devrait mieux marcher.
13/04/2008, 21h48
Javabad

Merci beaucoup je vais essayer. Sinon pour l'edit De mon message je serai prudent par la suite merci. Bonne soirée.

Salut

Ton i et j prennent des valeurs supérieur à N et M
voici une autre façon:

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
 
public class generaleatoire{
    public static final int MAX = 20 ;
 
 
      public static void creation(int[][]mat){
      int i,j;
      int N=mat.length;
      int M=mat.length;
 
      for(i=0;i<N;i++){
          for(j=0;j<M;j++){
       System.out.print(mat[i][j]+"\t");
          }
            System.out.println();
        }
      }
 
      public static void remplissagemat(int[][]mat)
      {
      	int i,j;
      	int N=mat.length;
      	int M=mat.length;      
 
	  for(i=0;i<N;i++)
	  {
          for(j=0;j<M;j++)
          {
          	mat[i][j]=(int) (Math.random()*MAX)+1;
          }
      }
    }
 
 
//Programme principal.
  public static void main(String [] args){
  System.out.println("Donnez la taille du plateau en commencant par le nombre de lignes puis en second le nombre de colonnes");
  int N=Clavier.readInt();
  int M=Clavier.readInt();
  int[][] mat1= new int[N][M];
  System.out.println("Le plateau est le suivant :");
  remplissagemat(mat1);
  creation(mat1);
  }
}

Bonne chance :)

14/04/2008, 10h43
Javabad

Merci mais je voudrais dans mon cas que tout le tableau ne soit pas remplis... Je voudrais qu'il y ait seulement 5 cases qui soient remplis par exemple et toutes les autres contiennent des 0... Je ne l'ai pas encore essayé mais je ne vois pas de limitations avec votre algo...;)
Bonne journée.
14/04/2008, 11h45
miloux32
Code:

1 2 3 4 5 if (mat[i][j]=0){ mat[i][j]=(int) (Math.random()*(MAX+1)); cpt=cpt+1; }
Code:

if (mat[i][j] = 0)

me pose souci pour 2 raisons ...
1/ a moins que je régresse et que je confonde avec d'autres langages ( et c'est possible hein ) il s'agit d'une affectation et non d'un test ...
ecris plutot mat[i][j] == 0
2/ Ta matrice n'est pas définie à 0 au départ il me semble. Ce n'est pas parce que tu fais un int[][] mat1= new int[N][M]; que les cases de ton plateau sont d'office à 0 ...

Personnellement j'aurais créé un classe Matrice avec un contructeur qui remplit de 0 (ou de nombre aléatoire comme tu veux )

PS: par habitude je te conseille d'écrire la phrase " Votre plateau est le suivant" apres création et remplissageMat ( je ne sais d'ailleurs pas a quoi sert creation) et de créer une méthode afficherMatrice qui s'occupe de ca ;)

Bonjour,

Code:

1
2
int N=mat.length;
int M=mat.length;

donne N=M !

Code:

1
2
3
4
5
6
 
int[][] test = new int[10][20];
System.out.println(test.length);
if(test.length>0) {
	System.out.println(test[0].length);
}

donne

Code:

14/04/2008, 13h24
Wydicule

Citation:

Envoyé par miloux32

2/ Ta matrice n'est pas définie à 0 au départ il me semble. Ce n'est pas parce que tu fais un int[][] mat1= new int[N][M]; que les cases de ton plateau sont d'office à 0 ...

Bah si, jusqu'a preuve du contraire, un entier non initialisé vaut 0, et un tableau d'objets utilise les valeurs par défaut pour s'initialiser.
14/04/2008, 13h43
miloux32

Citation:

Envoyé par Wydicule

Bah si, jusqu'a preuve du contraire, un entier non initialisé vaut 0, et un tableau d'objets utilise les valeurs par défaut pour s'initialiser.

Mes souvenirs doivent dater alors mais bon j'ai toujours gardé l'habitude d'initialiser explicitement mes variables on comprend mieux ce qu'on fait comme ça je trouve . ( et je me méfie des matrices et de l'initialisation par habitude :p )
14/04/2008, 13h50
Javabad

En essayant toutes les méthodes ca ne marche toujours pas...:( Une matrice est toujours initialisée à 0.;) fnobb je ne comprends pas où tu veux en venir...
14/04/2008, 14h01
miloux32
Citation:

Envoyé par Javabad

En essayant toutes les méthodes ca ne marche toujours pas...:( Une matrice est toujours initialisée à 0.;) fnobb je ne comprends pas où tu veux en venir...

Reprend ce que tu as écrit :

Ta matrice est de taille N*M ( mettons N= 5, et M = 6 pour l'exemple)
Or dans ton exemple tu fais
Code:

1 2 3 4 5 6 i=(int) (Math.random()*(MAX+1)); j=(int) (Math.random()*(MAX+1)); if (mat[i][j]=0){ mat[i][j]=(int) (Math.random()*(MAX+1)); cpt=cpt+1;
avec MAX = 20
Donc tu peux obtenir plus que 5 et 6 ....

Donc 2 solutions si tu veux des sous matrices :
- Tu initialises ta matrice de taille MAX*MAX et tu travailles sur ue nsous matrice N*M (que tu passes en paramètres )
- Soit tu changes la méthode de calcul de i et j ;)

De plus
Code:

1 2 int N=mat.length; // renvoie 5 dans notre cas int M=mat.length; // renvoie aussi 5 alors qu'on veut 6 !!!! voir fnobb

Il me semblais que c'était clair :roll:
En initialisant tes variables
dans la méthode
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 public static void remplissagemat(int[][]mat) { int i,j; int N=mat.length; int M=mat.length; for(i=0;i<N;i++) { for(j=0;j<M;j++) { mat[i][j]=(int) (Math.random()*MAX)+1; } }
en initialisant les variables N et M de la même façon
Code:

1 2 3 int N=mat.length; int M=mat.length;
tu sous entend que la matrice est carrée si elle n'est pas tu (par exemple int[20][16])
quand tu la parcours pour initialiser ses données (boucle j) tu fini par faire un mat[i][16] d'où un ArrayIndexOutOfBoundsException

le problème vient de la façon dont est calculé M (c'est pas mat.length mais mat[0].length) voir l'exemple de code

Merci j'ai en effet trouvé pour une matrice carrée et ca marche. Mais c'est vrai que ca ne marche pas pour une matrice non carrée... Donc j'ai essayé avec votre méthode. J'ai bien un affichage de la grille sans souci mais j'ai toujours un message d'erreur...

Voici mon code modifié :

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
// Génération aléatoire de valeurs.
public class generaleatoirejuste{
    public static final int MAX= 20 ;
 
 
      public static void creation(int[][]mat){
      int i,j;
      int N=mat[0].length;
      int M=mat[0].length;
 
      for(i=0;i<N;i++){
          for(j=0;j<M;j++){
       System.out.print(mat[i][j]+"\t");
          }
            System.out.println();
        }
      }
      public static void remplissagemat(int[][]mat,int n,int m){
      int i,j,cpt;      
	cpt=0;
	while(cpt!=5){
		i=(int) (Math.random()*((n-1)+1));
		j=(int) (Math.random()*((m-1)+1));
 
	if (mat[i][j]==0){
		mat[i][j]=(int) (Math.random()*(MAX+1));
		cpt=cpt+1;
 
        }
      }
    }
 
//Programme principal.
  public static void main(String [] args){
  System.out.println("Donnez la taille du plateau en commencant par le nombre de lignes puis en second le nombre de colonnes");
  int N=Clavier.readInt();
  int M=Clavier.readInt();
  int[][] mat1= new int[N][M];
  System.out.println("Le plateau est le suivant :");
  remplissagemat(mat1,N,M);
  creation(mat1);
  }
}

Merci.

:mouarf: l'erreur est la même

Code:

1
2
3
 
      int N=mat[0].length;
      int M=mat[0].length;

c'est

Code:

1
2
      int N=mat.length;
      int M=mat[0].length;

14/04/2008, 14h41
Javabad

En effet merci.:yaisse2: Le problème est résolu. Bonne journée.;)
14/04/2008, 14h46
Javabad

Une petite derniere question en fait : si je veux que ce soit l'utilisateur Qui donne le nombre d'objets qu'il veut, il me suffit de lui faire saisir une valeur A par exemple dans le programme principal et de faire while(cpt!=A) ?
Merci.

Edit : C'est bon en fait merci.
14/04/2008, 14h58
miloux32

Citation:

Envoyé par Javabad

Une petite derniere question en fait : si je veux que ce soit l'utilisateur Qui donne le nombre d'objets qu'il veut, il me suffit de lui faire saisir une valeur A par exemple dans le programme principal et de faire while(cpt!=A) ?
Merci.

oui a conditiion bien sur de le passer en argument de ta fonction ;)
14/04/2008, 15h01
Javabad

Oui c'est bon merci beaucoup. A bientot. ;)