Opened 5 years ago

Closed 5 years ago

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

WW #4

Reported by: shirokov.kirill Owned by: Дмитрий Свиридкин
Component: WW_mergesort Version: 2.0
Keywords: Cc:

Description


Change History (4)

comment:1 Changed 5 years ago by Дмитрий Свиридкин

Owner: changed from Дмитрий Свиридкин to shirokov.kirill

Тесты пройдены

  1. Именование функции нужно все-таки брать как указано в лабораторной
  1. В заголовке нужно подключить что-то, где объявлен size_t. stddef.h, например
  1. Разбейте main на три функции
  1. Не используйте variable-length arrays. Честно выделяйте память
  1. Идея выделять при слиянии пямять только для половины -- неплохая.
  1. Слияние лучше вынести в отдельную функцию.
  1. Можно использовать memcpy
  1. Расстояние между объявлением переменной и ее первым использованием следует делать как можно меньшим. Используйте C11.
  1. У индексных переменных у вас не самое тривиальное назначение, подумате об их переименовании. И о вспомогательных переменных для частей массива.

8

comment:2 Changed 5 years ago by Дмитрий Свиридкин

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

comment:3 Changed 5 years ago by shirokov.kirill

Owner: changed from shirokov.kirill to Дмитрий Свиридкин
Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:4 Changed 5 years ago by Дмитрий Свиридкин

Resolution: задача сдана
Status: assignedclosed
  1. Имя функции не поменяли
  1. Ну и что это за безобразие?

src/main.c:23:14: runtime error: index 4 out of bounds for type 'char *[*]'
00-smoke-str test failed (output).

  1. VLA так и не убрали.

4.

for (int j = argc-3; j >= 0; j--) {
        printf("%s ",array[j]);
}

Давайте сортировать в заданном порядке честно
  1. Процедуру слияния также не отделили
  1. Еще и форматирование поехало.

8.2

Note: See TracTickets for help on using tickets.