标签：load 文件 结对 loadtime 代码 编程 paper 使用 命名

9.13

优点

1 细节：读写文件时及时关闭了文件，这是很容易忘记的

 

 

 

 

2 命名：命名很规范，一眼就看出该变量的含义

 

 

 

3 注释的非常详细，对于下一步的操作，大多都有解析，但是有些解析起始没有必要，比如“生成随机数种子”这种注释

 

 

 

4 函数都使用驼峰命名法，易懂：

 

 

 

5 头文件排布合理

按照相关头文件-> C系统文件->C++ 系统文件->其他库的 .h 文件->本项目内 .h 文件来排序

 

 

不足

1 paper.h和load.h都没有版权信息：

 

 

 

 

 

2 paper.cc的版权信息格式不完整：

 

正确格式：Copyright [year] <Copyright Owner>

 

3 paper.h、load.h、paper.cc文件都使用了命名空间指示（namespace directive）：

 

这会污染命名空间

推荐写法是使用命名空间声明（using declaration）：在std的所有命名之前使用std::

 

4 load.h中，静态static和全局global的字符串变量在.h文件中是不被允许的，只有在.cc文件中才允许：

 

 

5 微小问题：load.h中有一个注释忘记了空一格，且结尾忘记放置换行符

 

除了load.h，其他文件也有类似的错误，这里不一一列出

 

6 load.h使用的流：只在日志时使用流，建议用scanf

 

 

 

在paper.h中也犯了同样的错误：不能使用ofstream流，推荐使用open,write,read三个函数，把流换成文件描述符

 

 

7 loadtime()没有loadtime_r的线程安全

 

建议改为loadtime_r()

同理，rand也没有rand_r安全

 

 

8 强制类型转换是不被允许的：

 

推荐使用static<unsigned int>time

 

标签：load,文件,结对,loadtime,代码,编程,paper,使用,命名
来源： https://www.cnblogs.com/squy/p/16688810.html

本站声明： 1. iCode9 技术分享网（下文简称本站）提供的所有内容，仅供技术学习、探讨和分享；
2. 关于本站的所有留言、评论、转载及引用，纯属内容发起人的个人观点，与本站观点和立场无关；
3. 关于本站的所有言论和文字，纯属内容发起人的个人观点，与本站观点和立场无关；
4. 本站文章均是网友提供，不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属；如您发现该文章侵犯了您的权益，可联系我们第一时间进行删除；
5. 本站为非盈利性的个人网站，所有内容不会用来进行牟利，也不会利用任何形式的广告来间接获益，纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。