Replaced switch in onOptionsItemSelected with if/else

This commit is contained in:
CappielloAntonio 2021-09-04 21:37:53 +02:00
parent 49e0e6231d
commit 5c67bbcc27
3 changed files with 18 additions and 27 deletions

View file

@ -100,15 +100,12 @@ public class DownloadFragment extends Fragment {
@Override @Override
public boolean onOptionsItemSelected(@NonNull MenuItem item) { public boolean onOptionsItemSelected(@NonNull MenuItem item) {
switch (item.getItemId()) { if (item.getItemId() == R.id.action_search) {
case R.id.action_search:
activity.navController.navigate(R.id.action_downloadFragment_to_searchFragment); activity.navController.navigate(R.id.action_downloadFragment_to_searchFragment);
return true; return true;
case R.id.action_settings: } else if (item.getItemId() == R.id.action_settings) {
activity.navController.navigate(R.id.action_downloadFragment_to_settingsFragment); activity.navController.navigate(R.id.action_downloadFragment_to_settingsFragment);
return true; return true;
default:
break;
} }
return false; return false;

View file

@ -114,15 +114,12 @@ public class HomeFragment extends Fragment {
@Override @Override
public boolean onOptionsItemSelected(@NonNull MenuItem item) { public boolean onOptionsItemSelected(@NonNull MenuItem item) {
switch (item.getItemId()) { if (item.getItemId() == R.id.action_search) {
case R.id.action_search:
activity.navController.navigate(R.id.action_homeFragment_to_searchFragment); activity.navController.navigate(R.id.action_homeFragment_to_searchFragment);
return true; return true;
case R.id.action_settings: } else if (item.getItemId() == R.id.action_settings) {
activity.navController.navigate(R.id.action_homeFragment_to_settingsFragment); activity.navController.navigate(R.id.action_homeFragment_to_settingsFragment);
return true; return true;
default:
break;
} }
return false; return false;

View file

@ -101,15 +101,12 @@ public class LibraryFragment extends Fragment {
@Override @Override
public boolean onOptionsItemSelected(@NonNull MenuItem item) { public boolean onOptionsItemSelected(@NonNull MenuItem item) {
switch (item.getItemId()) { if (item.getItemId() == R.id.action_search) {
case R.id.action_search:
activity.navController.navigate(R.id.action_libraryFragment_to_searchFragment); activity.navController.navigate(R.id.action_libraryFragment_to_searchFragment);
return true; return true;
case R.id.action_settings: } else if (item.getItemId() == R.id.action_settings) {
activity.navController.navigate(R.id.action_libraryFragment_to_settingsFragment); activity.navController.navigate(R.id.action_libraryFragment_to_settingsFragment);
return true; return true;
default:
break;
} }
return false; return false;