如何重构此C#代码以使其更易于阅读？

内容导读

互联网集市收集整理的这篇技术教程文章主要介绍了如何重构此C#代码以使其更易于阅读？，小编现在分享给大家，供广大互联网技能从业者学习和参考。文章包含5532字，纯文字阅读大概需要8分钟。

内容图文

这是我的Controller中的一个动作-在我的整个项目中,我的Controller中都有类似的大型方法.

我正在尝试学习将这些东西放在哪里以及如何清理它们.我是新手,如果我看到了一个很好的示例来更改自己的代码,那么它很可能会教我如何对大量代码进行更改.

这是我的动作：

public ActionResult Index(string sortOrder, string currentFilter, string searchString, int? page)
{
    ViewBag.CurrentSort = sortOrder;
    ViewBag.TitleSortParm = String.IsNullOrEmpty(sortOrder) ? "Title desc" : "";
    ViewBag.CreditsSortParm = sortOrder == "Credits" ? "Credits desc" : "Credits";
    ViewBag.ElectiveSortParm = sortOrder == "Elective" ? "Elective desc" : "Elective";
    ViewBag.InstructorSortParm = sortOrder == "Instructor" ? "Instructor desc" : "Instructor";
    ViewBag.YearSortParm = sortOrder == "Year" ? "Year desc" : "Year";
    ViewBag.AttendingDaysSortParm = sortOrder == "AttendingDays" ? "AttendingDays desc" : "AttendingDays";
    ViewBag.AttendanceCapSortParm = sortOrder == "AttendanceCap" ? "AttendanceCap desc" : "AttendanceCap";
    ViewBag.StartDateSortParm = sortOrder == "StartDate" ? "StartDate desc" : "StartDate";
    ViewBag.LocationSortParm = sortOrder == "Location" ? "Location desc" : "Location";
    ViewBag.ParishSortParm = sortOrder == "Parish" ? "Parish desc" : "Parish";
    ViewBag.DescriptionSortParm = sortOrder == "Description" ? "Description desc" : "Description";
    ViewBag.ApprovedSortPArm = sortOrder == "Approved" ? "Approved desc" : "Approved";
    ViewBag.CompletedSortPArm = sortOrder == "Completed" ? "Completed desc" : "Completed";
    ViewBag.ArchivedSortPArm = sortOrder == "Archived" ? "Archived desc" : "Archived";

    if (Request.HttpMethod == "GET")
    {
        searchString = currentFilter;
    }
    else
    {
        page = 1;
    }
    ViewBag.CurrentFilter = searchString;

    var courses = from s in db.Courses
                    select s;
    if (!String.IsNullOrEmpty(searchString))
    {
        courses = courses.Where(s => s.Title.ToUpper().Contains(searchString.ToUpper()));
    }
    switch (sortOrder)
    {
        case "Title desc":
            courses = courses.OrderByDescending(s => s.Title);
            break;
        case "Credits":
            courses = courses.OrderBy(s => s.Credits);
            break;
        case "Credits desc":
            courses = courses.OrderByDescending(s => s.Credits);
            break;
        case "Elective":
            courses = courses.OrderBy(s => s.Credits);
            break;
        case "Elective desc":
            courses = courses.OrderByDescending(s => s.Credits);
            break;
        case "Instructor":
            courses = courses.OrderBy(s => s.Instructor.LastName);
            break;
        case "Instructor desc":
            courses = courses.OrderByDescending(s => s.Instructor.LastName);
            break;
        case "Year":
            courses = courses.OrderBy(s => s.Year);
            break;
        case "Year desc":
            courses = courses.OrderByDescending(s => s.Year);
            break;
        case "AttendingDays":
            courses = courses.OrderBy(s => s.AttendingDays);
            break;
        case "AttendingDays desc":
            courses = courses.OrderByDescending(s => s.AttendingDays);
            break;
        case "AttendanceCap":
            courses = courses.OrderBy(s => s.AttendanceCap);
            break;
        case "AttendanceCap desc":
            courses = courses.OrderByDescending(s => s.AttendanceCap);
            break;
        case "StartDate":
            courses = courses.OrderBy(s => s.StartDate);
            break;
        case "StartDate desc":
            courses = courses.OrderByDescending(s => s.StartDate);
            break;
        case "Location":
            courses = courses.OrderBy(s => s.Location);
            break;
        case "Location desc":
            courses = courses.OrderByDescending(s => s.Location);
            break;
        case "Parish":
            courses = courses.OrderBy(s => s.Parish);
            break;
        case "Parish desc":
            courses = courses.OrderByDescending(s => s.Parish);
            break;
        case "Description":
            courses = courses.OrderBy(s => s.Description);
            break;
        case "Description desc":
            courses = courses.OrderByDescending(s => s.Description);
            break;
        case "Approved":
            courses = courses.OrderBy(s => s.Approved);
            break;
        case "Approved desc":
            courses = courses.OrderByDescending(s => s.Approved);
            break;
        case "Completed":
            courses = courses.OrderBy(s => s.Completed);
            break;
        case "Completed desc":
            courses = courses.OrderByDescending(s => s.Completed);
            break;
        case "Archived":
            courses = courses.OrderBy(s => s.Archived);
            break;
        case "Archived desc":
            courses = courses.OrderByDescending(s => s.Archived);
            break;
        default:
            courses = courses.OrderBy(s => s.Title);
            break;
    }
    int pageSize = 4;
    int pageNumber = (page ?? 1);
    return View(courses.ToPagedList(pageNumber, pageSize));
}

