Node.js 新手初学 Node.js,撸了一个有道词典命令行工具

kenshinji · 2017年09月15日 · 最后由 aldrich 回复于 2017年09月16日 · 6529 次阅读

欢迎大家来 code review,提 issue,代码很水,欢迎批评指正。 https://github.com/kenshinji/yddict

  • 考虑用 StandardJS 把代码格式规范化:https://standardjs.com/
  • 对 config 的处理:如果用户没有自定义的 config.json 文件,可以考虑在解析过文件之后就直接用一个对象字面量来提供配置的默认值。这样后面的代码就不需要操心 config 可能为空的问题了。
  • 可以考虑用一些很短小的函数来为某些操作明确表意:比如从页面里提取结果的部分,可以分别抽取成 parseCnResult($), parseEnResult($)
  • Don't Repeat Yourself: console.log(color_output( ... ));这部分结构重复出现了,可以考虑抽取成一个函数。
  • Program into the Language: JS 支持将函数作为对象来传递,当你做完了前两项重构之后,可以再简化代码:
const parser = isCn ? parseCnResult : parseEnResult
printWithColor(parser($))

暂时就这么多。

5long 回复

就喜欢这样指点,很到位

需要 登录 后方可回复, 如果你还没有账号请 注册新账号