Discussion :

[Dereck07]

Bonjour à tous,

Voila, je viens juste de me mettre à java. J'avais l'habitude de programmer en delphi du coup j'ai plus mes repères et je bute sur la creation de ma premiere classe.

Voici le code:
a) Classe Valeur (a priori ok):

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
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
 
public class Valeur {
    private String Date_;
    private float Vstart;
    private float Vmin;
    private float Vmax;
    private float Vend;
    private float Vmoy;
    private long Vol;
 
    public Valeur (){
    Date_="01/01/1900";
    Vstart=0.0f;
    Vmin=0.0f;
    Vmax=0.0f;
    Vend=0.0f;
    Vmoy=0.0f;
    Vol=0;
    }
 
 public Valeur (String date, float startV, float minV, float maxV, float endV, long volume){
    Date_=date;
    Vstart=startV;
    Vmin=minV;
    Vmax=maxV;
    Vend=endV;
    Vmoy=(minV+maxV)/2;
    Vol=volume;
    }   
 
 public String get_date()
 {
 return Date_;
 }
 
 public float get_start_v()
 {
 return Vstart;
 }
 
 public float get_min_v()
 {
 return Vmin;
 } 
 
 public float get_max_v()
 {
 return Vmax;
 } 
 
 public float get_end_v()
 {
 return Vend;
 } 
 
 public float get_moy_v()
 {
 return Vmoy;
 }
 
 public float get_norm_v()
 {
 return Vnorm;
 }
 
 public long get_volume()
 {
 return Vol;
 } 
}

b) Classe Action (Qui pose probleme):

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
import java.awt.List;
import java.util.LinkedList;

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */

/**
 *
 * @author TOM
 */
public class Action {
private String Name_;
private List<Valeur> list = new LinkedList<Valeur>();

//Constructeur par défaut
  public Action(String name){
      list = Null;
      Name_=name;
  }

  public void addValeur(String date, float startV, float minV, float maxV, float endV, long volume){
    Valeur Val = new Valeur(date, startV, minV, maxV, endV, volume);
    list.add(Val); 
   }
  
   public String DateFirstVal()
  {
    String Date="";
    if (list.isEmpty()==false)
    {
    list.get(0).get_date();   
    }
    else
    {
    Date="No Date";   
    }
    return Date;    
  } 
  
}

La classe action contient un nom et une liste de valeur de la calsse Valeur.
J'ai donc besoin de créer dans cette classe une liste d'objet Valeur et c'est là que je coince.
J'ai essayé de chercher sur le net sans succés ...

Est ce que quelqu'un peut me dire ce qui ne va pas dans mon code ?
BetBeans me sousligne private List<Valeur> list = new LinkedList<Valeur>(); avec le commentaire "Type List dosn't take parameter".
Après il n'aime pas non plus

Code :

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

list = Null;

. comment on initialise une liste null ?

Je vous remercie par avance ....

Dereck

[thelvin]

Est ce que quelqu'un peut me dire ce qui ne va pas dans mon code ?
BetBeans me sousligne private List<Valeur> list = new LinkedList<Valeur>(); avec le commentaire "Type List dosn't take parameter".

Ça c'est surtout la faute de Java -_-°.

Il ne fallait pas mettre

Code :

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

import java.awt.List;

mais

Code :

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

import java.util.List;

Les deux n'ont absolument rien à voir, et celui que tu voulais, c'est java.util.List.

Après il n'aime pas non plus

Code :

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

list = Null;

. comment on initialise une liste null ?

Sans majuscule -_-°.
En Java null est un mot-clé et les mots-clés n'ont pas de majuscule.

... Mais bon, tu ne devrais pas la mettre à null. Elle serait déjà à null par défaut, si tu ne lui avait rien assigné. Or tu lui as assigné new LinkedList<Valeur>() quand tu l'as déclarée. C'est donc que tu voulais qu'elle soit autre chose que null, alors ça ne sert à rien de la mettre à null tout de suite après -_-°. Il faut choisir.

