Comment passer une méthode en paramètre?

Comment passer une méthode en paramètre? - C#/.NET managed - Programmation

Marsh Posté le 01-03-2008 à 12:59:38    

Mes yeux sont maintenant rouge de sang, c'est maintenant 3h49 du matin (Vancouver) et j'ai plus le courage à chercher...  
 
Je souhaite définir un callback comme ceci:
 
 

Code :
  1. class Form1
  2. {
  3.     int compteur;
  4.     pubic Form1()
  5.     {
  6.         compteur = 0;
  7.         Prout prout = new Prout();
  8.         prout.SetCallback(this.IncCompteur)
  9.     }
  10.     public IncCompteur()
  11.     {
  12.         compteur++;
  13.     }
  14. }
  16. class Prout
  17. {
  18.     private delegate void callback();
  20.     public Prout()
  21.     {
  23.     }
  24.     public SetCallback(object tocall)
  25.     {
  26.         callback = new callback(tocall);
  27.     }
  28.     private UnThread()
  29.     {
  30.         while(1)
  31.         {
  32.             // Des choses...
  33.             // Encode des choses...
  34.             callback();
  35.             // Et puis encore des choses...
  36.         }
  37.     }
  38. }


 
Le problème c'est que cette partie là génère une erreur:
 

Code :
  1. public SetCallback(object tocall)
  2.     {
  3.         callback = new callback(tocall);
  4.     }


 
Cette partie là aussi d'ailleurs:
 

Code :
  1. prout.SetCallback(this.IncCompteur)


 
Pouvez-vous m'aider un peu?  
 
Merci d'avance

Reply

Marsh Posté le 01-03-2008 à 12:59:38   

Reply

Marsh Posté le 01-03-2008 à 17:55:22    

pour moi tu dois faire appel à un delegué. Renseigne toi la dessus.

Reply

Marsh Posté le 01-03-2008 à 21:34:49    

Oui c'est bien ce que je pensais c'est d'ailleurs ce que j'utilise dans mon exemple :)

Reply

Marsh Posté le 01-03-2008 à 22:18:55    

Bon j'avance gentillement mais c'est pas encore ca...  
 

Code :
  1. using System;
  2. using System.Collections.Generic;
  3. using System.Linq;
  4. using System.Text;
  5. using System.Threading;
  7. namespace ConsoleApplication1
  8. {
  9.     //---------------------- MAIN ---------------------//
  10.     class Program
  11.     {
  12.         // Main program
  13.         static void Main(string[] args)
  14.         {
  15.             // Set delegate
  16.             ActionHandler actionHandler = new ActionHandler(Action);
  18.             // Start service and set callback
  19.             Service service = new Service();
  20.             service.SetCallback(actionHandler);
  21.             service.Run();
  23.             // Do nothing
  24.             Console.ReadLine();
  25.         }
  27.         // Method called by the service...
  28.         delegate void ActionHandler();
  29.         static void Action()
  30.         {
  31.             Console.WriteLine("I was called!" );
  32.         }
  33.     }
  35.     //--------------------- SERVICE -------------------//
  37.     class Service
  38.     {
  39.         delegate void RemoteEvent();
  40.         RemoteEvent remoteEvent;
  42.         public Service()
  43.         {
  44.             remoteEvent = null;
  45.         }
  47.         public void Run()
  48.         {
  49.             Thread backgroundTask = new Thread(new ThreadStart(BackgroundTask));
  50.             backgroundTask.Start();
  51.         }
  53.         public void SetCallback(object remoteHandler)
  54.         {
  55.             RemoteEvent remoteEvent = (RemoteEvent)remoteHandler;
  56.         }
  58.         private void OnRemoteEvent()
  59.         {
  60.             if (remoteEvent != null)
  61.                 remoteEvent();
  62.         }
  64.         public void BackgroundTask()
  65.         {
  66.             for (int i = 0; i < 10; i++)
  67.             {
  68.                 Console.WriteLine("Hard Job, huh!" );
  69.                 System.Threading.Thread.Sleep(2000);
  70.                 OnRemoteEvent();
  71.             }
  72.         }
  73.     }
  74. }

Reply

Marsh Posté le 01-03-2008 à 22:25:32    

