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

Version imprimable

Voir 40 message(s) de cette discussion en une page

29/12/2005, 15h06
BiM
Non plus simplement :
Code:

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é.
29/12/2005, 15h26
ghohm
Citation:
Envoyé par BiMouXeTTe

Non plus simplement :

Code:

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 :

Citation:

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 ???
29/12/2005, 15h28
le y@m's

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:

((Voiture)this.listeChainee.get(i)).Affiche();
29/12/2005, 15h28
herve91

Citation:

Envoyé par BiMouXeTTe

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

Les noms de classes :wink:
29/12/2005, 15h29
BiM

Code:

((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).
29/12/2005, 15h34
ghohm

Citation:

Envoyé par herve91

Citation:

Envoyé par BiMouXeTTe

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

Les noms de classes :wink:

ah bah voilà ... j'me disais bien ... :roll: :mrgreen:

Citation:

Envoyé par BiMouXeTTe

Code:

((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 ... :sm:

Merci à tous ça fonctionne parfaitement :ccool:

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 ! :oops: :wink: [/EDIT]
29/12/2005, 15h52
BiM

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

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

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
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

29/12/2005, 16h26
BiM

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 :P
29/12/2005, 16h33
ghohm

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

J'étudirais ça un peu plus tard. :coucou:
29/12/2005, 16h34
vbrabant

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
29/12/2005, 16h48
BiM

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
29/12/2005, 16h54
vbrabant

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.
29/12/2005, 17h02
BiM

C'est maaaaaaaaaaaaaal lol.

Citation:

Envoyé par vbrabant

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

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
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();
    }     
}

Me revoilà :wink:

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 :

Citation:

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.

31/12/2005, 16h02
adiGuba
Salut,

Citation:

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:

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:

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:

1 2 3 for (Voiture v : voitures) { this.listeChainee.add(v); }
est "équivalent" à :
Code:

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 :ccool:

a++
31/12/2005, 16h17
ghohm

Merci adiGuba :oops:

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

Gôm

Une autre idée de code:

Code:

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:

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 :)

Voir 40 message(s) de cette discussion en une page