公司要定制自己的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数(竖屏就是竖着的方向)。