Changement d'extension de multiples fichiers

Résolu

ToxicHayabusa

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

- 13 avril 2018 à 17:22
vb95

Messages postés: 2629
Date d'inscription: samedi 11 janvier 2014
Statut: Modérateur
Dernière intervention: 13 mai 2022

- 14 avril 2018 à 17:35

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.

A voir également:

Extension fichier c#
C# extension de fichier - Meilleures réponses
C# change extension - Meilleures réponses
Extension fichier text ✓ - Forum - Visual Basic 6
Extension fichier disponible en upoad ✓ - Forum - PHP
Question extension fichier + pb float ✓ - Forum - C / C++ / C++.NET
Détecter extension de fichier dans un répertoire. ✓ - Forum - VB.NET
Extension fichier javascript ✓ - Forum - Javascript / DHTML

3 réponses

Réponse 1 / 3

Whismeril

Messages postés: 17326
Date d'inscription: mardi 11 mars 2003
Statut: Modérateur
Dernière intervention: 15 mai 2022

596
13 avril 2018 à 17:36

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

ToxicHayabusa

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

13 avril 2018 à 18:33

Oui ok, sauf que tout ça je l'ai déjà lu au par avant, et que je sais que

Path.GetFileNameWithoutExtension(f.Name);

ne retourne que le nom du fichier. Sauf que j'ai déjà essayé avec une précédente méthode (d'ailleurs la doc de microsoft ne sert pas à grand chose) où j'implémentais le chemin du fichier dans le nom, et le logiciel se contentait de chercher le fichier avec le chemin dans le nom..

ToxicHayabusa

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

Modifié le 13 avril 2018 à 18:35

D'ailleurs au début du code le chemin est déjà préciser

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

Donc je vois pas où est le problème ? Pour moi, le code est censé toujours chercher les fichiers dans ce chemin, puisqu'il les a bien trouvé au début.

Whismeril

Messages postés: 17326
Date d'inscription: mardi 11 mars 2003
Statut: Modérateur
Dernière intervention: 15 mai 2022

596
13 avril 2018 à 18:54

Ben non, et pour 2 raisons:

"toto.txt"

pour C# c'est un chemin relatif, ça veut dire

"répertoire de l'exe en cours\toto.txt"

, de même,

"tutu\toto.txt"

veut dire

"répertoire de l'exe en cour\tutu\toto.txt"

toto n'est pas un chemin du tout.

Ce que tu codes c'est

File.Move("toto", "toto.csv");

. Ça ne peut pas marcher, et contrairement à ce que tu dis, en lisant correctement la doc et en exécutant pas à pas tu aurais pu le voir par toi même.

De tête (et par conséquent sans test)

string ext = Path.GetExtension(f.Name);
string nouveauNom = f.Name.Replace(ext, ".csv");//là on converse le chemin complet
Path.Move(f.Name, nouveauNom);//là aussi

Mais vu, que Path propose une méthode qui fait le job, pourquoi ne pas tout simplement pas s'en servir?

ToxicHayabusa

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

13 avril 2018 à 19:04

Bref comme je le disais, tu viens exactement de me montrer une méthode que j'avais utilisée et qui ne fonctionnait pas car comme dit plus haut, le logiciel cherche donc le fichier avec le chemin dans son nom (au passage Path.Move n'existe pas, et Path.ChangeExtension (que j'ai essayé au cas où) ne fait juste rien du tout, ni erreur ni résultat)

Que veux-tu dire par "Path propose une méthode qui fait le job" ? Si tu parles de "Path.ChangeExtension", comme dis plus haut, il ne fait rien (et je ne sais pas pourquoi d'ailleurs)

ToxicHayabusa

Messages postés: 882
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

Whismeril

Messages postés: 17326
Date d'inscription: mardi 11 mars 2003
Statut: Modérateur
Dernière intervention: 15 mai 2022

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

Réponse 2 / 3

ToxicHayabusa

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

13 avril 2018 à 22:22

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.

vb95

Messages postés: 2629
Date d'inscription: samedi 11 janvier 2014
Statut: Modérateur
Dernière intervention: 13 mai 2022

151
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 !

Réponse 3 / 3

ToxicHayabusa

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

14 avril 2018 à 00:47

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

vb95

Messages postés: 2629
Date d'inscription: samedi 11 janvier 2014
Statut: Modérateur
Dernière intervention: 13 mai 2022

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

ToxicHayabusa

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

14 avril 2018 à 01:38

Ok, le problème venait simplement de l'anti slash, j'avoue ne pas avoir pensé à ça...Merci de l'aide !

Comme quoi, rien ne sert de s'énerver à m'expliquer 30 fois la même chose, le mieux serait encore de bien regarder mon code :) (surtout qu'il n'est pas très long..)

ToxicHayabusa

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

14 avril 2018 à 01:42

En revanche est-ce que vous connaîtriez un moyen d'inclure aussi les sous dossiers du dossier spécifié ?

vb95

Messages postés: 2629
Date d'inscription: samedi 11 janvier 2014
Statut: Modérateur
Dernière intervention: 13 mai 2022

151
14 avril 2018 à 01:56

En se servant de la récursivité !
Si on corrige ton code tout bêtement tu n'apprendras rien !
En te faisant comprendre tes erreurs tu progresseras

ToxicHayabusa

Messages postés: 882
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 ?

ToxicHayabusa

Messages postés: 882
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..

ToxicHayabusa

Messages postés: 882
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();
                        }
                    }
                }

vb95

Messages postés: 2629
Date d'inscription: samedi 11 janvier 2014
Statut: Modérateur
Dernière intervention: 13 mai 2022

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

Changement d'extension de multiples fichiers

3 réponses

Votre réponse

sur les mêmes thèmes