Array update speed
维护者通常 4 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 35/100
- Issue 类型
- 缺陷
- 描述清晰度
- 基本清楚
- 活跃度
- 停滞
- 技术栈
- javascript
- 领域
- performance
调研方向
从提供的 JavaScript 基准测试开始,在包含 10,000,000 个元素的数组上复现 filter、Immer's produce 配合 draft.splice 以及直接使用 Array.splice 之间的对比。追踪导致差异的 produce 和数组更新路径,然后使用相同的基准测试验证所报告的性能差距。当数组更新得到实质性改进,或预期的性能表现得到明确确立时,即视为完成。
由索引模型根据 Issue 内容生成。
描述
🐛 Bug Report
I thought that since immer updates arrays directly, it would be more or less as performant as updating the array directly, but this is not at all the case.
I simulated the pattern "remove the nth element of an array". Code pasted later. The results on my machine are:
filter: 186.852ms
immer: 15.871s
splice: 50.453ms
immer seems to be 84 slower than the standard way to remove the nth element of an array in a mutable way (which is looping throught all the elements to remove the given index). This, itself, is not an efficient method, yet in my example is only ~4 times slower then directly using Array.splice()
Link to repro
import {produce} from "immer"
const arr = Array.from({length: 10_000_000}, (i, idx) => ({n: idx}));
console.time('filter');
let remove10_000nth = arr.filter(((i, idx) => idx !== 10_000));
console.timeEnd('filter');
console.time('immer');
remove10_000nth = produce(arr, draft => {
draft.splice(10_000, 1)
})
console.timeEnd('immer');
console.time('splice');
arr.splice(10_000, 1);
console.timeEnd('splice');
Environment
- Immer version: 10.0.4
- I filed this report against the latest version of Immer
- Occurs with
setUseProxies(true) - Occurs with
setUseProxies(false)(ES5 only)
- 主要语言
- JavaScript
- 星标
- 29k
- 派生
- 905
- 平均合并
- 3 天 16 小时
- 30 天内合并 PR
- 22
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
immerjs/immer 的其他 Issue
-
Shrinking an array with `length =` and growing it back loses the patches for the truncated indices可能已有人在做 @RClaudiuM 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 75/100
维护者通常 4 天内回复
-
难度 3/5 1-2 天 新手友好度 35/100
维护者通常 4 天内回复
-
Docs: Chinese (zh-CN) website translation is outdated vs English docs可能重新可做 @lijiayou0728 于 91 天前认领,目前没有进行中的 PR。 未关闭
难度 4/5 3-5 天 新手友好度 55/100
维护者通常 4 天内回复
-
question
难度 5/5 一周以上 新手友好度 35/100
维护者通常 4 天内回复
-
难度 5/5 一周以上 新手友好度 35/100
维护者通常 4 天内回复
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 76/100
Automattic/node-canvas#2633 ·
-
难度 2/5 1-3 小时 新手友好度 78/100
capricorn86/happy-dom#2594 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
-
难度 2/5 1-3 小时 新手友好度 79/100
microsoft/monaco-editor#5575 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 84/100
维护者通常 1 天内回复