ChatKit-OC icon indicating copy to clipboard operation
ChatKit-OC copied to clipboard

关于#188 commit代码在目前最新代码中并没有生效

Open mokai opened this issue 8 years ago • 6 comments

Base Info for this issue

发现在聊天页面CPU Usage在90%以上,导致手机发烫。

通过这里的代码已优化好https://github.com/Maru-zhang/ChatKit-OC/commit/18848c4b0e58d17b8d758f8d3940ac0d19e87c5a

只是现在最新代码中并没有见过以上的变动,所以想问下是后面的commit有什么变动吗

mokai avatar Mar 16 '17 09:03 mokai

可能之前没有合并吧。

@Maru-zhang 我们能合并你的代码吗?看起来这个改进对性能很有帮助。

tang3w avatar Mar 16 '17 10:03 tang3w

@tang3w sorry,刚测试了下,发现刚进入聊天页面时头像圆角并不生效,需要滚动一下才能生效

mokai avatar Mar 16 '17 10:03 mokai

@tang3w 不知道这里提到的解决方案是否为最优方案。https://github.com/leancloud/ChatKit-OC/issues/188

self.avatarImageView.lcck_cornerRadius = avatarImageViewCornerRadius;
所执行的代码占绝大部分资源,改为
_avatarImageView.layer.cornerRadius = avatarImageViewCornerRadius;
_avatarImageView.clipsToBounds = YES;
CPU下降为30左右,绘制完成CPU资源释放。

mokai avatar Mar 16 '17 10:03 mokai

可以 Profiler 一下,看看 UI 上面什么地方渲染最耗时。UI 部分还有很多优化的空间,但我们可能得找出瓶颈在哪里。您也可以测试一下,这个改动是否能有效提升性能。

tang3w avatar Mar 16 '17 10:03 tang3w

刚测试了,正如https://github.com/leancloud/ChatKit-OC/issues/188 所说,6s Plus上可以保持CPU在30%以下,因为版本今天要发布,就不找lcck_cornerRadius异步渲染问题出在哪了。 到时修复了还望在这里告诉一声,感谢~

mokai avatar Mar 16 '17 10:03 mokai

希望能写一个回调让用户自行切圆角

speeddream avatar Mar 20 '17 06:03 speeddream