Code refactor

Philipp0002 · Jul 29, 2024 · ba9dab4 · ba9dab4
1 parent 24c333a
commit ba9dab4
Show file tree

Hide file tree

Showing 17 changed files with 38 additions and 32 deletions.
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
@@ -21,10 +21,10 @@
         android:name=".XADGPSApplication"
         tools:targetApi="31">
         <activity
-            android:name=".LoginActivity"
+            android:name=".ui.LoginActivity"
             android:exported="false" />
         <activity
-            android:name=".MainActivity"
+            android:name=".ui.MainActivity"
             android:exported="true">
             <intent-filter>
                 <action android:name="android.intent.action.MAIN" />

diff --git a/app/src/main/java/de/raffaelhahn/xadgps_client/background/NotificationWorker.java b/app/src/main/java/de/raffaelhahn/xadgps_client/background/NotificationWorker.java
@@ -21,8 +21,8 @@
 import java.util.Optional;
 import java.util.stream.Collectors;
 
-import de.raffaelhahn.xadgps_client.NotifyDevice;
-import de.raffaelhahn.xadgps_client.OperatingMode;
+import de.raffaelhahn.xadgps_client.model.NotifyDevice;
+import de.raffaelhahn.xadgps_client.model.OperatingMode;
 import de.raffaelhahn.xadgps_client.R;
 import de.raffaelhahn.xadgps_client.Constants;
 import de.raffaelhahn.xadgps_client.async.GetDeviceListAsync;

diff --git a/.../de/raffaelhahn/xadgps_client/Device.java → ...ffaelhahn/xadgps_client/model/Device.java b/.../de/raffaelhahn/xadgps_client/Device.java → ...ffaelhahn/xadgps_client/model/Device.java
@@ -1,13 +1,8 @@
-package de.raffaelhahn.xadgps_client;
+package de.raffaelhahn.xadgps_client.model;
 
 import android.content.Context;
-import android.util.Log;
 
