Changement d'extension de multiples fichiers [Résolu]

Question

Posez votre question Signaler

Changement d'extension de multiples fichiers [Résolu]

ToxicHayabusa

Messages postés: 924
Date d'inscription: dimanche 7 décembre 2014
Statut: Membre
Dernière intervention: 2 septembre 2018

- 13 avril 2018 à 17:22 - Dernière réponse : vb95

Messages postés: 1985
Date d'inscription: samedi 11 janvier 2014
Statut: Contributeur
Dernière intervention: 14 décembre 2019

- 14 avril 2018 à 17:35

C# / .NET Posez votre question Signaler

Bonjour,

Je rencontre récemment un problème dans un code C#. En effet mon but est de détecter automatiquement tout les fichiers présents dans un dossier donné et d'en changer leur extension.
J'ai donc essayé de faire ceci à partir de ce code :

           
            string fExt;
            string fnExt = ".xbxo";
            string fFromName;
            string fToName;
            
            string fPath = @"C:\Users\ToxicHayabusa\Desktop\Test";
            FileInfo[] files = new DirectoryInfo(fPath).GetFiles();
            foreach (var f in files)
            {
                fFromName = Path.GetFileNameWithoutExtension(f.Name);

                fExt = Path.GetExtension(f.Name);

                fFromName = string.Format("{0}", f.Name);

                fToName = string.Format("{0}{1}", f.Name, fnExt);

                File.Move(fFromName, fToName);
                
            }

Ce code est un mélange que j'ai pu faire à partir de deux codes trouvés sur internet...Malheureusement il ne fonctionne pas comme je le voudrais. En effet, là où ça bloque c'est à la dernière ligne, pour le File.Move, car lorsque la commande est donc censé remplacer les anciens fichiers avec les nouveaux portant la nouvelle extension, celui-ci cherche les fichiers dans le dossier local et non dans le dossier précisé au début du code..

Je ne sais pas si le code est maladroit ou mal fait, malgré mes recherches je n'ai pas trop réussit à comprendre les méthodes utilisé pour tout ça et donc il se peut que le code soit même totalement faux, c'est pour cela que je requiert votre aide, pour m'indiqué là où se trouverait le problème et même corriger mon code si possible.

Merci d'avance.

Afficher la suite

A voir également:

Extension fichier c#
C# get filename without extension - Meilleures réponses
C# getfilenamewithoutextension - Meilleures réponses
Extension fichier javascript ✓ - Forum - Javascript / DHTML
Récupérer type d'un fichier - Forum - C# / .NET
Extension fichier text ✓ - Forum - Visual Basic 6
Extension fichier disponible en upoad ✓ - Forum - PHP
Question extension fichier + pb float ✓ - Forum - C / C++ / C++.NET

3 réponses

Répondre Posez votre question

Whismeril · Answer 1 · 2018-04-13T17:36:09+0200

Signaler Commenter la réponse de Whismeril

Whismeril

Messages postés: 14045
Date d'inscription: mardi 11 mars 2003
Statut: Contributeur
Dernière intervention: 14 décembre 2019

324 13 avril 2018 à 17:36

0

Merci

Bonjour
trouver des bouts de code sur internet c'est une chose, mais les fusionner sans comprendre c'est pas le mieux.
Pour les comprendre, y'a 2 options (cumulables),

lire la doc en ligne https://msdn.microsoft.com/en-us/library/system.io.path.getfilenamewithoutextension%28v=vs.110%29.aspx
exécuter en pas à pas et regarder le résultat de chaque étapes.

Si tu avais appliqué une seule des ces 2 options, tu n'aurais pas posé cette question, car tu aurais vu que

Path.GetFileNameWithoutExtension(f.Name);

ne retourne que le nom du fichier et pas le chemin complet.

