以下のコードはきれいではないようです。コードを改善するための提案はありますか?
<li @if(ViewData["pagename"].ToString()=="Business details"){ <text>class="active" </text> } >
<a @if(ViewData["pagename"].ToString()=="Business details"){ <text>style="color: white; background-color: #08C; border: 1px solid #08C;" </text> }
href="@Url.Action("BusinessDetails", "Business")">Business Details</a>
</li>
<li @if (ViewData["pagename"].ToString() == "Booking policies"){ <text>class="active"</text> }>
<a @if (ViewData["pagename"].ToString() == "Booking policies")
{ <text>style="color: white; background-color: #08C; border: 1px solid #08C;" </text> }
href="@Url.Action("BookingPolicies", "Business")">Booking policies</a>
</li>
MVCには条件付き属性が組み込まれています...
<div @{if (myClass != null) { <text>class="@myClass"</text> } }>Content</div>
<div class="@myClass">Content</div>
@myClassがnullの場合、属性をまったく使用しません...
現在の問題を解決できるとは限りませんが、注目に値します!
http://weblogs.asp.net/jgalloway/archive/2012/02/16/asp-net-4-beta-released.aspx
<li class="@(ViewBag.pagename == "Business details" ? "active" : null)">
インラインstyle="..."
を別のクラス名に置き換え、そこで同じ構文を使用する必要があります。
ただし、ページ名とアクション名を受け取り、HTMLを一般的に生成するHTMLヘルパー拡張メソッドを個別に作成する方が簡単です。
値が空でない場合に条件付きで属性を追加し、ブール関数式がtrue
と評価される場合、条件付きで属性を追加する小さなヘルパーメソッドを使用します。
public static MvcHtmlString Attr(this HtmlHelper helper, string name, string value, Func<bool> condition = null)
{
if (string.IsNullOrEmpty(name) || string.IsNullOrEmpty(value))
{
return MvcHtmlString.Empty;
}
var render = condition != null ? condition() : true;
return render ?
new MvcHtmlString(string.Format("{0}=\"{1}\"", name, HttpUtility.HtmlAttributeEncode(value))) :
MvcHtmlString.Empty;
}
定義したら、Razorビューでこのメソッドを使用できます。
<li @(Html.Attr("class", "new", () => example.isNew))>
...
</li>
上記のコードは、<li class="new">...</li>
の場合はexample.isNew == true
をレンダリングし、そうでない場合はclass
属性全体を省略します。
MVC4で
<!DOCTYPE html>
<html>
<head>
</head>
<body>
@{
string css = "myDiv";
}
<div class='@css'></div>
</body>
</html>
または
<!DOCTYPE html>
<html>
<head>
</head>
<body>
@{
string css = "class=myDiv";
}
<div @css></div>
</body>
</html>
詳細はこちら: http://evolpin.wordpress.com/2012/05/20/mvc-4-code-enhancements/
TagWrap拡張メソッドを使用したアプローチ。質問のコードは次のようになります。
@using (Html.TagWrap("li", condition ? new { @class = "active" } : null))
{
var anchorAttrs = new Dictionary<string, object> { { "href", Url.Action("BusinessDetails", "Business") } };
if(condition)
{
anchorAttrs["style"] = "color: white; background-color: #08C; border: 1px solid #08C;";
}
using (Html.TagWrap("a", anchorAttrs))
{
<text>Business Details</text>
}
}
TagWrap拡張メソッド
microsoft.AspNetCore.Mvc.ViewFeaturesを使用します。
public static IDisposable TagWrap(this IHtmlHelper htmlHelper, string tagName, object data)
{
return htmlHelper.TagWrap(tagName, HtmlHelper.AnonymousObjectToHtmlAttributes(data));
}
public static IDisposable TagWrap(this IHtmlHelper htmlHelper, string tagName, IDictionary<string, object> data)
{
var tag = new TagBuilder(tagName);
tag.MergeAttributes(data);
htmlHelper.ViewContext.Writer.Write(tag.RenderStartTag());
return new DisposableAction(() =>
htmlHelper.ViewContext.Writer.Write(tag.RenderEndTag()));
}
Disposeの終了タグのレンダリングに使用されるヘルパークラス
public class DisposableAction : IDisposable
{
private readonly Action DisposeAction;
public DisposableAction(Action action)
{
DisposeAction = action;
}
public void Dispose()
{
DisposeAction();
}
}
ここでの解凍に基づいて、object
の代わりにstring
を使用して適応に答えます。
public static MvcHtmlString ConditionalAttr(this HtmlHelper helper, string attributeName, object value, Func<bool> condition)
{
if (string.IsNullOrEmpty(attributeName) || value == null)
{
return MvcHtmlString.Empty;
}
var render = condition != null ? condition() : true;
return render ?
new MvcHtmlString($"{attributeName}=\"{HttpUtility.HtmlAttributeEncode(value.ToString())}\"") :
MvcHtmlString.Empty;
}
この方法では、他のデータ型を渡す前に文字列で他のデータ型を有効にする必要がなく、わずかな.ToString()
を保存します。 thoに違いがあります:空の文字列を渡すとレンダリングされます。例として:
@Html.ConditionalAttr("data-foo", "", () => Model.IsFooNeeded)
// Ouput:
data-foo=""