Discussion :

[salma7]

bonsoir, j'ai un problème au niveau du remplissage de la matrice étant un attribut dans la classe individu, l'instaciation de Individu est dans la classe ClassMain. l'erreur est : java.lang.NullPointerException au niveau d la ligne: ms[i][j] = sc.nextInt();Mon code est:

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
import java.util.Scanner;
 
public class Individu {
	public static int id;
	public int ms[][];
	private static Scanner sc;
 
 
	public void creerMatriceS(int[][] ms){
		int c= MainClass.getNbClient();
		int p= MainClass.getNbPeriode();
		sc = new Scanner(System.in);
		for (int i=0; i<c; i++){
			for  (int j=0; j<p; j++){
				System.out.println("donnez la case "+j+ " de la ligne "+i+" de la matrice de stock");						
				ms[i][j] = sc.nextInt();
			}
		}
	}
	public Individu(){
		id++;
		this.creerMatriceS(ms);
		}			
		public static void main(String[] args) {
			// TODO Auto-generated method stub	
	}
}

merci

[joel.drigo]

Salut,

Le tableau ms est déclaré mais initialisé nulle part. Il ne suffit pas de créer une variable est de lui donner un type, il faut y mettre une référence à une instance d'objet pour pouvoir l'utiliser, sinon elle contient null, d'où l'exception NullPointerException.

Tu ne peux pas écrire la méthode comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public void creerMatriceS(int[][] ms){
		int c= MainClass.getNbClient();
		int p= MainClass.getNbPeriode();
		sc = new Scanner(System.in);
		for (int i=0; i<c; i++){
			for  (int j=0; j<p; j++){
				System.out.println("donnez la case "+j+ " de la ligne "+i+" de la matrice de stock");						
				ms[i][j] = sc.nextInt();
			}
		}
	}

Le paramètre est passé par copie : même si tu ajoutais dans cette méthode la ligne [c]ms = new int[p], à la sortie de la méthode, la nouvelle matrice ne serait pas disponible dans la variable ms de la méthode :

Code :

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

1
2
3
4
public Individu(){
		id++;
		this.creerMatriceS(ms);
		}

Il faut que tu fasses plutôt :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public int[][] creerMatriceS(){
		int c= MainClass.getNbClient();
		int p= MainClass.getNbPeriode();
                int[][] ms = new int[c][p];
		sc = new Scanner(System.in);
		for (int i=0; i<c; i++){
			for  (int j=0; j<p; j++){
				System.out.println("donnez la case "+j+ " de la ligne "+i+" de la matrice de stock");						
				ms[i][j] = sc.nextInt();
			}
		}
                return ms;
	}

Code :

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

1
2
3
4
public Individu(){
		id++;
		ms = this.creerMatriceS();
		}

Par ailleurs, évite d'instancier plusieurs Scanner sur System.in dans la même application : en théorie, on devrait lire les données à l'extérieur de la classe Individu, dans une classe dédiée à la lecture et qui stocke la seule instance de Scanner, et instancier un Individu avec les données lues en les passant en paramètre de son constructeur, et non pas faire que l'instanciation de la classe Individu déclenche une saisie au clavier.

[salma7]

oui ça a marché, merci joel pour votre aide, ainsi que vos précieux conseils.