c# - 覆盖 Asp.net WebControls、UserControls、Controls 上的 Render 方法时的最佳实践

Question

当创建一个自定义的WebControl、UserControl、Control并且需要覆盖各种Render方法时，主要有2种方法完成这个:

快速而肮脏的:

public override void RenderControl(HtmlTextWriter writer)
{
    writer.Write("<div class=\"{0}\"><a href={1}>{2}</a>", CssClass, "/someurl.aspx", LocalizedStrings.EditLinkText);
    base.RenderControl(writer);
    writer.Write("</div>");
}

又长又干净:

public override void RenderControl(HtmlTextWriter writer)
{
    writer.WriteBeginTag(HtmlTextWriterTag.Div.ToString());
    writer.WriteAttribute(HtmlTextWriterAttribute.Class.ToString(), CssClass);
    writer.Write(HtmlTextWriter.TagRightChar);
    var link = new HtmlAnchor();
    link.HRef = "/someurl.aspx";
    link.Title = LocalizedStrings.EditLinkTitle;
    link.InnerText = LocalizedStrings.EditLinkText; ;
    link.Attributes.Add(HtmlTextWriterAttribute.Class.ToString(), "someclass");
    link.RenderControl(writer);
    base.RenderControl(writer);
    writer.WriteEndTag(HtmlTextWriterTag.Div.ToString());
}

我想知道第二种方法是否值得。

Answer 1

作为 Clean Code 的粉丝，我会选择第一种方法。即使第二种方法更“照章办事”，它也缺乏第一种方法的清晰度。

源代码应该始终以清晰的方式传达意图(好吧，它也应该有效)，所以在证明无效之前，我会采用第一种方法。