remplissage d'une matrice

Version imprimable

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:

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

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:

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:

1 2 3 4 public Individu(){ id++; this.creerMatriceS(ms); }
Il faut que tu fasses plutôt :
Code:

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:

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.

09/04/2015, 15h36
salma7

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