Discussion :

[Loic Rebours]

Bonjour à tous,

J'ouvre cette discussion pour recueillir des avis, remarques et commentaires sur mon article : Windows Phone 7 Mango : découvrez les nouvelles tâches.

Bonne lecture.

[Samuel Blanchard]

Simple et efficace. Bon boulot

[Invité]

Comme l'a dit Samuel, simple et complet et du coup on est à l'aise lors de la lecture.

Par contre un truc qui me gêne.
Lorsque e.TaskResult est différent de TaskResult.OK, tu mets partout

Code C# :

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

MessageBox.Show("Erreur ....");

C'est Erreur qui me gêne en fait. Parce que le else peut correspondre aux autres valeurs de l'enum TaskResult telles que None (Aucune action effectuée), Cancel (Abandon) ou aucune valeur dans ce cas il s'agirait d'une erreur je pense.

Sinon impeccable de long en large. +1.

[Skyounet]

Une autre coquille : ShareLinkStatusTask -> ShareLinkTask

[Loic Rebours]

Comme l'a dit Samuel, simple et complet et du coup on est à l'aise lors de la lecture.

Par contre un truc qui me gêne.
Lorsque e.TaskResult est différent de TaskResult.OK, tu mets partout

Code C# :

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

MessageBox.Show("Erreur ....");

C'est Erreur qui me gêne en fait. Parce que le else peut correspondre aux autres valeurs de l'enum TaskResult telles que None (Aucune action effectuée), Cancel (Abandon) ou aucune valeur dans ce cas il s'agirait d'une erreur je pense.

Sinon impeccable de long en large. +1.

Effectivement, ce n'est pas obligatoirement une erreur, mais c'est volontaire. Le but était de simplifier le code au maximum et de se concentrer sur ce qui était réellement important ici, à savoir les tâches (propriétés, évènements et méthodes).

Merci pour ton retour !

Une autre coquille : ShareLinkStatusTask -> ShareLinkTask

C'est corrigé. Merci !