Android Launcher2里面的小bug

公司要定制自己的Launcher，于是对源代码里面的Launcher2做下修改。

 

最近需要将workspace每行只显示3个，于是修改了下workspace_screen.xml，将launcher:shortAxisCells="4"改为launcher:shortAxisCells="3"，可是一直报错，数组越界了（源码编译出来的之所以没错，因为他就是4行4列，怎么都不会有错的啦）。

 

这个问题好debug，仔细一看，原来记录workspace里面cell状态的mOccupied[][]二维数组是4列3行，但是应该是3列4行才对。

 

找到mOccupied初始化的地方就发现问题了：

 

 

if (mOccupied == null) {
            if (mPortrait) {
                mOccupied = new boolean[mShortAxisCells][mLongAxisCells];
            } else {
                mOccupied = new boolean[mLongAxisCells][mShortAxisCells];
            }
 }

 这里的mPortrait根本没有初始化！！！，只是在以后每次onMeasure里面给其赋值，但是没有初始化就默认到else里面执行了，这样就是默认Landscape即横屏方式的了，难怪。

 

改正很简单，在这句话上面先判断是横屏还是竖屏就行：

 

 

 mPortrait = this.getResources().getConfiguration().orientation == Configuration.ORIENTATION_PORTRAIT;

 

 

注意，Portrait也就是竖屏的情况下应该是mOccupied[mShortAxisCells][mLongAxisCells]，mShortAxisCells是比较短的那一方向的cell数（竖屏就是横着的方向），mLongAxisCells是比较长的那一方向的cell数（竖屏就是竖着的方向）。