Kr2.1 #25
Kr2.1 #25
Conversation
| @@ -0,0 +1,48 @@ | |||
| #include <stdlib.h> | |||
There was a problem hiding this comment.
Общие замечания:
- Комментарии к коммитам ужасны
- Надо проектные файлы. Тут вообще исходники раскиданы по папкам, собирать вручную — очень такое себе развлечение.
| char *copyString(char const * const string) | ||
| { | ||
| const size_t len = strlen(string); | ||
| char *copy = (char *)malloc(len * sizeof(char)); | ||
| if (copy != NULL) | ||
| { | ||
| strcpy(copy, string); | ||
| } | ||
| return copy; | ||
| } |
| #include <string.h> | ||
| #include <stdio.h> | ||
|
|
||
| char *getString(size_t * const len, FILE * file, char const endOfLine); |
There was a problem hiding this comment.
Надо комментарии к функциям из заголовочного файла
| switch (edge->firstNodeNumber == 0) | ||
| { | ||
| case false: | ||
| { | ||
| edge->firstNodeNumber = nodeNumber; | ||
| break; | ||
| } | ||
|
|
||
| case true: | ||
| { | ||
| edge->secondNodeNumber = nodeNumber; | ||
| break; | ||
| } | ||
|
|
||
| default: | ||
| break; | ||
| } |
There was a problem hiding this comment.
Лол.
| switch (edge->firstNodeNumber == 0) | |
| { | |
| case false: | |
| { | |
| edge->firstNodeNumber = nodeNumber; | |
| break; | |
| } | |
| case true: | |
| { | |
| edge->secondNodeNumber = nodeNumber; | |
| break; | |
| } | |
| default: | |
| break; | |
| } | |
| if (edge->firstNodeNumber == 0) | |
| { | |
| edge->secondNodeNumber = nodeNumber; | |
| } | |
| else | |
| { | |
| edge->firstNodeNumber = nodeNumber; | |
| } |
А то default для булевого выражения выглядит очень мило, особенно с break, который ничего не делает.
| List ** componentsTable = createComponentsTable; | ||
| if (componentsTable == NULL) | ||
| { | ||
|
|
| return memoryErrorFile; | ||
| } | ||
|
|
||
|
|
There was a problem hiding this comment.
Две пустые строки подряд --- нехорошо
|
|
||
| void printComponentsTable(Graph *graph, const size_t nodeAmount) | ||
| { | ||
| List ** componentsTable = createComponentsTable; |
There was a problem hiding this comment.
Это не скомпилится, аргументы не передаются :)
| Graph *createGraph(const size_t nodeAmount, const size_t edgeAmount); | ||
| void connectNode(Graph *graph, size_t index, size_t nodeNumber); | ||
| void printComponentsTable(Graph *graph, const size_t nodeAmount); | ||
| void deleteGraph(Graph * const graph); |
There was a problem hiding this comment.
Лучше так:
| void deleteGraph(Graph * const graph); | |
| void deleteGraph(Graph ** const graph); |
Чтобы потом иметь возможность присвоить NULL в graph и не оставлять у вызывающего указатель на удалённую область памяти.
| int getValueByIndex(List *list, size_t index, ListErrorCode *listErrorCode); | ||
|
|
||
| //checks if value is already in the list | ||
| const bool checkValue(List *list, int value) |
There was a problem hiding this comment.
Не компилится, тут точки с запятой не хватает. Причём тут дело даже не в различиях между компиляторами, так и в gcc, и в Visual Studio, и везде компилироваться не будет
|
|
||
| ListErrorCode append(List *const list, int const value) | ||
| { | ||
| Node **currentNode = list->head; |
There was a problem hiding this comment.
Это тоже вызывает ругань компилятора, list->head имеет тип Node*, а мы его присваиваем в Node **. Типы не сходятся.
2 participants
No description provided.