[Dereck07]

Merci thelvin
La situation c'est nettement ammélioré. J'ai plus d'erreur mais encore un petit warring.
Toujours sur

Code :

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

private List<Valeur> list = new LinkedList<Valeur>();

Netbeans me dit "redundant type agruments in new expression (use diamond operator instead)."
T'as une idée de ce que ca veut dire ?

Pour la list null en faite je veux simplement qu'a ca création la liste soit vide.
Si je comprends bien pour faire cela j'implemente le constructeur comme ceci:

Code :

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

1
2
3
4
5
6
7
8
9
public class Action {
private String Name_;
private List<Valeur> list = new LinkedList<Valeur>();
 
//Constructeur par défaut
  public Action(String name){
      Name_=name;
  }
//...

C'est correct ? (list est bien créé bien qu'il ne soit pas dans le constructeur)

Dereck

[Dereck07]

C'est bon j'ai remplacé par:

Code :

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

private List<Valeur> list = new LinkedList<>();

sur le net j'ai trouvé une explication:

The diamond operator in java 7 allows code like the following:
List<String> list = new LinkedList<>();

However in Java 5/6, I can simply write:
List<String> list = new LinkedList();

et

The diamond operator (<>) removes the need for explicit type arguments in constructor calls to generic classes, thereby reducing visual clutter. For example:
List<?> list = new ArrayList<>();

The compiler infers the type on the right side. So if you have a list of ?, the compiler will infer a list of Object.

Merci

Dereck

[thelvin]

Pour la list null en faite je veux simplement qu'a ca création la liste soit vide.
Si je comprends bien pour faire cela j'implemente le constructeur comme ceci:

C'est correct ? (list est bien créé bien qu'il ne soit pas dans le constructeur)

Oui. Les déclarations de variables d'instance s'exécutent avant le constructeur, mais s'exécutent chaque fois qu'on appelle le constructeur.

Et une nouvelle liste est toujours créée vide si on ne lui demande pas de contenir quelque chose à sa création.

C'est bon j'ai remplacé par:

Code :

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

private List<Valeur> list = new LinkedList<>();

Excellent.
Comme cela ne marche que depuis Java 7, que beaucoup de gens utilisent encore Java 6 pour commencer, et que la syntaxe Java 6 reste correcte bien que redondante, je ne t'ai pas fait de remarque là-dessus.

[Dereck07]

Je crois que j'avais fermé le sujet un peu rapidement.
Ma fonction DateFirstVal() ne fonctionne pas:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
   public String DateFirstVal()
  {
    String Date="";
    if (list.isEmpty()==false)
        {
        list.get(0).get_date();
        }
        else
        {
        Date="No Date";   
        }
    return Date;    
  }

Je sais que la list n'est pas vide car la fonction ne retoune pas "No Data" mais un string vide.

La maniere dont j'essaie d'acceder à l'objet ne dois pas être bonne car ce bout de code retourne bien "date":

Code :

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

1
2
3
4
 
Valeur Vt= new Valeur("date", 1.0f, 1.0f, 1.0f, 1.0f, 1l);
s=Vt.get_date();
System.out.println("s");

Normalement list contient des Objet de la classe Valeur.
Comment je fais pour acceder à un objet de la liste ?

Dereck

[thelvin]

Comment je fais pour acceder à un objet de la liste ?

Comme ça... Mais tu as oublié d'en faire quelque chose après y avoir accédé.

Dans le cas isEmpty(), tu fais :

Code :

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

Date="No Date";

et dans l'autre cas, il n'y a pas de

Code :

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

Date=

Pourquoi cette différence ?

Note à part ça : en Java les méthodes et les variables devraient commencer par des minuscules, convention respectée par tout le monde. Respecte-là, toi aussi, dès que possible. Ça aide beaucoup.

[Dereck07]

Haaaa oui, comment je n'ai pas pu le voir ...
Merci