1 回答

TA貢獻1810條經(jīng)驗 獲得超5個贊
不確定,但這是:
select new { Count = r.SiteHours, Value = r.SiteHours };
不應該是這樣的:(SiteName):
select new { Count = r.SiteHours, Value = r.SiteName };
另外,如果我可以說,您的命名非常糟糕。這不會讓你未來的工作更輕松。嘗試更具體地命名您的變量和對象。
編輯:
例如,您可以使用區(qū)域而不是代碼注釋,這將使代碼部分/部分的分離/排序/查看更加容易
更好地命名你的變量將使你的生活更輕松,其他人也更容易處理/使用你的代碼
我會將您當前的代碼更改為:
請注意,我沒有任何編輯器,并且可能存在語法錯誤
#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
我相信還有更多需要“改進”或“改變”的地方,但我認為這已經(jīng)是一種改進。
我相信其他人可以更好地闡述我在這里的建議:)
- 1 回答
- 0 關注
- 114 瀏覽
添加回答
舉報