Discussion :

[fabienfoo]

Bonjour,

J'ai en petit souci algorithmique sur la boucle suivante

Code :

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

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

[Skalp]

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.

[fabienfoo]

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

[fabienfoo]

Code :

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

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

[Arthis]

Pourquoi fais tu des boucles?

[fabienfoo]

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

[Arthis]

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

bon code à toi,

[Skalp]

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

Code :

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

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 :

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

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

[fabienfoo]

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