Discussion :

[toss.net]

Salut,
Dans une liste de contrat, je dois sortir dès j'ai trouvé un contrat en cours. Cependant j'ai une erreur me disant :

Absence de boucle englobant 'break' ou 'continue'

Code :

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

1
2
3
4
5
6
7
8
9
10
bool bOk=false;
oPers.colContrats.ForEach(delegate(cContrat c)
{
    if (c.DateEntree <= DateTime.Now & DateTime.Now <= c.DateFin)
    {
        oPers.Load(p, ref lisPers);
        bOk=true;
    }
    if (bOk == true)  break; //erreur sur ce Break
});

Ca doit pas être bien compliqué mais je vois pas le problème

[ostenhard]

Bonjour,

Je pense que le break ne fonctionne pas car il est dans une méthode ForEach.

Pourquoi ne pas voir les choses légèrement différemment, en récupérant le premier élément correspondant à votre test et ensuite appeler la méthode Load ?

Code :

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

1
2
3
cContrat contrat = oPers.colContrats.Fisrt(c => c.DateEntree <= DateTime.Now && DateTime.Now <= c.DateFin )
if(contrat != null)
	oPers.Load(p, ref lisPers);

[Invité]

Techniquement t'es pas dans une boucle foreach, mais une méthode Foreach qui utilise un delegate. Tu dois pouvoir le réécrire comme ça (au passage je ne vois pas l'intérêt du booléen :

Code :

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

1
2
3
4
5
6
7
8
9
 
foreach(var c in oPers.colContrats)
{
    if (c.DateEntree <= DateTime.Now & DateTime.Now <= c.DateFin)
    {
        oPers.Load(p, ref lisPers);
        break;
    }
});

Ca c'est si tu veux utiliser foreach.
Mais t'as la solution de ostenhard qui est plus élégante : tu utilises la méthode first qui te renvoie le premier élément satisfaisant la conditition en paramètre.

[toss.net]

Merci bcp pour vos réponses il est vrai que j'ai toujours bêtement fait mes boucles comme cela.. (j'avoue même sans me pencher sur l'utilisation du Delegate)

[Bluedeep]

Mais t'as la solution de ostenhard qui est plus élégante : tu utilises la méthode first qui te renvoie le premier élément satisfaisant la conditition en paramètre.

Plutôt SingleOrDefault que First car First va lever une exception si la condition n'est pas rencontrée.

[toss.net]

Tiens d'ailleurs si je mettais First, il me retournait :

'System.Collections.Generic.List<cContrat>' ne contient pas une définition pour 'Fisrt' et aucune méthode d'extension 'Fisrt' acceptant un premier argument de type 'System.Collections.Generic.List<cContrat>' n'a été trouvée (une directive using ou une référence d'assembly est-elle manquante ?)

En remplacant First par SingleOrDefault , nickel

[Bluedeep]

Tiens d'ailleurs si je mettais First, il me retournait :
'System.Collections.Generic.List<cContrat>' ne contient pas une définition pour 'Fisrt' et aucune méthode d'extension 'Fisrt'

Oui, pour 'Fisrt' ce n'est pas étonnant; mais peut être que avec First ça marche mieux

[toss.net]

Oui, pour 'Fisrt' ce n'est pas étonnant; mais peut être que avec First ça marche mieux