随处可见的JavaScript学习笔记-代码重构基础

我为简短的回答向庞大的问题致歉。
真理啊,不要太留意我。

题图--引自网络,侵删--请联系我

引言

模式和重构之间联系紧密,在一定程度上来说,设计模式的目的就是为许多重构行为提供目标。
在实际的项目开发中,一次到位是非常少见的事,无论时间是否充裕,第一版的代码往往单纯的以实现功能为目标。后续,如果有时间或者要求较高,可能会有一次,两次重构。如果是小企业兴许就会这么交差。然而对于我们开发者而言,长时间大量编写低质量代码假以时日就会变得只会写这种类型的代码,这对职业发展是不利的。
即使只以提升自身为目标,适当的了解有关代码重构的知识将对日后的开发工作大有脾益。
作为一个初级前端,本笔记旨在简单介绍一些重构的原则,目标,手段。会比较偏颇,只具备一定的参考价值。
这里重构主要针对JavaScript。

代码重构

提炼函数

函数是JavaScript的一等公民,平日的开发不是在和函数打交道就是在和对象打交道。
当然我知道函数也是一个对象,把它当对象来使用是毫无问题的,可以玩出很多花样。
针对函数有这么几点重构建议。

  1. 假如一个函数名本身就很长,包含了很多功能它就应该被重构。
  2. 可以独立出来的代码最好独立出来成为一个单独的函数,所谓拆分。
  3. 独立出来的函数需要良好的命名,它可以起到注释的作用。
    举例来说
function getInfo(){
  ...//一段业务逻辑,也许是数据处理
$.ajax({
      url:'xx',
      data:'xx',
      type:'post',
      success:function(data){
          ...//一段业务逻辑,也许是数据处理和展示
      }
  }) 
}
//这是一个很常见的函数,如果有必要,它可以拆分成
function getInfo(){
  handleData(data)
  $.ajax({
      url:'xx',
      data:'xx',
      type:'post',
      success:showData
  }) 
}
function handleData(data){
  ...//一段业务逻辑,也许是数据处理
}
function showData(data){
    ...//一段业务逻辑,也许是数据处理和展示
}
//就像我说的如果有必要..如果不需要考虑复用我会考虑把拆分出来的两个函数写到主函数里面去。

拆分出功能对于了解一个复杂函数内部在干什么很重要,同时维护分别的小函数,比维护一个大而复杂的函数要容易的多。

合并重复的条件片段

假如一个函数中有一些条件分支语句,而这些分支语句中有一些重复的语句,这样,我们就可以进行去重把这些重复语句提取出来。
这个没什么太多好说的很常见比如一个分页跳转。

function paging(currPage){
var jumpPage= 1;
  if(currPage <=0){
      jumpPage= 0;
      jump(jumpPage)
  }else if(currPage>totalpage){
      jumpPage= 10
      jump(jumpPage)
  }else{
      jumpPage=currPage
       jump(jumpPage)
  }
}
//虽然我个人认为不会有人写这么累赘的函数,这里只是为了举例可以把jump(jumpPage)提出来
function paging(currPage){
var jumpPage= 0;
  if(currPage <=0){
      jumpPage= 0;
  }else if(currPage>totalpage){
      jumpPage= 10
  }else{
      jumpPage=currPage
  }
    jump(jumpPage)
}
//类似这种感觉,只要是做数据前处理的,基本都以提出来
//如果有必要你可以把这里的重构成多态的形式,只是在这个例子里没有必要。

把条件分支语句提炼成函数

巨大而复杂的条件往往会使函数变得很难懂,简单来说就是把各种简单的小判断单独拉出来当作一个函数。
返回一个布尔值,通过这个布尔值来做判断。
例子:

function getInof(){
  if(一串长而复杂的判断条件){
     ...//一串独立的处理代码 
     return result;
  }
  return result;
}
//这里就可以把那串判断条件提出来
function getInfo(){
    if( judgeInfo()){
     ...//一串独立的处理代码 
     return result;
  }
  function judgeInfo(){
    return //一串长而复杂的判断条件。
  }
}

这样会使代码更好懂,当然不绝对,需要分情况讨论,在此不再展开。

