Kann diese Erweiterungsmethode umgestaltet werden?

Question

Ich habe folgende Erweiterungsmethode:

public static void ThrowIfArgumentIsNull<T>(this T value, string argument) 
    where T : class 
{ 
    if (value == null) 
    { 
     throw new ArgumentNullException(argument); 
    } 
} 

und dies ist ein Beispiel für die Verwendung ....

// Note: I've poorly named the argument, on purpose, for this question. 
public void Save(Category qwerty) 
{ 
    qwerty.ThrowIfArgumentIsNull("qwerty"); 
    .... 
} 

arbeitet zu 100% in Ordnung.

Aber ich mag nicht, wie ich den Namen der Variablen bereitstellen muss, nur um meine Ausnahmebedingung zu helfen.

ich mich gefragt, ob es möglich ist, die Erweiterung Methode Refactoring, so könnte es so ...

qwerty.ThrowIfArgumentIsNull(); 

und automatisch herausfindet aufgerufen werden, dass der Name der Variablen ‚QWERTY‘ und deshalb verwendet dies als Wert für die ArgumentNullException.

Möglich? Ich nehme an, Reflexion könnte das tun?

Answer 1

Nein, das geht nicht. Es wäre schön, aber es ist nicht möglich ohne eine Art von AOP zu beteiligen. Ich bin sicher, dass Postsharp kann einen guten Job machen, hoffentlich Attribute verwenden und in dem Code Verträge wäre es nur sein:

Contract.Requires(qwerty != null); 

Im Idealfall würde ich ein Postsharp Attribut wie der der Code Verträge nennen erzeugt - und ich werde spielen Sie irgendwann damit herum - aber bis dahin ist die Erweiterungsmethode die beste Methode, die ich gefunden habe ...

(Wenn ich jemals den Ansatz PostSharp + Code Contracts versuche, werde ich sicherlich blog über es, btw ... Mono Cecil könnte es auch relativ einfach machen.)

EDIT: Um Laurent Antwort zu erweitern, könnten Sie möglicherweise haben Dies hätte zur Reflexion verwenden, um herauszufinden, die Eigenschaften

new { qwerty, uiop, asdfg }.CheckNotNull(); 

:

new { qwerty }.CheckNotNull(); 

Und Sie konnten haben, wenn Sie viele Nicht-Nullable-Parameter hatte. Es gibt Möglichkeiten, die Reflektion bei jedem Zugriff zu vermeiden, indem Sie für jede Eigenschaft einen Delegierten erstellen und diesen im Allgemeinen in Schwung bringen. Ich kann dies für einen Blog-Post untersuchen ... aber es ist etwas eklig, und ich bevorzuge die Idee, in der Lage, nur die Parameter ...

EDIT: Code implementiert, und blog post gebührend gemacht. Ick, aber Spaß

Answer 2

In einem Wort: nein.

Der Erweiterungsmethode wird ein Wert übergeben. Es hat keine Ahnung, woher der Wert kommt oder welche Kennung der Aufrufer gewählt haben könnte, um darauf zu verweisen.

Answer 3

würde ich empfehlen, dass Sie eher Folgendes tun:

public static void ThrowIfArgumentIsNull(this object value, string argument) 
{ 
    if (value == null) 
    { 
     throw new ArgumentNullException(argument); 
    } 
} 

die Verwendung von Generika in diesem Fall scheint keinen Wert hinzuzufügen. Aber zu Ihrer ursprünglichen Frage glaube ich nicht, dass das möglich ist.

Answer 4

Siehe auch ArgumentNullException and refactoring für eine komplette Lösungen auf der gleichen Linie wie die Antwort.

Was:

public void Save(Category qwerty) 
{ 
    ThrowIfArgumentIsNull(() => return qwerty); 
    qwerty.ThrowIfArgumentIsNull("qwerty");  
    // .... 
} 

dann ThrowIfArgumentIsNull als

public static void ThrowIfArgumentIsNull(Expression<Func<object>> test) 
{ 
    if (test.Compile()() == null) 
    { 
     // take the expression apart to find the name of the argument 
    } 
} 

leider definiere ich im Detail oder geben Sie den vollständigen Code derzeit nicht über die Zeit zu füllen.

Answer 5

Ich finde es am einfachsten, dies mit einem Code-Snippet zu tun.

In Ihrem Beispiel kann ich tna<tab>qwerty<enter> eingeben. Hier

ist das Snippet:

<?xml version="1.0" encoding="utf-8" ?> 
<CodeSnippets xmlns="http://schemas.microsoft.com/VisualStudio/2005/CodeSnippet"> 
    <CodeSnippet Format="1.0.0"> 
     <Header> 
       <Title>Check for null arguments</Title> 
       <Shortcut>tna</Shortcut> 
       <Description>Code snippet for throw new ArgumentNullException</Description> 
       <Author>SLaks</Author> 
       <SnippetTypes> 
         <SnippetType>Expansion</SnippetType> 
         <SnippetType>SurroundsWith</SnippetType> 
       </SnippetTypes> 
     </Header> 
     <Snippet> 
       <Declarations> 
         <Literal> 
           <ID>Parameter</ID> 
           <ToolTip>Paremeter to check for null</ToolTip> 
           <Default>value</Default> 
         </Literal> 
       </Declarations> 
       <Code Language="csharp"><![CDATA[if ($Parameter$ == null) throw new ArgumentNullException("$Parameter$"); 
     $end$]]> 
       </Code> 
     </Snippet> 
    </CodeSnippet> 
</CodeSnippets> 

Answer 6

Ich mag Enforce vom Lokad Shared Libraries.

Basic-Syntax:

Enforce.Arguments(() => controller,() => viewManager,() => workspace); 

Dies wird eine Ausnahme mit dem Parameternamen und die Art, wenn eines der Argumente ist null werfen.