délégué simple ne fonctionne pas

Question

class SimpleDelegate 
{ 
    public delegate void LogHandler(string message); 

    public void Process(LogHandler logHandler) 
    { 
    if (logHandler != null) 
    { 
     Console.WriteLine("Process begin"); 
    } 

    if (logHandler != null) 
    { 
     Console.WriteLine("Process end"); 
    } 
    } 
} 

class FileLogger 
{ 
    FileStream fileStream; 
    StreamWriter writer; 

    public FileLogger(string fileName) 
    { 
    fileStream = new FileStream(fileName, FileMode.Create); 
    writer = new StreamWriter(fileStream); 
    } 

    public void Logger(string s) 
    { 
    writer.WriteLine(s); 
    } 

    public void Close() 
    { 
    writer.Close(); 
    fileStream.Close(); 
    } 
} 

class Program 
{ 
    static void Main(string[] args) 
    { 
    SimpleDelegate cp = new SimpleDelegate(); 

    FileLogger fl = new FileLogger(@"C:\TEMP\MyLog.log"); 

    SimpleDelegate.LogHandler handler = null; 
    handler += new SimpleDelegate.LogHandler(Logger); 
    handler += new SimpleDelegate.LogHandler(fl.Logger); 

    cp.Process(handler); 

    fl.Close(); 
    } 

    static void Logger(string s) 
    { 
    Console.WriteLine("writing s " + s); 
    } 
} 

sur l'exécution I get suivantes sortie:
processus commence
fin du processus
Appuyez sur une touche pour continuer. . .

En fait, il devrait être:
processus commence
fin processus
processus commence
fin du processus
Appuyez sur une touche pour continuer. . .

Je ne suis pas en mesure de comprendre l'erreur :(

Answer 1

vous n'appeler SimpleDelegate.Processune fois - pourquoi voudriez-vous attendre à voir la sortie deux fois

Vous n'êtes jamais en train d'appeler le délégué ... juste en le testant pour la nullité. Vous testez deux fois, une fois avant d'écrire "Début du processus" et une fois avant d'écrire "Fin du processus", mais c'est tout.

Answer 2

Je pense que c'est ce qui se passe

SimpleDelegate cp = new SimpleDelegate(); 

    FileLogger fl = new FileLogger(@"C:\TEMP\MyLog.log"); 

    cp.Process(null); 
    fl.Close(); 

C'est handler est nulle, de sorte que le code ne sera exécuté. Vous voyez votre message une fois en raison de l'appel à cp.Process(null)

Answer 3

Je ne sais pas si je reçois le point ...

Mais vous ne l'appelle LogHandler dans ce code:

public void Process(LogHandler logHandler) 
    { 
     if (logHandler != null) 
     { 
      Console.WriteLine("Process begin"); 
     } 

     if (logHandler != null) 
     { 
      Console.WriteLine("Process end"); 
     } 
    } 

Answer 4

Je crois que votre confusion est autour du fait que vous appelé "LogHandler + = ..." deux fois, mais n'obtiennent que deux sorties. Notez que la fonction Process que vous appelez ne fait pas de boucle qui utilise LogHandler! C'est pourquoi vous n'obtenez pas les résultats que vous attendez.

En outre, le modèle que vous utilisez est légèrement désactivé. Au lieu d'utiliser directement le délégué, il est préférable de créer un événement et de vous y abonner.

public delegate void LogHandler(string message); 
public event LogHandler OnLog; 
... 
OnLog += ...