23 / 23 / 11
Регистрация: 15.04.2012
Сообщений: 183
1

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

27.09.2012, 21:17. Показов 1628. Ответов 17
Метки нет (Все метки)

Author24 — интернет-сервис помощи студентам
Есть такая программулька:
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);
 
}
Проблема:
Когда нажимаю добавить элемент, и уже его ввожу, то вылетает ошибка и закрываеться программа, через отладчик прогонял он мне сказал "попытка получить доступ к неразрешённому объекту" что-то типо того....
Перед тем как вынести добавление в отдельную функцию, я это в самом свитч реализовывал циклически, и всё работало нормально. Но когда решил выпендриться то появились проблемы)....
Помогите пожалуйста. И просьба объяснить то место где я допустил ошибку подробнее.
Спасибо!
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
27.09.2012, 21:17
Ответы с готовыми решениями:

Ввожу текст в textarea, нажимаю Enter дальше пишу нажимаю Submit...
I vse chto ya vvel otpravlyau v FUNKCIU kotoraya poluchit etot text i v mestah gde ya najimal...

Вылетает, когда ввожу последний символ
#include &quot;stdafx.h&quot; #include &lt;stdio.h&gt; #include &lt;conio.h&gt; #include &lt;Windows.h&gt; #define N 6 ...

Когда запускаю и ввожу параметры, то, в момент когда программа должна выполнять действие, пишет ошибку
Недавно начали изучать c++. так как в школе не было программирования (преподу пофиг было) я в...

Когда нажимаю на кнопку, программа отключается...
Здрава всем. Учу QT, новинкии Я тут... питаюс скомпилировать вручную Есть вот такой код...

17
576 / 559 / 47
Регистрация: 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. вопросы могут не относиться к дефекту)
0
23 / 23 / 11
Регистрация: 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
0
576 / 559 / 47
Регистрация: 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; - почему статик? зачем это вам?
1
23 / 23 / 11
Регистрация: 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 не знает в какое множество элемент пихать\
0
576 / 559 / 47
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2012, 22:29 6
Странно. У меня это прекрасно работает. VS2010.
А если так - (*sets).add(elem);

Номер строки называл исходя из того кода, который вы выложили. Там ее номер был 73.
0
23 / 23 / 11
Регистрация: 15.04.2012
Сообщений: 183
27.09.2012, 22:29  [ТС] 7
Всё, догнал)

Не по теме:

лашара\\\\\

0
5498 / 4893 / 831
Регистрация: 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;
        };
0
576 / 559 / 47
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2012, 22:32 9
alsav22, закрывающая скобка там дальше вроде. Во всяком случае ошибок компиляции нет.
0
5498 / 4893 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
27.09.2012, 22:36 10
Удалил. Не разобрлся.
0
23 / 23 / 11
Регистрация: 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();
 
    };
Да это всё структура, а она находиться ещё в классе, а там ещё и массив элементов класса....)....Спасибо всем за помощь!

Не по теме:

тупанул:)


Не по теме:

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

0
5498 / 4893 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
27.09.2012, 22:48 12
Это я поспешил, не разобрался.
0
576 / 559 / 47
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2012, 22:55 13
Elfenlide, вам бы не мешало разнести реализацию и описание по разным файлам.

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

exit(1); - это тоже иногда не рекомендуют писать. Правда в вашем случае будет все ок. В любом случае, return сделает то же самое - тоже выйдет из программы
0
23 / 23 / 11
Регистрация: 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 ?

Не по теме:

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

0
576 / 559 / 47
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2012, 23:22 15
Про exit можно почитать тут - Прервать работу програмы
using namespace std; не стоит писать только в .h файлах. В .cpp можно. Некоторые пишут std:: просто потому, что так привыкли
0
23 / 23 / 11
Регистрация: 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], то это работает,правдо надо заменить "->" на "."
А вот с указателями не хочет..желательно с пояснением...спасибо
0
576 / 559 / 47
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2012, 23:45 17
Elfenlide, вы опять путаетесь в указателях.
cout<<*sets[0];

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

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

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

Я бы посоветовал побольше почитать про указатели. В чем разница между указателем и объектом и тд.
Спасибо вам огромное за помощь!) Да, у меня с указателями проблемы....мне ссылки как-то роднее...с ними проще...но пока времени указатели проработать нормально нету, поэтому учусь по ходу дела на ошибках...
0
27.09.2012, 23:54
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
27.09.2012, 23:54
Помогаю со студенческими работами здесь

Добавить элемент в конец уже существующего списка и вернуть его же
Добрый вечер. Есть список состоящий из буквенных символов. Обозначим его за X Допустим мы уже...

вроде программа работает, но когда ввожу значения
Составить программу для вычисления функции при постоянных х и у. Я написала программу, но когда...

ListBox выдает null когда нажимаю в "молоко" (Не в элемент "молоко", а просто в пустоту) и из-за этого ошибка
Нужно либо задать базовое значение или как-то это исправить. Пробовал, чтобы не выполнялось если...

Когда я ввожу номер теста у меня ошибка file not found
Почему когда я ввожу номер теста у меня ошибка file not found там вроде нет файлов Program Test;...

Когда ввожу размерность массива меньше нуля ошибка не вылезает
Всем здравствуйте. Программа заменяет max и min целочисленного массива средним арифметическим...

запустив прогу я ввожу число нажимаю энтер и прога закрывается
#include &lt;iostream&gt; using namespace std; int main() { double f; double m; int counter;...


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

Или воспользуйтесь поиском по форуму:
18
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2024, CyberForum.ru