Skip to content

HW4 CMake for Optimal sort#14

Open
DolzhenkoAlexa wants to merge 16 commits intomainfrom
HW4_OptimalSort_Dolzhenko
Open

HW4 CMake for Optimal sort#14
DolzhenkoAlexa wants to merge 16 commits intomainfrom
HW4_OptimalSort_Dolzhenko

Conversation

@DolzhenkoAlexa
Copy link
Owner

@DolzhenkoAlexa DolzhenkoAlexa commented Dec 20, 2025

Исправлены поправки преподавателя для дз Оптимальная сортировка
(в скобках прошлые комментарии преподавателя)

  1. Добавлен заголовочный файл (Нет заголовочника; зачем-то extern)
  2. Добавлен возврат переменной movedCount, которая показывает сколько элементов изменило свою позицию (Не выполнено "Кодом возврата приложения является количество элементов, участвовавших в сортировке и изменивших свою позицию.")
  3. Была убрана лишняя переменная (Зачем-то продублировано i и count)
  4. Инструкция перенесена в README.md (Инструкцию лучше в readme)

UPD: Изменена логика возвращения количества элементов (возвращало колво перестановок, что не верно), теперь сравнивает старый и новый массив
CMake починен

Comment on lines 7 to 8
src/hw4_optimalSort/main.c
src/hw4_optimalSort/sort.c

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это не будет работать. Пути в CMake указываются относительно CMakeLists.txt. Кстати, к остальным задачам по CMake это тоже относится.

Comment on lines 12 to 16
add_executable(hw4_optimalSort
main.c
sort.c
sort.s
)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Так это работать не будет. Вы вообще запускали программу перед тем как сдавать?


int moved = sortArray(array, count);

return moved;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Где-то у Вас ошибка. На вводе 1 8 2 15 получается 1, а должно быть 2.

Comment on lines 6 to 16
add_custom_command(
OUTPUT sort.s
COMMAND gcc -S -O2 sort.c -o sort.s
DEPENDS sort.c
)

add_executable(hw4_optimalSort
main.c
sort.c

)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Так тоже не сработает. Сейчас у Вас получается две цели: отдельно sort.s и отдельно hw4_optimalSort, который ассемблерный файл никак не использует. Надо либо включить поддержку ассемблера в CMake, либо прописать везде custom_command. Либо можно сделать обычный CMake, а инструкцию по сборке с использованием ассемблерного файла положить отдельно.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправила CMake (оставила только сборку без ассемблера) и написала 2 инструкции запуска в READme. Это то, что вы имели в виду? Или нет?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants