Opened 5 years ago

Closed 4 years ago

#416 closed ожидаются исправления (задача сдана)

WW #5 kreslavskiy.kirill

Reported by: Kreslavski Kirill Owned by: Kreslavski Kirill
Component: WW_c_io Version:
Keywords: Cc:

Description

I have done all the functions for working with text format. Binary format is not supported yet

Change History (8)

comment:1 Changed 5 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Kreslavski Kirill
Summary: WW #5 → WW #5 kreslavskiy.kirill
Type: ожидается проверка → ожидаются исправления

Корректность:

  1. vaglrind.
  2. Бинарные файлы.

Стиль:

  1. Читай отзыв на предыдущую лабу.

Развивай. Удачи на сессии!

comment:2 Changed 5 years ago by Kreslavski Kirill

Owner: changed from Kreslavski Kirill to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка

Binary files are now supported. I had no errors in valgrind. I can't find your rewiev on my codestyle, so i didn't know what to improve or change. As is seems to me it is quite ok.

comment:3 Changed 5 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Kreslavski Kirill
Type: ожидается проверка → ожидаются исправления

Корректность:

  1. VALGRIND

Стиль:

  1. data_kreslavsky. Хммммм. Плохое имя.
  2. Пробелы, странные пары пустых строк.
  3. Жёсткое дублирование кода в main.

7/10.

comment:4 Changed 5 years ago by Kreslavski Kirill

Owner: changed from Kreslavski Kirill to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка

I have improved code style, but i still can't find any valgrind errors. Can you please specify when exactly do you encounter a valgrind error.

comment:5 Changed 5 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Kreslavski Kirill
Type: ожидается проверка → ожидаются исправления

Память течёт, что) Наверное, к выходу программы надо память освободить.

Пробелы вокруг управляющих конструкций.
Дублирование осталось. Ну вынеси же чтение отдельно, последующие действия отдельно.

7/10.

comment:6 Changed 5 years ago by Kreslavski Kirill

Owner: changed from Kreslavski Kirill to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка

My bad, now there should be no memory leaks + i have restyled code a bit as you asked.

comment:7 Changed 4 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Kreslavski Kirill
Type: ожидается проверка → ожидаются исправления

Пробелы ж. int main() {, if () {. И не вынес общий код. И там по дороге много конструкций в одной строчке.

8/10.

comment:8 Changed 4 years ago by Дмитрий Лапшин (lapshin)

Resolution: → задача сдана
Status: assigned → closed
Note: See TracTickets for help on using tickets.