通过AngularFireDatabase在Firebase添加记录前校验条目数量

嘿，我来帮你梳理下这段代码里的潜在问题，然后给出更可靠的实现方案：

原代码的核心问题 #

• 重复订阅与内存泄漏：每次调用addQuote都会创建一个新的valueChanges()订阅，如果没有手动取消，这些订阅会一直存在，导致内存泄漏，而且多次调用会触发多次重复逻辑。
• 竞态条件风险：判断data.length < 3之后，push()是异步操作，在这个操作完成前，如果有另一个addQuote请求进来，此时读取到的data.length依然小于3，会导致额外的记录被添加，最终突破3条的限制。
• 冗余实例创建：每次操作都重新获取list实例，虽然不影响功能，但没必要。

基础修复方案（解决订阅问题，降低竞态概率） #

我们可以改用单次数据读取替代持续订阅，同时用async/await让异步逻辑更清晰：

async addQuote(q: any) {
  try {
    // 复用同一个列表引用
    const quoteListRef = this.angularFireDatabase.list(this.getUserPath('/savedQuoteList'));
    // 只获取一次当前数据，避免持续订阅
    const currentQuotes = await quoteListRef.valueChanges().pipe(take(1)).toPromise();
    
    console.log("Length = " + currentQuotes.length);
    if (currentQuotes.length < 3) {
      await quoteListRef.push(q);
      console.log("Quote saved.");
    } else {
      console.log("Too many quotes in list.");
    }
  } catch (error) {
    console.error("添加quote时出错:", error);
  }
}

关键优化点： #

• 使用take(1)配合toPromise()，只获取一次当前列表数据，订阅会自动完成并销毁，避免内存泄漏。
• async/await让异步操作顺序执行，确保在确认长度后再执行添加操作，降低竞态发生的概率。
• 复用列表引用，减少不必要的实例创建。

进阶方案：用事务彻底解决竞态条件 #

如果你的应用存在多用户同时添加记录的场景，上面的方案还是可能出现并发问题。这时候可以用Firebase的事务来实现原子性操作，确保无论多少并发请求，都不会突破3条的限制：

addQuote(q: any) {
  // 获取Firebase原生的数据库引用
  const quoteListRef = this.angularFireDatabase.database.ref(this.getUserPath('/savedQuoteList'));
  
  quoteListRef.transaction((currentQuotes) => {
    // 如果列表还不存在，初始化并添加第一条quote
    if (!currentQuotes) {
      return [q];
    }
    // 长度小于3时，添加新quote
    else if (currentQuotes.length < 3) {
      currentQuotes.push(q);
      return currentQuotes;
    }
    // 长度已满，返回null取消事务
    else {
      return;
    }
  })
  .then((result) => {
    if (result.committed) {
      console.log("Quote saved.");
    } else {
      console.log("Too many quotes in list.");
    }
  })
  .catch((error) => {
    console.error("通过事务添加quote时出错:", error);
  });
}

事务的优势： #

• 原子性：事务会先读取当前数据，执行修改逻辑，再写回数据。如果在这期间有其他请求修改了数据，Firebase会自动重试事务，确保最终结果符合预期。
• 彻底避免竞态：无论多少并发请求，都能严格保证列表长度不超过3条。

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