Sortir d'un ForEach

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 :

Citation:

---

Absence de boucle englobant 'break' ou 'continue'

---

Code:

---

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

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:

---

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

---

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:

---

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.

Merci bcp pour vos réponses :ccool: 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)

Citation:

---

Envoyé par Gastiflex

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.

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

Citation:

---

'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

Citation:

---

Envoyé par 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'

---

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

Citation:

---

Envoyé par Bluedeep

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

---

;)

Bonjour,

Il vaut mieux préférer FirstOrDefault à SingleOrDefault.
En effet ce dernier vérifie la contrainte d'unicité et renvoie une exception s'il trouve au moins deux résultats.
Corollaire du constat précédent, pour vérifier cette unicité toute la collection doit être itérée, contrairement au FirstOrDefault qui quittera au premier élément concluant trouvé (et renvoyant null si aucun élément ne correspondait).

Cordialement !