-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
我觉得可能存在内存溢出的风险。 #5992
Comments
刷新页面有内存增长,这个问题和框架或组件库应该是没有任何关系的,除非框架使用了持久化缓存相关的API(如:sessionStorage、localStorage...); |
我疑惑的点也在这里,通过 DevTools Memory 是没有任何内存泄漏现象出现。 |
我的意思就是观察网站(此标签页)的内存增长,刷新后使用DevTools Memory面板手动触发GC查看当前标签页内存波动,你会发现此标签页的内存会立马掉下来,那说明浏览器并没有立即回收刷新前标签页的内存,而是等待下次Major GC |
这不是内存泄露,但是这可能也是一个需要优化的地方,可能是native-ui某个逻辑或使用了某个API导致的。 |
我建议你使用vue-virtual-scroller来代替n-data-table。我已经使用native ui开发我的开源软件好几个月了,所以按照我的经验,你使用Native ui需要“扬长避短[doge]”,它适合用来快速开发静态布局,如果你用它搞大量的动态数据(里面还渲染一些图片等等大资源),那么内存将陷入火葬场,这在Electron中尤为明显[doge]。即使你使用了vue-virtual-scroller,你也不能在vue-virtual-scroller中内嵌native ui的组件(那么内存又将陷入火葬场,且gc不掉的[doge]) |
靠重构来这个issues的成本太高,所以使用时只能扬长避短,毕竟没有什么东西是万能的[doge] |
本人有利用过 NaiveUI 制作过自己的歌词编辑器,但是由于此库的几个潜在的内存泄露和滥用资源的问题已经放弃使用 NaiveUI 框架并迁移到 React 开发了,情况如下:
NaiveUI 正如其名,直到现在还只是一个还很稚嫩的组件库,提出这些并用比较偏激的用词并无贬低之意,但是如果作者能够优化改善,就能够在发展上走得更远了。 |
描述错误
vuejs/core#10976
每次刷新页面网站的内存都会很大幅度的增长。
复现步骤
打开 https://www.naiveui.com/zh-CN/os-theme/components/data-table 页面,在地址栏回车,刷新页面,观察tab处内存变化。
最小复现链接
https://www.naiveui.com/zh-CN/os-theme/components/data-table
系统信息
使用的包管理器
npm
验证
The text was updated successfully, but these errors were encountered: