Opened 3 years ago

Closed 3 years ago

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

WW #3

Reported by: Terova Valeriia Owned by: Святослав Власов
Component: WW_mergesort Version: 2.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 3 years ago by Святослав Власов

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

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

Стиль: 1/3

  1. Почему main.c не зависит от mergesort.h?
  2. В заголовочном файле mergesort.h должно быть определение только тех функций, которые объявлены в одном файле, а используются в другом. Это только функция mergesort, все остальные объявления там лишние.
  3. Длинные строки (>80 символов) нужно разбить.
  4. Зачем ты перед сортировкой половинок массива копируешь их в отдельные массивы а потом копируешь обратно? Можно ведь сортировать прямо на месте. Единственное место, где нужно выделять память -- это для буфера в функции merge.
  5. Инклюд-гарды нужны только внутри самого заголовочного файла, вокруг включения этого заголовочного файла гарды не нужны.
  6. Зачем ты объявляешь переменную comparator в функции main и потом никак её не используешь?
  7. Выделять отдельный массив под указатели на строки излишне, можно ведь сами argv передать в mergesort.
Last edited 3 years ago by Святослав Власов (previous) (diff)

comment:2 Changed 3 years ago by Terova Valeriia

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

Я все исправила, надеюсь, что по дороге ничего не сломала :(

comment:3 Changed 3 years ago by Святослав Власов

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

10/10

Note: See TracTickets for help on using tickets.