Bon finalement j'ai trouvé une solution mais Ô que c'est laid.... Il suffisait de déclarer le délégué en dehors de toute classe.  
 
Avez-vous une autre alternative plus "propre"
 
 

Code :
  1. using System;
  2. using System.Collections.Generic;
  3. using System.Linq;
  4. using System.Text;
  5. using System.Threading;
  7. namespace ConsoleApplication1
  8. {
  9.     delegate void RemoteEvent();
  11.     //---------------------- MAIN ---------------------//
  12.     class Program
  13.     {
  14.         // Main program
  15.         static void Main(string[] args)
  16.         {
  17.             // Set delegate
  18.             RemoteEvent actionHandler = new RemoteEvent(Action);
  20.             // Start service and set callback
  21.             Service service = new Service();
  22.             service.SetCallback(actionHandler);
  23.             service.Run();
  25.             // Do nothing
  26.             Console.ReadLine();
  27.         }
  29.         // Method called by the service...
  30.         //delegate void ActionHandler();
  31.         static void Action()
  32.         {
  33.             Console.WriteLine("I was called!" );
  34.         }
  35.     }
  37.     //--------------------- SERVICE -------------------//
  38.     class Service
  39.     {       
  40.         RemoteEvent remoteEvent;
  42.         public Service()
  43.         {
  44.             remoteEvent = null;
  45.         }
  47.         public void Run()
  48.         {
  49.             Thread backgroundTask = new Thread(new ThreadStart(BackgroundTask));
  50.             backgroundTask.Start();
  51.         }
  53.         public void SetCallback(RemoteEvent remoteHandler)
  54.         {
  55.             remoteEvent = remoteHandler;
  56.         }
  58.         private void OnRemoteEvent()
  59.         {
  60.             if (remoteEvent != null)
  61.                 remoteEvent();
  62.         }
  64.         public void BackgroundTask()
  65.         {
  66.             for (int i = 0; i < 10; i++)
  67.             {
  68.                 Console.WriteLine("Hard Job, huh!" );
  69.                 System.Threading.Thread.Sleep(2000);
  70.                 OnRemoteEvent();
  71.             }
  72.         }
  73.     }
  74. }

Reply

Marsh Posté le 01-03-2008 à 23:23:39    

Bon je continue sur la lancée...  
J'ai trouvé beaucoup plus propre avec l'utilisation des évènements. Il suffit de définir la méthode appelée lorsque l'évènement "Modified" est déclanché.
 

Code :
  1. using System;
  2. using System.Collections.Generic;
  3. using System.Linq;
  4. using System.Text;
  5. using System.Threading;
  7. namespace ConsoleApplication1
  8. {
  9.     delegate void RemoteEvent();
  11.     //---------------------- MAIN ---------------------//
  12.     class Program
  13.     {
  14.         // Main program
  15.         static void Main(string[] args)
  16.         {
  17.             // Start service and set event
  18.             Service service   = new Service();
  19.             service.Modified += new EventHandler(Action);
  20.             service.Run();
  22.             // Do nothing
  23.             Console.ReadLine();
  24.         }
  26.         // Method called by the service...
  27.         static void Action(object sender, EventArgs e)
  28.         {
  29.             Console.WriteLine("I was called!" );
  30.         }
  31.     }
  33.     //--------------------- SERVICE -------------------//
  34.     class Service
  35.     {
  36.         public event EventHandler Modified;
  38.         public Service()
  39.         {
  40.         }
  42.         public void Run()
  43.         {
  44.             Thread backgroundTask = new Thread(new ThreadStart(BackgroundTask));
  45.             backgroundTask.Start();
  46.         }
  48.         private void OnModified(EventArgs e)
  49.         {
  50.             if (Modified != null)
  51.                 Modified(this, e);
  52.         }
  54.         public void BackgroundTask()
  55.         {
  56.             for (int i = 0; i < 10; i++)
  57.             {
  58.                 // Hard Job
  59.                 Console.WriteLine("Hard Job, huh!" );
  60.                 System.Threading.Thread.Sleep(2000);
  62.                 // Fire the event
  63.                 OnModified(EventArgs.Empty);
  64.             }
  65.         }
  66.     }
  67. }

Reply

Marsh Posté le 02-03-2008 à 01:28:54    

