Форум программистов, компьютерный форум CyberForum.ru

Балланс скобок - C++

Восстановить пароль Регистрация
 
stark91
1 / 1 / 0
Регистрация: 07.05.2010
Сообщений: 55
05.10.2011, 22:54     Балланс скобок #1
Собственно задача заключается в том, что бы написать программу ,которая с помощью стека считывает последовательность скобок вида : (), {},[] и выводит YES или NO в случае если последовательность правильная или неправильная соответственно. Правильной считается последовательность где каждой открывающей скобки соответствует ровно одна закрывающая и наоборот.
Извлечение из пустого стека не происходит. Программа должна считывать данные из входного файла input.txt и выводить результат в выходной output.txt

Вот что у меня получилось:

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
#include <iostream>
#include <fstream>
#include <conio.h>
using namespace std;
 
struct element
{
    char bracket;
    element *p;
};
 
element *top;
element *bot;
 
ifstream in ("input.txt");
ofstream out("output.txt");
 
void add (char ch);
 
void del();
 
bool stack_empty();
 
bool proc();
 
void Work();
 
//______________MAIN FUNC_______________//
 
int main()
{
    Work();
    return 0;
}
 
//____________END MAIN_________________//
 
 
void Work()
{
 
//  _____________________________
//                               |
    element *pv = new element;// |
    pv->p = NULL;             // | Stack
    top = pv;                 // | initialisation...
//  _____________________________|  
 
    if (proc()) out << "YES!";
    else out << "NO!";
 
    in.close();
    out.close();
        
    getch();
}
 
bool proc()
{
    char ch;
        
            
    while (!in.eof())
    {
        in >> ch;
        switch (ch)
        {
            case '(': add(ch); break;
            case '{': add(ch); break;
            case '[': add(ch); break;
            case ')': if ((stack_empty()==0) && (top->bracket == '(')) {del(); break;}
                      else {return 0; break;} 
            case '}': if ((stack_empty()==0) && (top->bracket == '{')) {del(); break;} 
                      else {return 0; break;}
            case ']': if ((stack_empty()==0) && (top->bracket == '[')) {del(); break;} 
                      else {return 0; break;}
            default: break;
        }
    }
 
 
 
    if (stack_empty()) cout << "Yes!!!";
    else cout << "NO!!!";
    
    if (stack_empty()) return 1;
    else return 0;
}
 
 
void add(char ch)
{
    element *pv = new element;
    pv->bracket = ch;
    pv->p = top;
    top = pv;
}
 
void del()
{
    element *temp = top;
    top = top->p;
    delete temp;
}
 
bool stack_empty()
{
    if (top->p == NULL) return 1;
    else return 0;
}

Проблема заключается в том, что программа не вовремя выходит из цикла while (!in.eof()), то есть программа дополнительно еще раз считывает зачем-то последний символ, проверял на отладчике MSVS-10, не понимаю в чем ошибка. Помогите пожалуйста, буду признателен.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
05.10.2011, 22:54     Балланс скобок
Посмотрите здесь:

C++ Баланс скобок
Баланс скобок C++
C++ Баланс скобок
Баланс скобок C++
Выражения из скобок C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
grizlik78
Эксперт C++
 Аватар для grizlik78
1882 / 1414 / 101
Регистрация: 29.05.2011
Сообщений: 2,958
05.10.2011, 23:20     Балланс скобок #2
Вместо конструкции
C++
1
2
3
        while (!in.eof())
        {
                in >> ch;
используйте
C++
1
2
        while (in >> ch)
        {
stark91
1 / 1 / 0
Регистрация: 07.05.2010
Сообщений: 55
05.10.2011, 23:27  [ТС]     Балланс скобок #3
Огромное спасибо, все сразу встало на свои места, но все таки не понимаю в чем был минус моей конструкции, если вам не трудно разъясните пожалуйста.
grizlik78
Эксперт C++
 Аватар для grizlik78
1882 / 1414 / 101
Регистрация: 29.05.2011
Сообщений: 2,958
05.10.2011, 23:37     Балланс скобок #4
Не трудно. Флаг конца потока устанавливается не после считывания последнего элемента, а при попытке считать элемент после последнего. Поэтому проверять достижение конца потока есть смысл сразу после операции чтения, а не перед, как в исходном коде. Так что правильнее проверять успешность операции считывания (или состояние потока, как в моём коде).
stark91
1 / 1 / 0
Регистрация: 07.05.2010
Сообщений: 55
05.10.2011, 23:44  [ТС]     Балланс скобок #5
Спасибо, все понятно.
Yandex
Объявления
05.10.2011, 23:44     Балланс скобок
Ответ Создать тему
Опции темы

Текущее время: 04:45. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru