Opened 5 years ago

Closed 5 years ago

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

WW #4

Reported by: lebedev.egor Owned by: Vasily Alferov
Component: WW_mergesort Version: 2.0
Keywords: Cc:

Description


Change History (4)

comment:1 Changed 5 years ago by lebedev.egor

сделал

comment:2 Changed 5 years ago by Vasily Alferov

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

Круто. 9/10

Корректность: 7/7

  • В принципе, лучше было бы воспользоваться memcpy, а не писать его руками с копипастом. Но это можно не исправлять.

Стиль: 2/3

  • Табы с пробелами смешались. Более того, в одном файле табы, в другом преимущественно пробелы, но табы тоже есть.
  • Давай называть переменные без транслита, пожалуйста.

comment:3 Changed 5 years ago by lebedev.egor

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

comment:4 Changed 5 years ago by Vasily Alferov

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

Зачтено, +1 балл за обработку аллокации.

Замечание от меня лично. Гораздо круче было бы один раз выделить дополнительную память на n элементов, а потом каждый раз её переиспользовать: у тебя слишком много аллокаций. Ещё очень круто было бы написать нерекурсивную версию. Но никаких требований на это нет. Я просто рассказываю, как можно было бы сделать ещё лучше.

Note: See TracTickets for help on using tickets.