编程参考 - C++ Code Review: 一个计算器的项目

GitHub - jroelofs/calc: Toy Calculator

Toy Calculator

1,拿到一个project,第一眼看,没有配置文件,说明没有引入持续集成系统,continuous integration system。

2,然后看cmake文件,使用的子文件夹、编译标准、Unit Test、要编译的Target。

3,项目的clean命令。

4,先看测试功能的部分。

5,包含项目内的头文件时,使用的是相对路径。便于代码移植。

6,使用了gtest框架。

7,使用fuzz testing,在分析用户提供的输入功能时使用。(libFuzzer工具)。用户通过某个input Entry point来提供各种随机数据。

8,使用if语句,需要大括号,curly brace。单行语句时,

9,Team work时,你提交的代码,创建的pull request,有别人帮忙review,如果自己一个人的话,也最好遵循这个流程。就是自己给自己Review,然后再Merge。

10,如果析构函数为空,可以设置:virtual ~Lexer( ) = default;

11,可以删除copy构造函数。Lexer &operator=(const Lexer &) = delete;

12,还有Lexer(const Lexer &) = delete; 因为不需要,还会在多态时引起不必要隐患。

13,use powerline shell for bash

14,可以实用线上编译, Compiler Explorer

15,sliced: 一个全新创建的对象,只从派生类取得一部分内容,别的可以用到多态的内容就不要了。在构造或赋值时发生。

16,父类删除了copy constructor, move constructor, copy assignment, move assignment,子类中也会自动删除。

17,父类析构函数是virtual的,那子类的析构函数也自动就是virtual。

18,rule of zero is to avoid defining any special member function. 不想有任何的copy构造函数、copy assignment、move assignment、move构造以及虚析构函数等。

19,如果在一个项目里,使用pragma once,里面基本只是symlinks,你会很痛苦。

20,使用nvim编辑文件。

21,使用decltype( )

22,just be consistent, 比如给类取命时,是Vector_Lexer还是VectorLexer,都可以保持统一即可。

23,类的成员定义,两个变量还是放到两行较好。

24,类的成员定义: int Line{0};

25, [[nodiscard]] virtual bool empty( ) const = 0; 表示我调用这个函数,但不使用这个函数的返回值,则表示这是一个bug。

如下所示,编译时会报一个warning:

26,如上面函数,如果需要,就要加上const。还有前面加上constexpr (C++ 20).

\[nodiscard\]\] constexpr virtual bool empty( ) const = 0; 27,CMAKE_CXX_STANDARD is set C++ 17 28, std::isspace( static_cast\ (c ) ) 29, GCC里有个__builtin_unreachable( ); , 用在switch case中。这样影响移植。 30,如果switch case里有异常的case,可以实使用assert或者抛出异常。 参考: 1,Youtube C++ Weekly [https://www.youtube.com/watch?v=rFnRzW3vvsQ\&list=PLs3KjaCtOwSYh2X7z0mxUbArhW1O4_4z1](https://www.youtube.com/watch?v=rFnRzW3vvsQ&list=PLs3KjaCtOwSYh2X7z0mxUbArhW1O4_4z1 "https://www.youtube.com/watch?v=rFnRzW3vvsQ&list=PLs3KjaCtOwSYh2X7z0mxUbArhW1O4_4z1")

相关推荐
具***717 小时前
“气动弹性系统的能量图方法Matlab程序”及其相关内容介绍
代码复审
程序员龙一3 天前
自动驾驶规控算法工程师Code Review指南
人工智能·自动驾驶·代码复审·code review
询问QQ688238864 天前
探索Bandgap带隙基准:新手友好指南
代码复审
电子科技圈6 天前
SiFive车规级RISC-V IP获IAR最新版嵌入式开发工具全面支持,加速汽车电子创新
嵌入式硬件·tcp/ip·设计模式·汽车·代码规范·risc-v·代码复审
聊天QQ:6882388613 天前
59.基于matlab的全离散法单自由度稳定极限切深叶瓣图绘制、两自由度稳定极限切深叶瓣图绘制
代码复审
黑客思维者17 天前
智能配电系统代码审查详细设计与实战体系:从缺陷预防到架构守护
网络·架构·代码复审·代码评审
沟通QQ:48773927819 天前
西门子PLC一拖二换热站自控系统程序探索
代码复审
还是转转25 天前
AI Code Review 工具
人工智能·代码复审
奋进的电子工程师1 个月前
如何实现开源组件的安全与合规治理?
安全·开源·代码规范·设计规范·代码复审
mit6.8243 个月前
[code-review] 文件过滤逻辑 | 范围管理器
人工智能·代码复审