我应该对上面的代码做什么以使其更具可读性？我是否只是将其中的一部分作为单独的方法移出并移至Controller的底部？我是否将方法放在另一个文件中的某个位置,并在此处引用？

请记住,我正在学习并且很清楚.

解决方法:

更改代码的方式将纯粹是主观的,这是一个创造性的过程,您需要经历并奋斗才能成为更好的开发人员.

但是作为一个伪代码示例,目的是使您的方法更小.不,真的,甚至更小！

使函数名称非常有意义,并确保它们告诉程序员方法的真正含义是什么.这只是结构的一个示例：

public class Something {

   public ActionResult Index() {
       MakeCallToFunction();
       var something = GetSomething();
       var somethingElse = GetSomethingElse(something);
       var model = new SomeViewModel(somethingElse);
       return View(model);
   }

   private void MakeCallToFunction () {
       ...
   }

   private string GetSomething() {
       ...
       return someVal;
   }

   private SomeObject GetSomethingElse(string something) {
       ...
       return someBigObject;
   }
}

或者这可能意味着您要利用所有这些小功能并上一堂课

public class Something {

   public ActionResult Index(int id) {
       var model = new SomeRelatedStuff.Load(id);
       return View(model);
   }
}

private class SomeRelatedStuff {

   public SomeRelatedStuff()

   public static SomeRelatedStuff Load(int id) {
       var something = GetSomething();
       var somethingElse = GetSomethingElse(something);
       var model = new SomeViewModel(somethingElse);
       MakeObject();
       return this;
   }

   private string GetSomething() {
       ...
       return someVal;
   }

   private SomeObject GetSomethingElse(string something) {
       ...
       return someBigObject;
   }

   private void MakeObject () {
       ...
   }

}

另外请请罗伯特·马丁(罗伯特·马丁)(鲍勃叔叔)获得清洁代码,至少在本周末阅读前四章.然后再次阅读它们.

内容总结

以上是互联网集市为您收集整理的如何重构此C#代码以使其更易于阅读？全部内容，希望文章能够帮你解决如何重构此C#代码以使其更易于阅读？所遇到的程序开发问题。如果觉得互联网集市技术教程内容还不错，欢迎将互联网集市网站推荐给程序员好友。

