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

建议 #5

Open
xingag opened this issue Aug 25, 2015 · 14 comments
Open

建议 #5

xingag opened this issue Aug 25, 2015 · 14 comments

Comments

@xingag
Copy link

xingag commented Aug 25, 2015

首页没必要那么麻烦吧,用RadioGroup就可以吧

@tianshaojie
Copy link
Owner

我试试,应该可以省去一些代码,感谢建议~

@xingag
Copy link
Author

xingag commented Aug 25, 2015

还有,结合ViewPager,最近很流行用TabLayout,可以试试用这个来替换PagerSlidingTabStrip

@tianshaojie
Copy link
Owner

回头试试TabLayout。还有什么好东西?都建议一下

@xingag
Copy link
Author

xingag commented Aug 25, 2015

还在看你的代码,没看完,看完了再和你说

@tianshaojie
Copy link
Owner

欢迎。近期可能会把PullToRefresh换成Ultra-Pull-To-Refresh

@xingag
Copy link
Author

xingag commented Aug 25, 2015

代码大概都看完了,说说几个想法
1.代码总体来说,实现比较基础,好多都是第三方源码,不能算是一个框架,只能算一个学习例子
2.总体代码结构有点冗余,没有很好的利用各种设计模式
3.网络请求利用handle消息机制队列来发送请求结果,可能会用内存泄露,我觉得这里有点多余,使用抽象方法,加一个接口,是不是更好呢

一点小建议,可能有些不对的地方,望采纳

@tianshaojie
Copy link
Owner

  1. 主要是整合了一些常用组件,个别包装了一层,本身并没有特别的深度和难度,只是提供便利
  2. 你说的设计模式主要包含什么?在android里很少用到设计模式
  3. 第三点没看太明白,特别是内存泄漏,可否举例说明,这是个大问题!

@xingag
Copy link
Author

xingag commented Aug 25, 2015

2.主要用的比较多的是观察者,Build,工厂,单例,外观这些
3.Handle结合线程很容易内存泄露,这个处理要小心一点;仅仅是觉得这里使用Handle有点不合理,你可以用LC测试一下

@tianshaojie
Copy link
Owner

TabLayout,不好用,效果也不好

@xingag
Copy link
Author

xingag commented Aug 26, 2015

结合材料设计语言,还是不错的哦

@tianshaojie
Copy link
Owner

MD是方向,要用的,需要结合设计效果会很好

@teddyisme
Copy link

下拉刷新 用recycleView呢

@tianshaojie
Copy link
Owner

Ultra-Pull-To-Refresh支持各种布局下拉刷新

@ronanhardiman
Copy link

minSdkVersion 14
org.apache.http.HttpException 等相关的代码应该可以换掉吧.

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

4 participants