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

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
bambino
193 / 19 / 4
Регистрация: 05.08.2010
Сообщений: 229
#1

Список, непонятная ошибка - C++

14.06.2011, 17:48. Просмотров 458. Ответов 10
Метки нет (Все метки)

Всем привет.
У меня есть список, вроде все норм, но когда вызываю оператор сложения - run-time error почему-то, если не трудно, укажите где у меня ошибка.
Весь код
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
#include <iostream>
#include <stdexcept>
#include <cstdlib>
#include <algorithm>
 
 
using namespace std;
 
class List
{
private:
        typedef int value_type;
        typedef value_type* pointer;
 
          /*Узел списка*/
        class _list_node_base
        {
        public:
                _list_node_base* _next;
                value_type _data;
        };
 
          /*Указатель на начало списка*/
        _list_node_base* _start;
          /*Разчер списка*/
        int _size;
 
        _list_node_base* node(int n)
        {
                _list_node_base* _tmp = _start;
                for (int i=0; i<n; i++)
                        _tmp = _tmp->_next;
                return _tmp;
        }
 
public:
          /*Конструктор без параметров*/
        List() : _start(0), _size(0) {}
            /*Конструктор копирования*/
        List(const List& other)
        {
                List();
                clear();
                append(other);
        }
 
           /*Добавление елементов заданого списка в исходный*/
        void append(const List& other)
        {
                for (int i=0; i<other.size(); i++)
                        push_back(other[i]);
        }
 
         /*Добавление елемента в конец спика*/
        void push_back(const value_type& data)
        {
                _list_node_base* _new = new _list_node_base;
                _new->_data = data;
                _new->_next = 0;
                if (_start == 0)
                {
                        _start = _new;
                }
                else
                {
                        _list_node_base* _end = node(size()-1);
                        _end->_next = _new;
                }
                _size++;
        }
 
           /*Вставка елемента в заданую позицию*/
        void insert(const int n, const value_type& data)
        {
                if (n > _size)
                        throw out_of_range("List index out of bounds.");
                if (n == _size)
                        push_back(data);
                else
                {
                        _list_node_base* _new = new _list_node_base;
                        _new->_data = data;
                        _new->_next = 0;
                        if (n == 0)
                        {
                                _new->_next = _start;
                                _start = _new;
                        }
                        else
                        {
                                _list_node_base* _tmp = node(n-1);
                                _new->_next = _tmp->_next;
                                _tmp->_next = _new;
                        }
                        _size++;
                }
        }
 
          /*Удаляет елемент и заданой позции*/
        void remove(int n)
        {
                if (n>=size())
                        throw out_of_range("List index out of bounds.");
                _list_node_base* _this_node;
                if (n==0)
                {
                        _this_node = _start;
                        _start = _this_node->_next;
                }
                else
                {
                        _list_node_base* _prev_node = node(n-1);
                        _this_node = _prev_node->_next;
                        _prev_node->_next = _this_node->_next;
                }
                delete _this_node;
                _size--;
        }
 
         /*Возвращает размер списка*/
        int size() const { return _size; }
 
           /*Достуа к елементам через индекс*/
        value_type& at(int n) const
        {
                _list_node_base* _tmp = _start;
                for (int i=0; i<n; i++)
                        if (_tmp!=0)
                                _tmp = _tmp->_next;
                if (_tmp!=0)
                        return _tmp->_data;
                else
                        throw out_of_range("List index out of bounds.");
        }
 
           /*Операторы доступа к елементам списка*/
        value_type& operator[](int n) { return at(n); }
        value_type& operator[](int n) const { return at(n); }
            /*Вывод в поток*/
        friend ostream& operator<< (ostream& o, const List& l)
        {
                for(int i=0; i<l.size(); i++)
                        o << l[i] << " ";
                return o;
        }
 
          /*Ввод в поток*/
        friend istream &operator>> (istream &is, List &l)
        {
          int n, t;
          cout << "\nKolu4ectvo elemntov v cpicke = ";
          cin >> n;
          for(int i = 0; i < n; ++i)
          {
            cout << "\n> ";
            cin >> t;
            l.push_back(t);
          }
          return is;
        }
 
         /*Сложение списков*/
        List operator+ (const List &l) const
        {
          List r;
          int ln = std::min(size(), l.size());
          for(int i = 0; i < ln; ++i)
            r.push_back(at(i) + l.at(i));
          return r;
        }
 
          /*Оператор равно*/
        friend bool operator== (const List &l, const List &_l)
        {
          if( !(l.size() == _l.size()) )
            return false;
          for(int i = 0; i < l.size(); ++i)
            if(l.at(i) != _l.at(i)) return false;
          return true;
        }
 
          /*Оператор не равно*/
        friend bool operator!= (const List &l, const List &_l)
        {
          return !(l == _l);
        }
 
