编辑“︁C++ 代码审查”︁（章节）

== C++代码审查的关键检查点 ==

=== 1. 内存管理 ===
C++要求开发者手动管理内存，因此审查时需要特别注意：
* 检查是否存在内存泄漏
* 确认指针和引用的正确使用
* 验证资源获取即初始化（RAII）原则的应用

<syntaxhighlight lang="cpp">
// 不良实践：手动管理内存易出错
void badMemoryPractice() {
    int* arr = new int[100];
    // ...使用数组...
    // 忘记delete[] arr; 导致内存泄漏
}

// 良好实践：使用智能指针
void goodMemoryPractice() {
    auto arr = std::make_unique<int[]>(100);
    // ...使用数组...
    // 自动释放内存
}
</syntaxhighlight>

=== 2. 异常安全 ===
确保代码在异常发生时仍能保持一致性：

<syntaxhighlight lang="cpp">
// 不良实践：非异常安全
void unsafeFileOperation() {
    File* f = openFile("data.txt");
    processFile(f);  // 如果抛出异常，文件不会关闭
    closeFile(f);
}

// 良好实践：使用RAII
void safeFileOperation() {
    std::ifstream f("data.txt");
    processFile(f);  // 即使抛出异常，文件也会自动关闭
}
</syntaxhighlight>

=== 3. 代码风格一致性 ===
* 遵循一致的命名约定（如camelCase或snake_case）
* 保持适当的缩进和空格
* 限制行长度（通常80-120字符）

=== 4. 性能考量 ===
* 避免不必要的拷贝
* 评估算法复杂度
* 检查循环和递归的效率

<syntaxhighlight lang="cpp">
// 不良实践：不必要的拷贝
std::vector<int> duplicateData(const std::vector<int>& data) {
    std::vector<int> result = data;  // 不必要的拷贝
    // ...处理result...
    return result;
}

// 良好实践：移动语义
std::vector<int> processData(std::vector<int> data) {  // 按值传递
    // ...处理data...
    return data;  // 可能触发移动构造
}
</syntaxhighlight>