Opened 5 years ago

Closed 4 years ago

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

WW#4 samoylov.viktor

Reported by: samoylov.viktor Owned by: samoylov.viktor
Component: WW_mergesort Version:
Keywords: Cc:

Description


Change History (6)

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

Owner: changed from Дмитрий Лапшин (lapshin) to samoylov.viktor
Summary: WW#4 → WW#4 samoylov.viktor
Type: ожидается проверка → ожидаются исправления
Version: 1.0

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

  1. Твой стек переписали, твои караваны забрали, твои магические ограничения на длину строчек волшебны. Подумай, что строчки могут быть сииииииииильно длиннее.
  2. И массивы на стеке неизвестного размера такое себе.

Стиль:

  1. Вот всем, кто пользуется mergesort, нужен string.h? И malloc.h (это не стандарт лол)?
  2. Названия переменных можно лучше.
  3. Цикл копирования дублируется. Просится в фукнцию.
  4. Пробелы!

5/10.

comment:2 Changed 5 years ago by samoylov.viktor

Owner: changed from samoylov.viktor to Дмитрий Лапшин (lapshin)

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

Owner: changed from Дмитрий Лапшин (lapshin) to samoylov.viktor

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

  1. При работе со строчками утечка памяти.

Стиль:

  1. Компараторы теряют const.

8/10.

comment:4 Changed 5 years ago by samoylov.viktor

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

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

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

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

Стиль:

  1. Компараторы. Теряют. const.

9/10.

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

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