Opened 3 years ago

Closed 3 years ago

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

lab_10

Reported by: Terova Valeriia Owned by: Святослав Власов
Component: WW cpp_io Version: 3.0
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 3 years ago by Святослав Власов

Type: ожидается проверкаожидаются исправления

Тесты все попадали. Как минимум потому, что формат вывода не соответствует тому, что указан в задании. Проверь внимательно.

Стиль

  1. У тебя очень много каких-то бесполезных телодвижений в коде. Зачем в конструкторе Employee ты сначала инициализируешь _name(name), затем перезатираешь _name буфером выделенным под размер name, но name в него не копируешь. Очень странно. Да и вообще, зачем тебе конструктор от имени и зарплаты, если ты потом его не используешь? Зачем ты в конструкторе EmployeesArray объявляешь переменную типа вектора, с которой не делаешь потом ничего?
  2. Лучше вместо константного буфера на 101 символ воспользоваться всё таки std::string
  3. std::endl лучше использовать вместо '\n'
  4. Нет проверок на ошибки ввода
  5. employeesArray._employees[i]->text_out(out) << endl; -- зачем так? Ты ведь написала оператор вывода для Employee
  6. Вместо тернарного оператора создающего нужного сотрудника лучше сделай фабричный метод. Это и практичнее и избавляет от копи-пасты в операторе ввода и в main

comment:2 Changed 3 years ago by Terova Valeriia

Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:3 Changed 3 years ago by Святослав Власов

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

using namespace std; в заголовочном файле лучше не писать. Код, который инклюдит твой заголовочник вряд ли обрадуется, что ему в глобальное пространство имен заимпортили весь std.
using namespace допустимо использовать в .cpp файлах. Да и то я рекомендую привыкать писать std:: руками.

Посимвольно читать строку -- не очень хорошая идея. Есть getline который умеет читать строку до переданного символа-завершителя.

В остальном код стал сильно лучше. 9/10

comment:4 Changed 3 years ago by Terova Valeriia

Resolution: задача сдана
Status: closedreopened
Version: 2.03.0

Я думала про getline, но оно категорически не работало. Но я нашла, что именно не получалось, теперь работает :)

Про using namespace std; тоже поняла!

comment:5 Changed 3 years ago by Святослав Власов

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

Хорошо, что поняла, 10/10

Note: See TracTickets for help on using tickets.