clean up code more

joomer-ftxui-file-browser.cpp

@@ -37,6 +37,9 @@ void ReloadDirectory(ftxui::ScreenInteractive& screen, const std::filesystem::pa
             std::string name = entry.path().filename().string();            
             if (!name.empty() && name[0] == '.') // Works in C++11, C++14, and C++17
 				continue; // Skip hidden files and directories
+            if (std::filesystem::is_directory(entry.path())) {
+                name += "/"; // Append slash to directories for clarity
+			}
             entries.push_back(std::to_string(i) + ". " + name);
             clean_names.push_back(name);
             i++;
@@ -49,33 +52,37 @@ int main(int argc, char* argv[]) {
     int final_selected_index = -1; // Use -1 to indicate no selection was made        
     std::vector<std::string> entries;
     std::vector<std::string> clean_names; // New vector for logic
+
 	std::string current_path = std::filesystem::current_path().string();
     std::filesystem::path start_path = std::filesystem::current_path();
     
     if (argc > 1)
     {
-        //directory_path = argv[1];
         // Convert relative input (like "../../..") into a full absolute path
-		std::cout << "argv[1]: " << argv[1] << std::endl;
+        start_path = std::filesystem::absolute(argv[1]);		
-        start_path = std::filesystem::absolute(argv[1]).lexically_normal();		
+
-        directory_path = start_path.string(); // Update global        
+        // Convert to string        
-        std::cout << "directory_path: " << directory_path << std::endl;
+        start_path = start_path.lexically_normal();  
-        current_path = directory_path; // Sync local        
+        std::string path_str = start_path.string();
-        std::cout << "current_path: " << current_path << std::endl;        
+        // Manually remove trailing slash if it exists and isn't the root directory
-        //current_path = directory_path.string();
+        if (path_str.size() > 1 && (path_str.back() == '/' || path_str.back() == '\\')) {
-	    //current_path = directory_path;
+            path_str.pop_back();
-    }
+        }
-    std::cout << "Initial current_path after argc: " << current_path << std::endl;
+
+        current_path = path_str;
+
+    }     
+
+    directory_path = current_path; // Update global        
+
     // Check if the directory exists
     if (!std::filesystem::exists(directory_path) || !std::filesystem::is_directory(directory_path)) {
         std::cerr << "Error: Directory '" << directory_path << "' not found or is not a directory." << std::endl;
         return 1;
     }
     
-    auto screen = ftxui::ScreenInteractive::TerminalOutput();
+    auto screen = ftxui::ScreenInteractive::TerminalOutput();   
-    std::cout << "Initial current_path before reload: " << current_path << std::endl;
+    ReloadDirectory(screen, start_path, entries, clean_names);    
-    ReloadDirectory(screen, start_path, entries, clean_names);
-    std::cout << "Initial current_path after reload: " << current_path << std::endl;
     int selected = 0;
     auto menu = ftxui::Menu({
     .entries = &entries,
@@ -90,8 +97,7 @@ int main(int argc, char* argv[]) {
         if (event.is_character()) {
             char c = event.character()[0];
             if (std::isdigit(c)) {
-                input_buffer += c; // Add the digit to our "typing" buffer
+                input_buffer += c; // Add the digit to our "typing" buffer                
-                //std::cout << "Typed so far: " << input_buffer << std::endl; // For debugging
                 return true;
             }
         }
@@ -103,14 +109,16 @@ int main(int argc, char* argv[]) {
         }
 
         // Handle the 'Enter' key press
-        if (event == ftxui::Event::Return) {
+        if (event == ftxui::Event::Return) {			
-			std::cout << "Enter pressed. Input buffer: " << input_buffer << std::endl;  
             if (!input_buffer.empty()) {
-                // Convert buffer to integer and check bounds
+                try 
-                int requested_index = std::stoi(input_buffer);// -1;
+                {
-                if (requested_index >= 0 && requested_index < (int)clean_names.size()) {
+                    // Convert buffer to integer and check bounds
-                    selected = requested_index; // Move the highlight to the typed index
+                    int requested_index = std::stoi(input_buffer);// -1;
-                }
+                    if (requested_index >= 0 && requested_index < (int)clean_names.size()) {
+                        selected = requested_index; // Move the highlight to the typed index
+                    }
+                } catch (...) {}
                 input_buffer.clear(); // Clear buffer after use
             }
             std::string selected_name; 
@@ -128,13 +136,15 @@ int main(int argc, char* argv[]) {
 
             new_path = std::filesystem::absolute(new_path).lexically_normal();
             new_path.make_preferred();
+
             current_path = new_path.string();
 			directory_path = current_path; // Update global variable
-            
+            if (std::filesystem::is_regular_file(new_path))
+                screen.Exit(); // Exit the application
             // Call the function to switch and reload
             ReloadDirectory(screen, new_path, entries,clean_names);
 			selected = 0; // Reset selection to the first item
-            //screen.Exit(); // Exit the application
+            
             return true; // Event handled
         }
         
@@ -181,7 +191,8 @@ int main(int argc, char* argv[]) {
         });
 
     screen.Loop(main_renderer);
-
+    // This will print to your terminal AFTER you press 'q' to quit
+    std::cout << "Final Path on Exit: " << current_path << std::endl;
     // ----------------------------------------------------
     // --- OUTPUT AFTER THE LOOP HAS CLEANLY EXITED ---
     // ----------------------------------------------------
@@ -191,13 +202,5 @@ int main(int argc, char* argv[]) {
     } else {
         std::cout << "Selection cancelled or no valid entry selected." << std::endl;
     }
-  /*Element document = vbox({
+
-    text("left")   | border,
-    text("middle") | border | flex,
-    text("right")  | border,
-  });
- 
-  auto screen = Screen::Create(Dimension::Full(), Dimension::Fit(document));
-  Render(screen, document);
-  screen.Print();*/
 }