Bon cette fois-ci on s'écarte beaucoup de l'origine mais le principe reste le même.
 
Un service et un client. Le client appel le service et défini des "évènements" suivant:
 
- Quand le processus à progressé
- Quand le processus est terminé
 
J'ai rajouté un GUI pour que ce soit plus ceux qui veulent tester. Il suffit de créer un nouveau projet sous Visual Studio C#, de supprimer le fichier Form1.Designer.cs, faire clic droit sur Form1.cs et sélectionner View Code. Ensuite il suffit juste de coller le code suivant en remplacant tout.
 
J'ai atteint mon but et je suis content...  
 

Code :
  1. using System;
  2. using System.Collections.Generic;
  3. using System.ComponentModel;
  4. using System.Data;
  5. using System.Drawing;
  6. using System.Linq;
  7. using System.Text;
  8. using System.Windows.Forms;
  10. namespace WindowsFormsApplication1
  11. {
  12.     /// <summary>
  13.     /// Main Form
  14.     /// </summary>
  15.     public class Form1 : Form
  16.     {
  17.         #region Non Interesting...
  19.         private System.ComponentModel.IContainer components = null;
  20.         protected override void Dispose(bool disposing)
  21.         {
  22.             if (disposing && (components != null))
  23.             {
  24.                 components.Dispose();
  25.             }
  26.             base.Dispose(disposing);
  27.         }
  29.         #region Windows Form Designer generated code
  31.         private void InitializeComponent()
  32.         {
  33.             this.progressBar1 = new System.Windows.Forms.ProgressBar();
  34.             this.label2 = new System.Windows.Forms.Label();
  35.             this.textBox1 = new System.Windows.Forms.TextBox();
  36.             this.SuspendLayout();
  37.             //  
  38.             // progressBar1
  39.             //  
  40.             this.progressBar1.Location = new System.Drawing.Point(15, 25);
  41.             this.progressBar1.Name = "progressBar1";
  42.             this.progressBar1.Size = new System.Drawing.Size(267, 19);
  43.             this.progressBar1.TabIndex = 2;
  44.             //  
  45.             // label2
  46.             //  
  47.             this.label2.AutoSize = true;
  48.             this.label2.Location = new System.Drawing.Point(12, 9);
  49.             this.label2.Name = "label2";
  50.             this.label2.Size = new System.Drawing.Size(64, 13);
  51.             this.label2.TabIndex = 5;
  52.             this.label2.Text = "Current Job:";
  53.             //  
  54.             // textBox1
  55.             //  
  56.             this.textBox1.Location = new System.Drawing.Point(15, 50);
  57.             this.textBox1.Multiline = true;
  58.             this.textBox1.Name = "textBox1";
  59.             this.textBox1.ScrollBars = System.Windows.Forms.ScrollBars.Vertical;
  60.             this.textBox1.Size = new System.Drawing.Size(267, 167);
  61.             this.textBox1.TabIndex = 6;
  62.             //  
  63.             // Form1
  64.             //  
  65.             this.AutoScaleDimensions = new System.Drawing.SizeF(6F, 13F);
  66.             this.AutoScaleMode = System.Windows.Forms.AutoScaleMode.Font;
  67.             this.ClientSize = new System.Drawing.Size(292, 224);
  68.             this.Controls.Add(this.textBox1);
  69.             this.Controls.Add(this.label2);
  70.             this.Controls.Add(this.progressBar1);
  71.             this.Name = "Form1";
  72.             this.Text = "Test";
  73.             this.ResumeLayout(false);
  74.             this.PerformLayout();
  75.         }
  76.         #endregion
  78.         private System.Windows.Forms.ProgressBar progressBar1;
  79.         private System.Windows.Forms.Label label2;
  80.         private System.Windows.Forms.TextBox textBox1;
  82.         #endregion
  84.         Service service = new Service();
  86.         public Form1()
  87.         {
  88.             InitializeComponent();
  90.             // Start a background job
  91.             service.ProgressChanged = new ProgressChangedEventHandler(ProgressChanged);
  92.             service.CompletedEvent = new RunWorkerCompletedEventHandler(ProcessCompleted);
  93.             service.Run();
  94.         }
  96.         public void ProgressChanged(object sender, ProgressChangedEventArgs e)
  97.         {
  98.             progressBar1.Value = e.ProgressPercentage;
  99.             textBox1.AppendText("Current Value: " + service.Value.ToString() + Environment.NewLine);
  100.         }
  102.         private void ProcessCompleted(object sender, RunWorkerCompletedEventArgs e)
  103.         {
  104.             MessageBox.Show("Job Done!" );
  105.         }
  106.     }
  108.     /// <summary>
  109.     /// Service
  110.     /// </summary>
  111.     public class Service
  112.     {
  113.         private BackgroundWorker job;
  114.         public ProgressChangedEventHandler ProgressChanged;
  115.         public RunWorkerCompletedEventHandler CompletedEvent;
  116.         private delegate void Change(string str);
  118.         public int Value;
  120.         public Service()
  121.         {
  122.             job = new BackgroundWorker();
  123.             job.DoWork += new DoWorkEventHandler(backgroundJob_DoWork);         
  124.         }
  126.         public void Run()
  127.         {
  128.             job.RunWorkerAsync();
  129.             if (ProgressChanged != null)
  130.             {
  131.                 job.ProgressChanged += ProgressChanged;
  132.                 job.WorkerReportsProgress = true;
  133.             }
  134.             if (CompletedEvent != null)
  135.             {
  136.                 job.RunWorkerCompleted += CompletedEvent;
  137.             }
  138.            
  139.         }   
  141.         private void backgroundJob_DoWork(object sender, DoWorkEventArgs e)
  142.         {
  143.             for (int i = 0; i <= 100; i++)
  144.             {
  145.                 // Hard and long job
  146.                 System.Threading.Thread.Sleep(10);
  148.                 // Return values during process
  149.                 if (i % 10 == 0)
  150.                 {
  151.                     Value = i;
  152.                     (sender as BackgroundWorker).ReportProgress(i);
  153.                 }               
  154.             }
  155.         }     
  156.     }
  157. }

