Skip to content

Hw5 sorting station #4

Open
DolzhenkoAlexa wants to merge 4 commits intomainfrom
HW5_SortingStation_Dolzhenko
Open

Hw5 sorting station #4
DolzhenkoAlexa wants to merge 4 commits intomainfrom
HW5_SortingStation_Dolzhenko

Conversation

@DolzhenkoAlexa
Copy link
Owner

Добавлены файлы стэка и самого решения задачи "Сортировочная станция". Все комментарии внутри файлов и заголовочных файлов.

Долженко Александра

Choose a reason for hiding this comment

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

Надо удалить лишние файлы из PR.

// Параметр - Указатель на стек
// Вернет
//
int StackEmptiness(Stack *stack);

Choose a reason for hiding this comment

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

Функции, которые что-то проверяют обычно начинаются с Is или Does. Про это даже в стайлгайде сказано.

// Поддерживает операции: +, -, *, /, скобки () и цифры.
//

#ifndef TEST_SORTINGSTATION_H

Choose a reason for hiding this comment

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

Стайлгайд запрещает использование include guards. Вместо них надо использовать pragma once.
Да и вообще, include guards сейчас используются только в древних проектах, типа стандартной библиотеки C или ядра Linux. Сейчас все стараются переходить на pragma once.

Comment on lines +35 to +41
// Алгоритм:
// 1. Числа сразу добавляются в выходной массив
// 2. Открывающие скобки помещаются в стек
// 3. Закрывающие скобки выталкивают операторы из стека до открывающей скобки
// 4. Операторы выталкивают из стека операторы с высшим или равным приоритетом
// 5. В конце все оставшиеся операторы выталкиваются из стека
//

Choose a reason for hiding this comment

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

Описать используемый алгоритм -- очень хорошая идея, странно, что кроме Вас никто так не сделал. Ещё хорошо было бы здесь указать, что это алгоритм Сортировочной станции (или Дейкстры).

@p-senichenkov
Copy link

p-senichenkov commented Dec 8, 2025

Ваше решение очень похоже на решения Ивановой Алины и Разгуляевой Ады: https://github.com/ialina07/homework-c/pull/4/files, https://github.com/ada1ra/homework-c/pull/3/files.
Вы раньше не были замечены за списыванием, так что пока только -1 балл, но в следующий раз кара будет строгой для всех.

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