ubuntu上js如何进行代码审查

Ubuntu上JavaScript代码审查的实施方法

1. 静态代码分析：ESLint（核心工具）

ESLint是JavaScript/React项目首选的静态代码分析工具，可检测语法错误、代码风格不一致、潜在逻辑问题（如未使用的变量、不安全的DOM操作），并支持自定义规则。

• 安装与初始化：在Ubuntu终端中，进入项目目录，运行npm install eslint --save-dev安装ESLint；接着执行npx eslint --init，根据提示选择项目配置（如“Use a popular style guide”或“Answer questions about your style”），生成.eslintrc.js（或.eslintrc.json）配置文件。
• 运行检查：执行npx eslint .（检查当前目录及子目录所有JS文件）或npx eslint src/（仅检查src目录），终端会输出错误和警告信息（如no-unused-vars表示未使用变量）。
• 集成编辑器：推荐在VS Code中安装ESLint插件，实现实时代码检查（编辑时自动标记问题），提升开发效率。

2. 代码格式化：Prettier（辅助工具）

Prettier专注于代码格式统一（如缩进、引号、分号、换行），与ESLint配合使用可避免风格争议。

• 安装与配置：运行npm install --save-dev prettier安装；创建.prettierrc文件（如{ "semi": true, "singleQuote": true, "tabWidth": 2 }）定义格式规则。
• 格式化代码：执行npx prettier --write "src/**/*.{js,jsx}"，自动格式化src目录下所有JS/JSX文件。

3. Git钩子自动化：Husky + Lint-Staged

通过Husky设置Git钩子（如pre-commit），在提交代码前自动运行ESLint和Prettier，确保只有符合规范的代码能被提交；Lint-Staged则仅检查暂存的文件，提升效率。

• 安装依赖：运行npm install husky lint-staged --save-dev安装。
• 配置package.json：添加以下配置：

  "husky": {
    "hooks": {
      "pre-commit": "lint-staged"
    }
  },
  "lint-staged": {
    "*.{js,jsx}": [
      "eslint --fix",  // 自动修复可修复的问题
      "prettier --write",  // 格式化代码
      "git add"  // 将修复后的文件重新加入暂存区
    ]
  }
  

  此配置会在git commit时自动触发ESLint修复和Prettier格式化。

4. CI/CD集成：GitHub Actions（自动化审查）

将代码审查流程集成到GitHub Actions，实现每次推送（push）或拉取请求（Pull Request）时自动运行检查，确保主分支代码质量。

• 创建工作流文件：在项目根目录的.github/workflows目录下创建code-review.yml文件，内容如下：

  name: Code Review Workflow
  on:
    push:
      branches: ["main"]  # 推送至main分支时触发
    pull_request:
      branches: ["main"]  # 创建针对main分支的PR时触发
  jobs:
    review:
      runs-on: ubuntu-latest
      steps:
        - name: Checkout code
          uses: actions/checkout@v3
        - name: Set up Node.js
          uses: actions/setup-node@v3
          with:
            node-version: '18'  # 使用Node.js 18版本
            cache: 'yarn'  # 缓存yarn依赖，加速安装
        - name: Install dependencies
          run: yarn install --frozen-lockfile  # 安装项目依赖
        - name: Run ESLint and Prettier
          run: npm run lint  # 执行package.json中的lint脚本（需提前配置）
  

• 配置lint脚本：在package.json中添加lint脚本：

  "scripts": {
    "lint": "eslint src/ --ext .js,.jsx && prettier --check 'src/**/*.{js,jsx}'"
  }
  

  此脚本会检查src目录下所有JS/JSX文件的ESLint错误和Prettier格式问题。

5. 第三方代码审查服务

• Codacy：支持JavaScript静态分析、代码复杂度检测、重复代码查找，可与GitHub/GitLab集成。配置方法：在项目仓库中添加Codacy应用，授权后自动生成审查报告（如代码质量评分、问题列表）。
• Reviewable：专注于Pull Request审查，提供评论、线程讨论、状态跟踪功能，适合团队协作。集成方法：在GitHub仓库中安装Reviewable应用，连接至项目即可使用。

6. 自定义ESLint规则（高级需求）

若项目有特殊编码规范（如“函数参数不超过3个”“禁止使用alert”），可通过自定义ESLint规则实现。

• 创建自定义规则文件：在项目根目录创建eslint/rules/custom-rule.js，编写规则逻辑（如检查函数参数数量）：

  module.exports = {
    meta: {
      type: 'suggestion',
      docs: {
        description: '限制函数参数数量不超过3个',
        category: 'Best Practices',
        recommended: false
      },
      schema: [{ type: 'integer', minimum: 1 }],  // 允许配置最大参数数
      messages: {
        tooManyParams: '函数参数过多（{{count}}个），最多允许{{max}}个。'
      }
    },
    create(context) {
      const maxParams = context.options[0] || 3;  // 默认最大3个参数
      return {
        FunctionDeclaration(node) {
          if (node.params.length > maxParams) {
            context.report({
              node,
              messageId: 'tooManyParams',
              data: { count: node.params.length, max: maxParams }
            });
          }
        },
        FunctionExpression(node) {
          if (node.params.length > maxParams) {
            context.report({
              node,
              messageId: 'tooManyParams',
              data: { count: node.params.length, max: maxParams }
            });
          }
        }
      };
    }
  };
  

• 配置规则：在.eslintrc.js中引入自定义规则：

  module.exports = {
    rules: {
      'custom-rule': ['error', 3]  // 启用自定义规则，设置最大参数数为3
    },
    plugins: ['custom-plugin']  // 声明自定义规则插件（需创建eslint/plugins/custom-plugin/index.js导出规则）
  };
  

  自定义规则需通过eslint --print-config .验证是否生效。

7. 单元测试辅助（确保功能正确性）

代码审查不仅要检查风格和潜在错误，还需确保功能正确性。可使用Jest等单元测试框架编写测试用例，覆盖核心逻辑。

• 安装Jest：运行npm install --save-dev jest安装。
• 编写测试用例：创建__tests__/example.test.js文件，编写测试代码（如测试一个加法函数）：

  const add = require('../src/add');  // 假设add函数在src/add.js中
  test('adds 1 + 2 to equal 3', () => {
    expect(add(1, 2)).toBe(3);
  });
  

• 运行测试：执行npm test，Jest会输出测试结果（如通过/失败的用例数量）。