Comment marquer une méthode lancera inconditionnellement?

Question

Existe-t-il un moyen de décorer une méthode qui va faire de la journalisation, puis lancer une exception inconditionnellement, en tant que telle?

J'ai le code comme ceci:

void foo(out int x) 
{ 
    if(condition()) { x = bar(); return; } 

    // notice that x is not yet set here, but compiler doesn't complain 

    throw new Exception("missed something."); 
} 

Si je l'ai essayer d'écrire comme cela, je reçois un problème:

void foo(out int x) 
{ 
    if(condition()) { x = bar(); return; } 

    // compiler complains about x not being set yet 

    MyMethodThatAlwaysThrowsAnException("missed something."); 
} 

Toutes les suggestions? Merci.

Answer 1

Que pensez-vous de cela?

bool condition() { return false; } 
int bar() { return 999; } 
void foo(out int x) 
{ 
    if (condition()) { x = bar(); return; } 
    // compiler complains about x not being set yet 
    throw MyMethodThatAlwaysThrowsAnException("missed something."); 
} 
Exception MyMethodThatAlwaysThrowsAnException(string message) 
{ 
    //this could also be a throw if you really want 
    // but if you throw here the stack trace will point here 
    return new Exception(message); 
} 

Answer 2

Si vous savez que l'exception sera toujours levée, pourquoi est-ce important? Il suffit de définir la variable à quelque chose il peut donc compiler:

void foo(out int x) 
{ 
    if(condition()) { x = bar(); return; } 

    x = 0; 

    MyMethodThatAlwaysThrowsAnException("missed something."); 
} 

Answer 3

Il ne répond pas à votre question, mais lorsque vous utilisez des paramètres, il est toujours une bonne idée de les initialiser au début de la méthode. De cette façon, vous n'aurez pas des erreurs du compilateur:

void foo(out int x) 
{ 
    x = 0; 
    if(condition()) { x = bar(); return; } 
    MyMethodThatAlwaysThrowsAnException("missed something."); 
} 

Answer 4

x est un paramètre sur et doit être réglé avant que vous aller de l'avant

Answer 5

Il n'y a aucun moyen de marquer une méthode de cette façon.

Peut-être pas pertinent, mais le modèle dans votre exemple, utilisant un paramètre out, est un peu étrange. Pourquoi ne pas simplement avoir un type de retour sur la méthode à la place?

int Foo() 
{ 
    if (condition()) return bar(); 

    MyMethodThatAlwaysThrowsAnException("missed something."); 
} 

Answer 6

Si vous ne voulez pas avoir à définir x, pourquoi n'utilisez-vous pas simplement un paramètre ref à la place?

void foo(ref int x) 
{ 
    if(condition()) { x = bar(); return; } 

    // nobody complains about anything 

    MyMethodThatAlwaysThrowsAnException("missed something."); 
} 

Answer 7

Il est un fil très vieux, mais je veux juste vous ajouter devriez écrire différent du début:

void foo(out int x) 
{ 
    if (!condition()) 
     MyMethodThatAlwaysThrowsAnException("missed something."); 

    x = bar(); 
    // and so on... 
} 

Ce compilateur moyen ne se plaindra pas et votre code est beaucoup plus clair.