Discussion :

[karamaster]

Bonsoir,
Je suis débutant en java.
J'ai une ArrayList d'entier dans laquelle, je veux ajouter des nombres. Pour cela, j'ai deux classes : une classe EntierOrdonne qui implémente les méthodes insertion et affichage.
Et une classe CreerEntierOrdonne qui hérite de la classe EntierOrdonne.
Le problème c'est que mes insetions en marche pas.

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
 
public class EntierOrdonne{
 
	//Instanciation de la collection d'entier
	ArrayList<Integer> listEntier = new ArrayList<>();
 
	public void insertion(int x) {
		listEntier.add(x);
 
		//Tri de la liste
		Collections.sort(listEntier);
	}
 
       public void Affichage() {
 
		for(int i=0; i<= listEntier.size(); i++) {
			System.out.println(listEntier.get(i));
		}
	}
}

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
 
public class CreerEntierOrdonne extends EntierOrdonne{
 
	public static void main(String[] args) {
		int x, nbre = 0;
 
		System.out.println("Entrer le nombre d'entier ");
 
		Scanner scan = new Scanner(System.in);
		nbre = scan.nextInt();
 
		for (int i = 0; i <= nbre; i++) {
			x = scan.nextInt();
 
			new EntierOrdonne().insertion(x);
		}
 
		new EntierOrdonne().Affichage();
	}	
}

[thelvin]

Hello,

le problème c'est tu crées toujours un nouvel objet, puisque tu utilises toujours new.

Chaque objet contient ses propres données, et les autres objets contiennent d'autres données.

En l'occurrence donc, ta classe EntierOrdonne déclare parmi ses données, une ArrayList nommée listEntier. C'est très bien. Et donc chaque objet de classe EntierOrdonne aura sa propre ArrayList qui contient ses propres entiers. Et qui commence vide, puisque listEntier est initialisée avec new ArrayList<>(); donc création d'une nouvelle ArrayList vide.

Quand tu fais new EntierOrdonne().insertion(x); tu crées un nouvel objet de classe EntierOrdonne(), et tu y ajoutes bel et bien un entier.

Mais le problème c'est que cet objet, tu n'en fais plus rien ensuite, il se perd dans le ramasse-miettes. La prochaine fois que tu fais new EntierOrdonne().insertion(x); c'est un nouvel objet, qui n'a rien à voir avec le précédent, auquel tu ajoutes un premier entier. Et que tu perds juste après, comme à chaque fois.

Même chose avec new EntierOrdonne().Affichage(); tu crées un nouvel objet EntierOrdonne, qui est donc vide, et tu l'affiches. Forcément afficher un objet vide ça n'affiche rien.

Voici une manière d'utiliser la classe EntierOrdonne :

Code :

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

1
2
3
4
5
EntierOrdonne ordonneur = new EntierOrdonne();
ordonneur.insertion(2);
ordonneur.insertion(4);
ordonneur.insertion(1);
ordonneur.Affichage();

On notera avant tout :
- le fait que new EntierOrdonne() n'apparaît qu'une seule fois, et donc qu'un seul objet est créé.
- la déclaration d'une variable de type EntierOrdonne, qui permet de stocker l'objet créé et de s'en servir à chaque fois qu'on le veut.
- que cette déclaration de variable EntierOrdonne ressemble énormément aux déclarations de variables ArrayList. C'est normal, ce sont des classes, dont on crée des objets. Ça marche pareil.

[karamaster]

Merci beaucoup,
ça marche super.

Code :

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

1
2
3
4
5
6
7
8
9
 
EntierOrdonne listOrd = new EntierOrdonne();
 
    for (int i = 0; i <= nbre; i++) {
	x = scan.nextInt();
	listOrd.insertion(x);
   }
 
   listOrd.Affichage();