Discussion :

[lucas67]

Bonjour,

Je débute en C#, je viens de réaliser un programme qui permet de saisir un nombre dans une phrase dans un fichier texte, d'effectuer un calcul sur ce nombre puis je réécrit tout dans un nouveau fichier texte.

Je met tout le texte de mon fichier dans une chaine de caractère puis je fais un "split" de cette chaine. J'ai alors un tableau, je fais mes calculs puis au moment de rentrer mon tableau (avec les nouvelles valeurs) dans une nouvelle chaine, l'action dure très longtemps.
Voici le code qui prend du temps :

Code :

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

1
2
3
4
5
6
7
//Boucle pour reformer la chaine
                    for (int k = 2; k < cDefine.strgTablString.Length; k++)
                    {
                        // Définition de la nouvelle chaine qui contient tout le fichier copier et modifier
                        cDefine.strgNewCHAINE = cDefine.strgNewCHAINE + cDefine.strgTablString[k];
                        cDefine.strgTablString[k] = null;
                    }

La longueur de mon tableau ~122000 éléments
Est-ce que quelqu'un aurait une solution d'optimisation ?

[SaumonAgile]

Utilise un StringBuilder pour créer ta chaine de caractère ou écris directement les données dans le fichier au lieu de créer la chaine en mémoire avant.

[tomlev]

Utilise un StringBuilder pour créer ta chaine de caractère ou écris directement les données dans le fichier au lieu de créer la chaine en mémoire avant.

+1
A chaque fois que tu fais une concaténation, ça crée une nouvelle instance de String (les chaînes n'étant pas modifiables). Si tu le fais 122000 fois, forcément ça prend un peu de temps... Le StringBuilder sert justement à remédier à ce problème

[Tober]

Ce que je comprend pas, c'est pourquoi .Net change pas plutot sa facon de voir les strings plutot de creer un patch ?

[tomlev]

Ce que je comprend pas, c'est pourquoi .Net change pas plutot sa facon de voir les strings plutot de creer un patch ?

Quel patch ? Je me suis peut-être mal exprimé en disant que StringBuilder servait à "remédier à ce problème"... StringBuilder n'est pas un "patch" pour pallier une mauvaise conception, ça a été volontairement conçu comme ça, probablement pour de bonne raisons...
Après, si les développeurs n'utilisent pas correctement les classes à leur disposition, c'est un autre problème...

[Philippe Vialatte]

probablement pour de bonne raisons

vi, le but, c'est que les strings soient immutables, comme en Java...ca permets quelques optimisations du compilateur, si je me rappelle bien


C'est threadsafe, pas besoin de padding, trucs comme ca

[tomlev]

une discussion intéressante sur ce sujet (en anglais)
http://channel9.msdn.com/forums/Tech...mmutable-in-C/

[lucas67]

Merci pour toutes vos réponses, en effet mon programme est maintenant nettement plus rapide.