Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#3 closed ожидается проверка (задача сдана)

ha1 second milestone

Reported by: Sergei Shulman Owned by: Vladimir Rutsky
Priority: проверка Milestone:
Component: HA#1 matrices Version: 1.0
Keywords: Cc:

Description

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

Посмотрите, пожалуйста, первое домашнее задание.
http://trac.compscicenter.ru/svn/cpp16/shulman.sergej/ha1/

Жду Ваших замечаний.

С уважением,
Сергей

Change History (4)

comment:1 Changed 7 years ago by Vladimir Rutsky

Milestone: ha1-milestone1ha1-milestone2
Type: ожидается проверкаожидаются исправления

Здравствуйте, Сергей!

Замечания:

  1. Вместо приведения всех аргументов argv к std::string будет лучше привести Си-строки к std::string непосредственно там, где это необходимо (в вашем случае в двух местах сравнения аргументов с "--add" и "--mult").
  1. Не пишите void, если функция не принимает аргументов:
//size_t n_rows (void) const
size_t n_rows () const
  1. Используйте только ваш operator() для доступа к элементам матрицы:
//matrix_[i_row * n_columns_ + i_col] = other(i_row, i_col);
operator()(i_row, i_col) = other(i_row, i_col);
// или
(*this)(i_row, i_col) = other(i_row, i_col);

comment:2 in reply to:  1 Changed 7 years ago by Sergei Shulman

Summary: ha1 first milestoneha1 second milestone
Type: ожидаются исправленияожидается проверка

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

Всё исправлено.

Исправление 1 замечания также повлекло за собой ряд других изменений. Уменьшилась работа с динамической памятью. Сохранена ровно 1 переменная типа std::string, используемая в обоих сравнениях; изменена сигнатура функции ReadMatrix: теперь она использует Си-строки; подключение библиотеки <string> перенесено в main.cpp, поскольку теперь она больше нигде кроме него не используется.

Жду результатов второй проверки.

comment:3 Changed 7 years ago by Vladimir Rutsky

Resolution: задача сдана
Status: newclosed

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

comment:4 Changed 7 years ago by Vladimir Rutsky

Milestone: ha1-milestone2

Milestone ha1-milestone2 deleted

Note: See TracTickets for help on using tickets.