Opened 3 years ago

Closed 3 years ago

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

WW_mergesort Tulchinskiy lab_03

Reported by: tulchinskij.aleksandr Owned by: Антон Филатов
Component: WW_mergesort Version: 2.0
Keywords: Cc:

Description

Извинтие, что вместо таба 4 пробела (если будет неприятно смотреть код). Это фитча IDE, я пока не понял как фиксить.

Доп задание выполнено.

Change History (3)

comment:1 Changed 3 years ago by Антон Филатов

Type: ожидается проверкаожидаются исправления

Я тоже живу с 4 пробелами вместо таба (кроме случаев, когда я кодю в блокноте), мне так комфортнее
В целом правильно. Разве что можно быть более аккуратным с пробелами в конце строки (хорошо бы, чтобы их не было)
Корректность: 7/7
Стиль:

void* sorted_array = sorted_array = malloc((right - left) * element_size);

штаааа

if (sorted_array == NULL) {
     free(sorted_array);
     return -1;
}

Строго говоря, если sorted_array == NULL, то фри - необязателен. В задании имелось ввиду, что если сломалось что-то во время malloc, то надо удалить то, что было выделено ранее

3/3

Итог 10/10
Да, я докопался. Но проигнорировать это я не мог. Бонусное задание правильное, но стиль в нём с этими лишними free мне не нравится.
Кстати, странно, что вы на 76й в main.c строчке не делаете free(array)
Пофиксите это и бонус будет засчитан

comment:2 Changed 3 years ago by tulchinskij.aleksandr

Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:3 Changed 3 years ago by Антон Филатов

Resolution: задача сдана
Status: assignedclosed

Принимается. 11/10

Note: See TracTickets for help on using tickets.