ДЗ№1

[Кирилл]

"Настоятельно рекомендую не откладывать на последний день, иначе есть большая вероятность не сдать ДЗ."
Кажется, я не сдам:)

[Vladimir Rutsky]

Замечания:

1. Вы должны включать только заголовочные файлы:

#include "matrices.cpp"

а компилировать только единицы трансляции (main.cpp, huffman.cpp).

2. Ваше решение не компилируется в G++/clang (и согласно Стандарту не должно компилироваться):

In file included from main.cpp:1:
In file included from ./matrices.cpp:1:
./matrices.hpp:7:40: error: unknown type name 'ifstream'; did you mean 'std::ifstream'?
void open_file(const char* &file_name, ifstream* &fin);

вы используете имена из области видимости std явно не указывая его и не делая using namespace std;

3. Передавать указатели по константной ссылке бессмысленно --- указатели обычно реализуются через ссылки; следует использовать константный указатель (const char *).

void open_file(const char* &file_name, ifstream* &fin){

4. Я крайне не рекомендую создавать ifstream в динамической памяти. Обычно такое поведение свидетельствует о логической ошибке в построении кода. Передавайте везде либо имя файла (открывайте его непосредственно перед чтением), либо ссылку на ifstream с открытым файлом.

5. elements_ ничем не инициализируется, а в read(), вы безусловно делаете delete[] elements_.

6. Используйте Matrix вместо struct Matrix.

7. Выделение и освобождение памяти под матрицы лучше вынести в отдельные функции.

К сожалению, я не могу зачесть ваше решение с данными ошибками.

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

[Vladimir Rutsky]

Milestone ha1-deadline deleted