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

Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа - C++

Восстановить пароль Регистрация
 
Elfenlide
23 / 23 / 1
Регистрация: 15.04.2012
Сообщений: 183
27.09.2012, 21:17     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #1
Есть такая программулька:
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
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
//class_sets.h
using namespace std;
class Set
{
private:
    struct SetElement {
        enum {ATOM, SET} type;
        union {
            char atom;
            Set *set;
        };
 
        //перегрузка оператора == для сравнивания данных и определения типа
        bool operator==(const Set::SetElement &other);
        bool operator==(const char &element);
        bool operator==(const Set &set);
 
        //конструкторы подкласса
        SetElement();
        SetElement(const char &element);
        SetElement(const Set &set);
        SetElement(const Set::SetElement &other);
        ~SetElement();
 
    };
 
    SetElement *data;
    int size;
    int maxSize;
 
    static void printout(ostream &stream,const Set::SetElement &element);
 
public: 
    Set();
    Set(const Set &other);
 
    Set& add(const char &element); //Добавление элемента типа char
    Set& add(const Set &set);       //Добавление элемента типа Set
 
    bool operator==(const Set &other);   //перегрузка оператора == чтобы можно было сравнивать типы ATOM and SET
 
    bool contains(const char &element);  //Проверка содержимого на наличие элементов nbgf char
    bool contains(const Set &set);       //Проверка содержимого на наличие элементов типа Set
 
    bool deepContains(const char &element); //Проверка элемента-множества на элементы типа char
    bool deepContains(const Set &set); //Проверка элемента-множества на элементы типа Set
 
    friend
        ostream& operator<<(ostream &stream, const Set &set);
 
};
 
Set::Set()
    { 
        maxSize = 50;
        data = new SetElement[maxSize];
        size = 0;
    }
Set::SetElement::SetElement()
    { /*---------------------------*/ }
 
Set::Set(const Set &other) : data(other.data)
    {/*-------------------------------------------*/}
 
//Конструктор определяющий с помощью union что поступил элемент типа char
Set::SetElement::SetElement(const char &element)
{
    type = ATOM;
    atom = element;
}
 
//Конструктор определяющий с помощью union что поступил элемент типа Set
Set::SetElement::SetElement(const Set &set)
{
    type = SET;
    this->set = new Set(set);
}
 
//Конструктор чтобы проверять тип элемента множеств и элементов- множеств
Set::SetElement::SetElement(const Set::SetElement &other)
{
    this->type = other.type;
    if (type == ATOM)
        atom = other.atom;
    else
        this->set = new Set(*(other.set));
        
}
 
//Деструктор для элементов типа Set
Set::SetElement::~SetElement()
{
    if (type == SET)
        delete set;
}
 
//Добавляем элемент типа char
Set& Set::add(const char &element)
{
    if(!contains(element)) {
        data[size] = (SetElement(element));
        size++;
    }
    return *this;
}
 
//Добавляем элемент множество
Set& Set::add(const Set &set) {
    if(!contains(set)) {
        data[size] = (SetElement(set));
        size++;
    }
    return *this;
}
 
//Проверяем на присутствие одинаковых элементов типа char
bool Set::contains(const char &element) {
    bool yes = false;
    for (int i = 0; i < size; ++i) {
        if (data[i] == element) {
            yes = true;
            break;
        }
    }
    return yes;
}
 
//Проверяем на наличие одинаковых элементов - множеств 
bool Set::contains(const Set &set) {
    bool yes = false;
    for (int i = 0; i < size; ++i) {
        if (data[i] == set) {
            yes = true;
            break;
        }
    }
    return yes;
}
 
//Проверяем элемент - множество на элементы типа char
bool Set::deepContains(const char &element)
{
    bool yes = false;
    for (int i = 0; i < size; ++i) {
        if (data[i].type == Set::SetElement::ATOM) {
            if (data[i] == element) {
                yes = true;
                break;
            }
        }
        else {
            // Если наткнулись на множество, то рекурсивно проверяем,
            // есть ли искомый внутри него.
            if (data[i].set->deepContains(element)) {
                yes = true;
                break;
            }
        }
    }
    return yes;
}
//Если множество содержит множество то проверяем его элементы
bool Set::deepContains(const Set &set)
{
    bool yes = false;
    for (int i = 0; i < size; ++i) {
        if (data[i].type == Set::SetElement::SET) {
            if ((data[i] == set) || (data[i].set->deepContains(set))) {
                yes = true;
                break;
            }
        }
    }
    return yes;
}
 
//Перегружаем оператор == чтобы можно было сравнивать различные типы
bool Set::SetElement::operator==(const Set::SetElement &other)
{
    if (this->type != other.type) {
        return false;
    }
    else {
        if (this->type == ATOM) {
            return this->atom == other.atom;
        }
        else {
            return *(this->set) == *(other.set);
        }
    }
}
 
