How to use @Html.DisplayFor in ForEach statement

Question

I am trying to use @HTML.DisplayFor in @ForEach statement but not successful.

The following is my existing code:

@foreach (var cost in Model.CustomerCost)
{
    <tr>
        <td>@cost .CustomerName</td>
        <td>@(cost.Cost!= 0 ? string.Format("{0:C}", cost.Cost) + " USD" : "-")</td>
    </tr>
}

However, i am trying to replace the following line of code

<td>@(cost.Cost!= 0 ? string.Format("{0:C}", cost.Cost) + " USD" : "-")</td>

with

@HTMLDisplayFor

<td>@(cost.Cost!= 0 ? Html.DisplayFor(model => model.CustomerCost[cost].Cost + "USD":"-")</td>

What is wrong in @HTML.DisplayFor syntax?

score 10 · Accepted Answer · 2015-05-02 05:12:44Z

10

To access the Cost property of cost use

@(cost.Cost != 0 ? Html.DisplayFor(m => cost.Cost) + "USD":"-")

Side note: You may want to consider making property Cost nullable and using the DisplayFormat attribute on the property

[DisplayFormat(DataFormatString = "{0:C}USD", NullDisplayText = "-")]
public decimal? Cost { get; set; }

in which case you can simplify the view to just

@Html.DisplayFor(m => cost.Cost)

Note also the format you were using would be used in a for loop where cost is the indexer

for(int i = 0; i < Model.CustomerCost.Count; i++)
{
  @Html.DisplayFor(m => m.CustomerCost[i].Cost)
}

edited May 2, 2015 at 5:12

answered May 2, 2015 at 5:09

user3559349

Sign up to request clarification or add additional context in comments.

5 Comments

TheKingPinMirza Over a year ago

Thanks, for your explanation. i understand it now. Also thanks for telling handling nullable using data annotation.

user3559349 Over a year ago

Just added another explanation FYI :)

TheKingPinMirza Over a year ago

Thanks Stephen Muecke :)

TheKingPinMirza Over a year ago

@Stephan, i used [DisplayFormat(DataFormatString = "{0:C0}", NullDisplayText = "-")] in my model and than <td>@Html.DisplayFor(m => cost.Cost)</td> ------ It shows 0 for nullable but not "-"... Any idea? I think in my case it is not null but 0.0 so NullDisplayText won't work here. Correct me if i am wrong?

user3559349 Over a year ago

The use of NullDisplayText is only applicable for a nullable type, so your property would need to be public decimal? Cost { get; set; } (or Nullable<decimal>) and the value of the property would need to be null. I only added that side note because I was a bit puzzled why you would display a zero value as "-" rather than "$0.00USD"

Collectives™ on Stack Overflow

How to use @Html.DisplayFor in ForEach statement

1 Answer 1

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related