necessita di overload operator <e controllo nullo

Question

Sto sovraccaricando l'operatore lessthan in C# e mi chiedo se questo debba verificare se è nullo. Qui di seguito potete trovare un esempio:

public static bool operator <(MyClass x, MyClass y) 
{ 
    if (x == null && y == null) 
    { 
    return false; 
    } 
    if (x == null) 
    { 
    return true; //false? 
    } 
    if (y == null) 
    { 
    return false; //true? 
    } 
    return x.Value < y.Value; 
} 

o si tratta di corretta:

public static bool operator <(MyClass x, MyClass y) 
{ 
    return x.Value < y.Value; 
} 

mi hanno ragione't trovare alcuna istruzione su questo. Ma forse mi sono perso qualcosa.

Answer 1

La risposta dipende dal modello di utilizzo previsto. Se si prevede di avere valori nulli nel mix e si desidera considerare i valori null inferiori a valori non nulli, l'implementazione è corretta; se si desidera considerare i valori null superiori a quelli non nulli, è necessario utilizzare i valori restituiti commentati (false e true). Se non si prevede di consentire i null nel mix, lanciare un ArgumentNullException o consentire NullReferenceException sarebbe la scelta giusta.

Answer 2

Un operatore personalizzato è poco più di un metodo statico. Inoltre, gli operatori dei generali non dovrebbero normalmente lanciare eccezioni. Il che significa che hai bisogno di quei controlli null se MyClass è un tipo di riferimento.

A proposito, è normale che nulls sia inferiore a non nulls, il che rende l'implementazione proposta idiomatica.

Answer 3

Personalmente vorrei gettare un ArgumentNullException se uno o xy sono null, che dovrebbe essere una circostanza eccezionale.

Answer 4

Entrambi gli approcci sono corretti (per diversi valori di corretto).

Se x o sono probabilmente nulli e questo ha un significato valido nel tuo caso, quindi vai con il primo approccio.

Se è improbabile che il valore x e sia nullo, passare al secondo e lasciare propagare eventuali eccezioni al codice chiamante per la gestione.

Answer 5

1. E 'una cattiva idea di sovraccaricare gli operatori sulle classi. Però va bene per le strutture.

2. Se si decide di sovraccaricare un operatore in una classe, sarà necessario:
   a. Includi null check nella tua logica
   b. Genera eccezioni quando viene passato nulla in
   c. Fare controllo non nullo e consentire NullReferenceExceptions (cattivi)

In sostanza, si tratta di una cattiva idea di sovraccaricare un operatore su una classe. Trasformerei la tua classe in una struttura, o semplicemente implementare un'interfaccia come IComparable<T>/IEquatable<T> che ha delle linee guida quando i valori nulli sono usati nei confronti.