From d7b7dee6d119d6cc8808ff4c7574915b64aacf75 Mon Sep 17 00:00:00 2001 From: Preston Date: Tue, 28 Oct 2025 17:16:25 -0700 Subject: [PATCH 1/3] missing way to run commands and ensuring valid arguments in command so giberish doesnt run --- lib/commandLineParser.cpp | 67 +++++++++++++++++++++++++++++++++++++++ 1 file changed, 67 insertions(+) create mode 100644 lib/commandLineParser.cpp diff --git a/lib/commandLineParser.cpp b/lib/commandLineParser.cpp new file mode 100644 index 0000000..779e90c --- /dev/null +++ b/lib/commandLineParser.cpp @@ -0,0 +1,67 @@ +#include "commandLineParser.h" +#include +#include // for std::invalid_argument + +//implement hashmap for the code +CLI::CLI(int argc, char** argv){ + try{ + if (arc < 2) { + throw std::invalid_argument("Error: No input file provided.\n + Usage: "); + } + //sets program info + programName = argv[0]; + inputFile = argv[1] + + //checks that file has correct file extension + if(!(CLI::IsMarkupFile(inputFile))) { + throw std::invalid_argument("Error: Invalid file extension. Expected a '.md' (Markdown) file."); + } + //stores remaining arguments + for (int i = 1; i < argc; i++){ + args.push_back(argv[i]); + } + } + catch (const std::invalid_argument& e) { + std::cerr << e.what() << "\n"; + PrintHelp(); + std::exit(EXIT_FAILURE); + } + } + +bool CLI::IsMarkupFile(const std::string& filename) { + //markdown file extension + const std::string requiredExtension = ".md"; + //checks to see if it can even include extension + if (filename.size() <= requiredExtension.size()) { + return false; + } + // Extracts the last N characters of the filename (where N is the length of the required extension) + // and checks if they match the required file extension (e.g., ".md"). + // + // Example: + // filename = "notes.md" + // requiredExtension = ".md" + // filename.substr(filename.size() - requiredExtension.size()) == ".md" → true + // + // Reference: https://en.cppreference.com/w/cpp/string/basic_string/substr + return filename.substr(filename.size() - requiredExtension.size()) == requiredExtension; +} + + +//place to run commands but IDK IF WE SHOULD USE A HASMAP or how we are going to get each method to run +void CLI::RunCommands(){ + for (int i = 0; i < args.size(); i++){ + + } +} + +void CLI::PrintHelp() const{ + std::cout << "Usage:\n" + << " " << programName << " REQUIRED\n" + << "Flags:\n" + << " --o, --output , optional output filename\n" + << " --w, --watch, enables watchdog\n" + << " --s, --stop, stops watchdog\n"; +} + From 8f5a466a590f1778e616ac486132e2a2de5da5a3 Mon Sep 17 00:00:00 2001 From: Preston Date: Tue, 28 Oct 2025 17:18:58 -0700 Subject: [PATCH 2/3] added comments to clarify things --- lib/watchDog.cpp | 88 ++++++++++++++++++++++++++---------------------- 1 file changed, 48 insertions(+), 40 deletions(-) diff --git a/lib/watchDog.cpp b/lib/watchDog.cpp index e9bec9e..0fc08ff 100644 --- a/lib/watchDog.cpp +++ b/lib/watchDog.cpp @@ -6,63 +6,71 @@ #include "watchDog.h" namespace fs = std::filesystem; //makes it easier to read -void WatchDog::start() -{ - //checks if file exist - if(!fs::exists(path)) - { - //returns and sets parameters to false if file doesnt exist +void WatchDog::start() { + try { + // Try to get the last write time directly + lastWriteTime = fs::last_write_time(path); + + // If successful, start watching + watching = true; + hasInitialTime = true; + std::cout << "WatchDog: Started monitoring " << path << std::endl; + } + catch (const fs::filesystem_error& e) { + // If the file doesn't exist or another filesystem error occurs watching = false; hasInitialTime = false; - std::cout << "WatchDog: File does not exists: " << path << std::endl; - return; + std::cerr << "WatchDog: Cannot start. File error: " << e.what() << std::endl; + } + catch (const std::exception& e) { + // Catch any other unexpected errors + watching = false; + hasInitialTime = false; + std::cerr << "WatchDog: Unexpected error: " << e.what() << std::endl; } - //grabs intial write time - lastWriteTime = fs::last_write_time(path); - watching = true; - hasInitialTime = true; - std::cout << "WatchDog: Started" << std::endl; } void WatchDog::stop(){ watching = false; } -bool WatchDog::checkFile(){ - //If not watching returns false +//this is currently just telling you when the file is edited, the exact time its edited +bool WatchDog::checkFile() { if (!watching) return false; - //Checking if file was deleted - if(!fs::exists(path)) - { + try { + // Try to get last write time + fs::file_time_type currentWriteTime = fs::last_write_time(path); + + // File was just created + if (!hasInitialTime) { + lastWriteTime = currentWriteTime; + hasInitialTime = true; + std::cout << "WatchDog: File created: " << path << std::endl; + return true; + } + + // File modified + if (currentWriteTime != lastWriteTime) { + lastWriteTime = currentWriteTime; + std::cout << "WatchDog: File modified at " + << timePointToString(lastWriteTime) << std::endl; + return true; + } + + } catch (const fs::filesystem_error& e) { + // File deleted, inaccessible, or path invalid if (hasInitialTime) { - std::cout << "WatchDog: File was delete: " << path << std::endl; + std::cout << "WatchDog: File deleted or inaccessible: " << path << std::endl; hasInitialTime = false; return true; } return false; - } - - //Built in function with file system to check last write tim - fs::file_time_type currentWriteTime = fs::last_write_time(path); - //File was just created - if(!hasInitialTime) - { - lastWriteTime = currentWriteTime; - hasInitialTime = true; - std::cout << "WatchDog: File created: " << path << std::endl; - return true; + } catch (const std::exception& e) { + std::cerr << "WatchDog: Unexpected error checking file: " << e.what() << std::endl; + return false; } - //File modified - if (currentWriteTime != lastWriteTime) - { - lastWriteTime = currentWriteTime; - std::cout << "WatchDog: File modifed at " - << timePointToString(lastWriteTime) << std::endl; - return true; - } - - //No change + // No change return false; } From b38bb60e6217576c2a70a8362db0e1e05c36b3a4 Mon Sep 17 00:00:00 2001 From: Preston Date: Tue, 28 Oct 2025 17:19:27 -0700 Subject: [PATCH 3/3] added try and catch --- lib/commandLineParser.h | 80 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 80 insertions(+) create mode 100644 lib/commandLineParser.h diff --git a/lib/commandLineParser.h b/lib/commandLineParser.h new file mode 100644 index 0000000..88c9a09 --- /dev/null +++ b/lib/commandLineParser.h @@ -0,0 +1,80 @@ +/** + * @file commandLineParser.h + * @brief Parses command line + * @author Preston Shultz + * Sources: + * + */ + +#ifndef COMMAND_LINE_PARSER_H +#define COMMAND_LINE_PARSER_H + +#include +#include + +/** + * @brief CommandLine Helper class + * + * Parse input arguments, sets input and output files + * Lets user use commands with program + * + * @author Preston Shultz (shultzp1@my.erau.edu) + */ +class CLI{ +public: + /** + * @brief Takes in argc and argv and converts it to vector + * + * CLI constructor + * + * @author Preston Shultz (shultzp1@my.erau.edu) + */ + CLI(int argc, char **argv); + + /** + * @brief Runs Commands + * + * @author Preston Shultz (shultzp1@my.erau.edu) + */ + void RunCommands(); + + /** + * @brief Prints a list of commands that can be used + * + * @author Preston Shultz (shultzp1@my.erau.edu) + */ + void PrintHelp() const; + + /** + * @brief Returns input file + * + * @author Preston Shultz (shultzp1@my.erau.edu) + */ + std::string GetInputFile() const {return inputFile;} + + /** + * @brief Returns output file + * + * @author Preston Shultz (shultzp1@my.erau.edu) + */ + std::string GetOutputFile() const {return outputFile;} + +private: + + /** + * @brief Ensures the provided filename has a .markup extension + * + * @param filename The file name to validate + * @return true if file ends with .markup + * @return false otherwise + * @author Preston Shultz (shultzp1@my.erau.edu) + */ + static bool IsMarkupFile(const std::string& filename); + + std::string programName; + std::vector args; + std::string inputFile; + std::string outputFile; +}; + +#endif \ No newline at end of file