我写了一个HTML帮助器来生成YouTube嵌入链接。它有3个参数,YouTubeID,宽度和高度。我最初用StringBuilder编写它,但后来我决定尝试使用TagBuilder(好吧,其中一些)。
以下是两个不同的回报:
//Tag Builder
public static string YouTube(this HtmlHelper helper, string youtubeId, string width, string height)
{
const string vidSuffix = "&hl=en_US&fs=1";
var url = "http://www.youtube.com/v/" + youtubeId + vidSuffix;
var objBuilder = new TagBuilder("object");
objBuilder.MergeAttribute("width",width);
objBuilder.MergeAttribute("height",height);
var movParamBuilder = new TagBuilder("param");
movParamBuilder.MergeAttribute("name","movie");
movParamBuilder.MergeAttribute("value",url);
var fsParamBuilder = new TagBuilder("param");
fsParamBuilder.MergeAttribute("name","allowFullScreen");
fsParamBuilder.MergeAttribute("value","true");
var saParamBuilder = new TagBuilder("param");
saParamBuilder.MergeAttribute("name", "allowscriptaccess");
saParamBuilder.MergeAttribute("value", "always");
var embedBuilder = new TagBuilder("embed");
embedBuilder.MergeAttribute("src",url);
embedBuilder.MergeAttribute("type", "application/x-shockwave-flash");
embedBuilder.MergeAttribute("allowscriptaccess","always");
embedBuilder.MergeAttribute("allowfullscreen","true");
embedBuilder.MergeAttribute("width",width);
embedBuilder.MergeAttribute("height",height);
objBuilder.InnerHtml = movParamBuilder.ToString(TagRenderMode.Normal) +
fsParamBuilder.ToString(TagRenderMode.Normal) +
saParamBuilder.ToString(TagRenderMode.Normal) +
embedBuilder.ToString(TagRenderMode.Normal);
return objBuilder.ToString(TagRenderMode.Normal);
}
//StringBuilder
public static string YouTube(this HtmlHelper helper, string youtubeId, string width, string height)
{
const string vidSuffix = "&hl=en_US&fs=1";
var url = "http://www.youtube.com/v/" + youtubeId + vidSuffix;
sb.AppendFormat("<object width=\"{0}\" height=\"{1}\">", width, height);
sb.AppendLine();
sb.AppendFormat("<param name=\"movie\" value=\"{0}\">",url);
sb.AppendLine();
sb.Append("</param><param name=\"allowFullScreen\" value=\"true\">");
sb.AppendLine();
sb.AppendFormat("</param><param name=\"allowscriptaccess\" value=\"always\">");
sb.AppendLine();
sb.AppendFormat(
"</param><embed src=\"{0}\" type=\"application/x-shockwave-flash\" allowscriptaccess=\"always\" allowfullscreen=\"true\" width=\"{1}\" height=\"{2}\">",
url, width, height);
sb.AppendLine();
sb.Append("</embed></object>");
return sb.ToString();
}
除了StringBuilder在每个部分之后添加换行符之外,它们都会生成完全相同的代码。
哪种想法会更好,或者甚至更重要?谢谢!
答案 0 :(得分:2)
我想我会重构它更多的是有一个方法,也许是私有的,来生成每个参数标签,减少代码重复,否则我更喜欢StringBuilder。
private static string Param( this HtmlHelper helper, string name, string value )
{
var tagBuilder = new TagBuilder("param");
tagBuilder .MergeAttribute("name", name);
tagBuilder .MergeAttribute("value", value);
return tagBuilder.ToString( TagRenderMode.Normal );
}
public static string YouTube(this HtmlHelper helper, string youtubeId, string width, string height)
{
const string vidSuffix = "&hl=en_US&fs=1";
var url = "http://www.youtube.com/v/" + youtubeId + vidSuffix;
var objBuilder = new TagBuilder("object");
objBuilder.MergeAttribute("width",width);
objBuilder.MergeAttribute("height",height);
var embedBuilder = new TagBuilder("embed");
embedBuilder.MergeAttribute("src",url);
embedBuilder.MergeAttribute("type", "application/x-shockwave-flash");
embedBuilder.MergeAttribute("allowscriptaccess","always");
embedBuilder.MergeAttribute("allowfullscreen","true");
embedBuilder.MergeAttribute("width",width);
embedBuilder.MergeAttribute("height",height);
objBuilder.InnerHtml = helper.Param( "movie", url ) +
helper.Param( "allowFullScreen", "true" ) +
helper.Param( "allowscriptaccess", "always" ) +
embedBuilder.ToString(TagRenderMode.Normal);
return objBuilder.ToString(TagRenderMode.Normal);
}