//Перегружаем оператор == чтобы можно было сравнивать различные типы
bool Set::SetElement::operator==(const char &element)
{
    if (type != ATOM) {
        return false;
    }
    else {
        return atom == element;
    }
}
 
//Перегружаем оператор == чтобы можно было сравнивать различные типы
bool Set::SetElement::operator==(const Set &set)
{
    if (type != SET) {
        return false;
    }
    else {
        return *(this->set) == set;
    }
}
 
bool Set::operator==(const Set &other)
{
    if (this->size != other.size) {
        return false;
    }
    else {
        bool okay = true;
        for (int i = 0; i < size; ++i) {
            if (other.data[i].type == Set::SetElement::ATOM) {
                if (!contains(other.data[i].atom)) {
                    okay = false;
                    break;
                }
            }
            else {
                if (!contains(*(other.data[i].set))) {
                    okay = false;
                    break;
                }
            }
        }
        return okay;
    }
}
 
//Перегрузка оператора для вывода 
ostream& operator<<(ostream &stream, const Set &set)
{
    bool first = true;
    stream << "{";
    for (int i = 0; i < set.size; ++i) {
        if (!first) {
            stream << ", ";
        }
        Set::printout(stream, set.data[i]);
        first = false;
    }
    stream << "}";
    return stream;
}
 
void Set::printout(ostream &stream, const Set::SetElement &element)
{
    if (element.type == Set::SetElement::ATOM) {
        stream << element.atom;
    }
    else {
        stream << *(element.set);
    }
}
// union_reload.cpp
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
// union_reload.cpp: определяет точку входа для консольного приложения.
#include "stdafx.h"
#include <vector>
#include <iostream>
#include "class_sets.h"
using namespace std;
 
//Прототипы
void addElements(Set *sets, int &setNumber);
 
int _tmain(int argc, _TCHAR* argv[])
{
    setlocale(0,"");
    static int k = 1;
    int choice;
    Set *sets[4];
    do 
    {
        cout<<"\n1.Добавить множества"<<endl
            <<"2.Добавить элемент в множество."<<endl
            <<"0.Выход"<<endl;
        cin>>choice;
        switch (choice)
        {
            
        case 0: exit(1);
        case 1: {
            char choice1;
                    do 
                    {
                        if (k < 5)
                        {
                            sets[k] = new Set;
                            system("cls");
                            cout << "Количество множеств: " <<k<<endl; k++;
                            cout << "Желаете ли вы добавить ещё одно множество?(Y\\N)"; 
                            cin >> choice1;
                        }
                        else {
                            system("cls");
                            cout << "Хватить делать велосипед, создай vector и будь счастлив!" << endl; break; 
                        }
                    } while (choice1 == 'y' || choice == 'Y');
                break;
                }
        case 2: 
            {
                int set_choice;
                system("cls");
                    cout << "Выбирите множество от 1 до "<<k - 1<<" в которое хотите добавить элемент: ";
                cin >> set_choice;
                addElements(sets[set_choice],set_choice);
                break;
            }
        }
    } while (choice != 0);
    return 0;
}
 
void addElements(Set *sets,int &setNumber)
{
    char elem;
    do
    { 
        cout << "Введите элементы множества (Enter - выход): ";
        cin.sync();
        cin.get(elem);
        if (elem == '\n') {
            cin.sync(); 
            break;
        }
        sets[setNumber].add(elem);
        while (cin.get() != '\n');
    } while (true);
 
}
Проблема:
Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа, через отладчик прогонял он мне сказал "попытка получить доступ к неразрешённому объекту" что-то типо того....
Перед тем как вынести добавление в отдельную функцию, я это в самом свитч реализовывал циклически, и всё работало нормально. Но когда решил выпендриться то появились проблемы)....
Помогите пожалуйста. И просьба объяснить то место где я допустил ошибку подробнее.
Спасибо!
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
27.09.2012, 21:17     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа
Посмотрите здесь:

я ввожу любое число он его конвертирвует в бинарное C++
C++ программа для работы с бинарным деревом, реализующую функции: добавить элемент, удалить элемент, вывести в виде «дерева».В C++
C++ Закрываеться программа
C++ Программа закрываеться при запуске(не успеваешь прочитать сообщение об ошибке)
запустив прогу я ввожу число нажимаю энтер и прога закрывается C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
I.M.
 Аватар для I.M.
564 / 547 / 5
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2012, 21:44     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #2
Set *sets[4]; - этой строкой вы что хотели сделать?

static int k = 1; - почему статик? зачем это вам?

if (k < 5) - а если 4? у вас не вылетает при таком числе?

int &setNumber - зачем ссылку передаете?

