VPF::Проблема с входным потоком. - Форум программистов

Pogreb

Дата 29.10.2005, 17:58 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Awaiting Authorisation
Сообщений: 69
Регистрация: 17.10.2005

Репутация: нет
Всего: нет

При перегрузке оператора >> И << появляются проблемы.
При вводе текста в одном объкте тот же текст появляется и в объекте вводимом до него.
С адрессами вроде бы порядок.
Конструктор копирования отсутствует.
ПОМОГИТЕ!!!
Вот прога:

Это сообщение отредактировал(а) Pogreb - 29.10.2005, 17:59

Присоединённый файл ( Кол-во скачиваний: 9 )

LABKA.CPP 0,75 Kb

LPBOY

Дата 29.10.2005, 18:20 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 228
Регистрация: 12.7.2005

Репутация: 20
Всего: 20

Я не заметил в твоем коде ввода/вывода строки...
Так пойдет?

Код


#include<iostream.h>
#include<conio.h>
#include<string.h>
#include<stdio.h>
#include<iomanip.h>

class str
{
private:
    char* p;
    friend istream& operator>>(istream&, str&);
    friend ostream& operator<<(ostream&, const str&);
public:
    str(unsigned size)
    {
        p = new char[size];
    }
    ~str()
    {
        delete [] p;
        p = 0;
    }    
};

istream& operator>>(istream& in, str& obj)
{
    return in >> obj.p;
}

ostream& operator<<(ostream& out, const str& obj)
{
    return out << obj.p;
}

int main()
{
    clrscr();
    int i;
    cout<<"Vvedite dlinu stroki\n";
    cin>>i;
    str ob1(i),ob2(i);
    cin>>ob1;
    cin>>ob2;
    cout<<"Overload <<"<<endl;
    cout<<ob1<<endl;
    cout<<ob2<<endl;
    getch();
}

Добавлено @ 18:24

Цитата(LPBOY @ 29.10.2005, 18:20)

Я не заметил в твоем коде ввода/вывода строки...

Уже заметил smile

Промежуточный массив temp абсолютно не нужен.

--------------------

Каждый человек по-своему прав, а по-моему нет...

BreakPointMAN

Дата 29.10.2005, 18:25 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 725
Регистрация: 15.9.2004
Где: Saratov

Репутация: 14
Всего: 24

Код


istream &operator>> (istream &input,str &s)
{
 char temp[100];
 input>>setw(100)>>temp;
 s.st=temp;
 return input;
}

Бред.
Ты в оператор-функции operator>> определил локальный массив temp типа char, чем все дело и испортил... Вводя туда данные, а затем присваивая адрес этого массива указателю st, ты делаешь ошибку, в результате которой st после выхода из этой функции будет указывать на уже возможно разрушенную область памяти, т.к. локальные объекты уничтожаются в конце блока, а блоком для твоего temp является тело твоей оператор-функции, а указатель st указывает на него... Надо, вероятно, так:

Код


istream &operator>> (istream &input, const str &s)
{
 return input>>s.st;
}

+ ко всему я бы выделял память так:

Код


 str::str(int i)
 {
 st=new char[i+1];
 }

--------------------

"Разруха не в клозетах, а в головах." © Ф.Ф. Преображенский (М.Булгаков, "Собачье сердце")

LPBOY

Дата 29.10.2005, 18:34 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 228
Регистрация: 12.7.2005

Репутация: 20
Всего: 20

Цитата(BreakPointMAN @ 29.10.2005, 18:25)

Надо, вероятно, так:

Код


istream &operator>> (istream &input, const str &s)
{
 return input>>s.st;
}

const ты здесь зря добавил smile

- компилироваться не будет.

--------------------

Каждый человек по-своему прав, а по-моему нет...

BreakPointMAN

Дата 29.10.2005, 18:39 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 725
Регистрация: 15.9.2004
Где: Saratov

Репутация: 14
Всего: 24

Цитата(LPBOY @ 29.10.2005, 18:34)

const ты здесь зря добавил - компилироваться не будет.

ага... ошибся малость... smile

--------------------

Pogreb

Дата 30.10.2005, 13:53 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Awaiting Authorisation
Сообщений: 69
Регистрация: 17.10.2005

Репутация: нет
Всего: нет

Спасибо всё работает.

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Общие вопросы \| Следующая тема »