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

[alexey-malov]

• [x] Имена и структура тестов должны играть роль спецификации на тестируемую функцию или класс, а не просто перечислить ситуации, в которых вы проводите испытание класса или функции.

• Vector3d

  • имеет координаты, переданные в конструктор
  • имеет унарный плюс и минус
  • позволят складывать вектора
  • ...

• [x] Вывод тестов настройте лесенкой

   vector3d tests
2>      Vector3D gets value from constructor
2>      Vector3D reutrn length vector
2>      Vector3D have unary operators plus and minus
2>      Vector3D have binary operators plus
2>      Vector3D have binary operators minus
2>      Vector3D can multiply vector and scalar
2>      Vector3D can division vector and scalar
2>      Vector3D have operation equality and inequality
2>      Vector3D can multiply vectors
2>      Vector3D have scalar multiplication vectors
2>      Vector3D have operation normalizion
2>      Vector3D can read data from stream
2>      Vector3D can output data to stream

• [x] Имя родительского теста является началом имен тестов вложенных тестов. Смотрите, как сделано в примере TVSet в моем репозитории

[alexey-malov]

• [x] В тестах исходные вектора в тест-кейсах лучше задавать внутри самих тест-кейсов, а не в fixture, т.к. в противном случае приходится постоянно обращаться к верхней части исходник чтобы узнать значения веторов-аргументов

[alexey-malov]

• [x] что за деление скаляра на вектор? Выпилить

[alexey-malov]

• [x] Оператор != реализован неправильно

Следующая проверка не проходит

    BOOST_CHECK(CVector3D(1, 0, 0) != CVector3D(1, 1, 0));

[alexey-malov]

• [x] Тесты, проверяющие вектора на равенство и неравенство должны проверять как равные, так и неравные вектора.

[alexey-malov]

• [x] проект с тестами не загружается

C:\teaching\2016\old\veschev\OOP\LAB5\VECTOR\tests\tests.vcxproj(67,5): The imported project "C:\teaching\2016\old\veschev\OOP\LAB5\VECTOR\tests\boost.props" was not found. Confirm that the path in the <Import> declaration is correct, and that the file exists on disk.

[alexey-malov]

• [x] Корявый английский. Не Vector have, а Vector Has

В английском обратный порядок слов. длина вектора - vector length, а не length vector

[alexey-malov]

• [x] Вложенные test case-ы должны быть с отступами

[alexey-malov]

    BOOST_CHECK_EQUAL(Rounding(vector.GetLength()), 15.895);

• [x] Для приблизительного сравнения чисел с плавающей запятой используйте BOOST_CHECK_CLOSE или BOOST_CHECK_CLOSE_FRACTION

[alexey-malov]

    CVector3D& operator*=(const float scalar);

    CVector3D& operator/=(const float scalar);

CVector3D const operator*(const CVector3D & vector, const float scalar);

CVector3D const operator*(const float scalar, const CVector3D & vector);

CVector3D const operator/(const CVector3D & vector, const float scalar);

• [x] Скаляр следует передавать в виде числа двойной точности

[alexey-malov]

• [x] Тесты операторов сравнения таковы, что пропускают даже такую реализацию операторов:

bool const operator==(const CVector3D & v1, const CVector3D & v2)
{
/*
    if (abs(v1.x - v2.x) > DBL_EPSILON)
    {
        return false;
    }
*/
    if (abs(v1.y - v2.y) > DBL_EPSILON)
    {
        return false;
    }
/*
    if (abs(v1.z - v2.z) > DBL_EPSILON)
    {
        return false;
    }
*/
    return true;
}

[alexey-malov]

BOOST_AUTO_TEST_CASE(Vector3D_can_read_data_from_stream)
{
    CVector3D vector2 = { 10, 2, 3 };
    std::stringbuf stringBuff("5 2 1");
    std::istream is(&stringBuff);
    is >> vector2;
    CVector3D rightVector = { 5, 2, 1 };
    BOOST_CHECK(vector2 == rightVector);
}

BOOST_AUTO_TEST_CASE(Vector3D_can_output_data_to_stream)
{
    CVector3D vector = { 5.26, 0, -15 };
    output_test_stream output;
    output << vector;
    BOOST_TEST(!output.is_empty(false));
    BOOST_TEST(output.is_equal("5.26, 0, -15"));
}

• [x] Сейчас вектор не способен считывать данные из потока в том же формате, в каком выводит их туда
• [ ] нужно проверять, что при чтении вектора из потока при валидных входных данных поток находится в хорошем состоянии, а при ошибке - в ошибочном (выставлен fail bit)

[alexey-malov]

1>c:\teaching\2016\old\veschev\oop\lab5\vector\vector\vector.cpp(16): warning C4101: 'a': unreferenced local variable
1>  Generating Code...

• [x] Программа должна компилироваться без предупреждений компилятора во всех конфигурациях и платформах.

[alexey-malov]

2>c:/teaching/2016/old/veschev/oop/lab5/vector/tests/vector3dtests.cpp(158): error : in "Vector3D/can_read_data_from_stream": check vector2 == rightVector has failed
2>      can output data to stream
2>c:/teaching/2016/old/veschev/oop/lab5/vector/tests/vector3dtests.cpp(167): error : in "Vector3D/can_output_data_to_stream": check output.is_equal("5.26, 0, -15") has failed. Output content: "5.26, 0, -15

• [x] тесты не прошли