P.S. вопросы могут не относиться к дефекту)
Elfenlide
23 / 23 / 1
Регистрация: 15.04.2012
Сообщений: 183
27.09.2012, 22:03  [ТС]     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #3
Цитата Сообщение от I.M. Посмотреть сообщение
Set *sets[4]; - этой строкой вы что хотели сделать?

static int k = 1; - почему статик? зачем это вам?

if (k < 5) - а если 4? у вас не вылетает при таком числе?

int &setNumber - зачем ссылку передаете?

P.S. вопросы могут не относиться к дефекту)
Я создают массив множеств, тоесть 4 множества, нет не вылетает, 1,2,3,4.....4<5///

ссылку передаю потому что не вижу смысла передавать копию.....память тратить зря не надо...
а сама причина, это для того что бы определённому множеству элементы вводились, это переменная отвечает за номер множества для которого будет вызываться метод add
I.M.
 Аватар для I.M.
564 / 547 / 5
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2012, 22:14     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #4
В с/с++ нумерация с нуля идет. Да, у вас не вылетает. Вы просто память портите.

Измените 73 строку на sets->add(elem);

Добавлено через 47 секунд
Цитата Сообщение от Elfenlide Посмотреть сообщение
ссылку передаю потому что не вижу смысла передавать копию.....память тратить зря не надо...
Размер int& == размеру int. Никакой экономии нет.

Цитата Сообщение от I.M. Посмотреть сообщение
static int k = 1; - почему статик? зачем это вам?
Elfenlide
23 / 23 / 1
Регистрация: 15.04.2012
Сообщений: 183
27.09.2012, 22:22  [ТС]     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #5
Цитата Сообщение от I.M. Посмотреть сообщение
В с/с++ нумерация с нуля идет. Да, у вас не вылетает. Вы просто память портите.

Измените 73 строку на sets->add(elem);

Добавлено через 47 секунд

Размер int& == размеру int. Никакой экономии нет.
спасибо, не знал, учту.

статик сделал случайно....точнее копировал из предыдущей копии программы, а там нужно было так сделать...спасибо.
73 строка у меня кстати while (cin.get() != '\n');
если вы умели ввиду 72: сделал как вы сказали, теперь нормально, но куда теперь передаёться элемент? ведь теперь sets не знает в какое множество элемент пихать\
I.M.
 Аватар для I.M.
564 / 547 / 5
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2012, 22:29     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #6
Странно. У меня это прекрасно работает. VS2010.
А если так - (*sets).add(elem);

Номер строки называл исходя из того кода, который вы выложили. Там ее номер был 73.
Elfenlide
23 / 23 / 1
Регистрация: 15.04.2012
Сообщений: 183
27.09.2012, 22:29  [ТС]     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #7
Всё, догнал)

Не по теме:

лашара\\\\\

alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
27.09.2012, 22:29     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #8
У вас со скобками в этом месте что?
C++
1
2
3
4
5
6
7
8
9
class Set
{
private:
    struct SetElement {
        enum {ATOM, SET} type;
        union {
            char atom;
            Set *set;
        };
I.M.
 Аватар для I.M.
564 / 547 / 5
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2012, 22:32     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #9
alsav22, закрывающая скобка там дальше вроде. Во всяком случае ошибок компиляции нет.
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
27.09.2012, 22:36     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #10
Удалил. Не разобрлся.
Elfenlide
23 / 23 / 1
Регистрация: 15.04.2012
Сообщений: 183
27.09.2012, 22:39  [ТС]     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #11
Цитата Сообщение от alsav22 Посмотреть сообщение
Ошибок то нет, но что получается? Вот это всё, получается, структура SetElement:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
private:
    struct SetElement 
    {
        enum {ATOM, SET} type;
        union 
    {
            char atom;
            Set *set;
    };
 
        //перегрузка оператора == для сравнивания данных и определения типа
        bool operator==(const Set::SetElement &other);
        bool operator==(const char &element);
        bool operator==(const Set &set);
 
