Conversation
p-senichenkov
left a comment
p-senichenkov
left a comment
There was a problem hiding this comment.
Небольшие замечания по стилю.
Из критичного:
- оператор
->-- я уверен, что Вас этому учили на одном из первых занятий; - код из предыдущего задания.
6/loop_sorted_list.c
Outdated
|
|
||
| Node* head = list.head; | ||
|
|
||
| while ((*head).next != NULL) { |
There was a problem hiding this comment.
Почему Вы не пользуетесь оператором ->?
6/loop_sorted_list.c
Outdated
| @@ -0,0 +1,65 @@ | |||
| #include "sorted_list.h" | |||
|
|
|||
| void main() { | |||
There was a problem hiding this comment.
В стайлгайде сказано, что в определениях функций открывающая фигурная скобка должна быть на новой строке.
6/loop_sorted_list.c
Outdated
|
|
||
| Node* head = list.head; | ||
|
|
||
| while ((*head).next != NULL) { |
There was a problem hiding this comment.
На NULL принято проверять так: while (head->next). И стайлгайд тоже так требует.
6/loop_sorted_list.c
Outdated
| for (int i = 1; i <= n; i++) { | ||
| addValue(&list, i); | ||
| } | ||
|
|
There was a problem hiding this comment.
Отступов больше чем в одну строку внутри функции не должно быть.
6/loop_sorted_list.c
Outdated
| while (list.size > 1) { | ||
| if (num == m) { | ||
| (*prev).next = (*head).next; | ||
| //free(head); |
There was a problem hiding this comment.
Не надо оставлять закомментриванный код.
There was a problem hiding this comment.
Для этой задачи сортированный список не нужен -- достаточно обычного. И в задании не сказано использовать код из предыдущей задачи. Так что проще было бы реализовать связный список заново.
А если хотите переиспользовать предыдущее решение, то нужно растить одну ветку от другой, а не копировать код.
p-senichenkov
left a comment
p-senichenkov
left a comment
There was a problem hiding this comment.
Всё ещё нужно исправить замечания
2 participants
No description provided.