取消useEffect挂钩中的所有异步/等待任务以防止反应中发生内存泄漏的正确方法是什么?

时间:2019-10-25 16:35:17

标签: reactjs firebase async-await google-cloud-firestore use-effect

我正在开发一个反应chap应用程序,该应用程序从Firebase数据库中提取数据。在我的“仪表板”组件中,我有一个useEffect钩子来检查经过身份验证的用户,如果是,则从firebase中提取数据并设置电子邮件变量和chats变量的状态。我将abortController用于useEffect清理,但是,每当我第一次注销并重新登录时,都会收到内存泄漏警告。

  

index.js:1375警告:无法在已卸载的组件上执行React状态更新。这是空操作,但它表明应用程序中发生内存泄漏。要修复,请取消使用useEffect清理功能中的所有订阅和异步任务。

     

在控制台中(由Context.Consumer创建)

最初我没有abortController,我只是返回了控制台日志进行清理。做了更多的研究,发现abortController但是示例使用了提取和信号,我找不到与异步/等待结合使用的任何资源。我愿意改变数据的检索方式(无论是通过fetch,async / await还是任何其他解决方案进行检索),我只是无法使其与其他方法一起使用。

const [email, setEmail] = useState(null);
const [chats, setChats] = useState([]);

const signOut = () => {
    firebase.auth().signOut();
  };

useEffect(() => {
    const abortController = new AbortController();
    firebase.auth().onAuthStateChanged(async _user => {
      if (!_user) {
        history.push('/login');
      } else {
        await firebase
          .firestore()
          .collection('chats')
          .where('users', 'array-contains', _user.email)
          .onSnapshot(async res => {
            const chatsMap = res.docs.map(_doc => _doc.data());
            console.log('res:', res.docs);
            await setEmail(_user.email);
            await setChats(chatsMap);
          });
      }
    });

    return () => {
      abortController.abort();
      console.log('aborting...');
    };
  }, [history, setEmail, setChats]);

预期结果是正确清理/取消useEffect清理功能中的所有异步任务。一个用户注销后,相同或不同的用户重新登录后,我在控制台中收到以下警告

  

index.js:1375警告:无法在已卸载的组件上执行React状态更新。这是空操作,但它表明应用程序中发生内存泄漏。要修复,请取消所有订阅    和useEffect清理功能中的异步任务。

     

在控制台中(由Context.Consumer创建)

3 个答案:

答案 0 :(得分:4)

firebase 的情况下,您不是在处理async/await,而是进行流式处理。您只需在清理功能中退订Firebase流:

const [email, setEmail] = useState(null);
const [chats, setChats] = useState([]);

const signOut = () => {
    firebase.auth().signOut();
  };

useEffect(() => {
    let unsubscribeSnapshot;
    const unsubscribeAuth = firebase.auth().onAuthStateChanged(_user => {
        // you're not dealing with promises but streams so async/await is not needed here
      if (!_user) {
        history.push('/login');
      } else {
        unsubscribeSnapshot = firebase
          .firestore()
          .collection('chats')
          .where('users', 'array-contains', _user.email)
          .onSnapshot(res => {
            const chatsMap = res.docs.map(_doc => _doc.data());
            console.log('res:', res.docs);
            setEmail(_user.email);
            setChats(chatsMap);
          });
      }
    });

    return () => {
      unsubscribeAuth();
      unsubscribeSnapshot && unsubscribeSnapshot();
    };
  }, [history]); // setters are stable between renders so you don't have to put them here

答案 1 :(得分:0)

onSnapshot method不会返回承诺,因此等待其结果没有任何意义。相反,它开始监听数据(以及对该数据的更改),并使用相关数据调用onSnapshot回调。这可能会发生多次,因此无法返回承诺。侦听器将保持与数据库的连接,直到您通过调用从onSnapshot返回的方法来取消订阅该数据库为止。由于您从不存储该方法,更不用说调用它了,侦听器将保持活动状态,稍后将再次调用您的回调。这可能是内存泄漏的来源。

如果您要等待Firestore的结果,则可能正在寻找get() method。这样会一次获取数据,然后解决承诺。

await firebase
      .firestore()
      .collection('chats')
      .where('users', 'array-contains', _user.email)
      .get(async res => {

答案 2 :(得分:0)

取消async/await的一种方法是创建类似于内置AbortController的东西,该函数将返回两个函数:一个用于取消,另一个用于检查取消,然后在{{1 }}需要执行取消检查:

async/await
function $AbortController() {
  let res, rej;
  const p = new Promise((resolve, reject) => {
    res = resolve;
    rej = () => reject($AbortController.cSymbol);
  })
  
  function isCanceled() {
    return Promise.race([p, Promise.resolve()]);
  }
  
  return [
    rej,
    isCanceled
  ];
}

$AbortController.cSymbol = Symbol("cancel");

function delay(t) {
  return new Promise((res) => {
    setTimeout(res, t);
  })
}

let cancel, isCanceled;

document.getElementById("start-logging").addEventListener("click", async (e) => {
  try {
    cancel && cancel();
    [cancel, isCanceled] = $AbortController();
    const lisCanceled = isCanceled;
    while(true) {
      await lisCanceled(); // check for cancellation
      document.getElementById("container").insertAdjacentHTML("beforeend", `<p>${Date.now()}</p>`);
       await delay(2000);
    }
  } catch (e) {
    if(e === $AbortController.cSymbol) {
      console.log("cancelled");
    }
  }
})

document.getElementById("cancel-logging").addEventListener("click", () => cancel())