Discussion :

[Dadoue]

Bonsoir à tous,

Voila, je vous expose mon problème:

J'ai créé une fonction qui, à partir d'un objet de type véhicule, me retourne une collection (d'objets Vehicule) de ces véhicules qui ont besoins d'un entretien rapidement.

Seulement, il semble que ma fonction soit dans l'incapacité de me retourner cette fameuse collection...

le code de ma fonction est le suivant:

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
public CollectionVehicules vehiculesAentretenir()
        {
            int i;
            bool b;
            Vehicule unVehicule = new Vehicule();
 
            for (i = 1; i <= CollectionVehicules.ColVehicule.Count; i++)//Parcours de la collection
            {
                unVehicule = (Vehicule)CollectionVehicules.ColVehicule[i];//extraction de l'objet
                if (necessiteEntretien(unVehicule) == true)
                {
                    CollectionVehicules.ColVehicule.Add(unVehicule);
                    //Si l'objet vehicule necessite un entretien, ajouter cet objet à la collection.
                }
            }
            return (CollectionVehicules.ColVehicule);//erreur de conversion...
            /* 
            Le :
                 return((CollectionVehicules)CollectionVehicules.ColVehicule);
            ne fonctionne pas non plus, une erreur de conversion s'affiche...
           */
        }

• la fonction necessiteentretien retourne un booléen
• le problème semble venir de la dernière ligne, mais je ne voit pas où il peut être...

Merci à tous ceux qui me répondront!

[Nathanael Marchand]

Bonsoir à tous,

Voila, je vous expose mon problème:

J'ai créé une fonction qui, à partir d'un objet de type véhicule, me retourne une collection (d'objets Vehicule) de ces véhicules qui ont besoins d'un entretien rapidement.

Seulement, il semble que ma fonction soit dans l'incapacité de me retourner cette fameuse collection...

le code de ma fonction est le suivant:

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
public CollectionVehicules vehiculesAentretenir()
        {
            int i;
            bool b;
            Vehicule unVehicule = new Vehicule();
 
            for (i = 1; i <= CollectionVehicules.ColVehicule.Count; i++)//Parcours de la collection
            {
                unVehicule = (Vehicule)CollectionVehicules.ColVehicule[i];//extraction de l'objet
                if (necessiteEntretien(unVehicule) == true)
                {
                    CollectionVehicules.ColVehicule.Add(unVehicule);
                    //Si l'objet vehicule necessite un entretien, ajouter cet objet à la collection.
                }
            }
            return (CollectionVehicules.ColVehicule);//erreur de conversion...
            /* 
            Le :
                 return((CollectionVehicules)CollectionVehicules.ColVehicule);
            ne fonctionne pas non plus, une erreur de conversion s'affiche...
           */
        }

• la fonction necessiteentretien retourne un booléen
• le problème semble venir de la dernière ligne, mais je ne voit pas où il peut être...

Merci à tous ceux qui me répondront!

Pourquoi ne pas travailler avec des List<Vehicule> ?

Ca pourrait devenir:

Code :

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

1
2
3
4
public List<Vehicule> vehiculesAentretenir()
        {
            return CollectionVehicules.Where(vehicule=>necessiteEntretien(vehicule)).ToList();
        }

Plus compréhensible nan?
Bon ca c'était pour le côté "ouh ce code m'a l'air bien compliqué pour rien"

Pour le côté coup de main sur ton code, tant qu'on ne sait pas ce qu'est l'objet CollectionVehicules on peut rien dire

Erf tant qu'on y est je pense que y'a beaucoup plus que ca à changer dans ton code. Par exemple pourquoi necessiteEntretien n'est pas une méthode membre de Vehicule?

[hegros]

Pourquoi ne pas travailler avec des List<Vehicule> ?

Ca pourrait devenir:

Code :

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

1
2
3
4
public List<Vehicule> vehiculesAentretenir()
        {
            return CollectionVehicules.Where(vehicule=>necessiteEntretien(vehicule)).ToList();
        }

Plus compréhensible nan?

+1

Erf tant qu'on y est je pense que y'a beaucoup plus que ca à changer dans ton code. Par exemple pourquoi necessiteEntretien n'est pas une méthode membre de Vehicule?

Cela se discute. D'un point de vue grasp pattern on peut se poser la question si l'objet véhicule est l'expert de cette information. A priori je dirais que non et donc que cette méthode ne devrait pas être dans un véhicule. Du coup la signature NecessiteEntretien(Vehicule vehicule) me semble très bien à premier abord. Si en plus la méthode est dans une classe garage/centrediagnostic par exemple c'est encore mieux