-import org.json.JSONException;
-import org.json.JSONObject;
-
-import java.lang.reflect.Field;
-import java.util.Iterator;
+import de.raffaelhahn.xadgps_client.R;
 
 public class Device extends NotifyDevice {
     public String sn;

diff --git a/...ffaelhahn/xadgps_client/NotifyDevice.java → ...ahn/xadgps_client/model/NotifyDevice.java b/...ffaelhahn/xadgps_client/NotifyDevice.java → ...ahn/xadgps_client/model/NotifyDevice.java
@@ -1,6 +1,5 @@
-package de.raffaelhahn.xadgps_client;
+package de.raffaelhahn.xadgps_client.model;
 
-import android.content.Context;
 import android.util.Log;
 
 import org.json.JSONObject;

diff --git a/...faelhahn/xadgps_client/OperatingMode.java → ...hn/xadgps_client/model/OperatingMode.java b/...faelhahn/xadgps_client/OperatingMode.java → ...hn/xadgps_client/model/OperatingMode.java
@@ -1,4 +1,4 @@
-package de.raffaelhahn.xadgps_client;
+package de.raffaelhahn.xadgps_client.model;
 
 public enum OperatingMode {
 

diff --git a/app/src/main/java/de/raffaelhahn/xadgps_client/services/DeviceListService.java b/app/src/main/java/de/raffaelhahn/xadgps_client/services/DeviceListService.java
@@ -12,8 +12,8 @@
 import java.util.Timer;
 import java.util.TimerTask;
 
-import de.raffaelhahn.xadgps_client.Device;
-import de.raffaelhahn.xadgps_client.OperatingMode;
+import de.raffaelhahn.xadgps_client.model.Device;
+import de.raffaelhahn.xadgps_client.model.OperatingMode;
 import de.raffaelhahn.xadgps_client.async.AsyncCallback;
 import de.raffaelhahn.xadgps_client.Constants;
 import de.raffaelhahn.xadgps_client.async.GetDeviceListAsync;

diff --git a/app/src/main/java/de/raffaelhahn/xadgps_client/services/MovementMonitorService.java b/app/src/main/java/de/raffaelhahn/xadgps_client/services/MovementMonitorService.java
@@ -3,8 +3,8 @@
 import android.app.Activity;
 import androidx.core.app.ActivityCompat;
 
-import de.raffaelhahn.xadgps_client.Device;
-import de.raffaelhahn.xadgps_client.NotifyDevice;
+import de.raffaelhahn.xadgps_client.model.Device;
+import de.raffaelhahn.xadgps_client.model.NotifyDevice;
 import de.raffaelhahn.xadgps_client.R;
 import de.raffaelhahn.xadgps_client.Constants;
 import de.raffaelhahn.xadgps_client.background.BackgroundWorkService;

diff --git a/...elhahn/xadgps_client/DeviceInfoSheet.java → ...ahn/xadgps_client/ui/DeviceInfoSheet.java b/...elhahn/xadgps_client/DeviceInfoSheet.java → ...ahn/xadgps_client/ui/DeviceInfoSheet.java
@@ -1,7 +1,6 @@
-package de.raffaelhahn.xadgps_client;
+package de.raffaelhahn.xadgps_client.ui;
 
 import android.os.Bundle;
-import android.util.Log;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
@@ -18,6 +17,9 @@
 import java.util.ArrayList;
 import java.util.Optional;
 
+import de.raffaelhahn.xadgps_client.R;
+import de.raffaelhahn.xadgps_client.Utils;
+import de.raffaelhahn.xadgps_client.model.Device;
 import de.raffaelhahn.xadgps_client.services.DeviceListService;
 import de.raffaelhahn.xadgps_client.services.MovementMonitorService;
 

diff --git a/...elhahn/xadgps_client/DevicesFragment.java → ...ahn/xadgps_client/ui/DevicesFragment.java b/...elhahn/xadgps_client/DevicesFragment.java → ...ahn/xadgps_client/ui/DevicesFragment.java
@@ -1,4 +1,4 @@
-package de.raffaelhahn.xadgps_client;
+package de.raffaelhahn.xadgps_client.ui;
 
 import android.os.Bundle;
 
@@ -12,11 +12,10 @@
 import android.view.View;
 import android.view.ViewGroup;
 
-import org.osmdroid.util.GeoPoint;
-import org.osmdroid.views.overlay.Marker;
-
 import java.util.ArrayList;
 
+import de.raffaelhahn.xadgps_client.R;
+import de.raffaelhahn.xadgps_client.model.Device;
 import de.raffaelhahn.xadgps_client.services.DeviceListService;
 
 public class DevicesFragment extends Fragment implements DeviceListService.DeviceListUpdateListener {

diff --git a/...xadgps_client/DevicesRecyclerAdapter.java → ...gps_client/ui/DevicesRecyclerAdapter.java b/...xadgps_client/DevicesRecyclerAdapter.java → ...gps_client/ui/DevicesRecyclerAdapter.java
@@ -1,4 +1,4 @@
-package de.raffaelhahn.xadgps_client;
+package de.raffaelhahn.xadgps_client.ui;
 
 import android.view.LayoutInflater;
 import android.view.View;
@@ -9,6 +9,9 @@
 
 import java.util.ArrayList;
 
+import de.raffaelhahn.xadgps_client.R;
+import de.raffaelhahn.xadgps_client.model.Device;
+
 public class DevicesRecyclerAdapter extends RecyclerView.Adapter<DevicesRecyclerAdapter.DeviceViewHolder> {
 
     private ArrayList<Device> deviceList = new ArrayList<>();

diff --git a/...faelhahn/xadgps_client/LoginActivity.java → ...lhahn/xadgps_client/ui/LoginActivity.java b/...faelhahn/xadgps_client/LoginActivity.java → ...lhahn/xadgps_client/ui/LoginActivity.java
@@ -1,4 +1,4 @@
-package de.raffaelhahn.xadgps_client;
+package de.raffaelhahn.xadgps_client.ui;
 
 import androidx.appcompat.app.AppCompatActivity;
 
@@ -14,8 +14,11 @@
 
 import java.util.Locale;
 
+import de.raffaelhahn.xadgps_client.Constants;
+import de.raffaelhahn.xadgps_client.R;
 import de.raffaelhahn.xadgps_client.async.AsyncCallback;
 import de.raffaelhahn.xadgps_client.async.LoginAsync;
+import de.raffaelhahn.xadgps_client.model.OperatingMode;
 
 public class LoginActivity extends AppCompatActivity {
 

diff --git a/...ffaelhahn/xadgps_client/MainActivity.java → ...elhahn/xadgps_client/ui/MainActivity.java b/...ffaelhahn/xadgps_client/MainActivity.java → ...elhahn/xadgps_client/ui/MainActivity.java
@@ -1,4 +1,4 @@
-package de.raffaelhahn.xadgps_client;
+package de.raffaelhahn.xadgps_client.ui;
 
 import androidx.annotation.NonNull;
 import androidx.appcompat.app.AppCompatActivity;
@@ -14,6 +14,8 @@
 
 import org.osmdroid.config.Configuration;
 
+import de.raffaelhahn.xadgps_client.Constants;
+import de.raffaelhahn.xadgps_client.R;
 import de.raffaelhahn.xadgps_client.services.DeviceListService;
 
 public class MainActivity extends AppCompatActivity {

diff --git a/...lhahn/xadgps_client/TrackingFragment.java → ...hn/xadgps_client/ui/TrackingFragment.java b/...lhahn/xadgps_client/TrackingFragment.java → ...hn/xadgps_client/ui/TrackingFragment.java
@@ -1,4 +1,4 @@
-package de.raffaelhahn.xadgps_client;
+package de.raffaelhahn.xadgps_client.ui;
 
 import android.Manifest;
 import android.content.Context;
@@ -31,6 +31,9 @@
 
 import java.util.ArrayList;
 
+import de.raffaelhahn.xadgps_client.Constants;
+import de.raffaelhahn.xadgps_client.R;
+import de.raffaelhahn.xadgps_client.model.Device;
 import de.raffaelhahn.xadgps_client.services.DeviceListService;
 
 public class TrackingFragment extends Fragment implements DeviceListService.DeviceListUpdateListener {

diff --git a/app/src/main/res/layout/activity_login.xml b/app/src/main/res/layout/activity_login.xml
@@ -6,7 +6,7 @@
     android:layout_width="match_parent"
     android:layout_height="match_parent"
     android:padding="32dp"
-    tools:context=".LoginActivity">
+    tools:context=".ui.LoginActivity">
 
 
     <androidx.constraintlayout.widget.ConstraintLayout

diff --git a/app/src/main/res/layout/activity_main.xml b/app/src/main/res/layout/activity_main.xml
@@ -5,7 +5,7 @@
     android:layout_width="match_parent"
     android:layout_height="match_parent"
     android:background="?attr/colorSurface"
-    tools:context=".MainActivity">
+    tools:context=".ui.MainActivity">
 
     <androidx.fragment.app.FragmentContainerView
         android:id="@+id/fragment_container_view"

diff --git a/app/src/main/res/layout/fragment_devices.xml b/app/src/main/res/layout/fragment_devices.xml
@@ -3,7 +3,7 @@
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
-    tools:context=".DevicesFragment"
+    tools:context=".ui.DevicesFragment"
     android:fitsSystemWindows="true">
 
     <androidx.recyclerview.widget.RecyclerView

diff --git a/app/src/main/res/layout/fragment_tracking.xml b/app/src/main/res/layout/fragment_tracking.xml
@@ -4,7 +4,7 @@
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
-    tools:context=".TrackingFragment">
+    tools:context=".ui.TrackingFragment">
 
     <org.osmdroid.views.MapView
         android:id="@+id/map"