          /*Оператор меньше*/
        friend bool operator< (const List &l, const List &_l)
        {
            return l.size() < _l.size();
        }
 
          /*Оператор больше*/
        friend bool operator> (const List &l, const List &_l)
        {
            return l.size() > _l.size();
        }
 
          /*Вставка елемента в конец списка*/
        void operator<< (const int &el)
        {
          push_back(el);
        }
 
          /*Удаляет елемент и заданой позции*/
        void operator>> (int n)
        {
          remove(n);
        }
 
          /*Очистка списка*/
        void clear()
        {
                _list_node_base* _tmp;
                while (_start!=0)
                {
                        _tmp = _start->_next;
                        delete _start;
                        _start=_tmp;
                }
                _size = 0;
        }
 
            /*Оператор присваевания*/
        List& operator= (const List& other)
        {
                if (this!=&other)
                {
                        clear();
                        append(other);
                }
        }
 
         /*Деструктор*/
        ~List()
        {
                clear();
        }
};
 
 
int main(){
  List a, b;
  std::cout << "Vvedite 1 cpicok: \n";
  std::cin >> a;
  std::cout << "Vvedite 2 cpicok: \n";
  std::cin >> b;
  std::cout << "\nCymipyuwiu` cpicok: \n";
  std::cout << (a + b);
  /*std::cout << "\nVedite element kotupu` bydet dobavlen v cpicok 1: \n> ";
  int n;
  std::cin >> n;
  a << n;
  std::cout << "\nCpicok 1: \n" << a;
  std::cout << "\nVedite index elementa kotopuu` budet ydalen iz cpicka 2: \n> ";
  std::cin >> n;
  b >> n;
  std::cout << "\nCpicok 2: ";
  std::cout << b;*/
  system("pause >> void");
  return 0;
}


Только оператор сложения
C++
1
2
3
4
5
6
7
8
List operator+ (const List &l) const
{
          List r;
          int ln = std::min(size(), l.size());
          for(int i = 0; i < ln; ++i)
            r.push_back(at(i) + l.at(i));
          return r;
}

Кстате, g++ по этому поводу молчит, однако студия недовольна.
Всем спасибо!
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
14.06.2011, 17:48     Список, непонятная ошибка
Посмотрите здесь:

Непонятная ошибка - C++
Мне дана матрица коэффициентов системы линейных уравнений в необычном виде: Исходная СЛУ 4x1 + 2x2 + 9x3 = 1 x2 + 6x3 = 7 4x1 + 2x3...

Непонятная ошибка!? - C++
Есть задача: дан массив и число, переставить числа в массиве таким образом, чтобы слева от некоторой границы шли числа, меньшие или равные...

Непонятная ошибка - C++
Вообщем имеется такой код: // главная программа aka programm.cpp #include &quot;Sales_item.h&quot; using namespace std; ...