Reply

Marsh Posté le 05-03-2008 à 02:02:45    

Je suis un peu à la bourre, mais en gros...
 
Si j'ai bien pigé, t'as une application, et tu veux qu'un traîtement asynchrone tourne, et te prévienne de son avancement, c'est ça ?
 
Si oui, plutôt que de jouer avec des events, joue plutôt avec des Threads, c'est là pour ça.
 
Et qui plus est, les BackgroundWorker, qui sont d'une simplicité extrême à utiliser.
 
Exemple complet avec 3 boutons :
bouton1 = start
bouton2 = stop
bouton3 = pause/resume
Une progressbar qui indique où en est le worker
Un label qui indique l'état du worker
Une textbox qui indique jusqu'à combien compter (afin de provoquer une erreur si désiré)
 

Code :
  1. using System;
  2. using System.ComponentModel;
  3. using System.Windows.Forms;
  4.  
  5. namespace SampleBackgroundWorker
  6. {
  7.     public partial class Form1 : Form
  8.     {
  9.         int COUNTER_DONE;
  10.         BackgroundWorker bw = new BackgroundWorker();
  11.         byte WorkingCounter = 0;
  12.  
  13.         public Form1()
  14.         {
  15.             InitializeComponent();
  16.  
  17.             COUNTER_DONE = int.Parse(textBox1.Text);
  18.  
  19.             progressBar1.Minimum = 0;
  20.             progressBar1.Maximum = 100;
  21.  
  22.             bw.DoWork += new DoWorkEventHandler(bw_DoWork);
  23.             bw.ProgressChanged += new ProgressChangedEventHandler(bw_ProgressChanged);
  24.             bw.RunWorkerCompleted += new RunWorkerCompletedEventHandler(bw_RunWorkerCompleted);
  25.             bw.WorkerReportsProgress = true;
  26.             bw.WorkerSupportsCancellation = true;
  27.  
  28.             button3.Enabled = false;
  29.             button2.Enabled = false;
  30.         }
  31.  
  32.         private void button1_Click(object sender, EventArgs e)
  33.         {
  34.             button1.Enabled = false;
  35.             button2.Enabled = true;
  36.             button3.Enabled = true;
  37.             button3.Text = "Pause";
  38.             bw.RunWorkerAsync((object)false);
  39.         }
  40.  
  41.         private void button2_Click(object sender, EventArgs e)
  42.         {
  43.             if (bw.IsBusy)
  44.             {
  45.                 button2.Enabled = false;
  46.                 bw.CancelAsync();
  47.             }
  48.         }
  49.  
  50.         private void button3_Click(object sender, EventArgs e)
  51.         {
  52.             if (bw.IsBusy)
  53.             {
  54.                 button2.Enabled = false;
  55.                 button3.Text = "Resume";
  56.                 bw.CancelAsync();
  57.             }
  58.             else
  59.             {
  60.                 button1.Enabled = false;
  61.                 button3.Text = "Pause";
  62.                 bw.RunWorkerAsync((object)true);
  63.             }
  64.         }
  65.  
  66.         private void bw_DoWork(object sender, DoWorkEventArgs e)
  67.         {
  68.             bool resume = (bool)e.Argument;
  69.  
  70.             if (!resume)
  71.             {
  72.                 WorkingCounter = 0;
  73.             }
  74.  
  75.             while (WorkingCounter < COUNTER_DONE)
  76.             {
  77.                 Random rnd = new Random();
  78.                 if (WorkingCounter > (WorkingCounter += 10))
  79.                 {
  80.                     throw new Exception(string.Format("Dépassement du type du compteur : {0} pour {1}.", (byte)(WorkingCounter - 10) + 10, byte.MaxValue));
  81.                 }
  82.  
  83.                 int Percent = (int)WorkingCounter * 100 / COUNTER_DONE;
  84.  
  85.                 bw.ReportProgress(Percent, (object)false);
  86.                 System.Threading.Thread.Sleep(250);
  87.  
  88.                 if (bw.CancellationPending)
  89.                 {
  90.                     e.Cancel = true;
  91.                     bw.ReportProgress(Percent, (object)true);
  92.                     System.Threading.Thread.Sleep(500);
  93.                     break;
  94.                 }
  95.             }
  96.  
  97.             e.Result = (object)WorkingCounter;
  98.         }
  99.  
  100.         private void bw_ProgressChanged(object sender, ProgressChangedEventArgs e)
  101.         {
  102.             progressBar1.Value = e.ProgressPercentage;
  103.             if ((bool)e.UserState)
  104.             {
  105.                 label1.Text = "Le worker est en cours d'arrêt...";
  106.             }
  107.             else
  108.             {
  109.                 label1.Text = string.Format("Le worker fonctionne normalement... ({0}%)", e.ProgressPercentage);
  110.             }
  111.         }
  112.  
  113.         private void bw_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
  114.         {
  115.             if (e.Error != null)
  116.             {
  117.                 label1.Text = string.Format("Le worker s'est terminé avec une erreur :\n{0}.", e.Error.Message);
  118.             }
  119.             else
  120.             {
  121.                 if (e.Cancelled)
  122.                 {
  123.                     label1.Text = "Le worker s'est arrêté à la demande de l'utilisateur.";
  124.                     button3.Enabled = false;
  125.                 }
  126.                 else
  127.                 {
  128.                     label1.Text = string.Format("Le worker a fini de compter jusqu'à {0}.", e.Result);
  129.                     button3.Enabled = true;
  130.                 }
  131.             }
  132.             button1.Enabled = true;
  133.             button2.Enabled = false;
  134.         }
  135.  
  136.         private void textBox1_TextChanged(object sender, EventArgs e)
  137.         {
  138.             COUNTER_DONE = int.Parse(textBox1.Text);
  139.         }
  140.     }
  141. }


 
-- Merde, j'avais pas fait gaffe que c'est ce à quoi t'es arrivé au final :D
En tout cas, c'est clairement la façon la plus élégante de faire ce genre de choses.
Tu peux aussi gérer des thread à la main, mais c'est infiniment plus complexe pour des possibilités dont on n'a pas forcément besoin.
 
Attention, le test sur "e.Error" dans le completed est très important : si le worker raise une exception, il va la trapper tout seul comme un grand, et la stocker à dedans.
Toute tentative d'accès à "e.Result" avec "e.Error != null" provoque une "InvalidOperationException" !


Message édité par MagicBuzz le 05-03-2008 à 02:34:24
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed