Opened 6 years ago
Closed 6 years ago
#204 closed ожидаются исправления (задача сдана)
ha1
Reported by: | Alina Filimokhina | Owned by: | rutsky,grabovoy.philipp |
---|---|---|---|
Priority: | проверка | Milestone: | ha1-deadline |
Component: | HA#1 matrices | Version: | 1.0 |
Keywords: | Cc: |
Description
Проверьти, пожалуйста, мое домашнее задание.
Change History (3)
comment:1 Changed 6 years ago by
Milestone: | ha1-milestone1 → ha1-milestone2 |
---|
comment:2 Changed 6 years ago by
Milestone: | ha1-milestone2 → ha1-deadline |
---|---|
Type: | ожидается проверка → ожидаются исправления |
comment:3 Changed 6 years ago by
Resolution: | → задача сдана |
---|---|
Status: | new → closed |
Замечания:
- Из предыдущей проверки замечание про константность неизменяемых аргументов не исправлено до конца. Например, в
int make(char *file, Matrix &out)
стоит использоватьint make(char const *file, Matrix &out)
, т.к. буфер с именем файла не должен меняться внутриmake
.
Решение зачтено, но с минусом.
Note: See
TracTickets for help on using
tickets.
Замечания:
bool check_file(char *file);
вы передаёте буфер char (строку), который не планируете модифицировать - передавайте его константным:bool check_file(char const *file);
for (int i = 0; i < n; i++) {
matrices.hpp
отсутствует страж включения.matrices.hpp
вы используете типsize_t
, который определён в<cstddef>
, но последний вы не включаете (тем самым заголовочный файл у вас не самодостаточен).matrices.hpp
вmatrices.cpp
--- это хороший способ частично проверить, что заголовочный файл соответствует реализации.new[]
, должны быть освобождены с помощьюdelete[]
. Даже в случае возникновения ошибочных ситуаций (например, несоответствующий размер матриц), вся выделенная память должна быть аккуратно освобождена.check_operator()
используйте сравнение строк:1
,2
,3
вcheck_operator
лучше использоватьenum
. Но если вы исправите предыдущее замечание, то можно будет убрать функциюcheck_operator
и проверять текущую операцию на равенство со строками прямо в кодеmain()
.