Discussion :

[olibara]

Bonjour

Je me heurte a un probleme qui m'interpelle car j'ai encore un trou de compréhension qq part

Je definis une classe et dans son constructeur j'aimerais appeller une methode de désérialisation.
Mais j'ai le message :

Cannot assign to '<this>' because it is read-only

Est ce que cela signifie qu'une classe ne peut pas avoir son propre désérializer et que je suis obligé d'utiliser une methode statique externe ?

Voici l'exemple, c'est :

Code :

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

this = (C_Algemeen)xs.Deserialize(reader);

Qui pose problème

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
41
42
43
44
45
 public class C_Algemeen
  {
    public bool[] ForbidDays { get; set; }
    public List<Plage> ForbidTime {get;set;}
 
    public C_Algemeen()
    {
    }
 
    public C_Algemeen(string _file)
    {
       LoadXml(_file);
    }
    public void LoadXml(string FileName)
    {
      if (!File.Exists(FileName))
      {
        return;
      }
      try
      {
        using (StreamReader fs = new StreamReader(FileName))
        {
          XmlReaderSettings xmlSettings = new XmlReaderSettings();
          xmlSettings.CloseInput = true;
          xmlSettings.IgnoreWhitespace = true;
          xmlSettings.ConformanceLevel = ConformanceLevel.Document;
          xmlSettings.CloseInput = false;
          XmlReader reader = XmlReader.Create(fs, xmlSettings);
 
 
          XmlSerializerNamespaces ns = new XmlSerializerNamespaces();
          ns.Add("", "");
          XmlSerializer xs = new XmlSerializer(typeof(C_Algemeen));
          this = (C_Algemeen)xs.Deserialize(reader);
          reader.Close();
          fs.Close();
        }
      }
      catch
      {
      }
      return;
    }
  }

[Pol63]

this ca représente l'instance actuelle qui execute le code, tu ne peux pas la modifier ca serait un non sens ...

il suffit de mettre une variable à la place

[olibara]

il suffit de mettre une variable à la place

Mais ca c'est un autre non sens il me semble ?

Puisque l'idée c'est de charger la classe directement via le constructeur !
Mettre une variable a la place signifie que je construit une classe intermediaire avec le constructeur par défaut et qu'ensuite je substitue les propriétés de la premiere avec la deuxieme

Si c'est pour faire ca autant faire une methode statique hors de la classe a qui on passe le fichier Xml et qui construit elle meme la classe

Mais ca me perturbe un peu d'avoir une methode SaveXml interne a la classe et une methode LoadXml externe

[StormimOn]

Tu ne peux pas faire ça ainsi. Comme le dit Pol63 tu ne peux pas modifier this, et encore heureux.

Mais ca me perturbe un peu d'avoir une methode SaveXml interne a la classe et une methode LoadXml externe

Il suffit alors d'avoir que les méthodes Load et Save soient toutes les deux externes, dans une classe dédiée à la sérialisation des données. Car la classe que l'on sérialise au final et un bête container de données qui n'a pas besoin de savoir comment on charge ou on sauvegarde les données en question.

[olibara]

Il suffit alors d'avoir que les méthodes Load et Save soient toutes les deux externes, dans une classe dédiée à la sérialisation des données. Car la classe que l'on sérialise au final et un bête container de données qui n'a pas besoin de savoir comment on charge ou on sauvegarde les données en question.

Merci StorMimonn
Oui ca me semble le plus cohérent !

Néanmoins je trouvais plaisant de tout mettre dans le meme sac par souci de groupage !

Car si le bête container de données n'a pas besoin de savoir comment on charge ou on sauvegarde les données en question.

Le developpeur passe souvent a se demander "c'est quoi encore les methodes a employer ???"
Et quand on les a sous la main avec la classe grace a l'intellisense c'est quand meme sympa !

[Pol63]

le mieux serait d'utiliser un design pattern factory

rentre le constructeur privé pour que personne ne puisse l'appeler
mettre une fonction static qui retourne une instance de cette classe (qui s'occupe donc de la désérialisation)
et si d'autres personnes doivent utiliser le code, mettre en aide xml au début de la classe qu'il faut appeler le createur d'instance