Opened 3 years ago
Closed 3 years ago
#75 closed ожидается проверка (задача сдана)
WW_mergesort (lab_03) [volkov.ivan]
Reported by: | Ivan Volkov | Owned by: | Дмитрий Лапшин (lapshin) |
---|---|---|---|
Component: | WW_mergesort | Version: | |
Keywords: | Cc: |
Description
А можно вопрос: где в mergesort.c
лучше переходить от void*
к char*
- как можно выше (везде использовать char *
кроме самой mergesort
) или как можно ниже (везде использовать void*
кроме самых глубоких по вложенности операций типа cpy
)?
Change History (6)
comment:1 Changed 3 years ago by
Version: | → 1.0 |
---|
comment:2 Changed 3 years ago by
Version: | 1.0 |
---|
comment:3 Changed 3 years ago by
Owner: | changed from Дмитрий Лапшин (lapshin) to Ivan Volkov |
---|---|
Type: | ожидается проверка → ожидаются исправления |
comment:4 Changed 3 years ago by
Owner: | changed from Ivan Volkov to Дмитрий Лапшин (lapshin) |
---|
Вроде все (кроме большого количества вызовов malloc
) пофиксил, еще постарался избавиться от дублирования кода в main.c
; не уверен, что получилось лучше.
comment:5 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
Note: See
TracTickets for help on using
tickets.
Наружу, конечному пользователю, надо выставить
void *
. Дальше твоя свобода — ты прав, что удобнее для наших операций использоватьchar *
. Но это уже пространство для выражения мыслей, и пока там нет неверных идей, можно выбирать.Нарушаешь формат вывода.
Утечка памяти в
solve_XXX
при ошибке внутриmergesort
.Выделяешь память многовато раз, но это скорее примечание, ничего не стоящее в баллах.
Вижу в
merge
циклы копирования памяти по одному элементу.7/10.