Discussion :

[Javabad]

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]

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

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

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

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]

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

[bobmidou]

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