Замечания по программе Vector

[alexey-malov]

oop-course/Task2/VectorBasics/src/main.cpp

Lines 10 to 18 in bd479e6

	int main()
	{
	// Get first line from std::cin
	std::string inputLine;
	std::getline(std::cin, inputLine);
	std::vector<double> vectorOfDoubles;
	// Pass line with space separated values to form a vector
	SeparatedValuesToVector(inputLine, vectorOfDoubles);

• Программа должна считывать все числа, а не только первую строку

• SeparatedValuesToVector - дать более адекватное имя

[alexey-malov]

oop-course/Task2/VectorBasics/tests/vector_process_tests.cpp

Line 26 in bd479e6

TEST_CASE("{0, 1} vector should become {0, 2}")

oop-course/Task2/VectorBasics/tests/vector_process_tests.cpp

Line 36 in bd479e6

TEST_CASE("{1, 1} vector should become {2, 2}")

oop-course/Task2/VectorBasics/tests/vector_process_tests.cpp

Line 46 in bd479e6

TEST_CASE("Vector with single value: { 1 } -> { 2 }")

oop-course/Task2/VectorBasics/tests/vector_process_tests.cpp

Line 56 in bd479e6

TEST_CASE("1 to 5 vector")

• Неудачные имена тестов. Имя теста должно помогать понять, выполнение какого требования этот тест проверяет. Перечисление частных случаев не дает такого понимания. То есть желательно, чтобы тест звучал как требование к функции.