+ operatore concat stringa con un operando null

Question

Un collaboratore mi ha mostrato un comportamento molto strano e mi piacerebbe sapere se qualcuno potrebbe spiegarmi perché.

Un costruttore di base con 2 params stringa:

public MyClass(string str1, string str2) 
    { 
     this.s1 = str1; 
     this.s2 = str2; 
     this.s3 = Method(str2 + "._className", str1); 
    } 

metodo è:

public string Method(string key, string defaultValue) 
{ 
    List<string> list = _vars[key]; 
    if (list == null) return defaultValue; 
    string res = ""; 
    foreach (string s in list) 
    { 
     if (res != "") res += ","; 
     res += s; 
    } 
    return res; 
} 

Quando questo ctor viene chiamato all'interno di una pagina aspx con str2 come null, tutto funziona benissimo, perché se un operando di concatenazione di stringhe + è null, una stringa vuota viene sostituita.

Tuttavia, quando questo ctor viene chiamato con str2 come null in un thread in background, viene generato un NullReferenceException.

Il problema è stato risolto testando str2 != null prima di utilizzarlo, ma mi piacerebbe davvero sapere perché lo stesso codice genera talvolta un'eccezione, a volte no!

Ecco la traccia dello stack:

Exception: System.NullReferenceException 
Message: Object reference not set to an instance of an object. 
StackTrace: 
at MyClass..ctor(String str1, String str2) 
at AbandonedCartsNotificationJob.NotifyAbandonedCarts() in AbandonedCartsNotificationJobPartial.cs:line 39 
at AbandonedCartsNotificationJob.work() in AbandonedCartsNotificationJob.cs:line 15 
at MyRuntime.JobManager.run() 
at System.Threading.ThreadHelper.ThreadStart_Context(Object state) 
at System.Threading.ExecutionContext.runTryCode(Object userData) 
at System.Runtime.CompilerServices.RuntimeHelpers.ExecuteCodeWithGuaranteedCleanup(TryCode code, CleanupCode backoutCode, Object userData) 
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state) 
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state) 
at System.Threading.ThreadHelper.ThreadStart() 

Answer 1

C'era un oscuro bug nell'implementazione di .NET Framework di concatenazione di stringhe, ma ha interessato solo concatenazioni di 4 oggetti, in cui uno degli oggetti non è nullo e fornito un override di ToString che restituito null. Chiaramente quella situazione non è il caso qui.

Questa situazione è probabilmente causato da uno dei seguenti:

• _vars è nullo quando Method è chiamato
• causa di un uso improprio di _vars in un'applicazione multi-thread, lo stato interno del _vars è stato danneggiato, risultando in un NullReferenceException quando viene utilizzato l'operatore [].

Answer 2

Il problema sta nella realizzazione del Method oggetto. Dal momento che il + Operator implementation interprets a null value as an empty string. Il valore null di actuall non entra mai nel costruttore se impostato in str2. Sul lato opposto, str1 immette direttamente come valore nullo e può dipendere dall'implementazione causa un'eccezione di riferimento null.