2 回答

TA贡献1875条经验 获得超5个赞
我认为你已经非常接近预期的行为了。我的处理方法如下:
const Component = props => {
const { docs } = useFirestore("books");
const id = props.match.params.id;
// useMemo - whenever `docs` change, recalculate the book variable.
// If `docs` don't change, `book` will also not change.
// Your `docs` will probably change on every snapshot.
const book = useMemo(() => docs && docs.filter(doc => doc.id === id), [docs]);
console.log(book);
useEffect(() => {
if (book) {
// Do something with the book, e.g. loadContent(book).
// Keep in mind that this will run on every snapshot.
// If you only want to run this once, you'll need an
// extra state variable to store that the effect was
// already run, and check it here.
}
}, [book]); // The effect will run whenever book changes
};
这个useFirestore钩子看起来几乎没问题,只有一件事:现在即使你改变参数collection,快照监听器也不会改变。您可能想要这样做:
useEffect(() => {
const unsubscribe = firestore.collection(collection).onSnapshot(snapshot => {
const items = snapshot.map(doc => ({ ...doc.data(), id: doc.id }));
setDocs(items);
setLoading(false);
});
return unsubscribe;
// Whenever `collection` changes, `unsubscribe` will be called, and then this hook
// will subscribe to the new collection.
}, [collection]);
更新。
如果您希望useFirestore挂钩仅查询特定书籍,则需要更改挂钩以接受并使用文档 ID,如下所示:
const getDoc = doc => ({ ...doc.data(), id: doc.id });
const useFirestore = (collection, docId) => {
const [docs, setDocs] = useState([]);
const [loading, setLoading] = useState(true);
useEffect(() => {
let subject = firestore.collection(collection);
if (docId) {
// If docId is available, listen for changes to a
// particular document
subject = subject.doc(docId);
}
const unsubscribe = subject.onSnapshot(snapshot => {
// Notice here that if listening for a particular docId,
// the snapshot will be that document, not an array.
// To maintain the interface of the hook, I convert that
// document to an array with a single item.
const items = docId ? [getDoc(doc)] : snapshot.map(getDoc);
setDocs(items);
setLoading(false);
});
return unsubscribe;
}, [collection, docId]);
return { docs, loading };
};

TA贡献1842条经验 获得超12个赞
我需要有关“useFirestore”代码的更多信息,但您至少应该这样编写代码。
不要列出 firestore 上的每个文档来获取一个(您为每个读取请求付费)
在 useEffect 中加载文档,而不是在外部
useEffect 必须依赖于 id
const Component = (props) => {
const id = props.match.params.id;
const firestore = //;
const [book, bookSet] = useState(false);
useEffect(() => {
//Depending on useFirestore code
firestore.collections('books').doc(id)
.then( snapshot => {
if ( !snapshot.exists ) {
bookSet(null);
} else {
bookSet(snapshot.data());
}
});
}, [id]);
if( book === false) return <p>Loading</p>
if (!book) return <p>Not exists</p>
return <p>Display it</p>;
};
编辑
这是我对你的“useFirestore”钩子的猜测
const Component = (props) => {
const id = props.match.params.id;
const { docs, loading } = useFirestore('books');
useEffect(() => {
if( loading) console.log('Loading');
return;
const book = docs && docs.filter(doc => doc.id === id);
console.log({book});
},[loading, docs, id]);
};
添加回答
举报