Inclure les paramètres uniquement dans le but de signaler les exceptions dans les fonctions internes

Question

CreateDocument(string templatePath) 
{ 
    Document doc = OpenDocument(templatePath); 
    Picture pic = GetLogo(); 
    AddLogo(doc, pic, templatePath); 
} 

AddLogo(Document doc, Picture logo, string templatePath) 
{ 
    Picture placeholder = doc.FindLogoPlaceholder(); 
    if (placeholder.Size != logo.Size) 
    { 
     throw new ApplicationException(
      String.Format("Invalid template {0}, logo size: {1}, required: {2}", 
       templatePath, placeholder.Size, logo.Size 
      )); 
    } 
} 

Considérons le code ci-dessus comme un exemple que je viens de faire.

Notez que la seule raison templatePath est passée dans la méthode AddLogo est de faciliter la génération d'exception.

J'ai quelque chose dans mon code aujourd'hui où je devais faire ceci, et il me semble que c'est une odeur de code vraiment désagréable. Mais je ne connais pas très bien les schémas de gestion des exceptions et je ne vois pas vraiment de meilleur moyen de le faire. Je me demande ce que vous en pensez et s'il y a un meilleur modèle pour faire face à des situations comme celle-ci.

Answer 1

Créer l'exception à un niveau supérieur:

CreateDocument(string templatePath) 
{ 
    Document doc = OpenDocument(templatePath); 
    Picture pic = GetLogo(); 
    try { 
     AddLogo(doc, pic); 
    } catch (InvalidLogoSize e) { 
     throw new ApplicationException(
      String.Format("Invalid template {0}, logo size: {1}, required: {2}", 
       templatePath, e.pSize, e.lSize 
      )); 
    } 
} 

AddLogo(Document doc, Picture logo) 
{ 
    Picture placeholder = doc.FindLogoPlaceholder(); 
    if (placeholder.Size != logo.Size) 
    { 
     throw new InvalidLogoSizeException(placeholder.Size, logo.Size); 
    } 
} 

Answer 2

Vous serait généralement jeter l'exception, puis l'attraper à tout calque où vous pouvez ajouter plus d'informations, l'envelopper dans une nouvelle exception et jeter celui-là.

Answer 3

L'appelant de la fonction peut prendre un rethrow avec des informations supplémentaires:

CreateDocument(string templatePath) 
{ 
    Document doc = OpenDocument(templatePath); 
    Picture pic = GetLogo(); 
    try 
    { 
     AddLogo(doc, pic); 
    } 
    catch(Excpetion e) 
    { 
     throw new ApplicationException(templatePath, e); 
    } 
}