Enregistrer des données dans un arrayList

Version imprimable

Bonsoir à tous,

Je débute en Java et j'ai un problème que je n'arrive pas à résoudre.
On me demande d'enregistrer des élèves et leurs classes dans arraylist, cela est-il possible ?
J'arrive à enregistrer et à afficher, mais à chaque enregistrement le nouveau écrase l'ancien. Or j'aimerais conserver l'ensemble de mes enregistrements.

Voici mon code, je ne demande pas la solution, mais des pistes afin de mieux comprendre ;)
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 28 29 30 31 32 33 34 35 public static void main(String[] args) { String elv, cls; char reponse =' '; ArrayList<String> tableauDynamique = null; do{ //Entrer le nom prénom de l'élève Scanner sc =new Scanner(System.in); System.out.println("Introduction de l'élève"); elv = sc.nextLine(); //Entrer la classe Scanner sc1 =new Scanner(System.in); System.out.println("Introduction de la classe :"); cls = sc1.nextLine(); ArrayList tableauDynamique1 = new ArrayList(Arrays.asList(elv + cls)); for (int i=0; i<tableauDynamique1.size(); i++) { String str = (String)tableauDynamique1.get(i); System.out.println(tableauDynamique1.get(i)+"\n"); } do{ System.out.println("Voulez-vous introduire un autre élève ? O/N "); Scanner sc2 =new Scanner(System.in); reponse = sc2.nextLine().charAt(0); }while(reponse !='N' && reponse!='O'); }while(reponse=='O'); }
Je vous remercie d'avance pour l'attention que vous porterez à ma demande.

Stu

13/01/2018, 10h53
tchize_

Tu crée un nouvelle list à chaque tour de boucle, que tu stocke dans tableauDynamique1 qui n'existe qu'à l'intérieur d'un tour de boucle. Or le but de l'exercice c'est de stocker les donnée dans tableauDynamique qui se trouve à l'extérieur de la boucle et que tu n'utilise absolument jamais. C'est là ton erreur. Tu ne crée que des listede 1 éléments que tu jette à la fin de la boucle.
13/01/2018, 12h58
Stu76bel

Salut,

Merci de ta réponse, je débute en programmation Java et je n'ai plus programmé depuis 20 ans, il y a donc du taf ;)
Je vais analyser ce que tu me conseilles dès que je suis devant mon PC, merci encore pour ton message et pour le temps que tu m'as consacré.

Salut tchize_, j'ai tenu compte de tes remarques judicieuses et voici ce à quoi j'arrive. Le résultat me semble bon, en tout cas je réponds à la question.
Maintenant, j'aimerais sans vouloir abuser, avoir ton avis critique sur mon code.:)

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
28
29
30
31
32
33
34
35
 
public static void main(String[] args) {
		String elv, cls;
		char reponse =' ';
		ArrayList<String> tableauélève = new ArrayList<String>();
		do{
			System.out.println("Première étape, introduiser les noms et prénoms des élèves");
 
			//Entrer le nom prénom de l'élève
			Scanner sc =new Scanner(System.in);
			System.out.println("Taper d'abord le nom et en suite prénom");
			elv = sc.nextLine();
			//Entrer la classe
			Scanner sc1 =new Scanner(System.in);
			System.out.println("Taper la classe :");
			cls = sc1.nextLine();
			tableauélève.add(elv + cls);
 
			for (int i=0; i<tableauélève.size(); i++)
			{
				System.out.println(tableauélève.get(i));
			}
 
				do{
					System.out.println("Voulez-vous introduire un autre élève ? O/N ");
					Scanner sc2 =new Scanner(System.in);
					reponse = sc2.nextLine().charAt(0);
				}while(reponse !='N' && reponse!='O');
 
 
		}while(reponse=='O');
		System.out.println("Merci et au revoir ! ");	
	}
 
}

Je suppose qu'il y a moyen de faire plus simple, mais pour un débutant cela me semble pas trop mal.
Encore un tout grand merci pour ton aide. ;)

15/01/2018, 09h41
thelvin
Hello,

ça te dérange vraiment pas que les éléments de ta liste ressemblent à "Martin PaulCM2" ?

A part ça, rien de très transcendant.

# Il faudrait arrêter de créer un nouveau Scanner à chaque fois que tu veux lire un truc. Ton programme lit une seule console d'entrée, toujours le même System.in, donc il a besoin d'un seul Scanner pour la lire.

