前往小程序,Get更优阅读体验!
立即前往
首页
学习
活动
专区
工具
TVP
发布
社区首页 >专栏 >一些前端开发的代码审查和意见

一些前端开发的代码审查和意见

作者头像
Jerry Wang
发布2019-05-30 16:09:55
6380
发布2019-05-30 16:09:55
举报

现在做的web shop只有一套代码,里面是一些generic的逻辑,不能出现任何Nike Adi之类的硬编码。当前启动web shop的实例,具体是for Nike 还是Adi,通过启动时传入的参数指定。当时我拍脑袋说的是通过命令行,当时你问能不能用url,我说也可以。

但是现在的代码里,还是分别为Nike 和Adi各自做了一套UI啊。想一下将来假设CSO的同事为了突出demo效果,想同时开10个web shop,那么除了现在做好的Nike和Adi外,我们岂不是得复制粘贴8次?

想一下比如要遵循QDPR之类的规范,demo时不能出现真实客户名字,而是用ABCDEFGHIJ你做了10个 XX.vue, 岂不是每一个.vue文件进去后都要手动把真实客户名字改成ABCDEFGHIJ?

一个best practice:尽量把不变的东西抽象出来,固化到代码里,把易变的东西抽取出来,放到配置文件里。最理想的结果就是:无论老大们的需求怎么变,developer只需要修改配置文件即可应对,不需要修改代码,也不需要重新编译程序。

你现在这种为每个site做一个.Vue的做法,应对现在的demo我觉得也ok,只是我担心将来demo需要展示的web shop数量超过3个后,每次需求改动都会引入一些额外的本来可以通过更好的编程技巧去避免的工作量。

clipboard1
clipboard1

现在adi这个web shop : 支持的这些字段是硬编码的。

理想的情况是下图右边,你用#3 这个API询问后台,adi支持哪些字段?然后后台告诉你,你根据结果动态绘制vue页面。

如果觉得用vue实现这种动态绘制效果有难度,可以先按照现在这种编程方式做。但是缺点以前邮件提到过了,后期随着需求变动可能会带来一些体力活。

clipboard2
clipboard2
clipboard3
clipboard3
本文参与 腾讯云自媒体分享计划,分享自作者个人站点/博客。
原始发表:2019年03月25日,如有侵权请联系 cloudcommunity@tencent.com 删除

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

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

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

评论
登录后参与评论
0 条评论
热度
最新
推荐阅读
领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档