Стеки

@Aledes555 · Регистрация: 26.09.2019

Студворк — интернет-сервис помощи студентам

Здравствуйте, пожалуйста, помогите разобраться в паре ошибок.
Задание звучит так: Используя стек, решить следующую задачу. За один просмотр файла с данными и без использования дополнительных файлов напечатать элементы этого файла в обратном порядке: сначала – все числа, меньшие a, затем – все числа из отрезка [a, b], и наконец – все остальные числа. В каждой группе данные печатаются в обратном порядке.. a и b – вводятся с клавиатуры (a < b). Для реализации АТД Стек использовать массив.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
#include <iostream>
#include <fstream>
using namespace std;
#define SIZE 100
 
struct stack
{
    char mas[SIZE] = { 0 };
    int t = { 0 };
 
    stack() 
    {
         t = -1;
    }
};
 
 
int Top(stack* stack)
{
    return(stack->mas[stack->t]);
}
 
 
void push(stack* stack, int n)
{
    if (stack->t == SIZE) 
    {
        cout << "Попытка занесения значения в полный стек.";
        exit(EXIT_FAILURE);
    }
 
    else
    {
        stack->t++;
        stack->mas[stack->t] = n;
    }
}
 
 
bool empty(stack* stack)
{
    if (stack->t == -1)
        return(true);
    else
        return(false);
}
 
 
int pop(stack* stack)
{
 
    if (empty(stack)) 
    {
        cout << "стек пуст";
        exit(EXIT_FAILURE);
    }
    else
    {
        int val = 0;
        val = Top(stack);
        stack->t--;
 
        return (val);
    }
}
 
 
int main()
{
    setlocale(LC_ALL, "Rus");
 
    stack less_a, inside_ab, other;
    int n;
    int a, b;
    
    ifstream Filein("input.txt");
 
    if (Filein.is_open())
        cout << "файл input открыт\n";
 
    else
    {
        cout << "Открыть файл input не удалось\n";
        exit(EXIT_FAILURE);
    }
    cout << "Введите a (a < b):";
    cin >> a;
    cout << "Введите b (a < b):";
    cin >> b;
    
    while (!Filein.eof())
    {
        Filein >> n;
 
        if (n < a)
            push(&less_a, n);
 
        else
        {
            if (n >= a && n <= b)
                push(&inside_ab, n);
 
            else
                push(&other, n);
        }
        
    }
 
    cout << "\nЧисла, меньшие a:";
        
         while(&less_a != NULL)
        cout << " " << pop(&less_a);
 
    cout << "\nЧисла из отрезка [a, b] :";
        
        while (!empty(&inside_ab))
            cout <<" "<< pop(&inside_ab);
            
    cout << "\n Все остальные числа:";
        
        while (!empty(&other))
        cout << " " << pop(&other);
    
    Filein.close();
    return 0;
}

Комментарии от преподавателя были такие:1. Функция Top небезопасна по памяти. Возможно чтение из пустого стека.

2. Какие границы имеет массив размером 10 на Си\С++? Возможно ли чтение или запись в элемент с номером 10? Судя по Вашему коду, возможно.

Не очень понимаю, в чём заключаются ошибки.
Буду очень благодарен, если объясните суть ошибок.

Заранее спасибо

@oleg-m1973 · 29.12.2020, 13:26

Сообщение от Aledes555

2. Какие границы имеет массив размером 10 на Си\С++? Возможно ли чтение или запись в элемент с номером 10? Судя по Вашему коду, возможно.

Сообщение от Aledes555

if (stack->t == SIZE)

Чтобы это условие срабатывало, нужно, чтобы в t хранилось количество элементов в стеке (и это будет правильным), а не индекс последнего, как у тебя
Тогда уж сделай if (stack->t == SIZE - 1)

Добавлено через 3 минуты

Сообщение от Aledes555

. Функция Top небезопасна по памяти. Возможно чтение из пустого стека.

Это он херню какую-то спорол. Можно, конечно, туда добавить проверку на empty, но что тагда возвращать в этом случае?
Обычно такие фунции так и делаются, посмотри любой стандартный контейнер, никто там неичего не проверяет в аналогичных функциях front() и back(). Все проверки делаются снаружи, перед тем как их вызывать

Добавлено через 1 минуту
То же касается метода pop() - не надо внутри проверять на empty

Добавлено через 1 минуту
B push тоже - не надо внутри класса делать exit и что-то печатать. Просто верни true или false

@jurok_85 · 29.12.2020, 13:40

ну по первому пункту в функции top нужно как и в функции pop сначала проверить не пуст ли стэк

C++
1
2
3
4
5
6
7
8
9
10
11
12
int Top(stack* stack)
{
    if (empty(stack)) 
    {
        cout << "стек пуст";
        exit(EXIT_FAILURE);
    }
    else{
    return(stack->mas[stack->t]);
   }
 
}

ну а по второму пункту ваш препод придрался к функции push:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
void push(stack* stack, int n)
{
    if (stack->t == SIZE) 
    {
        cout << "Попытка занесения значения в полный стек.";
        exit(EXIT_FAILURE);
    }
 
    else
    {
        stack->t++;// допустим SIZE = 100, а до инкремента stack->t было равно 99
        stack->mas[stack->t] = n;// то здесь уже будете записывать в mas[100] 
    }
}

Новые блоги и статьи Все статьи Все блоги /
сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .
Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .	Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и источниками (напряжения, ЭДС и тока). Найти токи и напряжения во всех элементах. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и. . .