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: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
comment:3 Changed 3 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
using namespace std;
в заголовочном файле лучше не писать. Код, который инклюдит твой заголовочник вряд ли обрадуется, что ему в глобальное пространство имен заимпортили весь std.
using namespace
допустимо использовать в .cpp
файлах. Да и то я рекомендую привыкать писать std::
руками.
Посимвольно читать строку -- не очень хорошая идея. Есть getline
который умеет читать строку до переданного символа-завершителя.
В остальном код стал сильно лучше. 9/10
comment:4 Changed 3 years ago by
Resolution: | задача сдана |
---|---|
Status: | closed → reopened |
Version: | 2.0 → 3.0 |
Я думала про getline
, но оно категорически не работало. Но я нашла, что именно не получалось, теперь работает :)
Про using namespace std;
тоже поняла!
comment:5 Changed 3 years ago by
Resolution: | → задача сдана |
---|---|
Status: | reopened → closed |
Хорошо, что поняла, 10/10
Тесты все попадали. Как минимум потому, что формат вывода не соответствует тому, что указан в задании. Проверь внимательно.
Стиль
Employee
ты сначала инициализируешь_name(name)
, затем перезатираешь_name
буфером выделенным под размерname
, ноname
в него не копируешь. Очень странно. Да и вообще, зачем тебе конструктор от имени и зарплаты, если ты потом его не используешь? Зачем ты в конструктореEmployeesArray
объявляешь переменную типа вектора, с которой не делаешь потом ничего?std::string
std::endl
лучше использовать вместо'\n'
employeesArray._employees[i]->text_out(out) << endl;
-- зачем так? Ты ведь написала оператор вывода дляEmployee