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
comment:2 Changed 5 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Круто. 9/10
Корректность: 7/7
- В принципе, лучше было бы воспользоваться
memcpy
, а не писать его руками с копипастом. Но это можно не исправлять.
Стиль: 2/3
- Табы с пробелами смешались. Более того, в одном файле табы, в другом преимущественно пробелы, но табы тоже есть.
- Давай называть переменные без транслита, пожалуйста.
comment:3 Changed 5 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
comment:4 Changed 5 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Зачтено, +1 балл за обработку аллокации.
Замечание от меня лично. Гораздо круче было бы один раз выделить дополнительную память на n элементов, а потом каждый раз её переиспользовать: у тебя слишком много аллокаций. Ещё очень круто было бы написать нерекурсивную версию. Но никаких требований на это нет. Я просто рассказываю, как можно было бы сделать ещё лучше.
Note: See
TracTickets for help on using
tickets.
сделал