Discussion :

[emayen01]

Bonjour,
j'ai fait un test mais marche pas , probablement ya une erreur quelques part , merci pour vos interventions.

voici le code :

index = file.Name.IndexOf("_events || _monthly");
if (index >= 0)

{
File.Copy(directory + @"\" + file.Name.ToString(), FinalDirectory + @"\" + file.Name.ToString(), true);
}

Mon code marche bien quand je mets que (_events dans la condition ) mais quand je rajoute (_monthly) à la condition ca marche pas.
C bien le (||) qu'il faut faire , je vois pas l'erreur tout en sachant que ma condition doit verifier l'existence de l'une des deux .

merci

[Bluedeep]

Et où as tu été pecher que la syntaxe de recherche dans une chaine était la syntaxe C# ?

Le indexOf te cherche l'occurence d'une chaine, or ici il cherche la chaine que tu as spécifié, en l'occurence "_events || _monthly".

Il n'y a pas d'opération OR logique dans ton code, juste une chaine qui contient "||" entre autres.

Tu peux faire plusieurs IndexOf.

[jsd03]

Bonjour,

a mon avis tu devrais plutôt faire ça :

Code :

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

1
2
3
4
5
if (file.Name.IndexOf("_events") >= 0 || file.Name.IndexOf("_monthly") >= 0)
 
{
File.Copy(directory + @"\" + file.Name.ToString(), FinalDirectory + @"\" + file.Name.ToString(), true);
}

La prochaine fois pense à mettre ton code entre les balises CODE

[emayen01]

Bonjour,

a mon avis tu devrais plutôt faire ça :

Code :

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

1
2
3
4
5
if (file.Name.IndexOf("_events") >= 0 || file.Name.IndexOf("_monthly") >= 0)
 
{
File.Copy(directory + @"\" + file.Name.ToString(), FinalDirectory + @"\" + file.Name.ToString(), true);
}

La prochaine fois pense à mettre ton code entre les balises CODE

Merci bcp .
ca marche bien

[jsd03]

Juste pour ajouter un petit truc.

Le fait de mettre >= 0 dans ta condition va prendre tout les fichiers même si ils ne contiennent pas "_events" ou "_monthly". Tu devrais le remplacer par > 0 non ?

[Bluedeep]

Juste pour ajouter un petit truc.

Le fait de mettre >= 0 dans ta condition va prendre tout les fichiers même si ils ne contiennent pas "_events" ou "_monthly". Tu devrais le remplacer par > 0 non ?

Non.
Si la chaine est au début, l'index est 0. Si elle n'existe pas l'index est -1. Donc ">= 0" c'est correct.

[emayen01]

Oui je vois mais au début j'ai declaré :

int index = 0;du coup je devrais faire >=0
mais la ca marche aussi , et ta raison c unutil de la laisser

la chaine est à la fin.

[jsd03]

Non.
Si la chaine est au début, l'index est 0. Si elle n'existe pas l'index est -1. Donc ">= 0" c'est correct.

A oui correct j'ai craqué

[emayen01]

Sinon je veux ajouter un try et catch pour gerer les exceptions .
as tu une idée ?

foreach (FileInfo file in files)
{
if (file.Name.IndexOf("_events") > 0 || file.Name.IndexOf("_monthly") > 0)
{
File.Copy(directory + @"\" + file.Name.ToString(), FinalDirectory + @"\" + file.Name.ToString(), true);
}

}

Merci

[Antho42]

Sinon je veux ajouter un try et catch pour gerer les exceptions .
as tu une idée ?

Code :

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

1
2
3
4
5
6
7
8
9
 
foreach (FileInfo file in files)
 {
   if (file.Name.IndexOf("_events") > 0 || file.Name.IndexOf("_monthly") > 0)
          {
   File.Copy(directory + @"\" + file.Name.ToString(), FinalDirectory + @"\" + file.Name.ToString(), true);
            }
 
 }

Merci

Tu veux gérer quoi comme exceptions ?
sinon pour le try~catch, il suffit d'englober le code qui est susceptible de générer l'exception. A toi de voir quel type d'exception, dans quel cas tu précise, sinon il y a Exception en Général , qui attrape tout.

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
 
try
{
   foreach (FileInfo file in files)
   {
     if (file.Name.IndexOf("_events") > 0 || file.Name.IndexOf("_monthly") > 0)
     {
       File.Copy(directory + @"\" + file.Name.ToString(), FinalDirectory + @"\" + file.Name.ToString(), true);
     }
   }
}
catch(Exception ex)
{
      Console.WriteLine(ex.Message);
}

[emayen01]

merciiiiiiii
je sais pas encore quelle exception je dois gerer mais comment ca marche pour l'exception generale ?

[Antho42]

merciiiiiiii
je sais pas encore quelle exception je dois gerer mais comment ca marche pour l'exception generale ?

http://msdn.microsoft.com/fr-fr/libr...exception.aspx

Tu y trouveras toutes les réponses

[emayen01]

ok merci .