Utilisation de WebClient (attendre la fin du thread)

Version imprimable

Bonjour,

J'ai actuellement une application [Silverlight] qui utilise WebClient afin d'accéder à un fichier XML.

Ce fichier XML doit être charger au démarage de l'application car tout est basé sur sont contenu (mise en forme de l'appli/utilisation des données ...) or étant donnée que je fais appel à DownloadStringAsync l'application n'attend pas le chargement du fichier pour continuer ce qui la plante inévitablement.

Ma question est comment attendre le résultat de DownloadStringAsync afin de continuer ?

Voici mon code:

Code:

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
 
 
rivate string urlFile;
        private XElement sheets;
        public XmlFile()
        {
            UrlFile = "/Charts.xml";
            string host = (App.Current.Host.Source.AbsoluteUri).Substring(0, App.Current.Host.Source.AbsoluteUri.Length - App.Current.Host.Source.AbsolutePath.Length);
            TakeFileFromUri(host + UrlFile);
        }
        public void TakeFileFromUri(String URL)
        {
            Uri uri = new Uri(URL, UriKind.RelativeOrAbsolute);
            WebClient client = new WebClient();
            client.DownloadStringCompleted += new DownloadStringCompletedEventHandler(DownloadStringCallback2);
            client.DownloadStringAsync(uri);
        }
        private void DownloadStringCallback2(Object sender, DownloadStringCompletedEventArgs e)
        {
            string resultBlock;
            if (e.Error == null)
                resultBlock = e.Result;
            else
                resultBlock = e.Error.ToString() + "erreur";
            XmlReader XmlFile = XmlReader.Create(new System.IO.StringReader(resultBlock));
            this.Sheets = XElement.Load(XmlFile);
 
        }

Merci de votre aide

11/04/2011, 11h59
tomlev

Bah je comprends pas trop en fait... tu ne fais rien à la suite de TakeFileFromUri, donc il ne devrait plus rien se passer avant le DownloadStringCompleted ? Ou alors tu n'as pas mis le code complet...

Bonjour Tom,

En effet je n'ai pas mis le code complet, il manque les appelles à cet objet fait dans mon xaml.cs ainsi que les méthodes de l'objet XmlFile:

Le xaml.cs

Code:

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
 
       public LineChart()
        {
            InitializeComponent();
            this.initChart();
            this.filter();
            this.drawChart();
        }
        private void initChart()
        {
            //Telerik.Windows.Controls.Charting.EmptyPointBehavior
            XmlContentFile = new XmlFile();
            MyDataSource = XmlContentFile.getGridDataSource();
            this.LayoutRoot.Height = XmlContentFile.getContentHeight() + 25;
            this.LayoutRoot.Width = XmlContentFile.getContentWidth() + 10;
            this.radChart.Width = XmlContentFile.getContentWidth();
            this.radChart.Height = XmlContentFile.getContentHeight();
            this.radChart.DefaultView.ChartArea.SmartLabelsEnabled = true;
 
 
            this.radChart.DefaultView.ChartArea.AxisX.LabelRotationAngle = XmlContentFile.getLabelRotationAngle(); 
            myNodes = XmlContentFile.getSerieValues();
            //Source
            this.radChart.ItemsSource = MyDataSource;
            //Title
            this.radChart.DefaultView.ChartTitle.Content = XmlContentFile.getChartTitle();
            //Y axes
            radChart.DefaultView.ChartArea.AxisY.Title = XmlContentFile.getYaxesContent("yaxe1");
            if (XmlContentFile.getYaxesContent("yaxe2") != null)
            {
                radChart.DefaultView.ChartArea.AxisY.Title = XmlContentFile.getYaxesContent("yaxe1");
                AxisY axisY = new AxisY();
                axisY.AxisName = "yaxe2";
                axisY.Title = XmlContentFile.getYaxesContent("yaxe2");
                radChart.DefaultView.ChartArea.AdditionalYAxes.Add(axisY);
            }
 
        }

Les methodes du XmlFile sont du type:

Code:

1
2
3
4
5
6
 
   public float getContentHeight()
        {
            //XElement sheets = XElement.Load(UrlFile);
            return float.Parse(Sheets.Attribute("height").Value);
        }

Comme tu le constates les méthodes utilisent Sheets, qui est initialisé dans mon thread de lecture de fichier...

Merci pour ton aide.

OK... donc en fait, quand tu crées l'objet XmlFile, il faudrait que tu attendes qu'il soit "prêt" avant de commencer à l'utiliser.

