Cleanup using AndroidStudio analyze.

This commit is contained in:
Dennis Guse
2022-10-28 17:49:59 +02:00
parent 8073f12947
commit 2207865a70
7 changed files with 9 additions and 15 deletions
@@ -195,7 +195,7 @@ public class TrackRecordingServiceTestStateMachine {
@MediumTest @MediumTest
@Test @Test
public void cannotResume_non_existing_track() throws InterruptedException { public void cannotResume_non_existing_track() {
// given // given
assertFalse(service.isRecording()); assertFalse(service.isRecording());
assertEquals(TrackRecordingService.STATUS_DEFAULT, service.getRecordingStatusObservable().getValue()); assertEquals(TrackRecordingService.STATUS_DEFAULT, service.getRecordingStatusObservable().getValue());
@@ -78,7 +78,7 @@ public class TrackStatisticsUpdaterTest {
// when // when
TrackStatisticsUpdater subject = new TrackStatisticsUpdater(); TrackStatisticsUpdater subject = new TrackStatisticsUpdater();
data.trackPoints.forEach(it -> subject.addTrackPoint(it)); data.trackPoints.forEach(subject::addTrackPoint);
// then // then
TrackStatistics statistics = subject.getTrackStatistics(); TrackStatistics statistics = subject.getTrackStatistics();
@@ -146,9 +146,7 @@ public class TrackRecordingActivity extends AbstractActivity implements ChooseAc
finish(); finish();
return true; return true;
}); });
viewBinding.trackRecordingFabAction.setOnClickListener((view) -> { viewBinding.trackRecordingFabAction.setOnClickListener((view) -> Toast.makeText(TrackRecordingActivity.this, getString(R.string.hold_to_stop), Toast.LENGTH_LONG).show());
Toast.makeText(TrackRecordingActivity.this, getString(R.string.hold_to_stop), Toast.LENGTH_LONG).show();
});
viewBinding.bottomAppBar.setNavigationIcon(R.drawable.ic_baseline_arrow_back_24); viewBinding.bottomAppBar.setNavigationIcon(R.drawable.ic_baseline_arrow_back_24);
setSupportActionBar(viewBinding.bottomAppBar); setSupportActionBar(viewBinding.bottomAppBar);
@@ -77,10 +77,8 @@ public class StatisticsRecordingFragment extends Fragment {
} }
}; };
private final TrackRecordingServiceConnection.Callback bindChangedCallback = (service, unused) -> { private final TrackRecordingServiceConnection.Callback bindChangedCallback = (service, unused) -> service.getRecordingDataObservable()
service.getRecordingDataObservable() .observe(StatisticsRecordingFragment.this, this::onRecordingDataChanged);
.observe(StatisticsRecordingFragment.this, this::onRecordingDataChanged);
};
@Override @Override
public void onCreate(@Nullable Bundle savedInstanceState) { public void onCreate(@Nullable Bundle savedInstanceState) {
@@ -62,8 +62,8 @@ public class IntroductionActivity extends AbstractActivity {
private static class CustomFragmentPagerAdapter extends FragmentStateAdapter { private static class CustomFragmentPagerAdapter extends FragmentStateAdapter {
private static final List<FragmentCreator> fragmentCreators = List.of( private static final List<FragmentCreator> fragmentCreators = List.of(
(FragmentCreator) WelcomeFragment::newInstance, WelcomeFragment::newInstance,
(FragmentCreator) OSMDashboardFragment::newInstance OSMDashboardFragment::newInstance
); );
public CustomFragmentPagerAdapter(@NonNull FragmentActivity fa) { public CustomFragmentPagerAdapter(@NonNull FragmentActivity fa) {
@@ -73,9 +73,7 @@ public class ImportExportSettingsFragment extends PreferenceFragmentCompat {
private void setFilenameTemplate() { private void setFilenameTemplate() {
EditTextPreference preference = findPreference(getString(R.string.export_filename_format_key)); EditTextPreference preference = findPreference(getString(R.string.export_filename_format_key));
preference.setOnBindEditTextListener(t -> { preference.setOnBindEditTextListener(t -> t.setHint(getString(R.string.export_filename_format_default)));
t.setHint(getString(R.string.export_filename_format_default));
});
preference.setDialogMessage(TrackFilenameGenerator.getAllOptions()); preference.setDialogMessage(TrackFilenameGenerator.getAllOptions());
preference.setOnPreferenceChangeListener((p, newValue) -> new TrackFilenameGenerator(newValue.toString()).isValid()); preference.setOnPreferenceChangeListener((p, newValue) -> new TrackFilenameGenerator(newValue.toString()).isValid());
@@ -131,7 +131,7 @@ public class IntervalStatisticsModel extends AndroidViewModel {
OPTION_20(20), OPTION_20(20),
OPTION_50(50); OPTION_50(50);
static IntervalOption DEFAULT = OPTION_1; static final IntervalOption DEFAULT = OPTION_1;
private final double multiplier; private final double multiplier;