Блог

Я ищу код друзей для внедрения Google charts в мой проект mvc.

У меня есть модель, из которой я извлекаю данные:

 public partial class HoursPerSite
{
    public string SiteName { get; set; }
    public Nullable<decimal> SiteHours { get; set; }
}
  

Диаграмма отрисована, и выглядит она хорошо, но в легенде показаны SiteHours, а не SiteName, как мне бы хотелось.

Вот часть контроллера:

 // HOLIDAY PIE START
ViewBag.msg = db.HoursPerSites.Count().ToString();

var query = from r in db.HoursPerSites
            select new { Count = r.SiteHours, Value = r.SiteHours };

var result2 = query.ToList();

var datachart2 = new object[result2.Count];
int l = 0;
foreach (var i in result2)
{
    datachart2[l] = new object[] { i.Value.ToString(), i.Count };
    l  ;
}
string datastr2 = JsonConvert.SerializeObject(datachart2, Formatting.None);
ViewBag.dataj2 = new HtmlString(datastr2);

// HOLIDAY PIE END
  

Я бы хотел, чтобы в условных обозначениях круговой диаграммы отображался сайт, а не часы.

Не уверен, но так ли это:

 select new { Count = r.SiteHours, Value = r.SiteHours };
  

Не должно быть этого: (SiteName):

 select new { Count = r.SiteHours, Value = r.SiteName };
  

Кроме того, ваше именование, если можно так выразиться, очень плохое. Это не облегчит вашу будущую работу. Попробуйте назвать свои переменные и objects более конкретно.

Редактировать:

• Вы можете использовать регионы вместо комментариев к коду, например, что значительно упростит разделение / упорядочивание / просмотр частей / разделов кода
• Улучшение именования ваших переменных облегчит вашу жизнь, а также жизнь других людей, работающих над вашим кодом

Я бы изменил ваш текущий код на этот:

Пожалуйста, обратите внимание, что у меня нет никакого редактора и что могут быть синтаксические ошибки

 #region Holiday Pie Chart

ViewBag.msg = db.HoursPerSites.Count().ToString();

var queryHoursPerSites = from r in db.HoursPerSites
        select new { Count = r.SiteHours, Value = r.SiteHours };

var resultsQueryHoursPerSites = queryHoursPerSites.ToList(); // or HoursPerSitesCollection

var holidayPieChart = new object[resultsQueryHoursPerSites.Count];
int counter = 0; // 
foreach (var record in resultsQueryHoursPerSites)
{
    holidayPieChart[counter] = new object[] { record.Value.ToString(), record.Count };
counter  ;
}

string deserialisedResults = JsonConvert.SerializeObject(holidayPieChart, Formatting.None);

// no idea what dataj2 is here ...
ViewBag.dataj2 = new HtmlString(deserialisedResults);


#endregion
  

Я уверен, что есть еще что «улучшить» или «изменить», но это, на мой взгляд, уже было бы улучшением.

Я уверен, что другие могут еще лучше разработать то, что я предлагаю здесь 🙂