前往小程序,Get更优阅读体验!
立即前往
首页
学习
活动
专区
工具
TVP
发布
社区首页 >专栏 >代码重构——程序员应有的基因

代码重构——程序员应有的基因

作者头像
雪雁-心莱科技
发布2018-12-27 10:32:38
2930
发布2018-12-27 10:32:38
举报
文章被收录于专栏:magicodesmagicodes

去年帮另一个项目组赶项目时,总是有很多地方令我不舒服。本人编码有点小小的“洁癖”——即不喜欢见到冗余代码。即时现在忙,没时间整理,但是一有时间,我都尽可能的去重构。也许本人是个完美主义者吧。

个人认为重构有以下好处:

  • 改善软件的性能、质量。
  • 使架构更加合理。
  • 使代码更容易被理解。
  • 提高软件的扩展性和可维护性。
  • 提高代码的重用率,通用性。
  • 提高软件的开发速度。
  • 更容易发现BUG。
  • 可以提高开发人员的开发水平。

在我见过的很多代码中,很多程序员宁愿无限复制粘贴代码,也不愿意重构代码。比如经常可以看到这样一个Switch结构中,每个Case块除了某个参数不同,里面的代码几乎一模一样。看到这种代码,不知道是我的悲哀,还是他的悲哀。

这里就展开说说:

一、控件绑定

在开发表单的时候,我们经常会用到下拉列表。如果是服务器控件,我们通常会在后台绑定,但如果是html控件,我们通常需要写JavaScript来绑定,比如下面这个例子:

代码语言:javascript
复制
$.post(siteBase + "MktUser/Op.aspx?op=items", {}, function(res) {
    if (res != null) {
        // 事业别
        var html = "<option value=''></option>";
        $(res.Data.items).each(function(i, item) {
            html += "<option value='" + item.Id + "'>" + item.Name + "</option>";
        });
        $("#item_id").html(html);
    }
}, 'json');
代码语言:javascript
复制
这是一个绑定下拉列表(select)的例子,如果每次都写这么多,一个不太复杂的页面也会让这些代码占满,那么我们可以重构下,如下面的代码:
代码语言:javascript
复制
function setSelect(se, key, data, val, txt, seval, setf) {
    $.post("Op.aspx?op=" + key, data, function (res) {
        if (res != null) {
            var html = "<option value=''></option>";
            var jsonData = (res.Data.items !== undefined ? res.Data.items : res.Data);
            $(jsonData).each(function (i, item) {
                var jTxt = '';
                var jVal = '';
                $.each(item, function (j) {
                    if (j == val)
                        jVal = item[j];
                    else if (j == txt)
                        jTxt = item[j];
                });
                html += "<option value='" + jVal + "' " + ((seval !== undefined && seval == jVal) ? "selected='selected'" : "") + ">" + jTxt + "</option>";
            });
            $(se).html(html);
            if (setf !== undefined)
                setTimeout(setf, 100);
        }
    }, 'json');
}

二、代码简化、灵活运用

很多代码我们可以简化或者灵活运用。关于简化,比如使用三元运算符。关于灵活运用,比如很多朋友喜欢将不为空的判断写死在JS里,那么我们也可以换一种方式,比如使用html元素的自定义属性或者文本来判断,比如:

代码语言:javascript
复制
function valText() {
    var isFlag = true;
    $("td:contains('*')").each(function () {
        var lbl = $(this).text();
        $(this).next().find("input,select").each(function () {
            if ($(this).val() == '') {
                alert("请填写:o" + lbl);
                isFlag = false;
                return isFlag;
            }
        });
        if (!isFlag)
            return isFlag;
    });
    return isFlag;
}

三、代码归类

很多人经常羡慕别人有代码库,自己总是没有呢?其实在平常编码中,只要用心重构就会有了。

在你重构代码的时候,一般你就会思考,如何重构才能够更优,如何重构这段代码就能尽可能多的重复利用(以后也能用),于是乎,有时你会去找找相关的代码,有时你会把类似的处理逻辑的代码统一放在一起,比如邮件处理的代码、文件操作的代码等等。如果是C#,你可以归类,如果是JS,你可以考虑放到一个JS文件里。

关于这些,大家可以从这里《也把咱的小类库拿出来晒晒》打劫。

四、提取函数

提取函数很实用,在VS中使用快捷键——Ctrl+R+M就能实现提取方法了。提取方法一方面方便代码重用,另一方面也增加了发现问题的几率。比如看下面的异常:

代码语言:javascript
复制
_shutDownMessage=关键目录的更改通知。
bin dir change or directory rename
HostingEnvironment initiated shutdown
关键目录的更改通知。
bin dir change or directory rename
HostingEnvironment 导致关闭

