Opened 3 years ago
Closed 3 years ago
#93 closed ожидается проверка (задача сдана)
WW #3
Reported by: | predelina.anastasiya | Owned by: | Святослав Власов |
---|---|---|---|
Component: | WW_mergesort | Version: | 3.0 |
Keywords: | Cc: |
Description
Change History (5)
comment:1 Changed 3 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
Внесла поправки по замечаниям.
comment:3 Changed 3 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Стиль -- 2/3
- Не проще ли будет в swap написать цикл-for, вместо цикла-while? Да и вообще, нужно ли тебе именно менять два элемента местами?
- free(NULL) -- бессмысленная операция.
- Если в первом вызове mergesort память не выделится, то нет никакого смысла запускать вторую итерацию и merge, надо сразу выходить.
- ch1, ch2, ... -- это не самые лучшие имена для возвращаемых значений, лучше уж ret или err.
- Не нужно жалеть пустых строк, стоит расставлять их в коде для обозначения логических пауз и выделения логических блоков. У тебя их маловато.
- Функции swap и merge в заголовочном файле не нужны. Это внутренние детали реализации и во внешних файлах вызываться не будут.
В целом код стал выглядеть намного-намного лучше, молодец!
comment:4 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 2.0 → 3.0 |
Поправила
comment:5 Changed 3 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
11/11, молодец!
Note: See
TracTickets for help on using
tickets.
Корректность -- 7/7
Стиль -- 0/3
вместо этого достаточно написать
В целом код получился очень избыточным. Постарайся сделать его более опрятным и лаконичным. Убери ненужные переменные, ненужные аргументы функций, ненужные функции.
К примеру, зачем тебе нужна функция mergeSortRecursive? Единственное её отличие от исходной функции mergesort только в том, что принимает в качестве аргументов указатель на начало и конец массива вместо указателя на начало и массива и количество аргументов. Но ведь эти вещи легко конвертируются друг в друга. Можно вместо mergeSortRecursive пользоваться самой mergesort.
Касательно main.c: