Conversation
| #include "task2.h" | ||
|
|
||
|
|
||
| int CompareBin(bool A[], bool B[], int n) |
There was a problem hiding this comment.
Функции по стайлгайду называются в camelCase, переменные тоже.
Также к функциям принято писать комментарии. Здесь он очень нужен, потому что трактовка возвращаемого значения нетривиальна.
|
|
||
| int CompareBin(bool A[], bool B[], int n) | ||
| { | ||
| for (int i = 0; i < n; i++) { |
There was a problem hiding this comment.
А если у чисел разная длина? Какой тогда контракт?
| int main() | ||
| { | ||
| int n; | ||
| scanf("%d", &n); | ||
|
|
||
| bool A[n], B[n]; | ||
|
|
||
| for (int i = 0; i < n; i++) | ||
| scanf("%d", (int*)&A[i]); | ||
|
|
||
| for (int i = 0; i < n; i++) | ||
| scanf("%d", (int*)&B[i]); | ||
|
|
||
| int r = CompareBin(A, B, n); | ||
|
|
||
| if (r > 0) | ||
| printf("A > B\n"); | ||
| else if | ||
| (r < 0) printf("A < B\n"); | ||
| else | ||
| printf("A = B\n"); | ||
|
|
||
| return 0; | ||
| } |
There was a problem hiding this comment.
Зачем Вам в библиотеке интерактивный main? Почему тестов не хватает?
Ещё и с глупыми ошибками типа кривого форматирования и неициализированных переменных.
| @@ -0,0 +1,4 @@ | |||
| #pragma once | |||
| #include <stdbool.h> | |||
There was a problem hiding this comment.
Если здесь уже есть stdbool.h, то в .c файле он не нужен.
| Node* curr = head; | ||
| while (curr) { | ||
| Node* next = curr->next; | ||
| curr->next = prev; |
There was a problem hiding this comment.
Здесь можно записать в нулевой указатель
| Node* reverse(Node* head) { | ||
| Node* prev = NULL; | ||
| Node* curr = head; | ||
| while (curr) { |
There was a problem hiding this comment.
Если я правильно всё посчитал, то после одного шага цикла над списком из трех элементов, ссылка на последний элемент потеряется и всё.
shannami
dismissed
WoWaster’s stale review
The merge-base changed after approval.
2 participants
Шалахина Анна б43