Le membre statique ne conserve pas la valeur affectée

Question

OK J'ai une classe statique qui a deux membres statiques, une chaîne et un booléen.
Une méthode statique publique affecte des valeurs à ces membres en fonction de l'état des paramètres passés.

Une méthode statique privée est ensuite appelée pour traiter les membres statiques. Le problème est que, bien que le booléen conserve la valeur à laquelle il est défini dans la fonction publique, la chaîne ne le fait pas; il revient par défaut à sa valeur initialisée. Pourquoi est-ce?
Le code simplifié est ci-dessous.

static class MessageHandler 
{ 
    private static String m_messageToSend = String.Empty; 
    private static bool m_requiresACK = false; 


    public static void Send(String message) 
    { 
     //formatting etc (actual method sets more fields) 
     m_messageToSend = message; 
     m_requiresACK = true; 

     Send(); 
    } 

    private void static Send() 
    { 
     SendMessageDelegate sendDelegate = DoSend; 
     //At this point m_requiresACK remains true but m_messageToSend does not 
     //hold value of message; it is empty. 
     IAsyncResult ar = sendDelegate.BeginInvoke(m_messageToSend, m_requiresACK); 


     //rest of function 
    } 
} 

//some other class 
MessageHandler.Send("Hello"); 

Answer 1

Le thread "unsafetyness" de ce code pourrait être le problème, puisque d'autres threads pourraient appeler Send (chaîne) alors que votre thread est actuellement au milieu de la même méthode. Je suggère la réécriture suivante de la classe Message:

static class Message 
{ 
    public static void Send(String message) 
    { 
     Send(message, true); 
    } 

    private void static Send(string messageToSend, bool requiresACK) 
    { 
     SendMessageDelegate sendDelegate = DoSend; 
     IAsyncResult ar = sendDelegate.BeginInvoke(messageToSend, requiresACK); 

     //rest of function 
    } 
} 

Answer 2

Ceci est probablement dû à un autre thread appelant

Message.Send(""); 

ou votre AppDomain est déchargé. Sans plus d'informations, il est difficile de dire avec certitude.

Answer 3

Je serais très surpris si un champ gardait la même valeur, et l'autre pas.

S'il s'agit d'une application Web, cela peut se produire si l'application est recyclée.

Answer 4

Vous avez d'énormes problèmes de sécurité de thread là. Si vous voulez vraiment cette statique, il y a une solution insolente:

[ThreadStatic] 
private static String m_messageToSend = String.Empty; 
[ThreadStatic] 
private static bool m_requiresACK = false; 

Cela agit maintenant comme statique, mais est par thread. Crise évitée; mais c'est un peu ... eh bien, j'essaierais d'éviter le besoin, moi-même - mais ça marchera bien.

important: les initialisateurs sont par fil, pas par demande; Comme il est probable que vos threads seront réutilisés, assurez-vous d'initialiser l'état avant de l'utiliser, sinon vous pourriez avoir de vieilles ordures.

Answer 5

Je vois dans vos commentaires qu'après cette première ligne, la variable privée est vide au lieu de contenir la valeur du paramètre. Est-ce vrai dès que vous appuyez sur la première ligne de Send()?

Écrire un test d'unité ou un faisceau de test simple qui appelle Message.Send ("Hello World"); et fait une affirmation sur la sortie. L'isoler de votre base de code appelante pourrait faire la lumière si c'est votre classe de message qui se comporte anormalement, ou si c'est un consommateur envoyant des données mauvaises/inattendues. De même, à moins que votre // reste de la fonction n'inclue la réinitialisation de votre booléen, il sera toujours vrai après l'envoi du premier message.