Comments (4)
这个重复调用引入了任何bug么?在前端的世界里反复渲染不是一个罪大恶极的事,有的时候如果可以让代码变得更加简单,更“stateless”,反而是更好的设计。也就是说,渲染本身总是完全渲染,并不依赖之前的状态,任何时刻,给了同样的数据,渲染出的总是同样的界面。
只有在performance真的开始出问题的时候,这才开始成为一个问题,然后再去考虑怎么优化。一个函数被反复调用这件事不是啥大事。
from biliscope.
确实,这个不是 bug。但我觉得还是有必要重构这个函数的,updateData
现在的可读性和可维护性太差了。 如果其中调用的函数出现了 biliGet
或者 biliPost
函数,那就会出现大问题了。
from biliscope.
有很多函数如果它里面调用了不该调用的函数都会出大问题啊,这不是理由。你任何一个函数,在没有处理好的情况下调用自己都可能产生无限递归。
updateData就是把数据渲染成html的过程,有了数据,画图,这是一个非常清晰的概念,并且是stateless的,任何情况下,有什么data就渲染什么前端。加上state显然是把程序变得更加复杂了。
from biliscope.
那就不重构了吧,但是还是可以把意料之外的调用给修改掉。
from biliscope.
Related Issues (20)
- [FR] 增加 `make.bat` 用于 windows 平台的构建
- [Bug] edge 中搜索标签时无法加载头像 HOT 1
- [FR] 悬停在视频上显示当前视频的视频AI总结 HOT 4
- [FR] 增加显示“共同关注"的人数 HOT 2
- [FR] 增加候选标签至信息卡片的备注框下方 HOT 2
- [FR] 移除词云中的 aid, bvid 和年份 HOT 1
- [FR] 视频标签也按权重加入词云并辅助分词 HOT 1
- [BUG] 通过标签搜索用户搜不到时,永远无法显示出预设页面
- [BUG] 舰长的随机排列算法,并不完全随机。
- [BUG] 动态界面出现合集时,卡片对应的用户显示错误
- Load options before everything
- 希望增加的功能 HOT 9
- [FR] 增加关闭 up 主卡片的选项
- [FR] 为视频AI总结添加快捷键支持 HOT 5
- [FR] 在options里增加”反馈BUG“选项 HOT 1
- [FR] 增加评论历史记录和弹幕历史记录 HOT 2
- [FR] 增加收藏评论的功能 HOT 1
- [FIX] AI conclusion card could cover `view latter` button HOT 4
- 大大,好人做到底...写一个油管总结吧...油管那个怪怪的 HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from biliscope.