Дз 1 матрицы Лучко Сергей

[Luchko Serega]

Здравствуйте. Проверьте пожалуйста первое дз.
​http://trac.compscicenter.ru/svn/cpp16/luchko.sergej/

моя почта luga2012@…

[Vladimir Rutsky]

Здравствуйте!

Замечания:

1. Нельзя использовать exit() для нормального выхода из программы. При вызове exit() не раскручивается стек и не вызываются деструкторы, т.е. не происходит никакой (вероятно важной) деинициализации объектов.

2. В #pragma once не должно быть в конце точки с запятой:

#pragma once;

3. Используйте C++ версии C заголовочных файлов: <cstdlib> вместо <stdlib.h> (если вам вообще необходимы).

4. Ставьте отступ у содержимого класса (а вот public/private/protected обычно без отступа):

class Matrix {
    size_t rows;
    size_t cols;
    double ** data;

public:
    static double ** allocateMatrix(size_t rows, size_t cols);

    ...
}

5. Не включайте не являющиеся необходимыми заголовочные файлы. Например, <string> в matrices.hpp и matrices.cpp не нужен.

6. Не стоит явно вызывать деструктор класса:

     this->~Matrix();

учтите, что вызов деструктора у объекта дважды (а второй раз он скорее всего будет вызван, т.к. ваши объекты создаются на стеке) это undefined behavior.

Вынесите код для освобождения памяти в отдельную функцию и вызывайте её (в том числе из деструктора).

7. Не используйте альтернативное описание операторов в C++, используйте || вместо or:

     if (other.rows != rows or other.cols != cols)

Альтернативное написание операторов предполагается для систем, где сложно набирать символы, вроде |.

[Luchko Serega]

Здравствуйте. Поправил.

[Vladimir Rutsky]

Замечания:

1. Следует использовать тип size_t для индексов:

     for (int i = 1; i < rows; ++i) {
             data[i] = data[i - 1] + cols;
     }

Решение зачтено.

[Vladimir Rutsky]

Milestone ha1-milestone2 deleted