编程参考 - 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<unsigned char> (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

相关推荐
帅次24 天前
基于边缘计算的智能门禁系统架构设计分析
软件工程·团队开发·软件构建·需求分析·规格说明书·代码复审·极限编程
龙智DevSecOps解决方案1 个月前
DevOps实践:在GitLab CI/CD中集成静态分析Helix QAC的工作原理与优势
安全·ci/cd·代码复审·1024程序员节
孰能生巧-LWP1 个月前
Code Review Item
代码复审
C+ 安口木2 个月前
在 Gitlab 中使用 ChatGPT 进行 CodeReview
chatgpt·gitlab·代码复审
AI大模型_学习君2 个月前
基于大模型 + 知识库的 Code Review 实践
人工智能·深度学习·机器学习·embedding·知识库·代码复审·ai大模型
Codigger官方4 个月前
Ruby、Python、Java 开发者必备:Codigger之软件项目体检
bug·代码规范·代码复审
GKDf1sh4 个月前
代码审计:Bluecms v1.6
安全·web安全·代码复审
-无-为-4 个月前
科普文:CodeReview小结
程序人生·代码复审·codereview
逆风就重开5 个月前
数仓SQL如何做code review?
数据库·sql·代码复审