"Lazy initial state" is actually greedy
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- JavaScript
- Star
- 11.8k
- Fork
- 7.9k
- Merge trung bình
- 1 ngày 11 giờ
- Pull request đã merge (30 ngày)
- 11
Mô tả
Is there any good reason to call the useState(() => ...) feature "Lazy initial state"? The actual implementation is synchronous ("greedy") as far as I can tell, so I would propose to rename it to "Functional initial state" (for consistency with "Functional updates").
- docs: https://reactjs.org/docs/hooks-reference.html#lazy-initial-state
- implementation: https://github.com/facebook/react/blob/02da938fd51f6345ca185d571ea6628189ae81a7/packages/react-reconciler/src/ReactFiberHooks.new.js#L1135-L1142
Related "Lazy initialization" in useReducer section => I would propose "Initialization function"
- docs: https://reactjs.org/docs/hooks-reference.html#lazy-initialization
- implementation: https://github.com/facebook/react/blob/02da938fd51f6345ca185d571ea6628189ae81a7/packages/react-reconciler/src/ReactFiberHooks.new.js#L646-L657
I can make a PR unless someone points out some aspect of the naming that I missed.
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu với các phần tham chiếu về Hooks được liên kết trong issue: “lazy initial state” và “lazy initialization”. So sánh thuật ngữ của chúng với implementation ReactFiberHooks.new.js được tham chiếu và cách diễn đạt “Functional updates”. Được xem là hoàn tất khi thuật ngữ trong tài liệu đã được rà soát và mọi cách đặt tên đã thống nhất được áp dụng nhất quán trong cả hai phần.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- javascript, react
- Lĩnh vực
- documentation
- Loại issue
- Tài liệu
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức phù hợp với người mới
- 45/100