JS类中isNextJsFramework字段未被读取警告问题求助

问题原因 #

• TypeScript类型检查器触发警告的核心原因：你仅对isNextJsFramework字段进行赋值操作，但从未在代码的任何逻辑中读取或使用这个字段的值，因此被判定为“已声明但未使用”。
• 代码还存在几个隐性问题：
  • JSON.parse是同步方法，不需要添加await关键字
  • functionIAmStuckWith是Promise对象，你仅声明却未等待它执行完成，会导致后续逻辑可能在赋值操作前就运行
  • 使用forEach遍历依赖来判断存在性效率低下，直接用Object.hasOwn或对象属性访问更简洁

修复方案 #

方案1：确保字段被读取使用 #

在后续逻辑中明确使用this.isNextJsFramework的值，比如在Promise resolve时返回它，或者在其他方法中调用：

class Parser {
  private options: ParserOptions;
  private isNextJsFramework: boolean;

  constructor(options: ParserOptions) {
    this.options = options;
    this.isNextJsFramework = false; // 直接初始化默认值，无需通过构造函数参数传入
  }

  public parse() {
    return new Promise((resolve) => {    
      this.getPackageJson()
        .then((path: string[]) => {
          const packageJsonContent = fs.readFileSync(path[0], 'utf8');
          const packageJsonFile = JSON.parse(packageJsonContent);

          // 直接判断next依赖是否存在，无需遍历
          if (packageJsonFile?.dependencies) {
            this.isNextJsFramework = Object.hasOwn(packageJsonFile.dependencies, 'next');
          }

          // 在resolve时使用该字段，确保它被读取
          resolve({ isNextJs: this.isNextJsFramework });
        })
        .catch(() => {
          // 处理文件读取或解析错误
          resolve({ isNextJs: false });
        });
    });
  }
}

方案2：无需类字段，直接返回判断结果 #

如果不需要将结果持久化在类实例中，直接在Promise中返回布尔值更简洁：

public parse() {
  return this.getPackageJson()
    .then((path: string[]) => {
      const packageJsonContent = fs.readFileSync(path[0], 'utf8');
      const packageJsonFile = JSON.parse(packageJsonContent);
      // 直接判断next是否存在，双重感叹号转为布尔值
      return !!packageJsonFile?.dependencies?.next;
    })
    .catch(() => false); // 错误时默认返回false
}

额外优化建议 #

替换同步的fs.readFileSync为异步的fs.promises.readFile，避免阻塞事件循环：

public parse() {
  return this.getPackageJson()
    .then(async (path: string[]) => {
      const packageJsonContent = await fs.promises.readFile(path[0], 'utf8');
      const packageJsonFile = JSON.parse(packageJsonContent);
      return Object.hasOwn(packageJsonFile?.dependencies || {}, 'next');
    })
    .catch(() => false);
}

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