Most frequent element2 #9
Conversation
semester_1/homework1/substring.c
Outdated
There was a problem hiding this comment.
В PR не должно быть изменений в файлах, не связанных с задачей.
| #include <stdbool.h> | ||
| #include <stdlib.h> | ||
|
|
||
| int mostFrequentElement(int array[], int size); |
There was a problem hiding this comment.
Принято передавать массив в функцию по указателю
Размеры и индексы храним в size_t.
| void printArray(int array[], int size); | ||
|
|
||
| bool test(void); | ||
| bool testStarterForMostFrequentELement(int testArray[], int size, int answer); |
| bool testStarterForMostFrequentELement(int testArray[], int size, int answer); | ||
| bool testMostFrequentElementFunction(void); | ||
|
|
||
| int main() |
| if (!test()) | ||
| { | ||
| printf("\nSorry but the program does not work correctly\n"); | ||
| return 1; |
There was a problem hiding this comment.
коды ошибок и возврата выносим в define
| if (rightElement < leftElement) | ||
| { | ||
| return 2; | ||
| } |
| if (errorCode != 0) | ||
| { | ||
| return 1; | ||
| } | ||
| return 0; |
| smartQuickSort(array, currentSeparator + 1, rightElement); | ||
| } | ||
| return 0; | ||
| } No newline at end of file |
| int smartQuickSort(int array[], int leftElement, int rightElement); | ||
|
|
||
| // a function for testing the work of quick sort | ||
| bool testSmartQuickSort(void); No newline at end of file |
| return true; | ||
| } | ||
|
|
||
| bool testSmartQuickSortFunction(void) |
There was a problem hiding this comment.
static и все функции, которые не нужны вне модуля делаем static
semester_1/homework1/substring.c
Outdated
There was a problem hiding this comment.
убрать из PR все файлы не связанные с задачей
There was a problem hiding this comment.
тут не должно быть изменений в других файлах, не надо удалять сами файлы.
сейчас после заливки этого PR в main удалятся старые решения, такого быть не должно
научиться делать git rebase и исправить/удалить все коммиты, в которых добавлялись / удалялись эти файлы.
| return testSmartQuickSort() && testMostFrequentElement(); | ||
| } | ||
|
|
||
| ExitCode main() |
| && mostFrequentElement(testArray2, testSize2) == answer2 | ||
| && mostFrequentElement(testArray3, testSize3) == answer3; |
There was a problem hiding this comment.
отступы выровнять: все строки после первой на 4 пробела дальше, чем первая
| return memcmp(testArray1, answerArray1, testSize1) == 0 | ||
| && memcmp(testArray2, answerArray2, testSize2) == 0 | ||
| && memcmp(testArray3, answerArray3, testSize3) == 0; |
No description provided.