Conversation
src/sort_list.c
Outdated
There was a problem hiding this comment.
Разбейте решение на несколько файлов, добвьте отступов и отформатируйте по стайлгайду.
src/sort_list.c
Outdated
| free(p); | ||
| } | ||
| } | ||
| void del_elem(cyclic_list *elem, int *len){ |
There was a problem hiding this comment.
При удалении элемента len автоматически уменьшается, а при добавлении -- пользователь должен сам этим озаботиться. Это плохо продуманный интерфейс.
src/sort_list.c
Outdated
| cyclic_list *elem = &begin; | ||
| while (len >= m){ | ||
| if ((c + 1) % m == 0){ | ||
| if (elem->next == &begin) |
There was a problem hiding this comment.
Интерфейс циклического списка должен быть прозрачным, то есть пользователь никогда не должен отдельно обрабатывать конец или начало списка. Более того, пользователь вообще не должен знать, где у списка начало или конец.
По условию, сикарии встали в круг (без начала и конца), и нужно, чтобы с точки зрения main это выглядело так же.
src/sort_list.c
Outdated
| elem = elem->next; | ||
| } | ||
| } | ||
| printf_list(&begin); |
There was a problem hiding this comment.
Лучше здесь не использовать printf_list, а напрямую напечатать первый элемент, чтобы никого не вводить в заблуждение.
2 participants
No description provided.