Discussion :

[Hepil]

Bonjour à tous

1 - J'ai récupéré une liste de dossiers dans un tableau
2 - Puis je ne veux conserver que le nom des dossiers, sans le chemin
3 - Ensuite, je veux remplacer l'ancienne valeur (avec le chemin) par la nouvelle (sans le chemin)

Pour cela, j'ai ce code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
            Byte boff
            String folder
            String[] versionZips = Directory.GetDirectories(stringPath)


            foreach (string versionZip in VersionZips)
            {
                // Split the directory
                boff = Convert.ToByte(versionZip.LastIndexOf('\\'));
                folder = versionZip.Substring(boff + 1);
                versionZip = folder;

            }

L'erreur est : "Readonly local variable canot be used as an assigment target"

Je pense que l'objet versionzip dans la boucle foreach est d'office en readonly
Est-ce vrai ?

En plus, je pense que je peux faire cela bien mieux via une expression lambda ...
Pourriez-vous me mettre sur la voir svp ?
Quelles sont les solutions ?
Comment feriez-vous cela ?

je suis en C#, VS 2010, pour un service WCF... what else ?

[PatteDePoule]

Passe par une boucle for, tu n'auras pas ce problème.

[Hepil]

merci de ta réponse
en fait je viens de faire cela :

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
16
            Byte boff
            String folder
            String[] versionZips = Directory.GetDirectories(stringPath)

            List<string> newVersionZips = new List<string>();

            foreach (string versionZip in VersionZips)
            {
                // Split the directory
                boff = Convert.ToByte(versionZip.LastIndexOf('\\'));
                folder = versionZip.Substring(boff + 1);
                
                newVersionZips.Add(folder); 

            }

Je préfère passer par une liste, vu ses avantages.
Je peux ensuite rebasculer vers un tableau si j'en ai besoin me semble t-il

Mais il me semble que je peux transformer ce foreach en expression lambda à présent

Une idée ?

[Guulh]

Hello,
tu as raison de préférer les listes, les tableaux n'ont que très peu d'intérêt.

Mais il me semble que je peux transformer ce foreach en expression lambda à présent

Une idée ?

Of course, c'est la base de Linq

Code :

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

1
2
3
4
5
6
var result = VersionZips.Select(dir =>
            {
                // Split the directory
                boff = Convert.ToByte(dir.LastIndexOf('\\'));
                return dir.Substring(boff + 1);
            }).ToList();

Sinon, plutôt que tripatouiller des chaines, tu peux utiliser cette methode, qui te renvoie des DirectoryInfo, dont la propriete Name contient ce que tu veux.

Code :

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

var names = new DirectoryInfo(path).GetDirectories().Select(info => info.Name).ToList();

[Hepil]

Je savais qu'il y avait encore des gens doués sur terre ...
Mais où sont-ils?

Merci à toi ôhh Guulh

Plus sérieusement, je vais étudier de suite ton code et je t'en remercie
Je reviens ici après bien sûr ...

[Hepil]

PS : Pour avoir le Select du tableau, il faut ajouter la référence :

Code :

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

using System.Linq;

Mais j'ai un souci :

...

Code :

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

1
2
3
4
5
6
var result = VersionZips.Select(dir =>
            {
                // Split the directory
                boff = Convert.ToByte(versionZip.LastIndexOf('\\'));
                return versionZip.Substring(boff + 1);
            }).ToList();

...

dans ce cas de figure, on ne trouve plus versionZip ... il est en erreur, vu qu'il faisiat partie de la boucle foreach ...

[micka132]

Change par dir , ou sinon modifie dir par versionZip.

[Hepil]

Change par dir , ou sinon modifie dir par versionZip.

oui c'est ce que j'ai fait... pardon

dur dur le lundi matin ... comme tous les lundis d'ailleurs désolé

[Hepil]

Merci. Pour la suite, j'ai :

Code :

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

1
2
3
4
5
6
            var result = VersionZips.Select(versionZip =>
            {
                // Split the directory
                boff = Convert.ToByte(versionZip.LastIndexOf('\\'));
                return versionZip.Substring(boff + 1);
            }).ToList();

Mais j'ai aussi mon ancien code qui utilisait le tableau précédent.
Je dois à présent utiliser la nouvelle liste :

mon ancien code :

Code :

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

string[] cleanVerZip = Array.FindAll<string>(VersionZips, p => IsVersion(p));

J'essaie d'utiliser la nouvelle liste 'result'

PS : IsVersion(p) est juste une function (méthode) de validation annexe.



j'ai pensé à :

Code :

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

var cleanVerZip = result.Where((p=>IsVersion(p)));

[Guulh]

oups micka 132, t'as raison; je corrige mon post pour ne pas induire la postérité en erreur

Hepil: les méthodes comme Array.FindAll ou List.FindAll date d'une époque ou Linq n'était pas disponible. Elles fonctionnent bien sûr toujours, mais il est plus simple et plus flexible de passer par Linq.

Dans ton cas, tu peux donc écrire, en mettant tout ensemble:

Code :

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

1
2
3
4
5
6
var result = VersionZips.Select(dir =>
            {
                ...
            })
            .Where(dir => IsVersion(dir))
            .ToList();

Ou pour reprendre ma version simplifiée (pas testée j'avoue),

Code :

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

var names = new DirectoryInfo(path).GetDirectories().Select(info => info.Name).Where(IsVersion).ToList();

Linq permet de faire du code concis, ça vaut la peine de l'apprendre. Je te conseille notamment cette page, qui présente la plupart des opérateurs et leur usage.

[Hepil]

superbe

merci pour tes précisions, mais surtout de ton suivi et de ton écoute.
j'en prends note et je consulte la documentation de manière urgente

Je n'avais pas encore trouvé de doc "progressive" sur ce sujet

merci à tous les autres aussi