Fixed bugs

example.cpp

@@ -6,13 +6,13 @@ using namespace gopt;
 
 int main()
 {
-    Matrix<double> m(3, 4);
+    Vector<double> v(5);
+    v.fill([]() { return 122; });
+    Vector<double> u(v);
 
-    std::default_random_engine gen;
-    std::normal_distribution<double> dist(0.0, 1.0);
-    m.fill([&]() { return dist(gen); });
+    u = v;
 
-    std::cout << m << std::endl;
+    std::cout << v << ", " << u << std::endl;
 
     return 0;
 }

math/matrix.hpp

@@ -365,8 +365,6 @@ namespace gopt
 			m._rows = 0;
 			m._cols = 0;
 			m.data = nullptr;
-
-			std::cout << "Called" << std::endl;
 		}
 
 		~Matrix()
@@ -417,7 +415,7 @@ namespace gopt
 			return *this;
 		}
 
-		template <typename F, typename... Args>
+		template <typename F, typename... Args, typename = std::enable_if_t<std::is_invocable_v<F, Args...>>>
 		Matrix& fill(F&& f, Args... args)
 		{
 			for (int i = 0; i < _rows; i++)

math/vector.hpp

@@ -310,8 +310,14 @@ namespace gopt
 		Vector(const Vector& v)
 			: len(v.len), data(new T[v.len])
 		{
-			for (unsigned int i = 0; i < len; i++)
-				data[i] = v.data[i];
+			std::copy(v.data, v.data + len, data);
+		}
+
+		Vector(Vector&& v)
+			: len(v.len), data(v.data)
+		{
+			v.len = 0;
+			v.data = nullptr;
 		}
 
 		~Vector()
@@ -320,6 +326,15 @@ namespace gopt
 				delete[] data;
 		}
 
+		Vector& operator=(const Vector& v)
+		{
+			len = v.len;
+			data = new T[len];
+			std::copy(v.data, v.data + len, data);
+
+			return *this;
+		}
+
 		Vector& fill(const T& s)
 		{
 			std::fill_n(data, len, s);
@@ -343,7 +358,7 @@ namespace gopt
 			return *this;
 		}
 
-		template <typename F, typename... Args>
+		template <typename F, typename... Args, typename = std::enable_if_t<std::is_invocable_v<F, Args...>>>
 		Vector& fill(F&& f, Args... args)
 		{
 			for (int i = 0; i < len; i++)