SonarLint提示移除已使用的customParams参数的原因排查

这个告警是SonarLint静态检查规则对TypeScript参数属性语法识别不全导致的误报，你的业务代码本身没有问题，customParams确实在getNextUrl方法中正常参与了逻辑计算，不存在未使用的情况。
当你在构造函数参数前添加private readonly修饰符时，TypeScript会自动将该入参绑定为类的实例属性，不需要手动在构造函数内写this.xxx = xxx的赋值逻辑。但部分旧版本SonarLint不会跨方法追踪实例属性和构造函数参数的关联关系，只会检查构造函数体内有没有直接引用参数名：

• 构造函数里的count、limit两个参数，因为你在this.maxPage = Math.ceil(count / limit)逻辑中直接调用了参数名，所以SonarLint能识别到参数被使用
• customParams参数你仅在getNextUrl方法中通过this.customParams访问，没有在构造函数体内直接引用参数名，就被SonarLint误判为未使用

你可以任选以下一种方式消除告警，不会影响原有业务逻辑：

• 升级SonarLint版本
  最新版SonarLint已经修复了TypeScript参数属性的识别bug，升级后该误报会自动消失，不需要改动业务代码。
• 构造函数内显式赋值（推荐，可读性最好）
  去掉customParams参数前的访问修饰符，在类内单独声明对应属性，构造函数内手动完成赋值，让静态检查工具明确感知到参数被使用，修改后的代码如下：

  class Pagination {
    private readonly maxPage: number
    private readonly customParams: {}
  
    constructor (
      private readonly data: Array<{}>,
      private readonly currentPage: number,
      private readonly limit: number,
      private readonly count: number,
      private readonly urlRoute: string,
      customParams: {} = {}
    ) {
      this.customParams = customParams
      this.maxPage = Math.ceil(count / limit)
    }
  
    private getNextUrl (): string | null {
      if (this.currentPage >= this.maxPage) {
        return null
      }
      const query = toQueryString({
        page: this.currentPage + 1,
        limit: this.limit,
        ...this.customParams
      })
      return getUrl(this.urlRoute) + '?' + query
    }
  }
  

• 轻量修改适配规则
  如果不想调整现有属性声明结构，可以在构造函数末尾添加一行void customParams，显式标记该参数已被使用，也能快速消除告警。如果按照SonarLint提示把参数重命名为_customParams，注意要同步调整实例属性的赋值逻辑，避免出现属性名不一致的问题。

内容的提问来源于stack exchange，提问作者Jhony Mota