rearrange

day02.cpp

@@ -1,7 +1,7 @@
 #include <iostream>
 #include <utility>
 
-#include <intcode.hpp>
+#include <intcode/intcode.hpp>
 
 using namespace intcode;
 

day05.cpp

@@ -1,7 +1,7 @@
 #include <iostream>
 #include <utility>
 
-#include <intcode.hpp>
+#include <intcode/intcode.hpp>
 using namespace intcode;
 
 namespace {

day07.cpp

@@ -5,7 +5,7 @@
 #include <utility>
 #include <vector>
 
-#include <intcode.hpp>
+#include <intcode/intcode.hpp>
 using namespace intcode;
 
 namespace {

day09.cpp

@@ -1,7 +1,7 @@
 #include <iostream>
 #include <utility>
 
-#include <intcode.hpp>
+#include <intcode/intcode.hpp>
 using namespace intcode;
 
 namespace {

day11.cpp

@@ -3,7 +3,7 @@
 #include <tuple>
 #include <map>
 
-#include <intcode.hpp>
+#include <intcode/intcode.hpp>
 using namespace intcode;
 
 namespace {

day13.cpp

@@ -3,7 +3,7 @@
 #include <tuple>
 #include <set>
 
-#include <intcode.hpp>
+#include <intcode/intcode.hpp>
 using namespace intcode;
 
 namespace {

day15.cpp

@@ -6,7 +6,7 @@
 #include <deque>
 #include <vector>
 
-#include <intcode.hpp>
+#include <intcode/intcode.hpp>
 using namespace intcode;
 
 namespace {

lib/CMakeLists.txt

@@ -1,2 +1,2 @@
-add_library(intcode intcode.cpp)
+add_library(intcode src/Parser.cpp src/Interpreter.cpp src/Machine.cpp)
-target_include_directories(intcode PUBLIC include)
+target_include_directories(intcode PUBLIC include)

lib/include/intcode/Interpreter.hpp

@@ -0,0 +1,26 @@
+#ifndef INTCODE_INTERPRETER_HPP_
+#define INTCODE_INTERPRETER_HPP_
+
+#include "Machine.hpp"
+
+namespace intcode {
+
+struct Input {
+ValueType &pos;
+};
+
+struct Output {
+ValueType val;
+};
+
+struct Halt {};
+
+auto Step(Machine & m) -> std::variant<Input, Output, Halt>;
+
+struct BadInstruction : public std::runtime_error {
+  explicit BadInstruction(char const* what);
+};
+
+} // namespace
+
+#endif

lib/include/intcode/Machine.hpp

@@ -1,30 +1,13 @@
-#ifndef INTCODE_HPP_
+#ifndef INTCODE_MACHINE_HPP_
-#define INTCODE_HPP_
+#define INTCODE_MACHINE_HPP_
 
 #include <cstddef>
-#include <cstdint>
-#include <istream>
 #include <unordered_map>
-#include <stdexcept>
-#include <variant>
 #include <vector>
 
-template <class... Ts> struct overloaded : Ts... { using Ts::operator()...; };
-template <class... Ts> overloaded(Ts...) -> overloaded<Ts...>;
-
 namespace intcode {
 
-  using ValueType = std::int64_t;
+using ValueType = std::int64_t;
-
-  struct Input {
-    ValueType &pos;
-  };
-
-  struct Output {
-    ValueType val;
-  };
-
-  struct Halt {};
 
 class Machine {
   std::vector<ValueType> rom_;
@@ -51,14 +34,6 @@ public:
   auto Rebase(std::size_t offset) -> void;
 };
 
-auto Step(Machine & m) -> std::variant<Input, Output, Halt>;
+} // namespace
 
-struct BadInstruction : public std::runtime_error {
+#endif
-  explicit BadInstruction(char const* what);
-};
-
-auto ParseStream(std::istream &in) -> std::vector<ValueType>;
-
-}
-
-#endif // INTCODE_HPP_

lib/include/intcode/Parser.hpp

@@ -0,0 +1,15 @@
+#ifndef INTCODE_PARSER_HPP_
+#define INTCODE_PARSER_HPP_
+
+#include <istream>
+#include <vector>
+
+#include "Machine.hpp"
+
+namespace intcode {
+
+auto ParseStream(std::istream &in) -> std::vector<ValueType>;
+
+} // namespace
+
+#endif

lib/include/intcode/intcode.hpp

@@ -0,0 +1,11 @@
+#ifndef INTCODE_HPP_
+#define INTCODE_INTCODE_HPP_
+
+#include <intcode/Machine.hpp>
+#include <intcode/Parser.hpp>
+#include <intcode/Interpreter.hpp>
+
+template <class... Ts> struct overloaded : Ts... { using Ts::operator()...; };
+template <class... Ts> overloaded(Ts...) -> overloaded<Ts...>;
+
+#endif

lib/src/Interpreter.cpp

@@ -1,37 +1,10 @@
-#include "intcode.hpp"
+#include <intcode/Interpreter.hpp>
-#include <iostream>
 
 namespace intcode {
 
 BadInstruction::BadInstruction(char const* what)
     : std::runtime_error{what} {}
 
-Machine::Machine() : rom_{}, ram_{}, pc_{0}, base_{0} {}
-
-Machine::Machine(std::vector<ValueType> ram)
-    : rom_{ram}, ram_{}, pc_{0}, base_{0} {}
-
-auto Machine::At(std::size_t i) -> ValueType & {
-  return i < rom_.size() ? rom_[i] : ram_[i];
-}
-
-auto Machine::Rel(std::size_t i) -> ValueType & {
-  return At(base_ + i);
-}
-
-auto Machine::Rebase(std::size_t offset) -> void {
-  base_ += offset;
-}
-
-
-auto Machine::Next() -> ValueType & {
-  return At(pc_++);
-}
-
-auto Machine::Goto(std::size_t address) -> void {
-  pc_ = address;
-}
-
 auto Step(Machine & m) -> std::variant<Input, Output, Halt> {
   for (;;) {
     auto instruction = m.Next();
@@ -119,14 +92,4 @@ auto Step(Machine & m) -> std::variant<Input, Output, Halt> {
   }
 }
 
-auto ParseStream(std::istream &in) -> std::vector<ValueType> {
+} // namespace
-  ValueType x;
-  std::string str;
-  std::vector<ValueType> result;
-  while (std::getline(in, str, ',')) {
-    result.push_back(std::stol(str));
-  }
-  return result;
-}
-
-}

lib/src/Machine.cpp

@@ -0,0 +1,30 @@
+#include <intcode/Machine.hpp>
+
+namespace intcode {
+
+Machine::Machine() : rom_{}, ram_{}, pc_{0}, base_{0} {}
+
+Machine::Machine(std::vector<ValueType> ram)
+    : rom_{ram}, ram_{}, pc_{0}, base_{0} {}
+
+auto Machine::At(std::size_t i) -> ValueType & {
+  return i < rom_.size() ? rom_[i] : ram_[i];
+}
+
+auto Machine::Rel(std::size_t i) -> ValueType & {
+  return At(base_ + i);
+}
+
+auto Machine::Rebase(std::size_t offset) -> void {
+  base_ += offset;
+}
+
+auto Machine::Next() -> ValueType & {
+  return At(pc_++);
+}
+
+auto Machine::Goto(std::size_t address) -> void {
+  pc_ = address;
+}
+
+} // namespace

lib/src/Parser.cpp

@@ -0,0 +1,17 @@
+#include <intcode/Parser.hpp>
+
+#include <iostream>
+
+namespace intcode {
+
+auto ParseStream(std::istream &in) -> std::vector<ValueType> {
+  ValueType x;
+  std::string str;
+  std::vector<ValueType> result;
+  while (std::getline(in, str, ',')) {
+    result.push_back(std::stol(str));
+  }
+  return result;
+}
+
+} // namespace