_shutDownStack=   在 System.Environment.GetStackTrace(Exception e, Boolean needFileInfo)
   在 System.Environment.get_StackTrace()
   在 System.Web.Hosting.HostingEnvironment.InitiateShutdownInternal()
   在 System.Web.Hosting.HostingEnvironment.InitiateShutdown()
   在 System.Web.HttpRuntime.ShutdownAppDomain(String stackTrace)
   在 System.Web.HttpRuntime.OnCriticalDirectoryChange(Object sender, FileChangeEvent e)
   在 System.Web.FileChangesMonitor.OnCriticaldirChange(Object sender, FileChangeEvent e)
   在 System.Web.DirectoryMonitor.FireNotifications()
   在 System.Web.Util.WorkItem.CallCallbackWithAssert(WorkItemCallback callback)
   在 System.Web.Util.WorkItem.OnQueueUserWorkItemCompletion(Object state)
   在 System.Threading._ThreadPoolWaitCallback.WaitCallback_Context(Object state)
   在 System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
   在 System.Threading._ThreadPoolWaitCallback.PerformWaitCallbackInternal(_ThreadPoolWaitCallback tpWaitCallBack)
   在 System.Threading._ThreadPoolWaitCallback.PerformWaitCallback(Object state)
代码语言:javascript
复制
从上面文字中,我们可以看到异常捕获是到方法为止的,比如类似于“未将对象引用到对象的实例”异常,如果没有报出具体对象的话,茫茫代码中是很难查找的(在很多情况下无法调试或者无法模拟当时环境时),但是我们可以从最后提示的执行的方法名来限定问题代码范围,然后仔细检查推敲。
代码语言:javascript
复制
代码语言:javascript
复制

五、方法重载

有时候,可能要满足多种需求,但是又不想更改代码,那么方法重载就能派上用场了。比如下面两个方法:

代码语言:javascript
复制
   1:      /// <summary>
代码语言:javascript
复制
   2:      /// 将string类型的fDateTime转换为formatStr格式的日期类型
代码语言:javascript
复制
   3:      /// </summary>      
代码语言:javascript
复制
   4:      public static string GetStandardDateTime(string fDateTime, string formatStr)
代码语言:javascript
复制
   5:      {
代码语言:javascript
复制
   6:          DateTime s = Convert.ToDateTime(fDateTime);
代码语言:javascript
复制
   7:          return s.ToString(formatStr);
代码语言:javascript
复制
   8:      }
代码语言:javascript
复制
   9:   
代码语言:javascript
复制
  10:      /// <summary>
代码语言:javascript
复制
  11:      ///将string类型的fDateTime转换为日期类型 yyyy-MM-dd HH:mm:ss
代码语言:javascript
复制
  12:      /// </sumary>
代码语言:javascript
复制
  13:      public static string GetStandardDateTime(string fDateTime)
代码语言:javascript
复制
  14:      {
代码语言:javascript
复制
  15:          return GetStandardDateTime(fDateTime, "yyyy-MM-dd HH:mm:ss");
代码语言:javascript
复制
  16:      }

<!-- .csharpcode, .csharpcode pre { font-size: small; color: black; font-family: consolas, "Courier New", courier, monospace; background-color: #ffffff; /*white-space: pre;*/ } .csharpcode pre { margin: 0em; } .csharpcode .rem { color: #008000; } .csharpcode .kwrd { color: #0000ff; } .csharpcode .str { color: #006080; } .csharpcode .op { color: #0000c0; } .csharpcode .preproc { color: #cc6633; } .csharpcode .asp { background-color: #ffff00; } .csharpcode .html { color: #800000; } .csharpcode .attr { color: #ff0000; } .csharpcode .alt { background-color: #f4f4f4; width: 100%; margin: 0em; } .csharpcode .lnum { color: #606060; } -->

<!-- .csharpcode, .csharpcode pre { font-size: small; color: black; font-family: consolas, "Courier New", courier, monospace; background-color: #ffffff; /*white-space: pre;*/ } .csharpcode pre { margin: 0em; } .csharpcode .rem { color: #008000; } .csharpcode .kwrd { color: #0000ff; } .csharpcode .str { color: #006080; } .csharpcode .op { color: #0000c0; } .csharpcode .preproc { color: #cc6633; } .csharpcode .asp { background-color: #ffff00; } .csharpcode .html { color: #800000; } .csharpcode .attr { color: #ff0000; } .csharpcode .alt { background-color: #f4f4f4; width: 100%; margin: 0em; } .csharpcode .lnum { color: #606060; } -->

最后

写了这么多,就此结束吧。

如果你还沉浸在日复一日的重复编码中,那么就想想重构吧。重构也是个费力活,但是却可以让你更上一层楼。

本文参与 腾讯云自媒体分享计划,分享自作者个人站点/博客。
原始发表:2012-01-29 ,如有侵权请联系 cloudcommunity@tencent.com 删除

本文分享自 作者个人站点/博客 前往查看

如有侵权,请联系 cloudcommunity@tencent.com 删除。

本文参与 腾讯云自媒体分享计划  ,欢迎热爱写作的你一起参与!

评论
登录后参与评论
0 条评论
热度
最新
推荐阅读
目录
  • 一、控件绑定
  • 二、代码简化、灵活运用
  • 三、代码归类
  • 四、提取函数
  • 五、方法重载
  • 最后
领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档