Opened 3 years ago

Closed 3 years ago

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

Lab03 Novozhilov Aleksandr

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

Description


Change History (4)

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

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

Я впечатлён вашими познаниями ассемблера, искренне.
Но не могу не задать вопрос: а почему вы не воспользовались стандартной функцией memcpy
Корректность: 7/7
Стиль:

  • 28 строка в main.c: (void*)&args[2] вот на этом месте просится (void*)(args + 2)
  • signed main - штааа. ни разу не видел реального использования signed вместо int. И при этом вы не делаете return в конце main
  • right = (void*)((size_t)right + elem_size); - вот так делать не надо. вы кастуете указатель (там же под void* всё же указатель прячется) к обычному типу. Си же умеет в арифметику указателей: right += elem_size же сработает (ругнётся ворнингом, но скастуйте заранее right к char*, если хотите). Я смотрю у вас это очень много где. И везде следует исправить

Стиль: 1,5/3
Итого 8,5/10
Бонус буду проверять, когда будет 10/10

comment:2 Changed 3 years ago by novozhilov.aleksandr

Так в задании сказано всю работу с памятью реализовать вручную, или я не так понял?

comment:3 Changed 3 years ago by novozhilov.aleksandr

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

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

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

Так в задании сказано всю работу с памятью реализовать вручную, или я не так понял?

действительно, вы правы. Я проглядел этот кусок задания.

Стиль:

void* right = (void*)((size_t)array + left_cnt * element_size);void* right = (void*)((size_t)array + left_cnt * element_size);

один такой кусочек всё же остался

Бонусное задание зачесть не могу, поскольку вы вернёте наружу только результат одного из merge(кстати, подумайте, какого именно: первого или последнего). А если в середине какие-то ошибки произошли, то вы их проигнорируете (и память не освободите)

9,9/10

Note: See TracTickets for help on using tickets.