Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pull request Туманов Степан ПМИ 2 #9

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

StepaUUka
Copy link

No description provided.

@annapirova
Copy link
Owner

Степан, добрый день!

  1. Надо адаптировать все сортировки для работы с отрицательными числами.
    При A[i] = rand() % 100 - 50; у вас не все работает.
    image
  2. Merge падает, Быстрая сортировка не сортирует.
  3. Сортировка подсчетом какая-то странная, это явно другая сортировка. Надо сделать так, как я на парах рассказывала
  4. Печать массива надо вынести в отдельную функцию
  5. Надо сделать функцию проверки отсортированности массива, вызывать после каждой сортировки.

@annapirova
Copy link
Owner

image

@annapirova
Copy link
Owner

Может, у Вас старая реализация загружена?

Исправил работу Quick Sort (Быстрой сортировки), исправил работу сортировки Двойным пузырьком, сделал везде время работы, добавил везде подсчёт количества сравнений и перестановок, но вот вывод этих количеств есть везде. кроме Quick Sort, там есть их подсчёт, но вот как вывести, не знаю. Merge Sort не работает(
@StepaUUka
Copy link
Author

Увидел ваши сообщения только сейчас( Я загрузил новую версию, но, так как увидел ваши сообщения только сейчас, то там исправлены не все ошибки

@StepaUUka
Copy link
Author

@annapirova

Код контрольной по массивам Туманов Степан ПМИ2
@annapirova
Copy link
Owner

  1. Контрольная ок.
  2. Лабораторную проверять или ждать обновлений?

Сделайте, пожалуйста, отдельную папку для своих кодов. Как у других

@StepaUUka
Copy link
Author

Проверьте, пожалуйста, всё-таки лабораторную, чтобы хотя бы была не надпись: "Всё починить не работает". А я до четверга постараюсь все ошибки исправить.

@annapirova
Copy link
Owner

Проверено, замечания в таблице

@StepaUUka
Copy link
Author

Добавил лабораторную по Тейлору @annapirova

@annapirova
Copy link
Owner

Степан, вроде все ничего. Но надо что-то сделать с русским языком, тестировать программу в таком виде затруднительно.

@StepaUUka
Copy link
Author

По Тейлору: Добрый вечер, а что с русским языком не так?
По Кассе: И ещё есть вопросик, почему может не работать функция isdigit?
Такое чувство, что у меня функция isdigit совсем не работает, так как даже "5" она не сичтает числом, в чём дело?
@annapirova

@annapirova
Copy link
Owner

annapirova commented Dec 22, 2024 via email

@StepaUUka
Copy link
Author

StepaUUka commented Dec 22, 2024

  1. Ну у меня всё хорошо было с русским языком
  2. Я написал #include <ctype.h>, но почему-то всё равно не работает, если что, то код в файле laba_box_office
    @annapirova

Готовая лаба по "Кассе"
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