Discussion :

[LDamien]

Bonjour à tous,

Je souhaiterais un avis et éventuellement être corrigé car j'ai bricolé quelque chose pour sauvegarder un changement à partir d'un databinding mais j'ai l'intuition qu'il doit y avoir la possibilité de faire quelque chose de plus "propre", ou disons conventionnel.

Voici donc l'extrait où la propriété Nom de ma vue-modèle doit enregistrer le nouveau nom dans le fichier XML :

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
public string Nom
        {
            get { return nom; }
            set
            {
                if(nom != null)
                {
 
                    XmlSerializer xs = new XmlSerializer(typeof(Dt_Classes.Classe));
                    using (StreamReader rd = new StreamReader(dossierGestion + '\\' + "4eD.xml"))
                    {
 
                        uneClasse = xs.Deserialize(rd) as Dt_Classes.Classe;
 
                        var lEleve = from eleve in uneClasse.LesEleves
                                     where eleve.NomEleve == nom
                                     select eleve;
 
                        int Index;
 
                        foreach(Dt_Classes.Eleve eleve in lEleve)
                        { 
                            Index = uneClasse.LesEleves.IndexOf(eleve);
                            uneClasse.LesEleves[Index].NomEleve = value;
                        }
                    }
 
                   XmlSerializer xs2 = new XmlSerializer(typeof(Dt_Classes.Classe));
                   using (StreamWriter wr = new StreamWriter(dossierGestion + "\\" + "4eD.xml"))
                   {
                       xs2.Serialize(wr, uneClasse);
                   }
 
 
                }
                nom = value;
                OnPropertyChanged("Nom");
 
            }
        }

Merci beaucoup pour vos retours.

En vous souhaitant une bonne journée,

[daerlnaxe]

Salut, je ne suis qu'amateur dans le développement c# mais je ne suis pas fan de la manière dont tu abordes.

Déjà il faut savoir de l'autre côté quand va se produire ton changement, je t'invite si tu n'as pas géré cet aspect à regarder par toi même en prenant par exemple avec notepad++ quand ton fichier est modifié.
Egalement si tu voulais continuer à gérer ainsi, je pense que ça serait plus propre de mettre ça dans une méthode.

Enfin, je me trompe peut être mais j'ai l'impression que tu vas lire ton fichier à chaque changement, ce qui peut rapidement être super lourd et ça aurait tendance à se gérer autrement. De là à étudier selon les choix techniques :
- Si on est plusieurs je check la dernière version de modif si elle est postérieure à... je recharge le fichier.
- Si je suis seul, autant ne charger qu'une fois, et même selon bloquer l'accès derrière en écriture pourrait être intéressant pour éviter des modifs manuelles pouvant amener des erreurs.
- Si mon fichier est très gros autant en effet bosser sur quelque chose qui ne serait pas chargé en mémoire mais à ce compte je vais plutôt passer par du regex que par de la serialization (avis perso)

C'est juste des pistes de réflexion en attendant des avis d'experts.

Cordialement.