Discussion :

[quentin869]

Bonjour, je débute avec le wpf en C# et j'aurais besoin d'un petit peu d'aide ^^.
Je souhaiterais modifier un ou plusieurs contrôles de ma class principale (MainWindow), a partir d'une autre class mais je n'y arrive pas.
J'ai essayé d'instancier la class principale, mais sans succès, j'ai aussi essayé de mettre la propriété x:FieldModifier des mes controls en public, mais rien n’y fait .

Je vous remercie d'avance.
Cordialement Quentin.

[tomlev]

Je souhaiterais modifier un ou plusieurs contrôles de ma class principale (MainWindow), a partir d'une autre class mais je n'y arrive pas.

A mon avis tu pars sur une fausse piste... si tu commences à faire ce genre de choses, ton application va vite devenir un plat de spaghetti incompréhensible. Il y a une bonne raison pour laquelle les contrôles ne sont pas publics par défaut : l'encapsulation. Si tu commences à modifier les membres d'une classe à partir d'une autre, tu casses l'encapsulation, qui est un principe très important de la programmation orientée objet.

Quel est ton besoin exactement ? Est-ce que tu utilises le pattern MVVM ?

[quentin869]

Ah :/, enfaîte pour ne pas polluer ma class principale je comptais faire une autre class que j'ai intitulé download me permettant de faire tout ce dont j'ai besoin pour télécharger des fichiers successivement et donc a chaque téléchargement, notifié l'avancement dans une richtextbox (exemple: préparation, téléchargement en cours(%), téléchargement terminer ...).

[quentin869]

Je m'excuse d'avance pour le double message.

J'ai réussi a faire ce que je voulais comme ceci :

MainWindow:

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
 
        public MainWindow()
        {
            Logger.Info("Start application");
 
            InitializeComponent();
            boxlog = this.richTextBox1;
 
            download.GeneralLogs("Chargement de la version");
            Logger.Info("Chargement de la version");            
        }
 
        public static RichTextBox richTextBoxLogs
        {
            get;
            set;
        }

Download:

Code :

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

1
2
3
4
5
6
7
 
        public void GeneralLogs(string message)
        {
            TextRange range = new     TextRange(MainWindow.richTextBoxLogs.Document.ContentEnd, MainWindow.richTextBoxLogs.Document.ContentEnd);
            range.Text = "\r" + message;
            range.ApplyPropertyValue(TextElement.ForegroundProperty, Brushes.Black);
        }

Le problème et donc résolu !

[tomlev]

Le problème et donc résolu !

De la façon la plus crade qui soit

1. ça ne respecte toujours pas l'encapsulation, puisque la RichTextBox est directement accessible depuis l'extérieur
2. la propriété est statique, ce qui est peut-être pratique dans ton cas, mais qui conceptuellement n'a pas de sens : la RichTextBox appartient à une instance spécifique de MainWindow, pas à la classe MainWindow en général. En plus, qu'est-ce qui se passerait si tu avais plusieurs instances de MainWindow ? (certes, en l'occurrence c'est peu plausible, mais dans le cas général ça ne fonctionnerait pas)

La classe Download n'a pas besoin de (et ne devrait pas) connaitre en détail la façon dont un message est affiché. Il suffit qu'elle connaisse un objet qui sait comment afficher le message, de façon à bien séparer les responsabilités des classes.

Une solution propre serait de créer une interface ILogger (par exemple) :

Code :

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

1
2
3
4
interface ILogger
{
    void LogMessage(string message);
}

Faire implémenter cette interface par la classe MainWindow :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public class MainWindow : Window, ILogger
{
    ...
 
    public void LogMessage(string message)
    {
        TextRange range = new     TextRange(richTextBox1.Document.ContentEnd, richTextBox1.Document.ContentEnd);
        range.Text = "\r" + message;
        range.ApplyPropertyValue(TextElement.ForegroundProperty, Brushes.Black);
    }
}

Et passer l'instance de MainWindow à la classe de Download, via un paramètre de type ILogger :

Code :

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

1
2
 // (dans le code de MainWindow)
download = new Download(this);

De cette façon, la classe Download n'est pas dépendante de la MainWindow, ni d'une façon particulière d'afficher les logs. Si un jour tu dois afficher les logs autrement, il suffira de faire une autre implémentation de ILogger et de la passer à la classe Download.

Je t'encourage vivement à lire cet article, et d'autres sur la conception objet en général. L'approche que tu as utilisée semble sans doute plus facile à mettre en oeuvre, mais si ton application grossit et que tu continues à faire ce genre de choses, ça va rapidement devenir impossible à maintenir...