小编Pau*_*ick的帖子

这个问题可能以前发过,但我找不到了.

我已经写了这么长时间的东西,我坐下来写一些新东西,然后开始输入这个就好像它是我自己的模式.最近出现了一个项目,我发现自己正在查看自己的代码并开始思考它看起来有多臭.

 BackgroundInfoIfYouCare

在这个特定的库中,我需要向用户发送电子邮件.到目前为止,有13个固定电子邮件.

每封电子邮件都有自己的模板(我使用的是Razor解析器,因此模板是用cshtml编写的).每个电子邮件模板都有一个名称密钥字符串.每封电子邮件都有自己的EF4查询,以根据"成员资格"实体和所有相关数据返回模型.

我有一个接受字符串的类,这是一个电子邮件模板名称密钥.

该方法将运行相应的查询并返回列表,抓取电子邮件模板.

列表和模板将传递给解析器,以将每个成员资格合并到模板中,并返回列表电子邮件.

 EndOfBackgroundInfoIfYouCare

真正的问题是......最好的方法是什么？

一种方法是只使用一个开关

public List<Membership> Execute(string TemplateKey) {
switch (TemplateKey) 
        {
            case "SomethingExpired":
                QueryResult = new SomethingExpiredEmailQuery().ExecuteQuery();
                break;
            case "SomethingExpireIn30":
                QueryResult = new SomethingExpireIn30EmailQuery().ExecuteQuery();
                break;
            case "FirstTimeLoginThanks":
                QueryResult = new FirstTimeLoginThanksEmailQuery().ExecuteQuery();
                break;
            case "SecurityTraining":
                QueryResult = new SecurityTrainingEmailQuery().ExecuteQuery();
                break;
            case ETC ETC ETC...

}

另一种方法是使用接口

IEmailQuery
void ExecuteQuery()

但是如果我使用接口,我仍然需要实例化Query类.它不会保存任何代码,也不会使代码更易于维护.

通过反射,我可以使用模式命名所有电子邮件查询:电子邮件模板SecurityTraining的密钥具有SecurityTrainingEmailQuery的查询名称,我可以使用反射来实例化并调用ExecuteQuery方法.

如果不使用反射,是否有更清洁的连线方式？