Переписывание контрольной №1. Кальсина Яна#17
Open
yaprogrammer18-yanchi wants to merge 6 commits intomainfrom
Open
Переписывание контрольной №1. Кальсина Яна#17yaprogrammer18-yanchi wants to merge 6 commits intomainfrom
yaprogrammer18-yanchi wants to merge 6 commits intomainfrom
Conversation
WoWaster
approved these changes
Dec 17, 2025
Collaborator
There was a problem hiding this comment.
Офигеть! Столько функций на контрольной пугает...
Comment on lines
+29
to
+35
| /* | ||
| * функция удаления элемента по индексу | ||
| * принимает указатель на список и индекс элемента | ||
| * возвращает булевое значние, сообщающее, удачно ли прошла операция | ||
| * внутри себя освобождает память удаляемого элемента | ||
| */ | ||
| bool removeElement(List* list, int index); |
Comment on lines
+45
to
+52
| /* | ||
| * функция вывода списка в консоль | ||
| * принимает только указаетль на список | ||
| * ничего не возвращает | ||
| * печатает элементы списка через пробел | ||
| * в случае переданного пустого или несуществующего списка сообщает об этом пользователю | ||
| */ | ||
| void printList(List* list); |
Comment on lines
+70
to
+74
| * функция проверки, один ли элемент в списке | ||
| * возвращает true если да, false, если нет | ||
| * на вход принимает указатель на список | ||
| */ | ||
| bool oneElement(List* list); |
Comment on lines
+52
to
+53
| if (list == NULL || isEmpty(list) || index < 0) | ||
| return -1; |
Collaborator
There was a problem hiding this comment.
Неа, очень плохо, у Вас код ошибки --- элемент области допустимых значений
Comment on lines
+154
to
+178
| List* reverse(List* list) | ||
| { | ||
| if (list == NULL || isEmpty(list)) { | ||
| return NULL; | ||
| } | ||
|
|
||
| List* newOneList = newList(); | ||
| copy(list, newOneList); | ||
|
|
||
| if (newOneList == NULL || isEmpty(newOneList)) { | ||
| return NULL; | ||
| } | ||
| ListNode* prev = NULL; | ||
| ListNode* current = newOneList->head; | ||
| ListNode* next = NULL; | ||
|
|
||
| while (current != NULL) { | ||
| next = current->next; | ||
| current->next = prev; | ||
| prev = current; | ||
| current = next; | ||
| } | ||
| newOneList->head = prev; | ||
| return newOneList; | ||
| } |
Collaborator
There was a problem hiding this comment.
Похоже на правду, но доставая элементы из первого списка и перекладывая их во второй было бы сильно проще
Comment on lines
+192
to
+193
| deleteList(newOne1); | ||
| return false; |
Collaborator
There was a problem hiding this comment.
А здесь никакого сообщения об ошибке нет, странно
Comment on lines
+133
to
+143
| if (argc == 2 && strcmp(argv[1], "--test") == 0) { | ||
| bool allTestsPassed = tests(); | ||
| if (allTestsPassed) { | ||
| printf("Все тесты прошли успешно"); | ||
| } else { | ||
| printf("Некоторые тесты не были пройдены"); | ||
| } | ||
| printf("\n"); | ||
| return 0; | ||
| } | ||
| return 0; |
Collaborator
There was a problem hiding this comment.
Забавно, что здесь Вы решили попробовать посмотреть в аргументы, а в другой задаче нет. (в 3 задаче даже правильнее)
Comment on lines
+64
to
+71
| for (int i = 1; i < length; i++) { | ||
| if (arr[i] != 0) { | ||
| int temp = arr[0]; | ||
| arr[0] = arr[i]; | ||
| arr[i] = temp; | ||
| break; | ||
| } | ||
| } |
Collaborator
There was a problem hiding this comment.
Да блин, почему Вы решили, что нули надо использовать. Я говорил ровно обратное
| return result; | ||
| } | ||
|
|
||
| int getMinNumberFromDigits(unsigned num, int length) |
Collaborator
There was a problem hiding this comment.
Забавно, что на вход у Вас беззнаковое, а выход знаковый
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Добавила решения второй и третьей задач. Тесты все прописаны в самих исходных файлах.