C# Razor foreach loop syntax in Javascript

Question

I got the following javascript code from google chart:

function drawTimeChart() {
    var dataArray = generateTimeChartArray();
    var data = google.visualization.arrayToDataTable([
    ['2001', '25'],
      ['2002', 35],
      ['2003', 25],
      ['2004', 55]
    ]);

    var options = {
        title: 'Reported devices in MAUDE based on the term since 1990',
        backgroundColor: '#EFEEEF',
        hAxis: { showTextEvery: 4 }
    };

    var chart = new google.visualization.LineChart(document.getElementById('MaudeTimeChart'));

    chart.draw(data, options);
}

The data that I insert into data i want to be dynamicly added. I tried doing like this:

var data = google.visualization.arrayToDataTable([
                @foreach (var item in Model.NumberOfReportedRecords){
                    [@Html.DisplayFor(modelItem => item.year), @Html.DisplayFor(modelItem => item.Status)]
                 }
                ]);

Anyone know how to make this work?

Vladimirs · Accepted Answer · 2014-12-02 14:09:46Z

6

You don't need to use Html.DisplayFor as it will render template instead of just value. Use simply @item.year for getting that value like:

var data = google.visualization.arrayToDataTable([
                @foreach (var item in Model.NumberOfReportedRecords){
                    ['@item.year', '@item.Status']
                 }
                ]);

Update 1.

Also I have some doubts about how you defined foreach loop - it might not work but you can also try something like that (@: explicitly indicates the start of content):

@foreach (var item in Model.NumberOfReportedRecords) 
        {
            @:['@(item.year)', '@(item.Status)'],
        }

edited Dec 2, 2014 at 14:09

answered Dec 2, 2014 at 13:55

Vladimirs

8,6595 gold badges50 silver badges88 bronze badges

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

3 Comments

Rhumborl Over a year ago

you should also look at HttpUtility.JavaScriptStringEncode to sanitize the output

Ørjan Over a year ago

Thanks! This is what i thought as well. But when i write this i get a bunch of errors saying "; expected", "Invalid expression term ,", "Only assignment, call, increment, decrement, await, and new object expressions can be used as a statement" Got any clue? While if i leave the foreach loop empty or insert the values manually like [2014, 15] no error is displayed

Ørjan Over a year ago

Than you very much! The last update fixed the problem! @: was the thing I was looking for :)

Collectives™ on Stack Overflow

C# Razor foreach loop syntax in Javascript

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related