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

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<unsigned char> (c ) )

29, GCC里有个__builtin_unreachable( ); ，用在switch case中。这样影响移植。

30，如果switch case里有异常的case，可以实使用assert或者抛出异常。

参考：

1，Youtube C++ Weekly