为什么使用猫鼬回调会导致两次保存数据？

Question

我一直想知道为什么向 mongoose findOneAndUpdate 函数添加回调会导致将数据两次保存到 DB 中？

public async addPersonAsFavorite(userId: string, friendId: string) {
    if (!await this.isPersonAlreadyFriend(userId, friendId)) {
      const friendList = FriendsList.findOneAndUpdate(
        { _id: userId },
        { $push: { friendsList: friendId } },
        { upsert: true, new: true },
        (err, data) => {
         if (err) console.error(err);
         return data;
        }
      );
      return friendList;
    }}

  public async isPersonAlreadyFriend(userId: string, friendId: string) {
    let isFriendFound = false;
    await FriendsList.findById(userId, (err, data) => {
      if (data) {
        console.log(data.friendsList);
      }
      if (err) console.error(err);
      if (data && data.friendsList.indexOf(friendId) > -1) {
        isFriendFound = true;
        console.log('already friend');
      } else {
        console.log('not friend');
        isFriendFound = false;
      }
    })
    return isFriendFound;
  }

如果我删除回调，数据只会保存一次。

编辑：添加了第二段代码和新问题。 如果有人向按钮发送垃圾邮件以添加朋友。该朋友将被添加多次，因为在添加第一个朋友之前，可以进行检查以防止这种情况，它已经多次添加了该人。

在允许再次调用该函数之前，我如何确保它完成了对 DB 的写入？

Answer 1

也许问题出在 isPersonAlreadyFriend 方法中，因为您尝试使用 async await 调用它，但随后您传递了一个回调，导致该方法不返回承诺。在 mongodb 中使用 Promise 的正确方法应该是这样的：

public async isPersonAlreadyFriend(userId: string, friendId: string) {
    let isFriendFound = false;
    const data = await FriendsList.findById(userId);
    if (data) {
      console.log(data.friendsList);
    }
    if (data && data.friendsList.indexOf(friendId) > -1) {
      isFriendFound = true;
      console.log('already friend');
    } else {
      console.log('not friend');
      isFriendFound = false;
    }
    return isFriendFound;
  }

试试这个，如果有帮助，请告诉我