        //конструкторы подкласса
        SetElement();
        SetElement(const char &element);
        SetElement(const Set &set);
        SetElement(const Set::SetElement &other);
        ~SetElement();
 
    };
Да это всё структура, а она находиться ещё в классе, а там ещё и массив элементов класса....)....Спасибо всем за помощь!

Не по теме:

тупанул


Не по теме:

и всё это лежит в яйце....вот она жизнь кащея...

alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
27.09.2012, 22:48     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #12
Это я поспешил, не разобрался.
I.M.
 Аватар для I.M.
564 / 547 / 5
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2012, 22:55     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #13
Elfenlide, вам бы не мешало разнести реализацию и описание по разным файлам.

using namespace std; - это лучше в хедерах не писать. Иначе при подключении такого хедера, это будет распространяться на весь код.

exit(1); - это тоже иногда не рекомендуют писать. Правда в вашем случае будет все ок. В любом случае, return сделает то же самое - тоже выйдет из программы
Elfenlide
23 / 23 / 1
Регистрация: 15.04.2012
Сообщений: 183
27.09.2012, 23:13  [ТС]     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #14
Цитата Сообщение от I.M. Посмотреть сообщение
Elfenlide, вам бы не мешало разнести реализацию и описание по разным файлам.

using namespace std; - это лучше в хедерах не писать. Иначе при подключении такого хедера, это будет распространяться на весь код.

exit(1); - это тоже иногда не рекомендуют писать. Правда в вашем случае будет все ок. В любом случае, return сделает то же самое - тоже выйдет из программы
Я только учусь....как по разным файлам правельно разбивать пока не знаю...вот научился кидать в h и в cpp, как-то пробовал разбить ещё на кусочки, но не вышло....
Скажите а почему многие я заметил пишут всегда std:: и почему не стоит писать using namespace std; ?
это где-то потом может помешать?
вместо exit(1) просто писать return 0; break; ? почему лучше не писать exit ?

Не по теме:

я просто ещё совсем новичёк\

I.M.
 Аватар для I.M.
564 / 547 / 5
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2012, 23:22     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #15
Про exit можно почитать тут - Прервать работу програмы
using namespace std; не стоит писать только в .h файлах. В .cpp можно. Некоторые пишут std:: просто потому, что так привыкли
Elfenlide
23 / 23 / 1
Регистрация: 15.04.2012
Сообщений: 183
27.09.2012, 23:25  [ТС]     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #16
Цитата Сообщение от I.M. Посмотреть сообщение
Про exit можно почитать тут - Прервать работу програмы
using namespace std; не стоит писать только в .h файлах. В .cpp можно. Некоторые пишут std:: просто потому, что так привыкли
Понятно спасибо!
Может ещё подскажете:
что делать с выводом...вообщем я закоментил весь свитч на время чтобы остался кусок и дописал коеч-то:
C++
1
2
3
4
5
6
7
8
setlocale(0,"");
        setlocale(0,"");
    int setSize = 0;
    int choice;
    Set *sets[4];
    sets[0] = new Set;
    sets[0]->add('x');
    cout<<sets[0];
Тут выходит что элемент добавляется, ну это и в свитч было реализовано, а вот выводить не хочет...выводит какой-то код...походу шестнадцатиричный..вместо символа...
у меня там был перегружен оператор << для вывода.
Если сделать Set sets[4], то это работает,правдо надо заменить "->" на "."
А вот с указателями не хочет..желательно с пояснением...спасибо
I.M.
 Аватар для I.M.
564 / 547 / 5
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2012, 23:45     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #17
Elfenlide, вы опять путаетесь в указателях.
cout<<*sets[0];

Если подробнее:
Set* sets[4] равносильно (ну почти) Set** sets. Т.е. это некий двумерный массив. Матрица, если хотите.
следовательно sets[0] - это указатель на строку (или столбец, как больше нравится) этой матрицы.
Естественно если попытаться вывести на экран эту строку, то выведется просто адрес. Чтобы вывести элемент надо разыменовать указатель

Я бы посоветовал побольше почитать про указатели. В чем разница между указателем и объектом и тд.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
27.09.2012, 23:54     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа
Еще ссылки по теме:

C++ Хеширование: добавить элемент, найти его и удалить
Не читает файл в папке дебаг. Уже везде его пихал, а программа не работает C++
Не могу найти ошибку в коде, когда ввожу то переменная принимаєт только 1 символ. ето програма телефонная книг C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
Elfenlide
23 / 23 / 1
Регистрация: 15.04.2012
Сообщений: 183
27.09.2012, 23:54  [ТС]     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа #18
Цитата Сообщение от I.M. Посмотреть сообщение
Elfenlide, вы опять путаетесь в указателях.
cout<<*sets[0];

Если подробнее:
Set* sets[4] равносильно (ну почти) Set** sets. Т.е. это некий двумерный массив. Матрица, если хотите.
следовательно sets[0] - это указатель на строку (или столбец, как больше нравится) этой матрицы.
Естественно если попытаться вывести на экран эту строку, то выведется просто адрес. Чтобы вывести элемент надо разыменовать указатель

Я бы посоветовал побольше почитать про указатели. В чем разница между указателем и объектом и тд.
Спасибо вам огромное за помощь!) Да, у меня с указателями проблемы....мне ссылки как-то роднее...с ними проще...но пока времени указатели проработать нормально нету, поэтому учусь по ходу дела на ошибках...
Yandex
Объявления
27.09.2012, 23:54     Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа
Ответ Создать тему
Опции темы

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