Cleanup UserIntent

src/UserIntent.cpp

@@ -7,17 +7,7 @@
 #include "Archiver.hpp"
 
 namespace xwim {
-
+    unique_ptr<UserIntent> make_compress_intent(const UserOpt &userOpt) {
-    unique_ptr<UserIntent> make_intent(const UserOpt &userOpt) {
-        if (userOpt.wants_compress() && userOpt.wants_extract()) {
-            throw XwimError("Cannot compress and extract simultaneously");
-        }
-        if(userOpt.paths.size() == 0) {
-            throw XwimError("No input given...");
-        }
-
-        // compression intent explicitly specified
-        if (userOpt.wants_compress()) {
         if (userOpt.paths.size() == 1) {
             return make_unique<CompressSingleIntent>(
                 CompressSingleIntent{
@@ -37,9 +27,8 @@ namespace xwim {
         });
     }
 
-        // extraction intent explicitly specified
+    unique_ptr<UserIntent> make_extract_intent(const UserOpt &userOpt) {
-        if (userOpt.wants_extract()) {
+        for (const path& p: userOpt.paths) {
-            for (path p: userOpt.paths) {
             if (!can_handle_archive(p)) {
                 throw XwimError("Cannot extract path {}", p);
             }
@@ -52,53 +41,80 @@ namespace xwim {
         });
     }
 
-        // no intent explicitly specified, try to infer from input
+    unique_ptr<UserIntent> try_infer_compress_intent(const UserOpt &userOpt) {
+      if(!userOpt.out.has_value()) {
+        spdlog::debug("No <out> provided");
+        if(userOpt.paths.size() != 1) {
+          spdlog::debug("Not a single-path compression. Cannot guess <out> for many-path compression");
+          return nullptr;
+        }
 
+        spdlog::debug("Only one <path> provided. Assume single-path compression.");
+        return make_unique<CompressSingleIntent>(CompressSingleIntent{*userOpt.paths.begin(), userOpt.out});
+      }
+
+      spdlog::debug("<out> provided: {}", userOpt.out.value());
+      if(can_handle_archive(userOpt.out.value())) {
+          spdlog::debug("{} given and a known archive format, assume compression", userOpt.out.value());
+          return make_compress_intent(userOpt);
+      }
+
+      spdlog::debug("Cannot compress multiple paths without a user-provided output archive");
+      return nullptr;
+    }
+
+    unique_ptr<UserIntent> try_infer_extract_intent(const UserOpt &userOpt) {
       bool can_extract_all = std::all_of(
           userOpt.paths.begin(), userOpt.paths.end(),
-                [](path path) {
+          [](const path &path) { return can_handle_archive(path); });
-                    return can_handle_archive(path);
-                });
 
-        bool is_out_archive = userOpt.out.has_value() && can_handle_archive(userOpt.out.value());
+      if(!can_extract_all) {
-
+        spdlog::debug("Cannot extract all provided <paths>. Assume this is not an extraction.");
-        // out is explicitly specified and an archive, assume we want compression
+        for(const path& p: userOpt.paths) {
-        if(is_out_archive) {
+          if(!can_handle_archive(p)) {
-            if(userOpt.paths.size() == 1) {
+            spdlog::debug("Cannot handle {}", p);
-                return make_unique<CompressSingleIntent>(
+          }
-                        CompressSingleIntent{
-                           *userOpt.paths.begin(),
-                           userOpt.out
-                        });
         }
 
-            return make_unique<CompressManyIntent>(
+        return nullptr;
-                    CompressManyIntent{
-                        userOpt.paths,
-                        userOpt.out.value() // this is ok is_out_archive checks for has_value()
-                    }
-            );
       }
 
-        // all inputs are extractable archives, assume extraction intent
+      if(userOpt.out.has_value() && can_handle_archive(userOpt.out.value())) {
-        if (can_extract_all) {
+        spdlog::debug("Could extract all provided <paths>. But also {} looks like an archive. Ambiguous intent. Assume this is not an extraction.", userOpt.out.value());
-            return make_unique<ExtractIntent>(
+        return nullptr;
-                    ExtractIntent{
-                            userOpt.paths,
-                            userOpt.out
-                    });
       }
 
-        // at this point all we can hope for is that the intention is a single-path compression:
+      spdlog::debug("Could extract all provided <paths>. But also <out> looks like an archive. Ambiguous intent. Assume this is not an extraction.");
-        // we don't know how to extract it; we don't know (and can't guess) output for many-path compression;
+      return make_extract_intent(userOpt);
-        if(userOpt.paths.size() == 1) {
-            return make_unique<CompressSingleIntent>(
-                    CompressSingleIntent{
-                        *userOpt.paths.begin(),
-                        userOpt.out
-                    });
     }
 
+    unique_ptr<UserIntent> make_intent(const UserOpt &userOpt) {
+        if (userOpt.wants_compress() && userOpt.wants_extract()) {
+            throw XwimError("Cannot compress and extract simultaneously");
+        }
+        if(userOpt.paths.empty()) {
+            throw XwimError("No input given...");
+        }
+
+        // explicitly specified intent
+        if (userOpt.wants_compress()) return make_compress_intent(userOpt);
+        if (userOpt.wants_extract()) return make_extract_intent(userOpt);
+
+        spdlog::info("Intent not explicitly provided, trying to infer intent");
+
+        if(auto intent = try_infer_extract_intent(userOpt)) {
+          spdlog::info("Extraction intent inferred");
+          return intent;
+        }
+        spdlog::info("Cannot infer extraction intent");
+
+        if(auto intent = try_infer_compress_intent(userOpt)) {
+          spdlog::info("Compression intent inferred");
+          return intent;
+        }
+        spdlog::info("Cannot infer compression intent");
+
+
         throw XwimError("Cannot guess intent");
     }
 

src/UserIntent.hpp

@@ -32,9 +32,9 @@ private:
 
 public:
     ExtractIntent(set<path> archives, optional<path> out): archives(archives), out(out) {};
-    ~ExtractIntent() = default;
+    ~ExtractIntent() override = default;
 
-    void execute();
+    void execute() override;
 };
 
 /**
@@ -55,14 +55,13 @@ public:
 class CompressSingleIntent : public UserIntent {
 private:
     path in;
-    optional <path> out;
+    optional<path> out;
 
 public:
-    CompressSingleIntent(path in, optional <path> out) : UserIntent(), in(in), out(out) {};
+    CompressSingleIntent(path in, optional<path> out) : UserIntent(), in(in), out(out) {};
+    ~CompressSingleIntent() override = default;
 
-    ~CompressSingleIntent() = default;
+    void execute() override;
-
-    void execute();
 };
 
 /**
@@ -81,9 +80,9 @@ private:
 
 public:
     CompressManyIntent(set<path> in_paths, path out): UserIntent(), in_paths(in_paths), out(out) {};
-    ~CompressManyIntent() = default;
+    ~CompressManyIntent() override = default;
 
-    void execute();
+    void execute() override;
 };
 
 }  // namespace xwim