合理使用循环

在一个庞大而复杂的函数体内其实有很多代码是负责的重复的工作,比如最常见的通过js直接对Dom元素进行赋值和取值这样的操作.
很多人会写出...

function setValue(){
  document.getElementById('a').innerHTML='1';
  document.getElementById('b').innerHTML='2';
  document.getElementById('c').innerHTML='3';
.........
}
//这里只是举例它们有的时候可以有20~30行长,这是让人崩溃的。为什么不写一个简单的for in循环呢?
function setValue(){
  var list={
    a:1,
    b:2,
    c:3
  }
  for(i in list){
     document.getElementById(i).innerHTML=list[i];
  }
}

不仅减少了编码量,看起来更加优雅,还比较好懂,循环,你值得拥有。

合理的传参&&合理的退出&&合理的三目运算符

JavaScript函数的出口不一定非得限定为一个,虽然通常我喜欢让它只有一个出口使得结果更为可控。
但不一定每次都这样,比如进来一个判断,满足才需要往下执行的情况,我们就可以一进来不满足直接退出。这里需要灵活掌控。
关于参数,如果一个函数要接受很多参数,而你全部写上了,这无论是对扩展还是调用都很不利。
所以可以把它们写到一个对象中去。
一个简单的判断通常可以用三目来替代,比如给参数赋上默认值但是,如果复杂的If else也使用三目来替代代码会变得很不好懂,所以合理使用。
这里我无意举一个坏例子,只摆一个我认为合适的例子在这。

var param={
  isExist:true,
  age:12,
  name:'tom',
  vip:false
}
function setUserInof(param){
  if(!param.isExist){
    return 'notExist'
  }
  var vipState= param.vip || false
  ....//大量的业务逻辑,比如赋值什么的

  return result;
}

大概就是这种感觉..坏的列子完全可以通过阅读上面的文字来反推。
关于合理的退出,假如内部有一堆循环,且需要判断到某个条件break的话,直接return效果更好,通常可以节约一些判断,且增加代码可读性。

合理使用链式调用

链式调用,使用过jQuery的话,对此不会陌生,每个操作之后可以直接跟下一个操作。

//就像这样
$('#user').text('tom').attr('display','block').css('font-size','28px')....

我在使用jQuery的时候听说这是被建议的,我见过一整个页面都是链式调用的代码,一个代码块可能有30~50行,每个链条隔了几百米远.....看起来似乎很高端?然而实际阅读起来,非常的不方便,而且没有必要,在这种情况下使用链式调用节省下来的字符数似乎相对原本的长度而言显得非常微不足道,我不知道为什么它要这么写。但这种代码修改起来非常的不友好,可以说是高度耦合的。
所以通常来说,如果结构稳定,使用链式调用似乎并没什么不妥。
但是如果这个代码很有可能需要扩展和修改,链式调用只会徒增修改的不便罢了。

结语

今天,稍微对代码重构进行了一些探讨,算是管中窥豹..
毕竟这一块展开来说可以出本书了。
实际上,这篇笔记让我来写有些空洞,因为基本上我没有对我写的模块进行大规模重构过。
充其量就是在项目时间充裕的时候精炼下写法罢了,然而通常项目时间并不充裕。
重构别人写的模块倒是有过几次,只是那些时候通常来说搞清楚它在干嘛然后自己写一遍会更快。
然而,并不是说这篇笔记就没有意义,虽然是现学现卖,但是这上面提到的重构原则,我在实践中通常会在编写时就考虑好,它们很基础,基础而有用。
这两天有点累,这篇笔记就到这里吧,相对之前的笔记篇幅可能略微有些短,这是为了便于补充。
于是在此暂时的,我放下了键盘。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 213,928评论 6 493
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 91,192评论 3 387
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 159,468评论 0 349
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 57,186评论 1 286
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 66,295评论 6 386
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,374评论 1 292
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,403评论 3 412
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,186评论 0 269
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,610评论 1 306
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,906评论 2 328
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,075评论 1 341
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,755评论 4 337
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,393评论 3 320
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,079评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,313评论 1 267
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 46,934评论 2 365
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 43,963评论 2 351

推荐阅读更多精彩内容