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

是否可以暴露出接口 #12

Open
zixun opened this issue Mar 15, 2017 · 5 comments
Open

是否可以暴露出接口 #12

zixun opened this issue Mar 15, 2017 · 5 comments

Comments

@zixun
Copy link

zixun commented Mar 15, 2017

是否可以暴露出接口,我想后续将其整合到GodEye迭代版本中

@olddonkey
Copy link
Owner

好的,我改一下,另外其实在Xcode8.3中,Xcode会对initialize() 给warning了,我现在还在考虑怎么改会比较好,你怎么看?

@zixun
Copy link
Author

zixun commented Mar 16, 2017

这个还好,可以暴露start()stop()接口让接入方来自己决定什么时候开始。我在考虑的是checkThread()这边怎么暴露接口给外面,因为这里会调用很多次,换一个label的文案就会调用3次,怎么抛给外面比较好。因为当检测出在非主线程做UI操作的时候可以去dump线程的堆栈信息,要是每次都dump感觉有点虚

@olddonkey
Copy link
Owner

抱歉今天有点忙,终于赶在晚上睡觉前改了个小prototype,我push在了API-Prototype branch里,很粗糙的一个prototype,如果有问题的话请见谅。

多次Dump这个问题我想了很久,觉得可能只能多次dump了,感觉用flag之类的来解决容易漏掉情况,而我感觉是宁可多报错,也不能漏了,反正这也是在debug下跑,release里应该没事。。。

个人观点。。。

顺便GodEye做的真的好,我准备在公司的App里用,膜拜。顺便校友好,虽然我应该比你早毕业一点。。

@zixun
Copy link
Author

zixun commented Mar 16, 2017

原来是师兄~感谢师兄认可!才发现你在国外,还是Amazon,好厉害!
我的想法是不需要专门开辟一个manager类来处理,都在一个extension中即可,这样用起来比较方便。关于dump的问题是否可以从堆栈信息上面入手,应该能做到至少一行代码只报一次。
我后续试试,如果可行,我提一个PR到API-Prototype branch分支上

@olddonkey
Copy link
Owner

抱歉,职业病,直接没考虑就写了个manager。

从堆栈信息上看应该是最理想的处理办法了。

再次对你的GodEye表示感谢

Repository owner deleted a comment Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants