Opened 3 years ago

Closed 3 years ago

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

WW_mergesort kirill.kondratyuk

Reported by: kondratyuk.kirill Owned by: Дмитрий Лапшин (lapshin)
Component: WW_mergesort Version:
Keywords: Cc:

Description


Change History (3)

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

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

Неверное имя исполняемого файла.

Слегка нарушаешь формат вывода.

Результат malloc обязательно надо проверять, хотя бы assert.

Include guards?

mergesort выглядит как каша, там можно было бы пустыми строчками выделить блоки.

6/10.

comment:2 Changed 3 years ago by kondratyuk.kirill

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

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

Поправил имя исполняемого файла - теперь lab_03
Поправил формат вывода - я не заметил, что в конце нужен перевод строки, ведь так?)
Проверил результат malloc для каждого выделения и вывожу ту же фразу, что и для неудавшегося мержсорта
Настроил include guards. Я до сих пор побаиваюсь писать их не через ifndef:/
Выделил логические блоки в мержсорте пустыми строками(одной и двумя - насколько сильно отличаются)

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

Resolution: → задача сдана
Status: assigned → closed

На пустом вводе формат вывода всё ещё неверный.

Формально у тебя в main есть утечка памяти, если mergesort вернёт ошибку.

Твой стиль переноса аргументов встречается, но необычен, чаще переносят и первый аргумент:

void huge_procedure(
   int a, int b, int c,
   const char *name, const char *surname,
   even more cool stuff!
) {

размер отступов у парамертов при этом как обычно или двойной.

Но бонус сделан, давай зачту на 10.

Note: See TracTickets for help on using tickets.