Loop sur existence d'un fichier

Bonjour,

J'ai en petit souci algorithmique sur la boucle suivante

Code:

---

1 2 3 4 5 6 7 8 9

strPicFile = request.PhysicalApplicationPath + "GeneratedGraphs\\ProductImage" + ProdId + DateTime.Now.Ticks + ".jpg"; for (int i = 0; i < 6; i++) { while (!File.Exists(strPicFile)) { FactoryTools.generateImageFilefromUrl(ProdImage, strPicFile); } }

---

En gros tant que mon image n'est pas chargée, j'essaye de le faire seulement 5 fois de suite.
Mais si elle est chargée je passe 5 fois dans la boucle ce qui n'est pas optimal.

Auriez vous une idée pour optimiser ce bout de code, avec un compteur probablement.

Merci de votre aide précieuse

Tu as imbriqué un for et un while ?
As-tu testé ce morceau de code ?
Il me semble que tant que le fichier n'existe pas, l'exécution va rester dans la boucle while indéfiniment...
Dans ce cas, à quoi bon répéter la boucle 5 fois ?

Ceci dit, pour sortir d'une boucle quand tu le souhaites, tu peux utiliser le mot-clé break. Peut-être que cela répondra à ton besoin.

Mais c'est très juste dis donc
j'avais testé mais le fichier existait toujours!!
Mais comment utilises-tu ce break pour lui dire d'exécuter ce while un maximum de 5 fois

Merci d'avoir mis le doigt sur une si grosse aberration

Code:

---

1 2 3 4 5 6 7 8

int nbTry = 0; While (!File.Exists(strPicFile)) { FactoryTools.generateImageFilefromUrl(ProdImage, strPicFile); nbTry++; if(nbTry>5)break; }

---

ça pourrait être juste ce bout de code.
ça me donne le tournis ces boucle

Pourquoi fais tu des boucles?

En fait j'essaye d'enregistrer une image a partir d'une url, je veux lancer ma méthode un maximum de 5 fois, si je ne la récupère pas j'utilise une image par défaut.
Mais si tu as une idée pour ne pas utiliser de boucle je suis preneur aussi

merci

Au temps pour moi, j'ai compris.
et Je n'ai pas mieux.

bon code à toi,

Ben je vais passer mon post à résolu.
Merci d'avoir skalpé mon while imbriqué dans un for.
Je trouve tout de même cette boucle un peu laide mais elle fonctionne...
Enfin si vous avez des idées pour un code plus propre je suis encore preneur.

Tout de même répéter 5 fois l'infini ça avait qque chose de mystique:lol:

Citation:

---

Envoyé par fabienfoo

Mais comment utilises-tu ce break pour lui dire d'exécuter ce while un maximum de 5 fois ?

---

Citation:

---

Envoyé par fabienfoo

Code:

---

1 2 3 4 5 6 7

int nbTry = 0; while (!File.Exists(strPicFile)) { FactoryTools.generateImageFilefromUrl(ProdImage, strPicFile); nbTry++; if (nbTry > 5) break; }

---

---

Oui voilà, par exemple :)

Mais dans la mesure où tu utilises un compteur, autant utiliser une boucle for et sortir en cas de succès :

Code:

---

1 2 3 4 5

for (int i = 0; i < 5; i++) { FactoryTools.generateImageFilefromUrl(ProdImage, strPicFile); if (File.Exists(strPicFile)) break; }

---

Citation:

---

Envoyé par Skalp

Oui voilà, par exemple :)

Mais dans la mesure où tu utilises un compteur, autant utiliser une boucle for et sortir en cas de succès :

Code:

---

1 2 3 4 5

for (int i = 0; i < 5; i++) { FactoryTools.generateImageFilefromUrl(ProdImage, strPicFile); if (File.Exists(strPicFile)) break; }

---

---

While, for... c'est kif-kif... une fois compilé, ça devient while de toutes façons (il me semble) :oops:

ceci-dit, il y a surtout une différence dans la facilité de lecture...
Quoiqu'avec le compteur du for, on peu faire des choses qui ne sont pas forcément disponibles par un while, non ?