作家
登录

记一次C#代码审查

作者: 来源: 2017-05-16 14:06:46 阅读 我要评论

select new { 
  •  
  •  
  •     u 
  •  
  •  
  •    }).First(); 
  •  
  •  
  •    favoritesList.Add(user.u); 
  •  
  •  
  •   } 
  •  
  •  
  •  } 
  •  
  •  
  •  return favoritesList; 
  •  
  •  
  • 代码变得清楚并且我不再须要担心 Cookie。

    将代码划分为自力的功能

    有两个 DB 萌芽,都应用同样的 Context。 有如下代码:

    1. public IEnumerable < GitHubUser > FollowedGitHubUsers(int userId) { 
    2.  
    3.  using(var db = new GitHubContext()) { 
    4.  
    5.   var followedUserIds = this.GiveMeFollowedUsersIdFor(db, userId); 
    6.  
    7.   return this.FindUsers(db, followedUsersIds); 
    8.  
    9.  } 
    10.  

    代码是不是看起来既简荡竽暌怪美不雅? 所有这一切,归功于功能的提取,使得代码异常轻易浏览。在这个层次上,我不在乎若何获得用户。 我只须要专注一些功能和输入的编排。

    如不雅函数没有任何参数,那么它有什么感化呢? 它大年夜哪里获得输入和当缁ご态呢? 它可能是一个全局的 - 但这也是一个不和模式。函数须要纯输入和纯输出。

    如不雅我须要懂得更多的细节,我才会进入这个办法查看。

    1. private IEnumerable<GitHubUser>FindUsers(GitHubContext db, IEnumerable<int> userIds) 
    2.  
    3.  
    4.  
    5.  
    6.         List<GitHubUser> favoritesList = new List<GitHubUser>(); 
    7.  
    8.  
    9.         foreach (var result in results) 
    10.  
    11.  
    12.         { 
    13.  
    14.  
    15.             var user = (from u in db.GitUsers 
    16.  
    17.  
    18.                         where u.Id == result.GitUserId 
    19.  
    20.  
    21.                         select new { u }).First(); 
    22.  
    23.  
    24.             favoritesList.Add(user.u); 
    25.  

    26.   推荐阅读

        老外有话说 QubesOS初体验

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


      本文标题:记一次C#代码审查

      地址:http://www.17bianji.com/lsqh/35240.html

    关键词: 探索发现

    乐购科技部分新闻及文章转载自互联网,供读者交流和学习,若有涉及作者版权等问题请及时与我们联系,以便更正、删除或按规定办理。感谢所有提供资讯的网站,欢迎各类媒体与乐购科技进行文章共享合作。

    网友点评
    自媒体专栏

    评论

    热度

    精彩导读
    栏目ID=71的表不存在(操作类型=0)