Il faudrait créer un seul Scanner, au début du programme avant la boucle, et lire tout ce que tu as à lire avec lui.

# Il faudrait des noms plus clairs que "elv" et "cls".

# Il faudrait déclarer tes variables là où tu les initialises au lieu de le faire au début du programme :

Code:

String nom = sc.nextLine();

Cela évite de s'encombrer de variables à la portée inadaptées. Tu n'avais pas besoin de ces variables avant de leur donner une valeur, alors dans quel but les avoir quand même ?

# L'inverse pour ta variable reponse qui sert de condition à la boucle globale. Tu es bien obligé de la déclarer avant puisque tu dois pouvoir la lire dans le while();

Par contre rien ne t'oblige à lui donner une valeur avant que le premier nextLine() ne s'en occupe.

# Pour la boucle qui affiche le contenu de ta liste, tu préféreras peut-être un for-each :
Code:

1 2 3 for(String eleve : tableauélève) { System.out.println(eleve); }
C'est en général considéré plus clair, et ça évite une variable artificielle i.

# Dans beaucoup de temps, tu préféreras sans doute programmer de manière plus générique et remplacer ta liste par :

Code:

List<String> tableauélève = new ArrayList<String>();

Certes il faut bien choisir un type de liste et tu as choisi ArrayList et c'est très bien.
Mais ça c'est surtout au moment de créer la liste.

Dans le reste de ton programme on se fiche de savoir si c'est une ArrayList, une LinkedList, une synchronizedList ou quoi que ce soit d'autre. Il n'y a donc aucune raison d'imposer que le type de la variable soit ArrayList. List suffit.
15/01/2018, 10h26
Stu76bel

Salut thelvin,

Merci pour ta réponse et ton analyse, je regarde cela ce soir et je reviens vers toi. Mais tes remarques sont judicieuses, je t'avoue n'avoir jamais pensé à ça étant débutant.

Encore un tout grand merci,

Stu
15/01/2018, 10h54
thelvin

Ah ben oui pour du code débutant ça va plutôt bien c'est sûr.

Mais puisqu'il y a toujours des trucs à dire...

J'ai modifié le code selon tes remarques et effectivement plus simple, plus propres et plus compréhensible. Malheureusement, pour améliorer l'affichage cela me parait un peu complexe vu mon niveau pour l'instant. Par contre, je comprends pas ce que tu vaux dire avec ma ArrayList

Quand je fais :

Code:

1
2
 
List<String> tableauélève = new ArrayList();

Il me retourne une erreur :( et je comprends pas pourquoi.
Sinon voici mon code, je vais maintenant chercher à améliorer l'affichage.

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
28
29
30
31
32
33
34
35
36
37
38
 
public static void main(String[] args) {
		char reponse =' ';
		ArrayList<String> tableauélève = new ArrayList();
		Scanner sc =new Scanner(System.in);
 
 
		do{
			String elv, cls;
 
			System.out.println("Première étape, introduiser les noms et prénoms des élèves");
 
			//Entrer le nom prénom de l'élève
 
			System.out.println("Taper d'abord le nom et en suite prénom");
			elv = sc.nextLine();
 
			//Entrer la classe
 
			System.out.println("Taper la classe :");
			cls = sc.nextLine();
			//J'ajoute les entrées à mon tableau élève
			tableauélève.add(elv+" "+cls);
 
			// Boucle qui liste le contenu de mon tableau.
			for(String eleve : tableauélève) {
				  System.out.println(eleve);
				}
 
			// Demande à l'utilisateur s'il veut sortir O/N
				do{
					System.out.println("Voulez-vous introduire un autre élève ? O/N ");
					reponse = sc.nextLine().charAt(0);
				}while(reponse !='N' && reponse!='O');		
		}while(reponse=='O');
		System.out.println("Merci et au revoir ! ");	
	}
}

Encore merci pour le temps que tu me consacres.

15/01/2018, 14h24
Cincinnatus
Citation:
Envoyé par Stu76bel

Quand je fais :

Code:

1 2 List<String> tableauélève = new ArrayList();

Il me retourne une erreur :( et je comprends pas pourquoi.
Bonjour,

Est-ce une erreur du type :

Code:

List cannot be resolved to a type

?

Dans ce cas il faut importer le type List :

Code:

import java.util.List;

en début de fichier.