代码在 catch 语句捕获和错误之后运行，并且 returns 在 react native firebase 中运行

Question

每当我从一个函数中捕获到错误和 return 时，我都会遇到问题，在 catch 块仍然运行之后通过代码。这是我使用的两个函数：

    usernameTaken: async (username) => {
        const user = await firebase.firestore().collection("uniqueUsers").doc(username).get();
        if (user.exists) {
            alert("Username is taken. Try again with another username.");
            throw new Error('Username is taken. Try again with another username.');
        }
    },
    changeUsername: async (currentUsername, newUsername) => {
      try {
          var user = Firebase.getCurrentUser();
          Firebase.usernameTaken(newUsername);
      } catch (err) {
          alert(err.message);
          return;
      }
      await db.collection('uniqueUsers').doc(currentUsername).delete();
      await db.collection("users").doc(user.uid).update({username: newUsername});
      await db.collection("uniqueUsers").doc(newUsername).set({username: newUsername});
      alert("Congratulations! You have successfully updated your username.");
    }

对于此问题的任何帮助，我将不胜感激，因为我已经为此苦苦挣扎了 2 天多，似乎找不到解决方案。

Answer 1

试试这个检查你的值是否为空或未定义在 try 块中抛出一些错误，例如

cosnt user = Firebase.getCurrentUser();
const name = Firebase.usernameTaken(newUsername); 
// throwing error 
if(name == "") throw "is empty"; 
await db.collection('uniqueUsers').doc(currentUsername).delete();

Answer 2

在您的原始代码中，usernameTaken() 承诺是浮动的，因为您没有使用 await。因为它是浮动的，所以您的 catch() 处理程序永远不会捕获它的错误。

changeUsername: async (currentUsername, newUsername) => {
  try {
      const user = Firebase.getCurrentUser();
      /* here -> */ await Firebase.usernameTaken(newUsername);
  } catch (err) {
      alert(err.message);
      return;
  }
  /* ... other stuff ... */
}

---

加分

usernameTaken 应该 return 一个布尔值

您应该将 usernameTaken 更改为 return 布尔值。这可以说比使用 alert()（它会阻止代码的执行）或抛出错误更好。

usernameTaken: async (username) => {
  const usernameDoc = await firebase.firestore().collection("uniqueUsers").doc(username).get();
  return usernameDoc.exists; // return a boolean whether the doc exists
}

安全声明和释放用户名

根据您当前的代码，您没有任何保护措施可以防止有人出现并删除您数据库中的任何用户名或声明在您上次检查其可用性和您致电 set() 为新用户名。您应该保护您的数据库，以便用户只能写入他们拥有的用户名。

将所有者的 ID 添加到文档中：

"/uniqueUsers/{username}": {
  username: "username",
  uid: "someUserId"
}

这样您就可以将 edits/deletions 锁定到拥有该用户名的用户。

service cloud.firestore {
  match /databases/{database}/documents {
    
    match /uniqueUsers/{username} {
      // new docs must have { username: username, uid: currentUser.uid }
      allow create: if request.auth != null
                    && request.resource.data.username == username
                    && request.resource.data.uid == request.auth.uid
                    && request.resource.data.keys().hasOnly(["uid", "username"]);

      // any logged in user can get this doc
      allow read: if request.auth != null;

      // only the linked user can delete this doc
      allow delete: if request.auth != null
                    && request.auth.uid == resource.data.uid;

      // only the linked user can edit this doc, as long as username and uid are the same
      allow update: if request.auth != null
                    && request.auth.uid == resource.data.uid
                    && request.resource.data.diff(resource.data).unchangedKeys().hasAll(["uid", "username"]) // make sure username and uid are unchanged
                    && request.resource.data.diff(resource.data).changedKeys().size() == 0; // make sure no other data is added
    }
  }
}

自动更新您的数据库

您正在以可能损坏数据库的方式修改数据库。您可以删除旧用户名，然后无法更新您当前的用户名，这意味着您永远不会 link 您的新用户名。要解决此问题，您应该使用 batched write 将所有这些更改一起应用。如果任何一个失败，什么都不会改变。

await db.collection("uniqueUsers").doc(currentUsername).delete();
await db.collection("users").doc(user.uid).update({username: newUsername});
await db.collection("uniqueUsers").doc(newUsername).set({username: newUsername});

变成

const db = firebase.firestore();
const batch = db.batch();

batch.delete(db.collection("uniqueUsers").doc(currentUsername));
batch.update(db.collection("users").doc(user.uid), { username: newUsername });
batch.set(db.collection("uniqueUsers").doc(newUsername), { username: newUsername });

await batch.commit();

用户名不区分大小写

您当前的用户名区分大小写，如果您希望您的用户 type/write 出他们的个人资料 URL，则不推荐这样做。考虑一下 "example.com/MYUSERNAME"、"example.com/myUsername" 和 "example.com/myusername" 如何成为不同的用户。如果有人在一张纸上潦草地写下了他们的用户名，您会希望所有这些都进入同一用户的个人资料。

usernameTaken: async (username) => {
  const usernameDoc = await firebase.firestore().collection("uniqueUsers").doc(username.toLowerCase()).get();
  return usernameDoc.exists; // return a boolean whether the doc exists
},
changeUsername: async (currentUsername, newUsername) => {
  const lowerCurrentUsername = currentUsername.toLowerCase();
  const lowerNewUsername = newUsername.toLowerCase();

  /* ... */

  return lowerNewUsername; // return the new username to show success
}

结果

将所有这些结合在一起，得到：

usernameTaken: async (username) => {
  const usernameDoc = await firebase.firestore().collection("uniqueUsers").doc(username).get();
  return usernameDoc.exists; // return a boolean
},
changeUsername: async (currentUsername, newUsername) => {
  const user = Firebase.getCurrentUser();
  if (user === null) {
    throw new Error("You must be signed in first!");
  }

  const taken = await Firebase.usernameTaken(newUsername);
  if (taken) {
    throw new Error("Sorry, that username is taken.");
  }

  const lowerCurrentUsername = currentUsername.toLowerCase();
  const lowerNewUsername = newUsername.toLowerCase();
  const db = firebase.firestore();
  const batch = db.batch();
  
  batch.delete(db.collection("uniqueUsers").doc(lowerCurrentUsername));
  batch.update(db.collection("users").doc(user.uid), {
    username: lowerNewUsername
  });
  batch.set(db.collection("uniqueUsers").doc(lowerNewUsername), {
    username: lowerNewUsername,
    uid: user.uid
  });

  await batch.commit();

  return lowerNewUsername;
}

// elsewhere in your code
changeUsername("olduser", "newuser")
  .then(
    (username) => {
      alert("Your username was successfully changed to @" + username + "!");
    },
    (error) => {
      console.error(error);
      alert("We couldn't update your username!");
    }
  );

注意：如果您使用上述所有建议（如安全规则），batch.commit() 预期失败的一种方式是有人在当前用户之前使用用户名。如果您遇到权限错误，假设有人在您之前使用了用户名。