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 Vasily Alferov

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

Надо сделать. Номер версии можно не исправлять.

comment:2 Changed 5 years ago by Екатерина Иценко

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

comment:3 Changed 4 years ago by Vasily Alferov

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

Круто, зачтено, 10/10.

У меня есть два глобальных замечания, которые нужно учитывать при написании будущего кода. Не только и не столько в домашках, а скорее по жизни.

  1. if (argc > 2) {
      // содержательный код
    } else
      return 0;
    

Сразу два замечания. Во-первых, else не нужен, можно просто в конце return 0. Во-вторых, глобальный if на весь main увеличивает уровень вложенности и смотрится кривовато. Лучше вообще так:

if (argc <= 2) {
  return 0;
}

// содержательный код
  1. Было бы очень круто, если бы вместо аллокации на каждый вызов mergesort была одна аллокация на самом верхнем уровне, а потом буффер просто переиспользовался.
Note: See TracTickets for help on using tickets.