[Débutant] incrémentation du nom d'une variable dans un for

**BiM** · 29/12/2005, 15h06

Non plus simplement :

Code :

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

1
2
3
4
5
for (int i=0 ; i<listeChainee.size() ; i++)
{
                System.out.println("Voiture "+(i+1)+" :");
                listeChainee.get(i).Affiche();
}

Je ne comprend pas pourquoi tu utilises un tableau d'ailleurs puisque tu as déjà une liste

EDIT :
Par convention :
- les noms de méthodes s'écrivent "VoitureDeSport"
- les noms de variables s'écrivent "maVariable"
- les noms de méthodes "afficherSiege()"
- les noms de variables statiques "TAILLE"
- les getters et setters de maVariable s'écrivent "getMaVariable()" et "setMaVariable()"
Ce sont les principales conventions, tout le monde doit les utiliser. De plus pour certains framework (comme struts, il faut bien respecter cette syntaxe (au niveau de minuscule ou majuscule pour la premiere lettre). Et c'est aussi et surtout une question de lisibilité.

**ghohm** · 29/12/2005, 15h26

Envoyé par BiMouXeTTe

Non plus simplement :

Code :

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

1
2
3
4
5
for (int i=0 ; i<listeChainee.size() ; i++)
{
                System.out.println("Voiture "+(i+1)+" :");
                listeChainee.get(i).Affiche();
}

Je ne comprend pas pourquoi tu utilises un tableau d'ailleurs puisque tu as déjà une liste

EDIT :
Par convention :
- les noms de méthodes s'écrivent "VoitureDeSport"
- les noms de variables s'écrivent "maVariable"
- les noms de méthodes "afficherSiege()"
- les noms de variables statiques "TAILLE"
- les getters et setters de maVariable s'écrivent "getMaVariable()" et "setMaVariable()"
Ce sont les principales conventions, tout le monde doit les utiliser. De plus pour certains framework (comme struts, il faut bien respecter cette syntaxe (au niveau de minuscule ou majuscule pour la premiere lettre). Et c'est aussi et surtout une question de lisibilité.

J'ai essayé et il me dit ça :

init:
deps-jar:
Compiling 1 source file to E:\Projets\Java\TP_1-4\build\classes
E:\Projets\Java\TP_1-4\src\Garage2.java:44: not a statement
(Voiture)this.listeChainee.get(i).Affiche();
1 error
BUILD FAILED (total time: 0 seconds)

Et puis je ne comprends pas ton EDIT ? C'est ce que j'ai toujours fait non ???

**le y@m's** · 29/12/2005, 15h28

Affiche() est une methode de la classe Voiture il faut donc que le cast soit fait avant, pour cela il te suffit de rajouter des parenthèses :

Code :

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

((Voiture)this.listeChainee.get(i)).Affiche();

**herve91** · 29/12/2005, 15h28

Envoyé par BiMouXeTTe

Par convention :
- les noms de méthodes s'écrivent "VoitureDeSport"

Les noms de classes

**BiM** · 29/12/2005, 15h29

Code :

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

((Voiture)listeChainee.get(i)).Affiche();

Non, la méthode Affiche() commence par une majuscule (vade retro !!!).

J'oubliais les constructeurs commencent par une majuscule (comme les classes).

**ghohm** · 29/12/2005, 15h34

Envoyé par herve91

Envoyé par BiMouXeTTe

Par convention :
- les noms de méthodes s'écrivent "VoitureDeSport"

Les noms de classes

ah bah voilà ... j'me disais bien ...

Envoyé par BiMouXeTTe

Code :

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

((Voiture)listeChainee.get(i)).Affiche();

Non, la méthode Affiche() commence par une majuscule (vade retro !!!).

J'oubliais les constructeurs commencent par une majuscule (comme les classes).

Bon bon soit ...

Merci à tous ça fonctionne parfaitement

Gôm

[EDIT]J'ai été docile et j'ai modifié tous les noms de mes méthodes n'étant pas des constructeurs en enlevant la majuscule de la première lettre !

[/EDIT]

**BiM** · 29/12/2005, 15h52

Lol au temps pour moi... Aïeuh !!

**vbrabant** · 29/12/2005, 16h24

Personellement, vu que tu tournes avec un JDK 5, tu pourrais grandement simplier ton code, en faisant ceci:

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
46
47
48
49
50
public class Garage2 {
 
    protected String nomGarage;
    protected LinkedList<Voiture> listeChainee;
 
    public Garage2(String nomGarage, Voiture... voitures) {
        this.nomGarage = nomGarage;
        listeChainee = new LinkedList();
        for (Voiture v : voitures) {
            this.listeChainee.add(v);
        }
    }
 
    public void affiche() {
 
        Voiture [] tmpTab = null;
        //met la liste dans un tableau. Pas de boucle, pas de source d'erreur
        tmpTab = listeChainee.toArray(tmpTab);
 
        if (this.listeChainee.isEmpty())
            System.out.print("Ce garage NE contient PAS de voiture(s) :\n");
        else {
            System.out.print("Le garage "+this.nomGarage+" contient "+tmpTab.length+" voitures :\n");
            for (Voiture v : tmpTab) {
                System.out.print("Voiture "+v+"\n\n");
            }
        }
    }
 
    public static void main(String[] args) {
       Voiture car_1 = new Voiture("Renault", "Mégane", 13900.85, 2004);
       car_1.Affiche();
 
       Voiture car_2 = new Voiture("Peugeot", "407", 16900.85, 2005);
       car_2.Affiche();
 
       Voiture car_3 = new Voiture("Citroën", "C6", 28900.85, 2006);
       car_3.Affiche();
 
       Garage2 concess_1 = new Garage2("Dabireau", car_1, car_2, car_3);
       concess_1.affiche();
 
       Voiture [] tabTempVoiture= {car_1, car_2, car_3};
       Garage2 g1=new Garage2("Dabireau_bis", tabTempVoiture);
       g1.affiche();
 
       Garage2 concess_2 = new Garage2("Saroteau", car_1, car_2, car_3);
       concess_2.affiche();
    }     
}

Maintenant que Garage2 a un constructeur avec Voiture... tu peux lui passer ou bien un tableau de voiture ou bien une succession de voitures.

Et le fait d'avoir utilisé LinkedList<Voiture> te permet de ne plus devoir caster chaque fois en Voiture le résultat de ta liste.
Et tu as également une belle méthode pour transformer la linkedList<Voiture> en Voiture[].

Toujours à ta disposition dans les limites du possible

**BiM** · 29/12/2005, 16h26

Sauf que pas tout le monde tourne en 1.5 et que ca réduit nettement la portabilité. Les évolutions sont faites pour être utilisées, mais pas trop tôt quand même

**ghohm** · 29/12/2005, 16h33

très bien merci, mais là je manque de temps !

J'étudirais ça un peu plus tard.

**vbrabant** · 29/12/2005, 16h34

C'était juste pour lui montrer comment écrire cela avec du JDK 5.0. Et comme il tourne quand même avec un JDK 5.0, autant qu'il en profite. Non ?

Vincent

**BiM** · 29/12/2005, 16h48

Ca depend, s'il fait ca pour l'école et que la bas ca ne tourne pas en 1.5, ce sera la ****. J'ai connu ca lol

**vbrabant** · 29/12/2005, 16h54

Si à l'école il doit faire cela avec du JDK 1.4, alors il a effectivement intérêt à faire du 1.4 à la maison. C'est évident.

Mais comme il n'a jamais dit qu'il était obligé de rester en 1.4 et que sur un sujet posté dans le forum NetBeans, il apparaît clairement qu'il est avec un JDK 5.0, je me suis dit qu'un exemple en JDK 5.0 lui ferait certainement plaisir.

Mais bon, assez perdu de temps à discutailler sur 1.4 ou 5.0 entre nous. On ne fait que supposer, et lui seul connait la réponse :-)

Vincent

P.S. developpez.com est là pour qu'on aide et qu'on apprenne.
C'est jamais perdu de voir ce qu'on fait en 1.4 et comparer avec ce qu'on pourra faire en 1.5.

**BiM** · 29/12/2005, 17h02

C'est maaaaaaaaaaaaaal lol.

**ghohm** · 31/12/2005, 14h26

Envoyé par vbrabant

Personellement, vu que tu tournes avec un JDK 5, tu pourrais grandement simplier ton code, en faisant ceci:

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
46
47
48
49
50
51
52
53
54
public class Garage2 {
 
    protected String nomGarage;
    protected LinkedList<Voiture> listeChainee;
 
---------------------------------------------------------------------------
        public Garage2(String nomGarage, Voiture... voitures) {
        this.nomGarage = nomGarage;
        listeChainee = new LinkedList();
        for (Voiture v : voitures) {
            this.listeChainee.add(v);
        }
    }
---------------------------------------------------------------------------
 
    public void affiche() {
 
        Voiture [] tmpTab = null;
        //met la liste dans un tableau. Pas de boucle, pas de source d'erreur
        tmpTab = listeChainee.toArray(tmpTab);
 
        if (this.listeChainee.isEmpty())
            System.out.print("Ce garage NE contient PAS de voiture(s) :\n");
        else {
            System.out.print("Le garage "+this.nomGarage+" contient "+tmpTab.length+" voitures :\n");
---------------------------------------------------------------------------
               for (Voiture v : tmpTab) {
                  System.out.print("Voiture "+v+"\n\n");
            }
---------------------------------------------------------------------------
        }
    }
 
    public static void main(String[] args) {
       Voiture car_1 = new Voiture("Renault", "Mégane", 13900.85, 2004);
       car_1.Affiche();
 
       Voiture car_2 = new Voiture("Peugeot", "407", 16900.85, 2005);
       car_2.Affiche();
 
       Voiture car_3 = new Voiture("Citroën", "C6", 28900.85, 2006);
       car_3.Affiche();
 
       Garage2 concess_1 = new Garage2("Dabireau", car_1, car_2, car_3);
       concess_1.affiche();
 
       Voiture [] tabTempVoiture= {car_1, car_2, car_3};
       Garage2 g1=new Garage2("Dabireau_bis", tabTempVoiture);
       g1.affiche();
 
       Garage2 concess_2 = new Garage2("Saroteau", car_1, car_2, car_3);
       concess_2.affiche();
    }     
}

Maintenant que Garage2 a un constructeur avec Voiture... tu peux lui passer ou bien un tableau de voiture ou bien une succession de voitures.

Et le fait d'avoir utilisé LinkedList<Voiture> te permet de ne plus devoir caster chaque fois en Voiture le résultat de ta liste.
Et tu as également une belle méthode pour transformer la linkedList<Voiture> en Voiture[].

Me revoilà

J'ai essayé d'étudier ton code et certaines choses (encadrées par des tirets) me sont encore bien obscures ... te serait-il possible de m'éclairer encore une fois étant donné que :

Envoyé par vbrabant

Toujours à ta disposition dans les limites du possible

A moins que quelqu'un d'autre s'en sente capable.

Merci d'avance, Gôm.

**adiGuba** · 31/12/2005, 16h02

Salut,

Envoyé par ghohm

J'ai essayé d'étudier ton code et certaines choses (encadrées par des tirets) me sont encore bien obscures ...

Il s'agit de nouveautés dans le langages apparut avec Java 5.0, en particulier Les méthodes à arguments variables et les Les itérations simplifiées.

Pour faire bref le constructeur suivant :

Code :

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

public Garage2(String nomGarage, Voiture... voitures)

Accepte 0 ou plusieurs Voitures en paramètre, ce qui fait que tu peux l'utiliser de la manière suivante :

Code :

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

1
2
3
4
new Garage2( "nom", v1, v2, v3, v4, v5 );
new Garage2( "nom", v1, v2, v3 );
new Garage2( "nom", v1);
new Garage2( "nom" );

Et dans ton code c'est géré comme un simple tableau...

Enfin la nouvelle boucle for permet de parcourir tous les éléments d'un tableau/collection simplement... Ainsi le code suivant :

Code :

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

1
2
3
        for (Voiture v : voitures) {
            this.listeChainee.add(v);
        }

est "équivalent" à :

Code :

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

1
2
3
        for (int i=0; i<voitures.length; i++) {
            this.listeChainee.add(voitures[i]);
        }

Pour plus d'info sur Java 5.0 : Présentation de Java 5.0 (Tiger) par Lionel Roux

a++

**ghohm** · 31/12/2005, 16h17

Merci adiGuba

Un peu de pratique et j'espère que, ces notions je serais assimilié !

Gôm

**axelmeunier** · 02/03/2006, 01h58

Une autre idée de code:

Code :

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

1
2
3
4
5
6
7
8
 
public class ServeurAvance {
	//tableau qui stocke les threadServer
	ArrayList TSlist = new ArrayList();
	public ServeurAvance() {
			...
		}
	}

Et tu t'occupes pas du reste, dans ton constructeur de l'autre classe:

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
 
public class ThreadServer implements Runnable{
	...
 
	public ThreadServer(Socket s1, ServeurAvance SA) {
		...
		SA.TSlist.add((ThreadServer)this);
		...
 
	}
 
		public void run() {
		try {
			//String temporaire ou est stoquée le bufferread.
			String strread = BR1.readLine();		
			...
					}
			else {
				//ici on envoie à tous
				 for (Iterator it = SA.TSlist.iterator(); it.hasNext();)
				     ((ThreadServer) it.next()).Broadcast(strread); //ici appel d'une méthode directement par le tableau...
			}
			...

Ya pas trop de commentaire, mais c'est pas trop compliqué, et la tu règles tes problèmes d'incrémentation en utilisant des list d'éléments sans jamais nomer ces éléments!
Voilà, bon courage

[Débutant] incrémentation du nom d'une variable dans un for

Langage Java

Discussions similaires

Partager

Partager