heavy refactoring & moved content reciever to viewer activity

Author: Aman

app/src/main/AndroidManifest.xml

@@ -9,7 +9,6 @@
         <provider android:authorities="aman.loghub.provider" />
     </queries>
 
-
     <application
         android:name="Log"
         android:largeHeap="true"
@@ -28,8 +27,14 @@
                 <action android:name="android.intent.action.MAIN" />
                 <category android:name="android.intent.category.LAUNCHER" />
             </intent-filter>
-            
-            <!-- Intent filter to handle JSON files -->
+        </activity>
+
+        <activity
+            android:name=".ViewerActivity"
+            android:exported="true"
+            android:screenOrientation="portrait"
+            android:parentActivityName=".MainActivity">
+
             <intent-filter>
                 <action android:name="android.intent.action.VIEW" />
                 <category android:name="android.intent.category.DEFAULT" />
@@ -39,8 +44,7 @@
                 <data android:mimeType="application/json" />
                 <data android:mimeType="text/json" />
             </intent-filter>
-            
-            <!-- Additional intent filter for .json file extensions -->
+
             <intent-filter>
                 <action android:name="android.intent.action.VIEW" />
                 <category android:name="android.intent.category.DEFAULT" />
@@ -53,14 +57,9 @@
                 <data android:pathPattern=".*\\..*\\..*\\.json" />
                 <data android:pathPattern=".*\\..*\\..*\\..*\\.json" />
             </intent-filter>
-        </activity>
 
-        <activity
-            android:name=".ViewerActivity"
-            android:exported="false"
-            android:screenOrientation="portrait"
-            android:parentActivityName=".MainActivity" />
+        </activity>
 
     </application>
 
-</manifest>
+</manifest>

app/src/main/java/aman/jsonviewer/Fragments/CardViewFragment.java

@@ -32,8 +32,8 @@ public class CardViewFragment extends Fragment implements ViewerActivity.Searcha
     private CardAdapter adapter;
     private List<CardItem> items = new ArrayList<>();
     private String jsonData;
-    private ExecutorService executor = Executors.newSingleThreadExecutor();
-    private Handler mainHandler = new Handler(Looper.getMainLooper());
+    private final ExecutorService executor = Executors.newSingleThreadExecutor();
+    private final Handler mainHandler = new Handler(Looper.getMainLooper());
 
     public static CardViewFragment newInstance() {
         return new CardViewFragment();
@@ -64,11 +64,9 @@ public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle sa
     @Override
     public void onResume() {
         super.onResume();
-        // Apply current search query when fragment becomes visible
         if (getActivity() instanceof ViewerActivity) {
             String currentQuery = ((ViewerActivity) getActivity()).getCurrentSearchQuery();
             if (currentQuery != null && !currentQuery.isEmpty() && adapter != null) {
-                // Post with delay to ensure data is loaded
                 recyclerView.postDelayed(() -> {
                     if (adapter != null && !items.isEmpty()) {
                         adapter.filter(currentQuery);
@@ -105,7 +103,6 @@ private void parseJsonAsync() {
                 progressBar.setVisibility(View.GONE);
                 recyclerView.setVisibility(View.VISIBLE);
 
-                // Apply search filter if there's an active search
                 if (getActivity() instanceof ViewerActivity) {
                     String currentQuery = ((ViewerActivity) getActivity()).getCurrentSearchQuery();
                     if (currentQuery != null && !currentQuery.isEmpty()) {
@@ -181,7 +178,6 @@ public void onSearch(String query) {
         }
     }
 
-    // NEW: Public method to allow ViewerActivity to access adapter for navigation
     public CardAdapter getAdapter() {
         return adapter;
     }
@@ -196,7 +192,7 @@ static class CardItem {
         String key;
         String value;
         String type;
-        boolean matchesSearch = false; // NEW: Track search matches
+        boolean matchesSearch = false; 
 
         CardItem(String key, String value, String type) {
             this.key = key;
@@ -209,17 +205,15 @@ class CardAdapter extends RecyclerView.Adapter<CardAdapter.ViewHolder> {
         private List<CardItem> displayItems;
         private List<CardItem> allItems;
 
-        // NEW: Search navigation fields
         private String currentSearchQuery = "";
-        private List<Integer> searchMatches = new ArrayList<>();
+        private final List<Integer> searchMatches = new ArrayList<>();
         private int currentMatchIndex = -1;
 
         CardAdapter(List<CardItem> items) {
             this.allItems = items;
             this.displayItems = items;
         }
 
-        // Method to update all items (called after async loading)
         void updateAllItems(List<CardItem> newItems) {
             this.allItems = newItems;
             if (currentSearchQuery.isEmpty()) {
@@ -246,7 +240,6 @@ public void onBindViewHolder(@NonNull ViewHolder holder, int position) {
             int color = getTypeColor(item.type);
             holder.typeText.setTextColor(color);
 
-            // NEW: Highlight current match differently
             if (!searchMatches.isEmpty() && currentMatchIndex >= 0 && 
                 currentMatchIndex < searchMatches.size() &&
                 searchMatches.get(currentMatchIndex) == position) {
@@ -291,15 +284,13 @@ void filter(String query) {
             currentSearchQuery = query;
 
             if (query.isEmpty()) {
-                // Clear search
                 for (CardItem item : allItems) {
                     item.matchesSearch = false;
                 }
                 displayItems = allItems;
                 searchMatches.clear();
                 currentMatchIndex = -1;
             } else {
-                // Filter and mark matches
                 List<CardItem> filtered = new ArrayList<>();
                 String lowerQuery = query.toLowerCase();
                 for (CardItem item : allItems) {
@@ -312,7 +303,6 @@ void filter(String query) {
                 }
                 displayItems = filtered;
 
-                // NEW: Update search matches and scroll to first
                 updateSearchMatches();
                 if (!searchMatches.isEmpty()) {
                     currentMatchIndex = 0;
@@ -322,7 +312,6 @@ void filter(String query) {
             notifyDataSetChanged();
         }
 
-        // NEW: Update search matches list
         private void updateSearchMatches() {
             searchMatches.clear();
             for (int i = 0; i < displayItems.size(); i++) {
@@ -332,7 +321,6 @@ private void updateSearchMatches() {
             }
         }
 
-        // NEW: Scroll to current match
         private void scrollToCurrentMatch() {
             if (currentMatchIndex >= 0 && currentMatchIndex < searchMatches.size()) {
                 int position = searchMatches.get(currentMatchIndex);
@@ -344,14 +332,12 @@ private void scrollToCurrentMatch() {
             }
         }
 
-        // NEW: Navigate to next match
         public void nextMatch() {
             if (searchMatches.isEmpty()) return;
             currentMatchIndex = (currentMatchIndex + 1) % searchMatches.size();
             scrollToCurrentMatch();
         }
 
-        // NEW: Navigate to previous match
         public void previousMatch() {
             if (searchMatches.isEmpty()) return;
             currentMatchIndex--;
@@ -361,12 +347,10 @@ public void previousMatch() {
             scrollToCurrentMatch();
         }
 
-        // NEW: Get current match index
         public int getCurrentMatchIndex() {
             return currentMatchIndex;
         }
 
-        // NEW: Get total matches
         public int getTotalMatches() {
             return searchMatches.size();
         }
@@ -386,4 +370,4 @@ class ViewHolder extends RecyclerView.ViewHolder {
             }
         }
     }
-}
+}