Opened 3 years ago

Closed 3 years ago

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

WW_mergesort efremov.andrej

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

Description


Change History (5)

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

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

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

if () {
}
else {
}

редкий стиль, чаще делают } else {.

Формат вывода нарушаешь)

С точки зрения корректности: ты часто выделяешь память, на каждом уровне mergesort. Можно реже. Там ещё тогда одну оптимизацию можно воткнуть, но это уже на уровне финтов.

Результат вызова malloc не проверяется совсем. Плохо.

7/10.

comment:2 Changed 3 years ago by Andrey Efremov

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

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

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

Формат вывода.

Проверяй резуьтат malloc хотя бы assert-ом. Вижу бонус, но без базового правильно использования принять не могу.

8/10.

comment:4 Changed 3 years ago by Andrey Efremov

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

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

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

На будущее: обычно бонус пишется прямо в текущем решении. Я чудом заметил папочку бонус.

Там конечно всё ок, но include guards лучше бы не терять.

10 в сумме.

Note: See TracTickets for help on using tickets.