Probleme Patron strategie.

Version imprimable

Bonjour,
J'aimerais afficher ma listeView avec mes élements de ma classe " MusiqueDur"
Hors je suis bloquer dans ma classe ListeMusique;
Pourriez vous m'aider svp ?
Merci

Ma Classe Musique dur :

Code:

1
2
3
4
5
6
7
8
9
10
 
public class MusiqueDur extends MusiqueManager{
 
    @Override
    public ArrayList<Musique> LireMusique(Integer id,String titre,String artiste,String duree){
                ArrayList<Musique> listeTemporaire = new ArrayList();
                listeTemporaire.add(new Musique(3, "Ceci est un essai", "Joanny", "5m10"));
                return listeTemporaire;
    }
}

ma classe MusiqueManager :

Code:

1
2
3
4
5
public abstract class MusiqueManager{
 
    public abstract ArrayList LireMusique(Integer id,String titre,String artiste,String duree);         
 
}

Ma classe ListeDeMusique ( il semblerait que se soit elle qui pose pb ... )

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
public class ListeMusique {
    ObservableList<Musique> lesMusiquesObservables = FXCollections.observableArrayList();
 
    private final ListProperty<Musique> lesMusiques = new SimpleListProperty<>(lesMusiquesObservables);
        public ObservableList<Musique> getLesMusiques() {return lesMusiques.get();}
        public void setLesMusiques(ObservableList<Musique> value) {lesMusiques.set(value);}
        public ListProperty<Musique> lesMusiquesProperty() {return lesMusiques;}
 
 
    public ListeMusique(ListProperty<Musique> lesMusiques){
        lesMusiques=MusiqueDur();
    }
 
 
    public void lireMusique(){
        ArrayList<Musique> listeTemporaire = ListeMusique.lireMusique();
        lesMusiques.add();
    }
}

Ma classe Musqiue :

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
public class Musique {
 
    private final IntegerProperty id = new SimpleIntegerProperty();
    public int getId() {return id.get();} public void setId(int value) {id.set(value);}
    public IntegerProperty idProperty() {return id;}
 
    private final StringProperty titre = new SimpleStringProperty();
    public String getTitre() {return titre.get();}
    public void setTitre(String value) {titre.set(value);}
    public StringProperty titreProperty() {return titre;}
 
    private final StringProperty artiste = new SimpleStringProperty();
    public String getArtiste() {return artiste.get();}
    public void setArtiste(String value){artiste.set(value);}
    public StringProperty artisteProperty() {return artiste;}
 
    private final StringProperty duree = new SimpleStringProperty();
    public String getDuree() {return duree.get();}
    public void setDuree(String value) {duree.set(value);}
    public StringProperty dureeProperty() { return duree;}
 
 
    public Musique(Integer id, String titre, String artiste, String duree){
        this.id.set(id);
        this.titre.set(titre);
        this.artiste.set(artiste);
        this.duree.set(duree);
 
    }
 
 
 
 
 
 
}

03/12/2015, 13h47
Kazh Du
Certainement à la ligne 42. La réponse est toujours 42. (Ah, zut, aucune de tes classes n'a 42 lignes. Ton problème n'est pas la grande question de l'univers, désolé)
Sinon, plus sérieusement, c'est quoi le problème, ça ne compile pas ? Ça ne s’exécute pas ? Ça ne donne pas le bon résultat ?

A part ça, ce code me choque :
Code:

1 2 3 public ListeMusique(ListProperty<Musique> lesMusiques){ lesMusiques=MusiqueDur(); }
Tu essaies d'attribuer une instance de MusiqueDur à une variable de classe ListProperty<Musique>.

Si tout compile sauf ces 2 lignes car je ne vois vraiment pas cmment faire ...

Code:

1
2
3
4
5
6
7
8
9
public ListeMusique(ListProperty<Musique> lesMusiques){
        lesMusiques=();
    }
 
 
    public void lireMusique(){
        ArrayList<Musique> listeTemporaire = ListeMusique.lireMusique();
        lesMusiques.add();
    }

Code:

1 2 3 4 5 6 7 8 9 10 11 public ListeMusique(ListProperty<Musique> lesMusiques){ this.lesMusiques=lesMusiques; // ou this.lesMusiques.set(lesMusiques.get()) } // ou bien public ListeMusique(ListProperty<Musique> unAutreNom){ lesMusiques=unAutreNom; // ou lesMusiques.set(unAutreNom.get()) }
Je pense que c'est mieux comme ça. A toi de choisir.
Code:

1 2 3 4 public void lireMusique(){ ArrayList<Musique> listeTemporaire = ListeMusique.lireMusique(); lesMusiques.add(); }
Par contre là, je ne comprends pas tout. Pourquoi lesMusiques.add(); ? Que veux-tu rajouter ? La méthode add prend obligatoirement un paramètre.
Ensuite ListeMusique.lireMusique(); n'est pas statique, donc tu ne peux pas l'appeler de cette façon. La déclarer statique n'est pas non plus la solution à mon avis, je pense que tu devrais passer par le pattern singleton :
Code:

1 2 3 public void lireMusique(){ ArrayList<Musique> listeTemporaire = ListeMusique.getSingleton().lireMusique(); }

03/12/2015, 14h13
fazexx

En fait, mon musiqueadd sert a ajouter les musiques qui sont dans la classe MusiqueDur.
J'aimerais avoir le lien entre MusiqueDur et la listeView ..
03/12/2015, 14h24
Kazh Du
Ok, on va y arriver.

Déjà il faut typer la liste de résultat dans ta classe MusiqueManager (pas non plus besoin de signaler explicitement qu'il s'agit d'une ArrayList au passage) :

Code:

public abstract List<Musique> lireMusique(Integer id,String titre,String artiste,String duree);

Ensuite dans ta classe ListeMusique :
Code:

1 2 3 4 public void lireMusique(){ List<Musique> listeTemporaire = MusiqueManager.getSingleton().lireMusique(); lesMusiques.addAll(listeTemporaire); }
03/12/2015, 14h33
fazexx

Mais il faut forcement utiliser le singleton ??
Deplus, dans le MusiqueDur, il me souligne le MusiqueDur dans

Code:

public class MusiqueDur extends MusiqueManager{
03/12/2015, 14h46
Kazh Du

Souligne ? :weird:
Il faut être sûr que les méthodes en override aient exactement le même intitulés que les méthodes abstraites de la classe mère (j'ai par exemple mis une minuscule au début du nom de la méthode dans mon exemple).

Ce n'est pas obligé de mettre en singleton mais je te le recommande d'un point de vue général, les managers sont des classes qui s'occupent de la cohérence des données, si tu les multiplie, tu risques d'avoir des conflits avec plusieurs listes d'objets théoriquement identiques mais au final contradictoires. C'est juste une bonne habitude à prendre.
03/12/2015, 14h53
fazexx

MAis je dois créer une classe Singleton non ? Car ça me propose une methode mais je ne pense pas ...
03/12/2015, 15h14
Kazh Du

En y réfléchissant, le pattern singleton ne doit pas être utilisé de cette manière.

Le but était de te proposer un moyen d'accéder à ton manager. Le passer en attribut dans ta classe Liste Musique ne me semble pas être la meilleure solution. Après c'est à toi de voire, du moment que tu récupères tes données.

Tu peux aussi qui utiliser une sorte de super classe (appelons la Application par exemple) en singleton qui un getter sur ton manager. Bref, cherche la solution qui te convient le mieux, le but étant de récupérer la liste de Musique que tu cherches.