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

Меняется значение указателя - C++

Восстановить пароль Регистрация
 
Mukue
0 / 0 / 0
Регистрация: 08.02.2013
Сообщений: 35
20.12.2013, 19:49     Меняется значение указателя #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
#include "stdafx.h"
#include <iostream>
using namespace std;
template <class T> class queue{ //класс для реализации односвязного списка типа element
    class element
    {
        public:
        T value;
        element *next; //указатель на следующий
        element (T smth = 0){value = smth; next=NULL;}
    };
public:
    element *begin; //указатель на начало
    queue(const queue& ob)
    {
    }
    queue()
    {
        begin=NULL; //обнуляю, чтобы проверить, есть ли элементы в списке("очереди" в будущем))
    }
    void operator =(const queue& a)
    {
        this->begin=a.begin;
    }
    void operator +(const T& a)
    {
        bool flag= true;
 
        
        if (begin==NULL)//если нет элементов в списке - добавляем новый
        {
            begin=&element(a);
        }
        else //если есть
        {
            element *temp=begin;
                do
                {
                    if (temp->next==NULL)//проверяем, есть ли следующий
                    {
                        temp->next=&element(a);
                        flag=false;
                    }
                    else//идем дальше
                    {
                        temp=temp->next;
                    }
                }while(flag);
        }
    }
    void operator --()
    {
        cout<<"Deleting: "<<begin.value<<endl;
        begin=begin.next;
    }
    void print()//здесь происходит самое плохое!!!
    {
        cout<<"Queue contents: "<<endl;
        element *temp2;
        temp2=begin;
        do
        {
            cout<<temp2->value<<endl;
        }
        while(temp2->next!=NULL);
    }
 
};
 
int main()
{
    
    queue<int> elem;
    elem+3;
    elem+7;
    elem.print();
    system("pause");
    return 0;
}
После передачи управления методу print адрес первого элемента остается таким же, значение указателя next и значение поля value изменяются(( Как исправить программу, чтобы не изменялись?

Добавлено через 32 минуты
Прошу помочь=(

Добавлено через 10 минут
ну или хотя бы почему так происходит скажите?
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
20.12.2013, 20:04     Меняется значение указателя #2
C++
1
begin=&element(a);
В очередь помещаете адреса локальных объектов (созданных в operator+), после выхода из operator+ их уже нет.
Mukue
0 / 0 / 0
Регистрация: 08.02.2013
Сообщений: 35
20.12.2013, 20:14  [ТС]     Меняется значение указателя #3
ну и что, адрес же я присвоил, по крайней мере после перехода в мэйн и до вызова принт я эти элементы вижу в контрольных значениях.
Сделал так:
C++
1
2
3
4
5
queue<int> elem;
    queue<int> elem2;
    elem+3;
    elem+7;
    elem2=elem;
та же ошибка, т.е. выходит, что проблема в копировании объекта?
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
20.12.2013, 20:20     Меняется значение указателя #4
Цитата Сообщение от Mukue Посмотреть сообщение
ну и что, адрес же я присвоил,
Адрес чего? Переменной, память под которой уже считается свободной. У меня (в студии) вылет происходит при втором сложении.
Mukue
0 / 0 / 0
Регистрация: 08.02.2013
Сообщений: 35
20.12.2013, 20:23  [ТС]     Меняется значение указателя #5
Цитата Сообщение от alsav22 Посмотреть сообщение
Адрес чего? Переменной, память под которой уже считается свободной. У меня (в студии) вылет происходит при втором сложении.
не знаю, ну а как в этом примере нормально конструктор копирования сделать?
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
20.12.2013, 20:30     Меняется значение указателя #6
Вот вам +:
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
void operator +(const T& a)
    {
        bool flag= true;
 
        if (begin==NULL)//если нет элементов в списке - добавляем новый
        {
            begin = new element(a);
        }
        else //если есть
        {
            element *temp=begin;
                do
                {
                    if (temp->next==NULL)//проверяем, есть ли следующий
                    {
                        temp->next = new element(a);
                        flag=false;
                    }
                    else//идем дальше
                    {
                        temp=temp->next;
                    }
                } while(flag);
        }
    }
Вот вам print():
C++
1
2
3
4
5
6
7
8
9
10
11
void print()
    {
        cout << "Queue contents: "<<endl;
        element *temp2;
        temp2 = begin;
        while (temp2)
        {
            cout << temp2 ->value << endl;
            temp2 = temp2 ->next;
        }
    }
На скрине - работа.
Миниатюры
Меняется значение указателя  
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
20.12.2013, 21:09     Меняется значение указателя
Еще ссылки по теме:

Как проверить значение (существование) указателя? C++
Ввести с клавиатуры значение указателя C++
Разыменование указателя, не передается правильное значение C++

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

Или воспользуйтесь поиском по форуму:
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
20.12.2013, 21:09     Меняется значение указателя #7
Если в очереди будут объекты, созданные в динамической памяти, то не забудьте освобождать память при удалении очереди (или при удалении элементов очереди: operator --()).
C++
1
2
3
4
5
6
7
8
9
10
11
12
void operator --()
{
    if (begin != 0)
    {
         cout <<"Deleting: " << begin ->value << endl;
         element* temp;
         temp = begin ->next;
         delete begin;
         begin = temp;
    }
    else cout << "Queue is empty!" << endl;
}
Yandex
Объявления
20.12.2013, 21:09     Меняется значение указателя
Ответ Создать тему
Опции темы

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