Par exemple, tu pourrais exposer dans XmlFile un évènement Loaded qui se déclenche quand le chargement est terminé. Et c'est seulement quand cet évènement se produirait que tu exécuterais la suite.

Par contre, il vaudrait mieux que tu ne lances pas le chargement dans le constructeur de XmlFile : sinon il y a un risque que le chargement soit terminé avant que LineChart se soit abonné à l'évènement. Tu pourrais par exemple ajouter une méthode Load() sur XmlFile, qui lancerait le chargement. Et tu l'utiliserais comme ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 public LineChart() { InitializeComponent(); XmlContentFile = new XmlFile(); XmlContentFile.Loaded += XmlContentFile_Loaded; XmlContentFile.Load() } private void XmlContentFile_Loaded(object sender, EventArgs e) { // Pour exécuter la méthode sur le thread de l'UI if (Dispatcher.CheckAccess()) { Dispatcher.Invoke(new EventHandler(XmlContentFile_Loaded), sender, e); return; } this.initChart(); this.filter(); this.drawChart(); }

11/04/2011, 14h52
scharly3

Merci encore pour tes réponses,

En effet tu as très bien compris ma problèmatique.

Citation:

Envoyé par tomlev

tu pourrais exposer dans XmlFile un évènement Loaded qui se déclenche quand le chargement est terminé

Pourais-tu illuster l'implémentation de l'évènement Loaded dans XmlFile ?

Je trouve cela bien compliqué alors qu'il me suffirait d'avoir une méthode non-asynchrone pour n'avoir aucun problème :( je suis assez surpris d'avoir cette problèmatique.

Citation:

Envoyé par scharly3

tu pourrais exposer dans XmlFile un évènement Loaded qui se déclenche quand le chargement est terminé

Pourais-tu illuster l'implémentation de l'évènement Loaded dans XmlFile ?

Voilà :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 public event EventHandler Loaded; private void OnLoaded() { EventHandler handler = Loaded; if (handler != null) handler(this, EventArgs.Empty); } private void DownloadStringCallback2(Object sender, DownloadStringCompletedEventArgs e) { string resultBlock; if (e.Error == null) resultBlock = e.Result; else resultBlock = e.Error.ToString() + "erreur"; XmlReader XmlFile = XmlReader.Create(new System.IO.StringReader(resultBlock)); this.Sheets = XElement.Load(XmlFile); // Déclencher l'évènement OnLoaded(); }
Citation:

Envoyé par scharly3

Je trouve cela bien compliqué alors qu'il me suffirait d'avoir une méthode non-asynchrone pour n'avoir aucun problème :( je suis assez surpris d'avoir cette problèmatique.

Bah c'est le problème de Silverlight, tout doit être asynchrone pour pas bloquer l'UI... Mais tout ça deviendra plus simple avec C# 5 ;)

11/04/2011, 17h19
scharly3
Encore une fois merci pour le temps que tu prends pour m'aider.

2 Problèmes :)
Code:

1 2 Dispatcher.Invoke(new EventHandler(XmlContentFile_Loaded), sender, e);
Me retourne :

Citation:

Erreur 1 'System.Windows.Threading.Dispatcher' ne contient pas une définition pour 'Invoke' et aucune méthode d'extension 'Invoke' acceptant un premier argument de type 'System.Windows.Threading.Dispatcher' n'a été trouvée (une directive using ou une référence d'assembly est-elle manquante ?) C:\Users\v-chsaug\Dropbox\ReportingSolution v0.3\ReportingSolution\Views\LineChart.xaml.cs 49 28 ReportingSolution

Tu veux appeller: XmlFile.Load(), mais je n'ai pas de méthode .Load ?
Code:

1 2 3 XmlContentFile = new XmlFile(); XmlContentFile.Loaded += XmlContentFile_Loaded; XmlContentFile.Load()
11/04/2011, 20h25
tomlev
Citation:
Envoyé par scharly3

Code:

1 2 Dispatcher.Invoke(new EventHandler(XmlContentFile_Loaded), sender, e);

Me retourne :
Ah oui, Silverlight n'a pas la méthode Invoke (j'utilise WPF d'habitude)...
Utilise BeginInvoke à la place (même syntaxe à part ça)

Citation:

Envoyé par scharly3

Tu veux appeller: XmlFile.Load(), mais je n'ai pas de méthode .Load ?

Il faut la créer et y mettre le code qui était dans le constructeur
12/04/2011, 10h01
scharly3
Bonjour,

Me revoici à l'attaque :)

J'ai bien appliqué toutes tes modifications mais cela ne semble pas fonctionner car je reste bloqué dans :
Code:

1 2 3 4 5 6 7 if (Dispatcher.CheckAccess()) { Dispatcher.BeginInvoke(new EventHandler(XmlContentFile_Loaded), sender, e); return; }
Pourtant la méthode OnLoaded(); est bien appelée.

Merci encore une fois pour votre support.
12/04/2011, 10h30
tomlev

Citation:

Envoyé par scharly3

J'ai bien appliqué toutes tes modifications mais cela ne semble pas fonctionner car je reste bloqué dans :

Comment ça tu "restes bloqué" ? Aucune de ces instructions n'est bloquante :?
Il se passe quoi exactement quand tu exécutes en pas à pas ?

Je suis pas bloqué a proprement parlé en effet, mais la méthode:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
        private void XmlContentFile_Loaded(object sender, EventArgs e)
        {
            // Pour exécuter la méthode sur le thread de l'UI
            if (Dispatcher.CheckAccess())
            {
                Dispatcher.BeginInvoke(new EventHandler(XmlContentFile_Loaded), sender, e);
                return;
            }
            this.initChart();
            this.filter();
            this.drawChart();
        }

Et appelée sans que jamais this.initChart() ... soit executé.

Pour rappel voici ou j'en suis :)

Dans le xaml.cs

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
public LineChart()
        {
            InitializeComponent();
            XmlContentFile = new XmlFile();
            XmlContentFile.Loaded += XmlContentFile_Loaded;
            XmlContentFile.load();
            //this.initChart();
            //this.filter();
            //this.drawChart();
        }
        private void XmlContentFile_Loaded(object sender, EventArgs e)
        {
            // Pour exécuter la méthode sur le thread de l'UI
            if (Dispatcher.CheckAccess())
            {
                Dispatcher.BeginInvoke(new EventHandler(XmlContentFile_Loaded), sender, e);
                return;
            }
            this.initChart();
            this.filter();
            this.drawChart();
        }

Dans le XmlFile.cs

Code:

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
 
       public event EventHandler Loaded;
        public XmlFile()
        {
 
        }
        public void load()
        {
            UrlFile = "/Charts.xml";
            string host = (App.Current.Host.Source.AbsoluteUri).Substring(0, App.Current.Host.Source.AbsoluteUri.Length - App.Current.Host.Source.AbsolutePath.Length);
            TakeFileFromUri(host + UrlFile);
 
        }
        public void TakeFileFromUri(String URL)
        {
            Uri uri = new Uri(URL, UriKind.RelativeOrAbsolute);
            WebClient client = new WebClient();
            client.DownloadStringCompleted += new DownloadStringCompletedEventHandler(DownloadStringCallback2);
            client.DownloadStringAsync(uri);
        }
        private void DownloadStringCallback2(Object sender, DownloadStringCompletedEventArgs e)
        {
            string resultBlock;
            if (e.Error == null)
                resultBlock = e.Result;
            else
                resultBlock = e.Error.ToString() + "erreur";
            XmlReader XmlFile = XmlReader.Create(new System.IO.StringReader(resultBlock));
            this.Sheets = XElement.Load(XmlFile);
            OnLoaded();
 
        }
        private void OnLoaded()
        {
            EventHandler handler = Loaded;
            if (handler != null)
                handler(this, EventArgs.Empty);
        }

12/04/2011, 11h14
tomlev

Oops, désolé, en fait j'ai oublié un "not" :

Code:

if (!Dispatcher.CheckAccess())

Ca devrait mieux marcher comme ça ;)
12/04/2011, 11h20
scharly3

Voila qui Fonctionne Parfaitement !

Citation:

Envoyé par tomlev

Oops, désolé, en fait j'ai oublié un "not" :

Je ne pense pas qu'il te sois nécessaire de t'excuser alors qu'en bonne assisté, j'aurais pu / du trouver l'erreur par moi même ...

Merci encore pour ton aide !
12/04/2011, 11h27
kheironn

Citation:

Envoyé par scharly3

Voila qui Fonctionne Parfaitement !

Je ne pense pas qu'il te sois nécessaire de t'excuser alors qu'en bonne assisté, j'aurais pu / du trouver l'erreur par moi même ...

Merci encore pour ton aide !

Ben justement, un bon assisté ne peut pas la trouver... ;)