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 Workflowon:push:branches: ["main"]# 推送至main分支时触发pull_request:branches: ["main"]# 创建针对main分支的PR时触发jobs:review:runs-on: ubuntu-lateststeps:- name: Checkout codeuses: actions/checkout@v3- name: Set up Node.jsuses: actions/setup-node@v3with:node-version: '18'# 使用Node.js 18版本cache: 'yarn'# 缓存yarn依赖，加速安装- name: Install dependenciesrun: yarn install --frozen-lockfile# 安装项目依赖- name: Run ESLint and Prettierrun: 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会输出测试结果（如通过/失败的用例数量）。