内容备注

版权声明：本文内容由互联网用户自发贡献，该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务，不拥有所有权，不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容，请发送邮件至 gblab@vip.qq.com 举报，一经查实，本站将立刻删除。

内容手机端

扫描二维码推送至手机访问。

本文链接：https://qyyshop.com/info/674405.html

来源：【匿名】

【上一篇】C#-Twitter OAuth流程-对oob / 3足身份验证和一般流程感到困惑,不需要PIN吗？【下一篇】assemblyinfo.csC#Assembly类访问程序集信息

更多 ►

【如何重构此C#代码以使其更易于阅读？】教程文章相关的互联网学习教程文章

C#连接ODBC数据源代码// Namespaces, variables, and constantsusing System;using System.Configuration;using System.Data;using System.Data.Odbc;private void CButton_Click(object sender, System.EventArgs e){ // Create the DataAdapter.#codego.net # codego.net# String sql = "SELECT ID, Name FROM Products"; OdbcDataAdapter da = new OdbcDataAdapter(sql,ConfigurationSettings.AppSettings["connection...

分页存储过程 + C#后台代码

ALTER proc [dbo].[p_page](@tableName varchar(8000),--必须@selectFields varchar(8000)=‘*‘,@andWhere varchar(4000),--不带where ，只写条件如 and 1=1@orderByFields varchar(100),--必须 row_number() over(order by)用@pageIndex int=1,@pageSize int=20,@totalCount int=0 output,@orderType varchar(5)=‘asc‘,--desc,asc@sql nvarchar(4000) output--返回当前执行的sql语句)asbegin if @selectFields=‘‘ or @select...

C#控制鼠标代码实例【代码】

1获得当前屏幕中鼠标的位置 int i = MousePosition.X;int j = MousePosition.Y; 这是control类中的方法。2移动鼠标首先引入dll [System.Runtime.InteropServices.DllImport("user32")] private static extern int mouse_event(int dwFlags, int dx, int dy, int cButtons, int dwExtraInfo); const int MOUSEEVENTF_MOVE = 0x0001; const int MOUSEEVENTF_LEFTDOWN = 0x0002; const in...

【转】编写高质量代码改善C#程序的157个建议——建议109：谨慎使用嵌套类【代码】

