Simplify SwitchPage() within GameListActivity.java. We no longer need the first switch statement, since case 0 will be hit when SwitchPage(0) is called from the folder browser. This means we can also get rid of recreateFragment(), since the only time this needs to be called is within the onCreate() function. When SwitchPage(0) is called, SwitchPage will perform basically the same actions as recreateFragment() would; thus the reason it can be removed.

This commit is contained in:
Lioncash 2013-09-27 17:16:59 -04:00
parent ad73efda57
commit 69d2ecff52

View File

@ -66,6 +66,7 @@ public final class GameListActivity extends Activity
mDrawerLayout = (DrawerLayout) findViewById(R.id.drawer_layout); mDrawerLayout = (DrawerLayout) findViewById(R.id.drawer_layout);
mDrawerList = (ListView) findViewById(R.id.left_drawer); mDrawerList = (ListView) findViewById(R.id.left_drawer);
// Construct list of items to add to the side menu.
List<SideMenuItem> dir = new ArrayList<SideMenuItem>(); List<SideMenuItem> dir = new ArrayList<SideMenuItem>();
dir.add(new SideMenuItem(getString(R.string.game_list), 0)); dir.add(new SideMenuItem(getString(R.string.game_list), 0));
dir.add(new SideMenuItem(getString(R.string.browse_folder), 1)); dir.add(new SideMenuItem(getString(R.string.browse_folder), 1));
@ -76,16 +77,16 @@ public final class GameListActivity extends Activity
mDrawerList.setAdapter(mDrawerAdapter); mDrawerList.setAdapter(mDrawerAdapter);
mDrawerList.setOnItemClickListener(mMenuItemClickListener); mDrawerList.setOnItemClickListener(mMenuItemClickListener);
// enable ActionBar app icon to behave as action to toggle nav drawer // Enable ActionBar app icon to behave as action to toggle nav drawer
getActionBar().setDisplayHomeAsUpEnabled(true); getActionBar().setDisplayHomeAsUpEnabled(true);
getActionBar().setHomeButtonEnabled(true); getActionBar().setHomeButtonEnabled(true);
// ActionBarDrawerToggle ties together the the proper interactions // ActionBarDrawerToggle ties together the the proper interactions
// between the sliding drawer and the action bar app icon // between the sliding drawer and the action bar app icon
mDrawerToggle = new ActionBarDrawerToggle( mDrawerToggle = new ActionBarDrawerToggle(
this, /* host Activity */ this, /* Host Activity */
mDrawerLayout, /* DrawerLayout object */ mDrawerLayout, /* DrawerLayout object */
R.drawable.ic_drawer, /* nav drawer image to replace 'Up' caret */ R.drawable.ic_drawer, /* Navigation drawer image to replace 'Up' caret */
R.string.drawer_open, /* "open drawer" description for accessibility */ R.string.drawer_open, /* "open drawer" description for accessibility */
R.string.drawer_close /* "close drawer" description for accessibility */ R.string.drawer_close /* "close drawer" description for accessibility */
) { ) {
@ -99,11 +100,7 @@ public final class GameListActivity extends Activity
}; };
mDrawerLayout.setDrawerListener(mDrawerToggle); mDrawerLayout.setDrawerListener(mDrawerToggle);
recreateFragment(); // Display the game list fragment.
}
private void recreateFragment()
{
mCurFragment = new GameListFragment(); mCurFragment = new GameListFragment();
FragmentManager fragmentManager = getFragmentManager(); FragmentManager fragmentManager = getFragmentManager();
fragmentManager.beginTransaction().replace(R.id.content_frame, mCurFragment).commit(); fragmentManager.beginTransaction().replace(R.id.content_frame, mCurFragment).commit();
@ -120,23 +117,9 @@ public final class GameListActivity extends Activity
if (mCurFragmentNum == toPage) if (mCurFragmentNum == toPage)
return; return;
switch (mCurFragmentNum)
{
// Folder browser
case 1:
recreateFragment();
break;
case 0: // Game List
case 2: // Settings
case 3: // About
/* Do Nothing */
break;
}
switch(toPage) switch(toPage)
{ {
case 0: case 0: // Game list
{ {
mCurFragmentNum = 0; mCurFragmentNum = 0;
mCurFragment = new GameListFragment(); mCurFragment = new GameListFragment();
@ -146,7 +129,7 @@ public final class GameListActivity extends Activity
} }
break; break;
case 1: case 1: // Folder Browser
{ {
mCurFragmentNum = 1; mCurFragmentNum = 1;
mCurFragment = new FolderBrowser(); mCurFragment = new FolderBrowser();
@ -156,14 +139,14 @@ public final class GameListActivity extends Activity
} }
break; break;
case 2: case 2: // Settings
{ {
Intent intent = new Intent(this, PrefsActivity.class); Intent intent = new Intent(this, PrefsActivity.class);
startActivity(intent); startActivity(intent);
} }
break; break;
case 3: case 3: // About
{ {
mCurFragmentNum = 3; mCurFragmentNum = 3;
mCurFragment = new AboutFragment(); mCurFragment = new AboutFragment();