Skip to content
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

大项目实战 跑Annotree 遇到的初见反馈 #46

Open
julystone opened this issue Oct 10, 2024 · 3 comments
Open

大项目实战 跑Annotree 遇到的初见反馈 #46

julystone opened this issue Oct 10, 2024 · 3 comments

Comments

@julystone
Copy link

初见feedback~

大佬好,从b站视频过来的,感觉up非常有水平,于是尝试使用了annotree。这里给您一个初见的feedback~

做了次小尝试

扫描了一下参与开发过的游戏脚本项目。400+个文件夹,7000+个文件(其中有非常多的脚本定位文件)算是做了一次小小的性能测试吧
image
开始扫描后占用内存会非常的高,且将所有的目录都扫描并展开了,这与我刚上手使用此软件的期望相比,有点出乎所料了

使用前的期望:

  1. 这是一款轻量的,占用内存不会太大的项目注释工具,随用随走
  2. 扫描完毕项目后,首先展示最外级总目录便以最外层根目录先注释,其次用户再做进一步子文件夹的展开与注释
  3. 扫描完毕项目后,会存有tree缓存(我也不太懂该如何称呼),展开关闭文件夹很快不会太卡顿

然而。。

  1. 7000多文件全部扫描内存占用了很大,后来发现可以在全局设置中选择忽视文件,只扫描文件夹
    • 这里是否考虑修改全局配置的默认值为不扫描文件呢,以让First Try更轻松、轻便?
    • image
  2. 所有文件夹目录都展开到了最里面一层,400个文件夹我找不到总目录层级了,感觉有些吃力不讨好
    • 这里是否考虑修改全局配置的默认值为不展开文件夹,tree遍历依旧遍历,但是不做自动的展开。
    • 或者说对于轻小项目,展开文件夹体验更好的话,能做到扫描前预加载项目的文件夹深度,过深的则不予默认展开处理
  3. 扫描完毕后,对文件夹的展开关闭操作,依然有很久的延迟时间。
    • 这里没了解代码实现,这里可以缓存tree吗,使得开启关闭不用再做扫描?
    • image

结语

感谢大佬能看到这里,本想以多个issue的方式提的,但由于与期望出乎所料地有些多,害怕您看不到相关用户故事,会有信息差,就详细描述了一番。

希望Annotree越做越好~

@itchaox
Copy link
Owner

itchaox commented Oct 11, 2024

@julystone 非常感谢听到这么用心的反馈~ 💗💗💗

  1. Annotree 扫描超大文件的性能问题,一直都有用户反馈,也在寻找解决方式,但是还没解决。
  2. 诸如 1万+ 的文件注释,我感觉正常来说不太常见,但是后续需要去解决,如何提高大文件扫描时的用户体验。
  3. 比如正常情况,一个真实的项目,可能就1-200 个文件需要注释吧,这种情况,使用 Annotree 就没有什么问题。
  4. 然后很多时候,都可以通过提前设置全局忽略的东西,来提高扫描效率,但是很多新用户不太清楚如何操作,这个后续也需优化。

展示最外级总目录 -> 这个需求有用户提过,后续会加。

这里是否考虑修改全局配置的默认值为不扫描文件呢,以让First Try更轻松、轻便? -> 不考虑修改全局配置的默认值为不扫描文件,因为很多时候,一来就需要展示文件,而且这个配置可以自行修改,然后会自动保存配置。

所有文件夹目录都展开到了最里面一层 -> 这个我后续考虑一下,展开层级问题。

文件夹的展开关闭操作,延迟问题 -> 后续我再验证,考虑下,应该是子文件太多,dom 渲染的问题。

非常感谢支持 🎉🎉🎉

@julystone
Copy link
Author

wow,大佬回复得好快!!
另外问一下项目两个contributor @小鱼儿 也是您本人吗

@itchaox
Copy link
Owner

itchaox commented Oct 11, 2024

@julystone 另一个不是我,是 Annotree 的共建者小伙伴,也很棒~
也期待你的 pr~ 💗

你加微信群了吗? 哈哈哈~
有需求也可以在微信群里,沟通~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants