首页 » 漏洞 » 巧用匿名函数重构你的代码

巧用匿名函数重构你的代码

 

巧用匿名函数重构你的代码

匿名函数最早是LISP语言引入,后面发展为不仅是函数式语言所特有,在解释型语言和编译型语言中也越来越多匿名函数的身影,或许有个更潮的名字叫 lambda 表达式。

闭包多是用匿名函数实现,在匿名函数中引用了外部变量,那这个匿名函数就形成了闭包。由于闭包和匿名函数之间有着千丝万缕的关系,所以经常会把两者搞混淆。其实在 Js 中匿名函数、闭包、自执行函数、回调函数、箭头函数,这些概念似乎相同,却又不同,请读者朋友自行了解。

匿名函数拥有可动态编程的执行过程。巧妙使用可以让你的代码简约而不失优雅,灵活而不失约束。好了,正式切入本文的正题,巧用匿名函数重构代码。按照重构的惯例,先指出代码中的坏味(Bad Smell):

  • 定义冗长的重复配置
  • 条件多变的集合过滤
  • 说一不二的方法调用

定义冗长的重复配置

在写配置代码时,经常会遇到大量重复的配置,如果要修改一些内容,所有的都要修改,着实很累,而且容易遗漏。比如:

{   html: `     <label><input type="checkbox" name="apple" /> Apple</label>     <label><input type="checkbox" name="banana" /> Banana</label>     <label><input type="checkbox" name="orange" /> Orange</label>   ` }

这里面有三个选项,三个选项的结构完全一样,如果要给所有选项加一个 title ,那你就要把这件事重复做三遍。

{   html: `     <label><input type="checkbox" name="apple" title="Apple" /> Apple</label>     <label><input type="checkbox" name="banana" title="Banana" /> Banana</label>     <label><input type="checkbox" name="orange" title="Orange" /> Orange</label>   ` }

程序猿是个奇怪的群体,他们宁愿做五件不同的事,也不愿重复三次做同一件事。所以这怎么能忍,把配置中不同的内容提取出来:

{   html: (function(fruits) {     return fruits.map(it => `<label><input type="checkbox" name="${it.name}" title="${it.title}" /> ${it.title}</label>`).join('');   }([     {name: 'apple', title: 'Apple'},     {name: 'banana', title: 'Banana'},     {name: 'orange', title: 'Orange'}   ])) }

这样的修改对配置的调用方来说是完全透明的,但是对于配置的维护者来说,将结构和数据分离开,要改结构就改自执行方法体,要改数据就改传入自执行方法的参数,可以大大减少犯错的风险,又避免一些无脑的复制粘贴。

虽然在配置中写代码逻辑不是特别推荐的做法,但相对于代码可维护性来说这不算啥了。

条件多变的集合过滤

集合过滤是个非常常见的需求,假设有个学生集合:

let students = [     {name: 'Lucy', age: 20, sex: 'female'},   {name: 'LiLei', age: 21, sex: 'male'},   {name: 'Jim', age: 18, sex: 'male'} ]

现在要过滤出年龄 20 岁的同学:

function filterByAge(list, age) {     let filtered = [];   for (let i = 0; i < list.length; i++) {     if (list[i].age === age) {       filtered.push(list[i]);     }   }   return filtered; }

又要过滤出姓名叫 LiLei 的同学:

function filterByName(list, name) {     let filtered = [];   for (let i = 0; i < list.length; i++) {     if (list[i].name === name) {       filtered.push(list[i]);     }   }   return filtered; }

还要过滤出性别为男的同学:

function filterBySex(list, sex) {     let filtered = [];   for (let i = 0; i < list.length; i++) {     if (list[i].sex === sex) {       filtered.push(list[i]);     }   }   return filtered; }

就在你觉得大功告成,可以站起

伸腰的时刻,突然被一股强大的力量按回了座椅,帮我找出姓名以 “L” 开头的童鞋。虽然你内心是 mmp~,但还能怎么办,写啊,于是又吭哧吭哧加了如下方法:

function filterByNameStart(list, nameStart) {     let filtered = [];   for (let i = 0; i < list.length; i++) {     if (list[i].name.indexOf(nameStart) === 0) {       filtered.push(list[i]);     }   }   return filtered; }

于是乎,这个调用方式:

filterByName(filterBySex(filterByAge(students, 21), 'male'), 'LiLei');

就可以理解为找出年龄为 21 岁的男性 LiLei。好吧,可读性还不算太差。

但是,不难看出以上 filterByAgefilterByNameStart 等这一系列方法中,除了过滤条件不同,其他逻辑完全一样,造成了大量代码的重复;并且没有任何灵活性可言,调用方改需求,你就要加方法。

我们现在就使用匿名函数把不同的部分抽出去,让调用方想怎么过滤就过滤。 filter 方法的主干逻辑只关心当前的元素是不是要放入结果集合中,其他的判断逻辑都交给匿名函数 fn 去做:

function filter(list, fn) {     let filtered = [];   for (let i = 0; i < list.length; i++) {     if (fn(list[i], i) === true) {       filtered.push(list[i]);     }   }   return filtered; }

上面的例子会变成这种写法:

filter(students, function(member) {     return member.name === 'LiLei' && member.age === 21 && member.sex === 'male'; });

使用箭头方法可以更简洁:

filter(students, member => member.name === 'LiLei' &&       member.age === 21 &&      member.sex === 'male');

现在调用方再提一些变态的过滤方式,你可以回一个眼神,自己写去。

说一不二的方法调用

假设有个 Api 接口,将传入的数字翻倍并返回,这个接口支持单个和批量的方式。

传入 5

  • 执行成功返回 {status: 'success', data: 10}
  • 执行失败返回 {status: 'failed', error: 'xxx'}

传入 [2, 3]

  • 执行成功返回 {status: 'success', data: [{status: 'success', data: 4}, {status: 'success', data: 6}]}
  • 执行失败返回 {status: 'success', data: [{status: 'failed', error: 'xxx'}, {status: 'failed', error: 'xxx'}]}

也就是单个输入会按单个格式输出,批量输入会按批量格式输出。三下五除二,实现了下面这个版本:

function multiple(inNum) {     if (Array.isArray(inNum)) {     // 处理批量情况     return {       status: 'success',       data: inNum.map(it => {         if (isNaN(parseFloat(it))) {           return {             status: 'failed',             error: 'The input is not a number'           };         }           return {           status: 'success',           data: it * 2         }       })     };   } else {     // 处理单个情况     if (isNaN(parseFloat(inNum))) {       return {         status: 'failed',         error: 'The input is not a number'       };     }      return {       status: 'success',       data: inNum * 2     };   } }

这里面单个和批量两种方式除了输入和输出格式不同,其他逻辑完全一样。如果要将乘 2 改成乘 3,两个地方都要改。那看下怎么使用匿名函数来避免两处修改:

function execute(data, fn) {     // 最小执行单元   let single = it => {     try {       return {         status: 'success',         data: fn(it)       };     } catch (e) {       return {         status: 'failed',         error: e.toString()       }     }   };    if (Array.isArray(data)) {     return {       status: 'success',       data: data.map(single)     }   } else {     return single(data);   } }  function multiple(inNum) {     return execute(inNum, it => {     if (isNaN(parseFloat(it))) {       throw new Error('The input is not a number');     }      return it * 2;   }); }

现在 execute 方法只管输入输出的格式和错误处理,包揽了所有脏活累活; multiple 方法则只关心业务的具体实现,也不用关心输入的是单个元素还是数组。如果要改乘 3,只要修改 multiple 方法最后一个 return 。如此一来, execute 还可以被其他的 Api 方法复用,可谓一举两得。

总结

本文的目的只是抛砖引玉,代码中可利用匿名函数重构的坏味还有很多,这种重构方式不只是适用于 Js 中。大家只要多思考、多动手,代码质量一定会day day up~~

关于重构我还想多说一点,重构的过程应该是渐进的方式,当你改第一次的时候可能觉得还ok,第二次就要想下是不是有更好的方式来实现。如果修改对调用方透明那是最好了,实在不行让调用方配合修改也是值得的,当然这其中还要权衡时间成本。Your boss is watching you :rage:

Happy Code :grin:

原文链接:巧用匿名函数重构你的代码,转载请注明来源!

0