непонятная ошибка( - C++
// 1.cpp : main project file. #include &quot;stdafx.h&quot; #include &lt;cstdio&gt; // бібліотека введення-виводу #include &lt;cstring&gt; //...

Непонятная ошибка - C++
Всем привет! Вообщем вот функция, которая переводит двоичное число в десятичное, + ещё пару кусков кода, чтобы было понятно, что и откуда...

непонятная ошибка - C++
#include &lt;stdio.h&gt; #include &lt;math.h&gt; main() { float x,pi,s,e,q,n; pi=3.14 ; x= pi/6 ; e=0,0001; s=0 ;

После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
pito211
186 / 173 / 8
Регистрация: 22.03.2010
Сообщений: 612
14.06.2011, 17:56     Список, непонятная ошибка #2
строка 169, вызывается копирующий конструктор, а в копирующем конструкторе вызывается clear() и внём происходит ошибка, щас посмотрю как исправить можно

Добавлено через 1 минуту
стоп копирующий коструктор вобще кривой
bambino
193 / 19 / 4
Регистрация: 05.08.2010
Сообщений: 229
14.06.2011, 18:10  [ТС]     Список, непонятная ошибка #3
Цитата Сообщение от pito211 Посмотреть сообщение
стоп копирующий коструктор вобще кривой
А как будет правильно ?
silent_1991
Эксперт С++
4960 / 3036 / 149
Регистрация: 11.11.2009
Сообщений: 7,027
Завершенные тесты: 1
14.06.2011, 18:11     Список, непонятная ошибка #4
Конструктор копии должен выглядеть так:
C++
1
2
3
4
5
6
7
List(const List& other):
_start(0),
_size(0)
{
    clear();
    append(other);
}
То, что в нём раньше вызывался List, никакой погоды не делало - объект создавался и тут же уничтожался, а в объекте, для которого был вызван конструктор копии, указатель и размер так и оставались не инициализированными.
pito211
186 / 173 / 8
Регистрация: 22.03.2010
Сообщений: 612
14.06.2011, 18:12     Список, непонятная ошибка #5
вот короче копируй это. Довёл до рабочего состояния
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
265
266
267
268
269
// test.cpp: определяет точку входа для консольного приложения.
//
 
#include "stdafx.h"
#include <iostream>
#include <stdexcept>
#include <cstdlib>
#include <algorithm>
 
 
using namespace std;
 
class List
{
private:
        typedef int value_type;
        typedef value_type* pointer;
 
          /*Узел списка*/
        class _list_node_base
        {
        public:
                _list_node_base* _next;
                value_type _data;
        };
 
          /*Указатель на начало списка*/
        _list_node_base* _start;
          /*Разчер списка*/
        int _size;
 
        _list_node_base* node(int n)
        {
                _list_node_base* _tmp = _start;
                for (int i=0; i<n; i++)
                        _tmp = _tmp->_next;
                return _tmp;
        }
 
public:
          /*Конструктор без параметров*/
        List() : _start(0), _size(0) {}
            /*Конструктор копирования*/
        List(const List& other)
        {
                _size = 0;
                _start = 0;
                append(other);
        }
 
           /*Добавление елементов заданого списка в исходный*/
        void append(const List& other)
        {
                for (int i=0; i<other.size(); i++)
                        push_back(other[i]);
        }
 
         /*Добавление елемента в конец спика*/
        void push_back(const value_type& data)
        {
                _list_node_base* _new = new _list_node_base;
                _new->_data = data;
                _new->_next = 0;
                if (_start == 0)
                {
                        _start = _new;
                }
                else
                {
                        _list_node_base* _end = node(size()-1);
                        _end->_next = _new;
                }
                _size++;
        }
 
           /*Вставка елемента в заданую позицию*/
        void insert(const int n, const value_type& data)
        {
                if (n > _size)
                        throw out_of_range("List index out of bounds.");
                if (n == _size)
                        push_back(data);
                else
                {
                        _list_node_base* _new = new _list_node_base;
                        _new->_data = data;
                        _new->_next = 0;
                        if (n == 0)
                        {
                                _new->_next = _start;
                                _start = _new;
                        }
                        else
                        {
                                _list_node_base* _tmp = node(n-1);
                                _new->_next = _tmp->_next;
                                _tmp->_next = _new;
                        }
                        _size++;
                }
        }
 
          /*Удаляет елемент и заданой позции*/
        void remove(int n)
        {
                if (n>=size())
                        throw out_of_range("List index out of bounds.");
                _list_node_base* _this_node;
                if (n==0)
                {
                        _this_node = _start;
                        _start = _this_node->_next;
                }
                else
                {
                        _list_node_base* _prev_node = node(n-1);
                        _this_node = _prev_node->_next;
                        _prev_node->_next = _this_node->_next;
                }
                delete _this_node;
                _size--;
        }
 
         /*Возвращает размер списка*/
        int size() const { return _size; }
 
           /*Достуа к елементам через индекс*/
        value_type& at(int n) const
        {
                _list_node_base* _tmp = _start;
                for (int i=0; i<n; i++)
                        if (_tmp!=0)
                                _tmp = _tmp->_next;
                if (_tmp!=0)
                        return _tmp->_data;
                else
                        throw out_of_range("List index out of bounds.");
        }
 
           /*Операторы доступа к елементам списка*/
        value_type& operator[](int n) { return at(n); }
        value_type& operator[](int n) const { return at(n); }
            /*Вывод в поток*/
        friend ostream& operator<< (ostream& o, const List& l)
        {
                for(int i=0; i<l.size(); i++)
                        o << l[i] << " ";
                return o;
        }
 
          /*Ввод в поток*/
        friend istream &operator>> (istream &is, List &l)
        {
          int n, t;
          cout << "\nKolu4ectvo elemntov v cpicke = ";
          cin >> n;
          for(int i = 0; i < n; ++i)
          {
            cout << "\n> ";
            cin >> t;
            l.push_back(t);
          }
          return is;
        }
 
         /*Сложение списков*/
        List operator+ (const List &l) const
        {
          List r;
          int ln = std::min(size(), l.size());
          for(int i = 0; i < ln; ++i)
            r.push_back(at(i) + l.at(i));
          return r;
        }
 
          /*Оператор равно*/
        friend bool operator== (const List &l, const List &_l)
        {
          if( !(l.size() == _l.size()) )
            return false;
          for(int i = 0; i < l.size(); ++i)
            if(l.at(i) != _l.at(i)) return false;
          return true;
        }
 
          /*Оператор не равно*/
        friend bool operator!= (const List &l, const List &_l)
        {
          return !(l == _l);
        }
 
          /*Оператор меньше*/
        friend bool operator< (const List &l, const List &_l)
        {
            return l.size() < _l.size();
        }
 
          /*Оператор больше*/
        friend bool operator> (const List &l, const List &_l)
        {
            return l.size() > _l.size();
        }
 
          /*Вставка елемента в конец списка*/
        void operator<< (const int &el)
        {
          push_back(el);
        }
 
          /*Удаляет елемент и заданой позции*/
        void operator>> (int n)
        {
          remove(n);
        }
 
          /*Очистка списка*/
        void clear()
        {
            if (_size) return;
                _list_node_base* _tmp;
                while (_start!=0)
                {
                        _tmp = _start->_next;
                        delete _start;
                        _start=_tmp;
                }
                _size = 0;
        }
 
            /*Оператор присваевания*/
        List& operator= (const List& other)
        {
                if (this!=&other)
                {
                        clear();
                        append(other);
                }
        }
 
         /*Деструктор*/
        ~List()
        {
                clear();
        }
};
 
 
int main(){
  List a, b;
  std::cout << "Vvedite 1 cpicok: \n";
  std::cin >> a;
  std::cout << "Vvedite 2 cpicok: \n";
  std::cin >> b;
  std::cout << "\nCymipyuwiu` cpicok: \n";
  std::cout << (a + b);
 
  /*std::cout << "\nVedite element kotupu` bydet dobavlen v cpicok 1: \n> ";
  int n;
  std::cin >> n;
  a << n;
  std::cout << "\nCpicok 1: \n" << a;
  std::cout << "\nVedite index elementa kotopuu` budet ydalen iz cpicka 2: \n> ";
  std::cin >> n;
  b >> n;
  std::cout << "\nCpicok 2: ";
  std::cout << b;*/
  system("pause >> void");
  return 0;
}
silent_1991
Эксперт С++
4960 / 3036 / 149
Регистрация: 11.11.2009
Сообщений: 7,027
Завершенные тесты: 1
14.06.2011, 18:12     Список, непонятная ошибка #6
Ну и к слову - храните в списке указатель не только на начало, но и на конец, а то искать этот самый конец каждый раз при вставке в конец (одной из самых популярных операций) совсем не комильфо.
pito211
186 / 173 / 8
Регистрация: 22.03.2010
Сообщений: 612
14.06.2011, 18:14     Список, непонятная ошибка #7
лучше как сайлент предложил, через списки инициализации, но тогда ещё и clear сотри, он бесполезен
bambino
193 / 19 / 4
Регистрация: 05.08.2010
Сообщений: 229
14.06.2011, 18:16  [ТС]     Список, непонятная ошибка #8
Еще вопросик, оператор присваевания правилный ?
silent_1991
Эксперт С++
4960 / 3036 / 149
Регистрация: 11.11.2009
Сообщений: 7,027
Завершенные тесты: 1
14.06.2011, 18:18     Список, непонятная ошибка #9
pito211, да, код глянул - clear бесполезен, цикл в нём не выполнится ни разу (в силу _size == 0). Просто в коде особо не разбирался.

Добавлено через 1 минуту
bambino, нет, в конце надо добавить
C++
1
return *this;
bambino
193 / 19 / 4
Регистрация: 05.08.2010
Сообщений: 229
14.06.2011, 18:19  [ТС]     Список, непонятная ошибка #10
Спасибо всем!
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
14.06.2011, 18:21     Список, непонятная ошибка
Еще ссылки по теме:

Непонятная ошибка - C++
Доброго времени суток! Не могу понять в чём ошибка. #include &quot;stdafx.h&quot; #include &lt;iostream&gt; using namespace std; void...

Непонятная ошибка - C++
Здравствуйте, есть задача с графами - соединенные точки и тп, вот попытался сделать структуры и классы для этой задачи и получил ошибку...

Непонятная ошибка - C++
Пробую написать подключаемый модуль к файловому менеджеру ФАР. Программа, не хочет компилится, затсряла на непонятной ошибке Плагин...

непонятная ошибка - C++
Почему здесь выбивает ошибку? Помогите последнюю лабу сделать) if (x&lt;(*Node)-&gt;l) Add(x,&amp;(*Node)-&gt;l); #include &lt;iostream.h&gt; ...


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

Или воспользуйтесь поиском по форуму:
pito211
186 / 173 / 8
Регистрация: 22.03.2010
Сообщений: 612
14.06.2011, 18:21     Список, непонятная ошибка #11
ну вроде работает, возвращает this и защищён от самоприсваивания, чё ещё надо

Добавлено через 1 минуту
а ну да, у тебя он this не возвращает)
Yandex
Объявления
14.06.2011, 18:21     Список, непонятная ошибка
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru