我的第一个应用非常糟糕。事实上,它糟糕得以致于我从应用市场上删除它,同时我甚至都不会在我的简历上罗列出它。如果我在开发之前能够知道一些Android开发的事情,也不会糟糕到这步田地。
当然,正如Code Standards所说:如果你所做的工作和你作为学生开发的Android应用类似,你很有可能会讨厌你的应用。
如果一年前你写的代码对于你来说感觉还不错,你很大程度上没有进行足够的学习。
——Code Standards 2015.5.21
1. 不要持有Context的静态引用
public class MainActivity extends LocationManagingActivity implements ActionBar.OnNavigationListener, GooglePlayServicesClient.ConnectionCallbacks, GooglePlayServicesClient.onConnectionFailedListener { //... private static MeTrackerStore mMeTrackerStore; //... @Override protected void onCreate(Bundle savedInstanceState) { //... mMeTrackerStore = new MeTrackerStore(this); }}如果MeTrackerStore通过它的构造函数保持一个指向Activity的引用,这个Activity将不会被垃圾回收(GC),除非静态变量被从新分配到不同的Activity。这是因为mMeTrackerStore是静态变量,而静态变量的内存是不会被回收,直到应用程序退出才回收。如果你正在试图做这样的事情,你的代码很有可能有严重的错误。寻找帮助吧,可能看看Google的Udacity课程“Android Development for Beginners”能够帮助你。
注:从技术上说,你可以对一个Application Context进行静态变量引用而不引起内存泄露,但我不建议你这样做。
这段代码有很多问题,但我现在只会把重点问题放在“隐式引用”那些问题上。在Java中,(非静态)内部类有个对外部类实例有个隐式引用。
GetLatAndLongAndUpdateCameraAsyncTask对生命周期我们无法控制的Fragment对象有一个隐式引用。Android SDK负责创建和销毁Fragment,如果GetLatAndLongAndUpdateCameraAsyncTask 因为正在运行而不能被垃圾回收,那么DefineGeofenceFragment也将因为具有隐式引用而保留不能被垃圾回收。
3. 使用Android Studio进行工作
public ViewPager getmViewPager { return mViewPager;}(如果你想知道为什么’m’是实例变量的名称的第一个字母,’m’往往是实例变量的公认约定。它代表了’member'(成员)的意思)。
Android Studio能够做的远不止于此。学习Android Studio从学习快捷键和模版是不错的开始。
在我写的众多类中,有一个类的一个方法我便写了有100多行。这类的方法是非常难以读懂、修改和重用,努力让一个方法只做一件事情。显然,这意味着你应该对超过20行的方法持有怀疑态度。这里,你可以使用Android Studio来帮助你发现有问题的方法:
这可能听起来微不足道,但是这是我开发第一个应用时候犯下的错误。
阅读Pragmatic Programmer,然后阅读Effective Java。这两本书会帮助你避免开发新手常犯的错误。在你学习了这两本书后,不停地寻找聪明的人并向他们学习。
当你开发应用的时候,你可能会遇到一些聪明人和有经验人已经解决过的问题。而且,许多这些问题的解决方案是可以作为开源库的,充分利用它们。
总结
第一时间掌握最新移动开发相关信息和技术,请关注mobilehub公众微信号(ID: mobilehub)。
