2017-02-22 76 views
-2

我正在创建一个计划,在设计团队中为引导项目进行代码审查,以减少缺陷并提高质量。
我已经为代码审查清单起草了一些问题,我在下面粘贴它。
此检查列表中还可以包含哪些问题或指出检查开发人员的代码以确保代码的质量?如何为Twitter Bootstrap项目进行代码审查?

  1. bootstrap.css是不是修改?
  2. 选的插件,意义和要求
  3. 显示多模态提示,避免
  4. 放置模式的HTML之前或之后
  5. 其次引导的命名约定?
  6. 最新版本的bootstrap被使用了?
  7. 将容器流体类用于全宽度行而不是全宽度行?
  8. 您是否根据引导CSS使用正确的媒体查询断点?
  9. Bootstrap网格用于使设计响应?

这些清单问题尚未完成,需要您的指导和帮助完成全面的代码审查。

回答

1
  1. Bootstrap设计为移动第一种方法,因此您应该始终检查是否遵循移动第一种方法。
  2. Bootstrap是一个庞大的库,因此您应该检查是否所有组件都已正确使用。例如。有像accordion, panels, carousel这样的许多组件可以使您的设计更加紧凑和美观。
  3. 检查是否所有的列都包含在<div class="row"></div>之内,因为这是使用引导程序的正确方法。
  4. 检查引导程序cdns是否在页面中使用了两次。原因,如果使用包括bootstrap cdns两次在一个pge,将干扰某些元素的正常运作。

以下应该是引导cdns的顺序 -

<link rel="stylesheet" href="http://maxcdn.bootstrapcdn.com/bootstrap/3.3.6/css/bootstrap.min.css"> 
     <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.12.4/jquery.min.js"></script> 
     <script src="http://maxcdn.bootstrapcdn.com/bootstrap/3.3.6/js/bootstrap.min.js"></script> 

这些都是在做审查,你应该检查一些基本的东西。

编辑

出于性能

结帐这个答案https://webmasters.stackexchange.com/questions/52502/how-can-i-speed-up-the-download-time-of-a-bootstrap-themed-website

文档

The Official website

许多其他文档可在线获得。 check out this

编码标准

现在对我来说这是根据需要一个相对的概念和变化。

注意

刚刚从我个人的角度观看一个建议,不要太死板关于在审查的规则。如果一段代码看起来很明显,但优雅地解决了一个问题,我会接受。

希望这会有所帮助!

+0

真的很有帮助。是的,这也将帮助我进一步检查您在答案中提到的几点,以便使代码更有效。我们可以添加一些与性能,编码标准,文档等有关的清单,以及不应该为了更好的代码做什么? –