建议109：谨慎使用嵌套类使用嵌套类的原则是：当某类型需要访问另一个类型的私有成员时，才将它实现为嵌套类。一个典型的例子是在实现集合时，要为集合实现迭代器，这时用到了嵌套类。代码如下所示：publicclass ArrayList : IList, ICollection, IEnumerable, ICloneable {//省略publicvirtual IEnumerator GetEnumerator(){returnnew ArrayListEnumeratorSimple(this);}[Serializable]privatesealedclass ArrayListEnumeratorSim...

委托和事件C#演示代码【代码】

class Cat {privatestring _name;public Cat(string name){_name = name;}publicvoid Shout(){Console.WriteLine("瞄，我是"+_name+"，我来了！");CatShout(); //第三步，触发点，事件要加括号 }publicdelegatevoid CatShoutEventHandler(); //第一步，声明委托，要加圆括号publicevent CatShoutEventHandler CatShout; //第二步，声明事件，类型为以上委托类型}class Mouse {publicstring _name;public Mouse...

vs2010中使用Nunit测试c#代码结果的正确性【图】

本文转载自：http://blog.csdn.net/pukuimin1226/article/details/8112151http://www.nunit.org/index.php?p=download上面地址下载，有安装版的，有直接解压版的，只有里面的nunit.framewor.dll就可以做简单的测试了。安装版的路径默认为：C:\Program Files \NUnit 2.6\bin\framework\nunit.framewor.dll从vs2010中直接下载安装插件：安装完后提示重启vs,然后就可以打开用了，在视图中可以打开：然后写好测试代码之后，就能够看到结...

编写高质量代码改善C#程序的157个建议——建议14：正确实现浅拷贝和深拷贝【代码】

建议14：正确实现浅拷贝和深拷贝为对象创建副本的技术称为拷贝（也叫克隆）。我们将拷贝分为浅拷贝和深拷贝。浅拷贝　将对象中的所有字段复制到新的对象（副本）中。其中，值类型字段的值被复制到副本中后，在副本中的修改不会影响到源对象对应的值。而引用类型的字段被复制到副本中的是引用类型的引用，而不是引用的对象，在副本中对引用类型的字段值做修改会影响到源对象本身。深拷贝　同样，将对象中的所有字段复制到新的对象中...

C#获取当前运行的源代码的文件名和当前源代码的行数的方法【图】

1,.声明：本文转载自http://www.cnblogs.com/lvdongjie/p/5777330.html2.工程代码： class Program { static void Main(string[] args) { Console.WriteLine(GetCurSourceFileName()+ ","+GetLineNum()); Console.ReadLine(); } /// 取得当前源码的哪一行 /// </summary> /// <returns></returns> public static int GetLineNum() { S...

C#用正则表达式获取网页源代码标签的属性或值【代码】

原文地址：http://blog.csdn.net/lhfly/article/details/7684319 整理两个在C#中，用正则表达式获取网页源代码标签的属性或值的方法：1、获取标签中的值： <a href="www.csdn.net" class="main" >CSDN</a> 结果：CSDN///<summary>/// 获取字符中指定标签的值///</summary>///<param name="str">字符串</param>///<param name="title">标签</param>///<returns>值</returns>publicstaticstring GetTitleContent(string str, stri...

提取HTML代码中文字的C#函数

/// <summary> /// 去除HTML标记 /// </summary> /// <param name="strHtml">包括HTML的源码 </param> /// <returns>已经去除后的文字</returns> public static string StripHTML(string strHtml) { string [] aryReg ={ @"<script[^>]*?>.*?</script>", @"<(\/\s*)?!?((\w+:)?\w+)(\w+(\s*=?\s*(([""‘])(\\[""‘tbnr]|[^\7])*?\7|\w+)|.{0})|\s)*?(\/\s*)?>", @"([\r\n])[\s]+", @"...

webbrowser代理c#代码实现

微软webbrowser控件也就是IE插件，他的所有功能就像IE类似，当然设置也是一样的，下面介绍下webbrowser如何设置代理，可不要用这个对抗广告联盟哦You can change the proxy with InternetSetOption method from the wininet.dll, here is a example to set the proxy:using System.Runtime.InteropServices;Public struct Struct_INTERNET_PROXY_INFO { public int dwAccessType; public IntPtr proxy; public IntPtr proxyBypa...

【译】最大限度地降低多线程 C# 代码的复杂性【代码】

原文:【译】最大限度地降低多线程 C# 代码的复杂性分支或多线程编程是编程时最难最对的事情之一。这是由于它们的并行性质所致，即要求采用与使用单线程的线性编程完全不同的思维模式。对于这个问题，恰当类比就是抛接杂耍表演者，必须在空中抛接多个球，而不要让它们相互干扰。这是一项重大挑战。然而，通过正确的工具和思维模式，这项挑战是能应对的。本文将深入介绍我为了简化多线程编程和避免争用条件、死锁等其他问题而编写的一...

基于c#的工控软件版本v1.03_11.25（附源代码工程下载地址）最新版本【图】

代码地址：http://pan.baidu.com/share/link?shareid=3381776290&uk=588847378点击打开链接新版本做了3个方面修改1，修改了原有的打开关闭的链接图片，改到本地工程中2，添加了定时发送机制3，添加的帮助栏using System; using System.Collections.Generic; using System.ComponentModel; using System.Data; using System.Drawing; using System.Linq; using System.Text; using System.Windows.Forms; using System.Collections;...

旺财C# .NET代码生成器之DTcms4/5代码批量生成功能【图】

经近半年的实战积累、于2017年6月底，增加对第三方开源软件DTcms(4和5都支持)的代码生成支持。可以一键生成的DTcms4/5代码如下：1.Model经典三层实体2.DAL经典三层数据库访问层3.BLL经典三层业务层4.Web/Admin/list列表页面及代码5.Web/Admin/edit编辑页面及代码6.Web/tools/ashx增删改服务页面及代码7.Web.UI/Page/列表、编辑、详情明细显示的前端配套代码8.Web.UI/Label/配套前端的相关标签代码DTcms数据库表及前缀等设置界面：D...

WPF中C#代码触发鼠标点击事件【代码】【图】

1.如下代码;1<Button x:Name="btnTest" Click="btnTest_Click">2<Button.Triggers>3<EventTrigger RoutedEvent="Button.Click">4<BeginStoryboard>56</BeginStoryboard>7</EventTrigger>8</Button.Triggers>9</Button>View Code在Button， btnTest按钮上既绑定了点击事件的处理方法(btnTest_Click())，也添加了路由事件的触发器(点击时执行动画)，意思是：当用户点击btnTest按钮时，同时执行btnTest_Click(...

C# - 技术教程分类

C# 教程 C# 简介 C# 环境 C# 程序结构 C# 基本语法 C# 数据类型 C# 类型转换 C# 变量 C# 常量 C# 运算符 C# 判断 C# 循环 C# 封装 C# 方法 C# 可空类型 C# 数组（Array） C# 字符串（String） C# 结构体（Struct） C# 枚举（Enum） C# 类（Class） C# 继承 C# 多态性 C# 运算符重载 C# 接口（Interface） C# 命名空间（Namespace） C# 预处理器指令 C# 异常处理 C# 特性（Attribute） C# 反射（Reflection） C# 属性（Property） C# 索引器（Indexer） C# 委托（Delegate） C# 事件（Event） C# 集合（Collection） C# 泛型（Generic） C# 匿名方法 C# 不安全代码 C# 多线程 c# 全部

C# - 最热教程

c#编程连接oracle数据库无法加载DLL(oc...一个适合新手C#程序员编写练习的小项目...C# 异步UDP发送接收数据 C#程序执行时间长和慢查询解决：线程并...c# aspose操作word文档 C#继承讲解以及对象的创建 jquery+ajax+C#实现无刷新操作数据库数...C#精髓 GridView72大绝技学习gridview...c# static的全部用法收集整理 js模拟实现类似c#下的hashtable的简单功...

首页 / C# / 如何重构此C#代码以使其更易于阅读？

如何重构此C#代码以使其更易于阅读？

内容导读

内容图文

内容总结

内容备注

内容手机端

【如何重构此C#代码以使其更易于阅读？】教程文章相关的互联网学习教程文章

C#连接ODBC数据源代码

分页存储过程 + C#后台代码

C#控制鼠标代码实例【代码】

【转】编写高质量代码改善C#程序的157个建议——建议109：谨慎使用嵌套类【代码】

委托和事件C#演示代码【代码】

vs2010中使用Nunit测试c#代码结果的正确性【图】

编写高质量代码改善C#程序的157个建议——建议14：正确实现浅拷贝和深拷贝【代码】

C#获取当前运行的源代码的文件名和当前源代码的行数的方法【图】

C#用正则表达式获取网页源代码标签的属性或值【代码】

提取HTML代码中文字的C#函数

webbrowser代理c#代码实现

【译】最大限度地降低多线程 C# 代码的复杂性【代码】

基于c#的工控软件版本v1.03_11.25（附源代码工程下载地址）最新版本【图】

旺财C# .NET代码生成器之DTcms4/5代码批量生成功能【图】

WPF中C#代码触发鼠标点击事件【代码】【图】

C# - 技术教程分类

C# - 最新教程

C# - 最热教程