Simplify move exercise.

For many students, it took too long to read the NVector implementation.
In this version, we use a class with one pointer to internal storage to
compare copy and move. The solution follows the copy and swap idiom
discussed in the lecture.

Fix #340.

Author: hageboeck

code/move/CMakeLists.txt

@@ -8,8 +8,8 @@ include( "${CMAKE_CURRENT_SOURCE_DIR}/../CompilerSettings.cmake" )
 include( "${CMAKE_CURRENT_SOURCE_DIR}/../SolutionTarget.cmake" )
 
 # Create the user's executable.
-add_executable( trymove "NVector.hpp" "trymove.cpp" )
+add_executable( trymove "trymove.cpp" )
 
 # Create the "solution executable".
-add_executable( trymove.sol EXCLUDE_FROM_ALL "solution/NVector.sol.hpp" "solution/trymove.sol.cpp" )
+add_executable( trymove.sol EXCLUDE_FROM_ALL "solution/trymove.sol.cpp" )
 add_dependencies( solution trymove.sol )

code/move/Makefile

@@ -4,8 +4,8 @@ solution: trymove.sol
 clean:
 	rm -f *o trymove trymove.sol *~ callgrind.out.*
 
-trymove : trymove.cpp NVector.hpp
-	${CXX} -g -std=c++11 -O0 -Wall -Wextra -L. -o $@ $<
+trymove : trymove.cpp
+	${CXX} -g -std=c++17 -O2 -Wall -Wextra -L. -o $@ $<
 
-trymove.sol : solution/trymove.sol.cpp solution/NVector.sol.hpp
-	${CXX} -g -std=c++11 -O0 -Wall -Wextra -L. -o $@ $<
+trymove.sol : solution/trymove.sol.cpp
+	${CXX} -g -std=c++17 -O2 -Wall -Wextra -L. -o $@ $<

code/move/NVector.hpp

@@ -1,13 +1,10 @@
-
 ## Prerequisites
-
-* be able to use simple templated code
+- Copy and move constructors
+- Copy and move assignment operators
 
 ## Instructions
 
-* look at the code and run it with callgrind
+* look at the code and run it with callgrind or time it e.g. using `/usr/bin/time ./trymove`
 * understand how inefficient it is
-* implement move semantic the easy way in NVector
-* run with callgrind and see no improvement
 * understand why and fix trymove.cpp
 * see efficiency improvements