[关闭]
@captainjack 2018-03-20T09:10:50.000000Z 字数 291 阅读 235

前端review代码总结

js 前端 总结


  1. 需求变化,逻辑变化后导致代码变化,但是代码并没有做重新设计.
    eg: image.png-72kB

    确认弹窗的需求是后来确认的,实际上后面添加代码的时候,代码的结构发生了变化,就有了重构的必要性,但是这个时候,开发为了图快,就省略了代码重构的步骤,导致后期代码的难以维护.

  2. 注意标签的语义化 使用标签之前 应多一些思考
    eg: image.png-74.4kB

    这里用p标签来包裹input标签等,值得商榷.p标签里面不能再块级元素,这造成了布局的局限性,在语义上也比较牵强.根据主流的方案,应该用div标签包裹.同时在类名的命名上,也主要更多的思考,basic-info__p换成basic-Info__item更为合理.

添加新批注
在作者公开此批注前,只有你和作者可见。
回复批注