Opened 5 years ago
Closed 4 years ago
#306 closed ожидается проверка (задача сдана)
WW #4
Reported by: | Екатерина Иценко | Owned by: | Vasily Alferov |
---|---|---|---|
Component: | WW_mergesort | Version: | 2.0 |
Keywords: | Cc: |
Description
Change History (3)
comment:1 Changed 5 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 5 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
comment:3 Changed 4 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Круто, зачтено, 10/10.
У меня есть два глобальных замечания, которые нужно учитывать при написании будущего кода. Не только и не столько в домашках, а скорее по жизни.
-
if (argc > 2) { // содержательный код } else return 0;
Сразу два замечания. Во-первых,
else
не нужен, можно просто в концеreturn 0
. Во-вторых, глобальныйif
на весьmain
увеличивает уровень вложенности и смотрится кривовато. Лучше вообще так:
if (argc <= 2) { return 0; } // содержательный код
- Было бы очень круто, если бы вместо аллокации на каждый вызов mergesort была одна аллокация на самом верхнем уровне, а потом буффер просто переиспользовался.
Note: See
TracTickets for help on using
tickets.
Надо сделать. Номер версии можно не исправлять.