Order `TagHelperExecutionContext.TagHelpers` without `.OrderBy`.

- By using linq we created several `OrderedEnumerable` allocations.

#601
This commit is contained in:
N. Taylor Mullen 2015-11-19 14:42:41 -08:00
parent 6b43ebc88b
commit 43aabcb2d6
2 changed files with 27 additions and 6 deletions

View File

@ -135,7 +135,7 @@ namespace Microsoft.AspNet.Razor.Runtime.TagHelpers
/// <summary> /// <summary>
/// <see cref="ITagHelper"/>s that should be run. /// <see cref="ITagHelper"/>s that should be run.
/// </summary> /// </summary>
public IEnumerable<ITagHelper> TagHelpers public IList<ITagHelper> TagHelpers
{ {
get get
{ {

View File

@ -2,6 +2,7 @@
// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information. // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
using System; using System;
using System.Collections.Generic;
using System.Linq; using System.Linq;
using System.Threading.Tasks; using System.Threading.Tasks;
using Microsoft.AspNet.Razor.TagHelpers; using Microsoft.AspNet.Razor.TagHelpers;
@ -31,11 +32,12 @@ namespace Microsoft.AspNet.Razor.Runtime.TagHelpers
executionContext.AllAttributes, executionContext.AllAttributes,
executionContext.Items, executionContext.Items,
executionContext.UniqueId); executionContext.UniqueId);
var orderedTagHelpers = executionContext.TagHelpers.OrderBy(tagHelper => tagHelper.Order);
foreach (var tagHelper in orderedTagHelpers) OrderTagHelpers(executionContext.TagHelpers);
for (var i = 0; i < executionContext.TagHelpers.Count; i++)
{ {
tagHelper.Init(tagHelperContext); executionContext.TagHelpers[i].Init(tagHelperContext);
} }
var tagHelperOutput = new TagHelperOutput( var tagHelperOutput = new TagHelperOutput(
@ -46,12 +48,31 @@ namespace Microsoft.AspNet.Razor.Runtime.TagHelpers
TagMode = executionContext.TagMode, TagMode = executionContext.TagMode,
}; };
foreach (var tagHelper in orderedTagHelpers) for (var i = 0; i < executionContext.TagHelpers.Count; i++)
{ {
await tagHelper.ProcessAsync(tagHelperContext, tagHelperOutput); await executionContext.TagHelpers[i].ProcessAsync(tagHelperContext, tagHelperOutput);
} }
return tagHelperOutput; return tagHelperOutput;
} }
private static void OrderTagHelpers(IList<ITagHelper> tagHelpers)
{
// Using bubble-sort here due to its simplicity. It'd be an extreme corner case to ever have more than 3 or
// 4 tag helpers simultaneously.
ITagHelper temp = null;
for (var i = 0; i < tagHelpers.Count; i++)
{
for (var j = i + 1; j < tagHelpers.Count; j++)
{
if (tagHelpers[j].Order < tagHelpers[i].Order)
{
temp = tagHelpers[i];
tagHelpers[i] = tagHelpers[j];
tagHelpers[j] = temp;
}
}
}
}
} }
} }