Erreur de compilation avec Java

**Javabad** · 13/04/2008, 11h49

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 :

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
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.

**Clorish** · 13/04/2008, 12h41

Tu peux preciser ou est l'erreur et quel est elle ? ma boulle de cristal est au SAV

**Javabad** · 13/04/2008, 18h40

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.

**Nuro** · 13/04/2008, 19h04

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 :

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

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 :

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

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.

**Javabad** · 13/04/2008, 21h48

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

**bobmidou** · 14/04/2008, 01h10

Salut

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

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
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

**Javabad** · 14/04/2008, 10h43

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.

**miloux32** · 14/04/2008, 11h45

Code :

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

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

Code :

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

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

**fnobb** · 14/04/2008, 12h02

Bonjour,

Code :

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

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

donne N=M !

Code :

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

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 :

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

**Wydicule** · 14/04/2008, 13h24

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.

**miloux32** · 14/04/2008, 13h43

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 )

**Javabad** · 14/04/2008, 13h50

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...

**miloux32** · 14/04/2008, 14h01

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 :

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

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 :

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

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

**fnobb** · 14/04/2008, 14h04

Il me semblais que c'était clair

En initialisant tes variables
dans la méthode

Code :

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

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 :

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

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

**Javabad** · 14/04/2008, 14h33

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 :

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
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.

**fnobb** · 14/04/2008, 14h37

l'erreur est la même

Code :

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

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

c'est

Code :

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

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

**Javabad** · 14/04/2008, 14h41

En effet merci.

Le problème est résolu. Bonne journée.

**Javabad** · 14/04/2008, 14h46

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.

**miloux32** · 14/04/2008, 14h58

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

**Javabad** · 14/04/2008, 15h01

Oui c'est bon merci beaucoup. A bientot.

Erreur de compilation avec Java

avec Java

Discussions similaires

Partager

Partager