记一次C#代码审查

你想在这诚实现什么?

有什么请求?

这是启动每个代码审查过程的最佳方法。一请求做代码审查的人已经知道这些问题的谜底。此函数返回一个 GitHub 收藏的用户列表。在 GitHub 上，有一个追踪用户的才能。

- LINQ 表示我们正在做一些萌芽。

看起来是很通俗的代码。大年夜哪儿开端呢? 高低文! 高低文是重点。这很明显。但起首，你须要提出以下问题：

- 你想在这诚实现什么?

- 有什么请求?

- HttpContext 和 Cookies 在 ASP.NET 范围内履行。

初步看法：

- 我看到 Git，但没有GitHub API客户端调用。

- Context 关键字是实体框架(ORM 拜访数据)。

- 没有参数的函数返收受吸收藏夹列表。

问题：

- 在哪里调用 GitHub API? 这是一个存储在外部数据存储区中的 GitHub 数据的快照。

定名商定

幻想的代码是易于懂得的代码。我确切偏爱 KISS 的原则; 这是一个务实的做法。一个简单而优雅的代码成功一切。我欲望读一份代码就像读一本论述书。实际中必定也存在某种机制，可以使我们的代码更好读。让我们大年夜函数名开端。

public IEnumerable < GitHubUser > FavoritesList() { 
 
 
 CookieHelper cookieHelper = new CookieHelper(this.HttpContext); 
 
 
 HttpCookie httpCookie = cookieHelper.SetAndGetHttpCookies(); 
 
 
 MyCookie myCookie = new MyCookie() { 
 
 
  ID = Convert.ToInt32(httpCookie["Id"]) 
 
 
 }; 
 
 
 List < GitUser > favoritesList = new List < GitUser > (); 
 
 
 using(var db = new GitHubContext()) { 
 
 
  var results = (from ch in db.CookiesHistory where ch.UserId == myCookie.ID select new { 
 
 
   GitUserId = ch.GitUserId 
 
 
  }); 
 
 
  foreach(var result in results) { 
 
 
   var user = (from u in db.GitUsers where u.Id == result.GitUserId select new { 
 
 
    u 
 
 
   }).First(); 
 
 
   favoritesList.Add(user.u); 
 
 
  } 
 	
			 1/4    1 2 3 4 下一页 尾页	
			

　　推荐阅读
　　老外有话说 QubesOS初体验
            
            【51CTO.com快译】众所周知，应用GUN/Linux体系的安然性要比微软windows赶过不少，但这并不料味着应用GUN/Linux就万无一掉。用户对体系安然越来越高的请求，正促使着体系套件的推想出新，>>>详细阅读


本文标题：记一次C#代码审查
地址：http://www.17bianji.com/lsqh/35240.html
 1/2    1