Donc tu aurais peut-être demandé comment changer ton extension et là encore, en lisant la doc en ligne ou même en ayant un peu de curiosité avec la class Path (qu'est ce propose intellisense que je tape Path. ?), tu aurais pu voir la méthode ChangeExtension https://msdn.microsoft.com/en-us/library/system.io.path.changeextension(v=vs.110).aspx

Afficher les 13 commentaires

Signaler Commenter la réponse de ToxicHayabusa

ToxicHayabusa

Messages postés: 924
Date d'inscription: dimanche 7 décembre 2014
Statut: Membre
Dernière intervention: 2 septembre 2018

- 13 avril 2018 à 19:26

Même avec ce code j'obtient la même erreur, il cherche le fichier dans le dossier local..

            string fExt;
            string fnExt = ".pwned";
            string fFromName;
            string fToName;

            string fPath = @"C:\Users\ToxicHayabusa\Desktop\Test";
            FileInfo[] files = new DirectoryInfo(fPath).GetFiles();
            foreach (var f in files)
            {
                //get the filename without the extension
                fFromName = Path.GetFileNameWithoutExtension(f.Name);
                //get the file extension
                fExt = Path.GetExtension(f.Name);
                

                fToName = Path.ChangeExtension(fPath + fFromName, fnExt);

                //rename the file by moving to the same place and renaming
                File.Move(fFromName + fExt, fToName);
            }

https://image.noelshack.com/fichiers/2018/15/5/1523640390-894ef.png

Signaler Commenter la réponse de ToxicHayabusa

ToxicHayabusa

Messages postés: 924
Date d'inscription: dimanche 7 décembre 2014
Statut: Membre
Dernière intervention: 2 septembre 2018

- 14 avril 2018 à 00:40

Ok, au pire si tu lisais mieux ce que je te répondais :
• je prend les conseilles, j'ai même testé tes codes et je t'ai montré qu'ils
ne fonctionnaient pas (je les ai vraiment testé)
• J'ai déjà dis plus haut que j'avais déjà lu ces docs et que je ne les
avaient pas trouvé utiles.

Tu parles de bénévolat, mais ce que tu te contentes de faire c'est de suivre l'exemple de Microsoft dans leur doc alors que comme tu pourras le voir, dans la doc ils renvoient juste une valeur string en echo, donc forcément qu'ils vont avoir le chemin vers le dossier, en revanche ils ne mettent jamais en pratique la commande pour remplacer l'extension d'un fichier. Et la seule commande que tu m'as donné était une version compacté de la mienne, et malgré ça je l'ai essayée, et ça ne fonctionnait pas non plus.

Signaler Commenter la réponse de Whismeril

Whismeril

Messages postés: 14045
Date d'inscription: mardi 11 mars 2003
Statut: Contributeur
Dernière intervention: 14 décembre 2019

324 - 14 avril 2018 à 13:06

• je prend les conseilles, j'ai même testé tes codes et je t'ai montré qu'ils
ne fonctionnaient pas (je les ai vraiment testé)

ben non, moi je t'ai proposé

File.Move(f.Name, nouveauNom);

et toi tu m'as répondu essayer

File.Move(fFromName + fExt, fToName);

aucun rapport

si on imagine que ton fichier est C:\Toto\tutu.txt
Ce que je te proposais revenait à

File.Move("C:\Toto\tutu.txt", "C:\Toto\tutu.csv");

chemin complet d'un fichier qui existe vers un nouveau nom.
Et toi

File.Move("tutu" + "txt", "C:\Toto\tutu.csv");

ou p'tet

File.Move("tutu" + ".txt", "C:\Toto\tutu.csv");

(j'ai toujours pas testé, j'ai pas plus d'EDI qu'hier sous la main) fichier qui n'existe pas.
Je te l'ai dit dès le premier message.

Y'a la raison évidente que je t'ai donnée, à savoir Path.GetFileNameWithoutExtension. Mais peut être aussi que f.Name est un chemin relatif (toujours pas d'EDI pour vérifier).
Je t'ai aussi conseillé d'exécuter en pas à pas et de regarder le contenu des variables à chaque étape.
Si tu avais fait ça (tout court? sérieusement?), tu aurais du voir que d'une part, tu n'avais pas le chemin complet et que d'autre part il manquait un "\" quelque part.

D'autre part, c'est pas parce que tu n'as pas compris la doc Microsoft qu'elle n'est pas utile.
C'est pas toujours super clair, je te l'accorde (traduction parfois malheureuse je suppose), mais y'a généralement tout ce qu'il faut dedans, et là c'est le cas.

Dans l'exemple de File.Move, il y a des chemins complets
https://msdn.microsoft.com/fr-fr/library/system.io.file.move(v=vs.110).aspx

Dans celle de Path.GetFileNameWithoutExtension y'a bien écrit qu'on perd le chemin du dossier.

Pour Path.ChangeExtension

dans la doc ils renvoient juste une valeur string en echo

oui!!!! C'est File.Move qui renomme un fichier, il faut donc lui donner un nom à appliquer

File.Move(f.Name, Path.ChangeExtension(f.Name, '.csv"));

Donc oui, j'ai cherché à te faire réfléchir par toi même, parce que d'une part c'est la règle de ce forum et d'autre part sans EDI pas de code certifié efficace.

Par contre, je peux te proposer d'utiliser Directory.GetFiles() https://msdn.microsoft.com/fr-fr/library/system.io.directory.getfiles(v=vs.110).aspx, plutôt que new DirectoryInfo(fPath).GetFiles() parce que je suis sûr que ça retourne les chemins complets et qu'en plus y'a une option pour obtenir le contenu de tous les sous répertoires.
Ça tient en 2 lignes et ça évite la récursivité (toujours pas testé)

foreach(string f in Directory.GetFiles(@"C:\Users\ToxicHayabusa\Desktop\Test","*.txt",SearchOption.AllDirectories))
           File.Move(f, Path.ChangeExtension(f, '.csv"));

Signaler Commenter la réponse de vb95

vb95

Messages postés: 1985
Date d'inscription: samedi 11 janvier 2014
Statut: Contributeur
Dernière intervention: 14 décembre 2019

79 - 14 avril 2018 à 15:37

bonjour Whismeril
Je viens d'essayer avec ton code juste au-dessus !
cela fonctionne très bien
Moi je l'ai fait avec "*.*" en second paramètre ce qui me change toutes les extensions trouvées dans le répertoire et tous les sous-répertoires

foreach(string f in Directory.GetFiles(@"C:\Users\ToxicHayabusa\Desktop\Test","*.*",SearchOption.AllDirectories))
           File.Move(f, Path.ChangeExtension(f, '.csv"));

comme quoi que chercher dans MSDN est une mine d'informations
A ++

Signaler Commenter la réponse de ToxicHayabusa

ToxicHayabusa

Messages postés: 924
Date d'inscription: dimanche 7 décembre 2014
Statut: Membre
Dernière intervention: 2 septembre 2018

- 14 avril 2018 à 16:39

Je t'ai dis que j'avais essayé avec ton code, je n'ai pas forcément partager le code. J'essayais juste de multiples codes, et je partageais ceux que je faisais moi même pour comprendre ce qui n'allait pas, mais je testais aussi ceux proposés à côté. Merci de bien lire mes réponses.

Je n'ai pas dis que je ne comprenais pas les docs, juste qu'ils ne faisaient pas assez de mise en pratique dans leurs exemples, ce qui serait tout de même plus intéressant. J'ai quant à moi trouvé bien plus d'informations utiles sur des sites comme StackOverflow ou CodeProject.

Signaler Commenter la réponse de ToxicHayabusa

ToxicHayabusa

Messages postés: 924
Date d'inscription: dimanche 7 décembre 2014
Statut: Membre
Dernière intervention: 2 septembre 2018

- 14 avril 2018 à 17:00

En revanche je te remercie pour le temps consacré, je sais que ce ne doit pas être facile d'aider sans donner de code, surtout avec moi, car j'ai souvent besoin d'exemples concrets pour mieux comprendre des codes.. (je préfère analyser un code complet pour savoir l'utiliser que de voir des petits exemples pas très concrets..)

ToxicHayabusa · Answer 2 · 2018-04-13T22:22:41+0200

Signaler Commenter la réponse de ToxicHayabusa

ToxicHayabusa

Messages postés: 924
Date d'inscription: dimanche 7 décembre 2014
Statut: Membre
Dernière intervention: 2 septembre 2018

13 avril 2018 à 22:22

0

Merci

Voici l'image.

En revanche le code donné résume simplement mon code actuel, pour preuve je l'ai testé, et il fait exactement la même erreur que mon code.

Signaler Commenter la réponse de vb95

vb95

Messages postés: 1985
Date d'inscription: samedi 11 janvier 2014
Statut: Contributeur
Dernière intervention: 14 décembre 2019

79 - Modifié le 13 avril 2018 à 23:09

Bonsoir !
La solution est ici


            string newextension = ".xbxo";
            string filename;
            string fpath = @"D:\Test\";
            FileInfo[] files = new DirectoryInfo(fpath).GetFiles();
            foreach (FileInfo f in files)
            {
                filename = Path.ChangeExtension(f.Name, newextension);
                File.Move(fpath + f.Name, fpath + filename);
            }

Lors du File.Move il faut préciser le répertoire des fichiers sinon C# prend le dossier de l'application par défaut : c'est cela que l'on appelle un chemin relatif comme dit Whismeril !

ToxicHayabusa · Answer 3 · 2018-04-14T00:47:38+0200

Signaler Commenter la réponse de ToxicHayabusa

ToxicHayabusa

Messages postés: 924
Date d'inscription: dimanche 7 décembre 2014
Statut: Membre
Dernière intervention: 2 septembre 2018

14 avril 2018 à 00:47

0

Merci

J'apprécie l'aide proposée, en revanche, comme je l'ai dis plus haut cette méthode ne fonctionne pas. Je ne suis pas débile non plus, j'ai très bien compris la méthode et comment elle est censée fonctionner, et j'ai aussi essayé de mon côté plusieurs possibilités de code, toutes menant à la même erreur.

(Au passage, avec votre méthode VB95, le programme rajoute même le nom du dossier en plus du chemin au nom du fichier qu'il cherche..)

Afficher les 13 commentaires

Signaler Commenter la réponse de vb95

vb95

Messages postés: 1985
Date d'inscription: samedi 11 janvier 2014
Statut: Contributeur
Dernière intervention: 14 décembre 2019

79 - Modifié le 14 avril 2018 à 01:46

Bonsoir !
J'ai moi-même testé sur 2 fichiers dans un répertoire et elle fonctionne très bien
As-tu bien mis un antislash à la fin de la String fpath ?
Car entre Test et 456.pxd il n'y est pas ce qui fait qu'il ne peut pas trouver les fichiers !

regarde cette ligne doit être


string fPath = @"C:\Users\ToxicHayabusa\Desktop\Test\";

et non

string fPath = @"C:\Users\ToxicHayabusa\Desktop\Test";

et aussi remplace var par FileInfo dans le foreach car f est une variable de type FileInfo

De plus tu écris : (Au passage, avec votre méthode VB95, le programme rajoute même le nom du dossier en plus du chemin au nom du fichier qu'il cherche..)

le nom du dossier c'est le chemin et pour chercher un fichier il faut
1) son chemin ( le répertoire où il se trouve )
2) son nom avec l'extension

filename et f.name ne sont que les noms des fichiers . il faut rajouter fpath qui est le chemin de ces fichiers ( le répertoire qui les contient)

Signaler Commenter la réponse de ToxicHayabusa

ToxicHayabusa

Messages postés: 924
Date d'inscription: dimanche 7 décembre 2014
Statut: Membre
Dernière intervention: 2 septembre 2018

- Modifié le 14 avril 2018 à 02:30

Oui mais bon, pour ce genres de choses, je ne vois pas en quoi je n'aurais rien appris...Vu que c'était juste un caractère oublié.
Sinon j'ai essayé un code comme ceci :

            
            string fnExt = ".pwned";
            string filename;
            string npath;

            string fPath = @"C:\Users\ToxicHayabusa\Desktop\Test\";
            FileInfo[] files = new DirectoryInfo(fPath).GetFiles();
            string[] folders = Directory.GetDirectories(fPath);
            foreach (var f in files)
            {
                filename = Path.ChangeExtension(f.Name, fnExt);
                File.Move(fPath + f.Name, fPath + filename);
            }
            foreach (var d in folders)
            {
              npath = Path.GetDirectoryName(d);
              FileInfo[] nfiles = new DirectoryInfo(npath).GetFiles();
                foreach (FileInfo f in nfiles)
                {
                    filename = Path.ChangeExtension(f.Name, fnExt);
                    File.Move(npath + f.Name, npath + filename);
                }
            }

Mais ça n'a pas l'air de fonctionné du tout...Est-ce que je suis dans la mauvaise direction ou bien y'a des choses qui pourraient fonctionner dedans ? (et donc dans ce cas il faut que je continue à chercher à modifier le code pour qu'il fonctionne..)

(ps : je viens de voir que les balises de code C# semblent un peu buggées..)

EDIT:Je viens de lire la doc Microsoft sur Path.GetDirectoryName, je comprend mieux pourquoi ça ne fonctionne pas, mais y a-t-il un moyen d'obtenir le nom du dossier concerné ? Et non sa racine ?

Signaler Commenter la réponse de ToxicHayabusa

ToxicHayabusa

Messages postés: 924
Date d'inscription: dimanche 7 décembre 2014
Statut: Membre
Dernière intervention: 2 septembre 2018

- Modifié le 14 avril 2018 à 03:02

Bon j'ai finis par trouvé par moi même, j'avais seulement deux petites choses à changer dans le code ! (et pour vous faire plaisir j'ai retiré ce "var" ;) )
Voici le code final !

            string fnExt = ".pwned";
            string filename;
            string npath;

            string fPath = @"C:\\Users\\ToxicHayabusa\\Desktop\\Test\\";
            FileInfo[] files = new DirectoryInfo(fPath).GetFiles();
            string[] folders = Directory.GetDirectories(fPath);
            foreach (FileInfo f in files)
            {
                filename = Path.ChangeExtension(f.Name, fnExt);
                File.Move(fPath + f.Name, fPath + filename);
            }
            foreach (string d in folders)
            {
              npath = Path.GetFullPath(d);
              FileInfo[] nfiles = new DirectoryInfo(npath).GetFiles();
                int pathlg = npath.Length;
                npath = npath.Insert(pathlg , "\\");
                foreach (FileInfo f in nfiles)
                {
                    filename = Path.ChangeExtension(f.Name, fnExt);
                    File.Move(npath + f.Name, npath + filename);
                }
            }

ps: afin de rendre le code plus lisible sur le site, j'ai temporairement modifié le chemin "@"C:\Users\ToxicHayabusa\Desktop\Test\" en "@"C:\\Users\\ToxicHayabusa\\Desktop\\Test\\" "

EDIT: Finalement je me suis rendu compte que mon code se limitait aux sous fichiers au premier degré, et ne prend donc pas en compte les sous-sous fichiers ect..
Du coup je me retrouve de nouveau bloqué, car à moins de faire une boucle de la même méthode des dizaine de fois pour être sûr que ça prendra en compte tout les sous dossiers possibles, il faut que je trouve une méthode plus viable..

Signaler Commenter la réponse de ToxicHayabusa

ToxicHayabusa

Messages postés: 924
Date d'inscription: dimanche 7 décembre 2014
Statut: Membre
Dernière intervention: 2 septembre 2018

- Modifié le 14 avril 2018 à 03:51

En revanche pour la méthode que vous me proposez c'est malheureusement d'un niveau trop élevé pour moi, je ne sais même pas comment tester si cela peut être un fichier ou un répertoire (d'ailleurs, ne fait-il pas déjà la différence ? Puisqu'il me semble que FileInfo ne prend en compte que les fichiers et non dossiers..)

En suivant de mon mieux votre méthode, j'ai donc élaboré ce code, qui ne fonctionne pas pour autant, et je ne comprend pas pourquoi..

            string fnExt = ".pwned";
            string filename;
            string npath;

            string fPath = @"C:\\Users\\ToxicHayabusa\\Desktop\\Test\\";
            FileInfo[] files = new DirectoryInfo(fPath).GetFiles();
            string[] folders = Directory.GetDirectories(fPath);

            foreach (FileInfo f in files)
            {
                filename = Path.ChangeExtension(f.Name, fnExt);
                File.Move(fPath + f.Name, fPath + filename);
            }
            
            
                foreach (string d in folders)

                {
                    void ChangeExt()
                    {
                        npath = Path.GetFullPath(d);
                        FileInfo[] nfiles = new DirectoryInfo(npath).GetFiles();
                        string[] nfolders = Directory.GetDirectories(npath);
                        int pathlg = npath.Length;
                        npath = npath.Insert(pathlg, "\\");
                        foreach (FileInfo f in nfiles)
                        {
                            filename = Path.ChangeExtension(f.Name, fnExt);
                            File.Move(npath + f.Name, npath + filename);
                        }
                        foreach (string x in nfolders)
                        {
                            ChangeExt();
                        }
                    }
                }

Signaler Commenter la réponse de vb95

vb95

Messages postés: 1985
Date d'inscription: samedi 11 janvier 2014
Statut: Contributeur
Dernière intervention: 14 décembre 2019

79 - 14 avril 2018 à 04:04

Cadeau !
Et j'ai testé

private void Button1_Click(object sender, EventArgs e) 
        {
            Changeextension(@"D:\Test");  // répertoire source
        }

        private void Changeextension(string fpath) 
        {
            string newextension = ".xbxo";
            string filename;
            // on recherche les fichiers et les sous-répertoires du répertoire courant
            String[] files = Directory.GetFileSystemEntries(fpath);
            foreach (String file in files)
            {
                // Vérification si c'est un fichier ou un répertoire 
                FileInfo f = new FileInfo(file);
                fpath = f.Directory + "\\";
                FileAttributes Fa = File.GetAttributes(f.FullName);
                if ((Fa & FileAttributes.Directory) == FileAttributes.Directory)
                   // répertoire 
                   Changeextension(f.FullName);
                else
                   // fichier 
                {
                filename = Path.ChangeExtension(f.Name, newextension);
                File.Move(fpath + f.Name, fpath + filename);
                }
            }
        }

Signaler Commenter la réponse de vb95

vb95 - Modifié le 14 avril 2018 à 04:22

Comme tu le vois la fonction Changeextention s'appelle elle-même dès que la fonction trouve un répertoire .
C'est ça la récursivité

Attention le répertoire d'origine n'a pas d'antislash ici

Changeextension(@"D:\Test");  // répertoire source

L'antislash est rajouté dans la fonction elle-même

Signaler Commenter la réponse de ToxicHayabusa

ToxicHayabusa

Messages postés: 924
Date d'inscription: dimanche 7 décembre 2014
Statut: Membre
Dernière intervention: 2 septembre 2018

- Modifié le 14 avril 2018 à 04:37

Franchement merci ! J'avais compris pour la récursivité en revanche, comme vous pouviez le voir dans mon essai de code précédent.. Mais il me manquait pas mal de notions (comme FileAttributes et Directory.GetFileSystemEntries (qui d'ailleurs je n'aurais jamais soupçonné servir à ce à quoi il sert vu le nom..)) Sur ce je vais bien étudié ce code dans le but de bien le comprendre, et je vais enfin pouvoir continuer mon long travail qui m'attend..

Merci encore.

ps: Même avec un antislash dans le répertoire d'origine, cela fonctionne

Signaler Commenter la réponse de ToxicHayabusa

ToxicHayabusa

Messages postés: 924
Date d'inscription: dimanche 7 décembre 2014
Statut: Membre
Dernière intervention: 2 septembre 2018

- Modifié le 14 avril 2018 à 16:43

J'ai remarqué que dans le code il manquait une chose en revanche, si le fichier est utilisé par un processus en cours, le logiciel ne réussira pas à le modifier et risque de planter, j'ai donc rajouté un "try {} catch{}" au niveau de

                {
                filename = Path.ChangeExtension(f.Name, newextension);
                File.Move(fpath + f.Name, fpath + filename);
                }

ce qui donne (pour ceux venant chercher de l'aide sur ce code)

         {
                    try
                    {
                        filename = Path.ChangeExtension(f.Name, newextension);
                        File.Move(fpath + f.Name, fpath + filename);
                    }
                    catch { }
                }

De cette manière il ne prendra pas en compte les erreurs et continuera le code.

Signaler Commenter la réponse de vb95

vb95

Messages postés: 1985
Date d'inscription: samedi 11 janvier 2014
Statut: Contributeur
Dernière intervention: 14 décembre 2019

79 - Modifié le 14 avril 2018 à 17:41

Bonjour !
Tout à fait judicieux !
Si tu en avertissait l'utilisateur

         {
                    try
                    {
                        filename = Path.ChangeExtension(f.Name, newextension);
                        File.Move(fpath + f.Name, fpath + filename);
                    }
                    catch ex as Exception
                   {
                      MessageBox.Show(ex.message + Environment.newline +"Changement extension non fait sur" + fpath + f.Name + " (fichier utilisé)")

                   }    
     }

As-tu regardé les messages 25 et 26 qui proposent une alternative sans récursivité visible ?
Mais ainsi pas de Try Catch possible

Changement d'extension de multiples fichiers [Résolu]

Votre réponse