Организация ввода собственного типа

@TheTangro · Регистрация: 24.09.2015

Author24 — интернет-сервис помощи студентам

Здравствуйте, создаю собственный класс для работы со строками. Так вот, встал такой вопрос: как ввести строку произвольного размера? Перегружаю оператор ">>" вот код:

C++

istream& operator>>(istream& os, const strings& obj)
{
    os >> obj.s;
    return os;
}

Так вот, если прописать в main что-то вроде

C++

#include "strings.h"
void main()
{
    strings  s(10);
    cin >> s;
    cout << s;
}

Все работает. Если же вот так:

C++

#include "strings.h"
void main()
{
    strings  s;
    cin >> s;
    cout << s;
}

То выскакивает ошибка, что строка пишется за пределами буфера.
Оно и понятно, ведь строки как таковой не создано.
Вопрос вот в чем:
Как сделать возможность объявлять переменную

C++

1	strings s;

И иметь возможность вводить строку произвольного размера?
Помогите, пожалуйста

nd2 · 16.10.2016, 21:09

Класс покажи.

@TheTangro · 16.10.2016, 21:11 **[ТС]**

Прототипы:

C++

#include <iostream>
#include <stdio.h>
#include <cstring>
using namespace std;
class strings
{
private:
    char *s;
public:
    strings();
    explicit strings(int n);
    strings(char *str);
    strings(const strings&);
    ~strings();
    strings operator= (strings &obj);
    strings operator= (char *str);
    strings operator+ (strings obj);
    strings operator+ (char *str);
    strings operator+= (strings obj);
    strings operator+= (char *str);
    strings operator- (strings obj);
    strings operator- (char *str);
    strings operator-= (strings obj);
    strings operator-= (char *str);
    bool operator> (strings obj);
    bool operator> (char *str);
    bool operator< (strings obj);
    bool operator< (char *str);
    bool operator== (strings obj);
    bool operator== (char *str);
    strings operator++ ();
    strings operator-- ();
    strings operator() (int, int);
    char operator [] (int);
    friend ostream& operator<<(ostream& os, const strings&);
    friend istream& operator>>(istream& os, const strings&);
};

Реализация:

C++

#include "strings.h"
strings::strings()
{
    s = new char[1];
    s[0] = '\0';
}
strings::strings(int n)
{
    s = new char[n];
}
strings::strings(char *str)
{
    s = new char[strlen(str)+1];
    strcpy(s, str);
}
strings::~strings()
{
    delete [] s;
}
strings::strings(const strings& copy)
{
    s = new char[strlen(copy.s) + 1];
    strcpy(s, copy.s);
}
strings strings::operator= (strings &obj)
    {
        if (&obj != this)
        {
            delete[] s;
            s = new char[strlen(obj.s) + 1];
            strcpy(s, obj.s);
        }
        return *this;
    }
strings strings::operator= (char *str)
{
    delete[] s;
    s = new char[strlen(str) + 1];
    strcpy(s, str);
    return *this;
}
strings strings::operator+ (strings obj)
{
    strings temp(strlen(this->s) + strlen(obj.s) + 1);
    strcat(strcpy(temp.s,this->s), obj.s);
    return temp;
}
strings strings::operator+ (char *s)
{
    strings temp(strlen(this->s) + strlen(s) + 1);
    strcat(strcpy(temp.s, this->s), s);
    return temp;
}
strings strings::operator+= (char *str)
{
    char *s = new char[strlen(this->s) + 1];
    strcpy(s, this->s);
    delete[] this->s;
    this->s = new char[strlen(str) + strlen(this->s) + 1];
    strcat(strcpy(this->s, s), str);
    delete[] s;
    return *this;
}
strings strings::operator+= (strings obj)
{
    char *s = new char[strlen(this->s) + 1];
    strcpy(s, this->s);
    delete[] this->s;
    this->s = new char[strlen(obj.s) + strlen(this->s) + 1];
    strcat(strcpy(this->s, s), obj.s);
    delete[] s;
    return *this;
}
strings strings::operator- (strings obj)
{
    if (strstr(s, obj.s) != NULL)
    {
        strings temp(strlen(s) - strlen(obj.s) + 1);
        strncpy(temp.s, s, strstr(s, obj.s)-s);
        temp.s[strstr(s, obj.s) - s] = '\0';
        strcat(temp.s, (strstr(s, obj.s) + strlen(obj.s)));
        return temp;
    }
    else return *this;
}
strings strings::operator- (char *str)
{
    if (strstr(s, str) != NULL)
    {
        strings temp(strlen(s) - strlen(str) + 1);
        strncpy(temp.s, s, strstr(s, str) - s);
        temp.s[strstr(s, str) - s] = '\0';
        strcat(temp.s, (strstr(s, str) + strlen(str)));
        return temp;
    }
    else return *this;
}
strings strings::operator-= (strings obj)
{
    if (strstr(s, obj.s) != NULL)
    {
        char *temp = new char[strlen(this->s) + 1];
        strcpy(temp, this->s);
        delete[] this->s;
        s = new char[strlen(temp) - strlen(obj.s) + 1];
        strncpy(s, temp, strstr(temp, obj.s) - temp);
        s[strstr(temp, obj.s) - temp] = '\0';
        strcat(s, (strstr(temp, obj.s) + strlen(obj.s)));
        delete[] temp;
    }
    return *this;
}
strings strings::operator-= (char *str)
{
    if (strstr(s, str) != NULL)
    {
        char *temp = new char[strlen(this->s) + 1];
        strcpy(temp, this->s);
        delete[] this->s;
        s = new char[strlen(temp) - strlen(str) + 1];
        strncpy(s, temp, strstr(temp, str) - temp);
        s[strstr(temp, str) - temp] = '\0';
        strcat(s, (strstr(temp, str) + strlen(str)));
        delete[] temp;
    }
    return *this;
}
bool strings::operator> (strings obj)
{
    if (strcmp(s, obj.s) > 0)
        return true;
    else return false;
}
bool strings::operator> (char *str)
{
    if (strcmp(s, str) > 0)
        return true;
    else return false;
}
bool strings::operator< (strings obj)
{
    if (strcmp(s, obj.s) < 0)
        return true;
    else return false;
}
bool strings::operator< (char *str)
{
    if (strcmp(s, str) < 0)
        return true;
    else return false;
}
bool strings::operator== (strings obj)
{
    if (strcmp(s, obj.s) == 0)
        return true;
    else return false;
}
bool strings::operator== (char *str)
{
    if (strcmp(s, str) == 0)
        return true;
    else return false;
}
strings strings::operator++()
{
    for (int i = 0; i < strlen(s); i++)
        s[i]++;
    return *this;
}
strings strings::operator--()
{
    for (int i = 0; i < strlen(s); i++)
        s[i]--;
    return *this;
}
strings strings::operator()(int n, int m)
{
    strings temp(n+m+3);
    int p = 0;
    for (int i = n; i < (m-1); i++, p++)
        temp.s[p] = s[i];
    temp.s[m-n-1] = '\0';
    return temp;
}
char strings::operator[] (int n)
{
    return s[n];
}
ostream& operator<<(ostream& os, const strings& obj)
{
    os << obj.s;
    return os;
}
istream& operator>>(istream& os, const strings& obj)
{
    strings ob;
    os >> obj.s;
    return os;
}

nd2 · 16.10.2016, 21:14

Сообщение от TheTangro

Как сделать возможность объявлять переменную

C++

1	strings s;

И иметь возможность вводить строку произвольного размера?

Выделять в конструкторе по умолчанию память произвольного размера.

@TheTangro · 16.10.2016, 21:15 **[ТС]**

А как это? Простите за невеждество

@GbaLog- · 16.10.2016, 21:17

Сообщение от TheTangro

C++

strings::strings()
{
* * s = new char[1];
* * s[0] = '\0';
}

Сообщение от TheTangro

А как это? Простите за невеждествo

C++

3	s = new char[16];

@hoggy · 16.10.2016, 21:21

Сообщение от TheTangro

istream& operator>>(istream& os, const strings& obj)

C++

1	istream& operator>>(istream& os, strings& obj)

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

Сообщение от TheTangro

os >> obj.s;

это не будет работать.

вместо char* s используйте std::string

@TheTangro · 16.10.2016, 21:22 **[ТС]**

Понятно, что можно прописать по умолчанию new char[100500]. Но разве это рациональное использование оперативной памяти? Пока на ум приходит только на вводе считывать каждый символ и забивать его в FIFO очередь. Потом же из этого формировать строку, точно зная сколько символов считалось

@GbaLog- · 16.10.2016, 21:25

Сообщение от TheTangro

Но разве это рациональное использование оперативной памяти?

А std::string что делает?

@TheTangro · 16.10.2016, 21:27 **[ТС]**

Мне не нужно использовать встроенный класс. Мне нужно написать свой. А что он делает?

@GbaLog- · 16.10.2016, 21:29

Хотя нет, std::string изначально 0 символов хранит.
Ресайзится при надобности.

nd2 · 16.10.2016, 21:35

Сообщение от TheTangro

Но разве это рациональное использование оперативной памяти?

C++

istream& operator>>(istream& os, strings& obj)
{
    char ch;
    os.get(ch);
    if(cin)
    {
        os.putback(ch);
        int count = os.rdbuf() ->in_avail(); 
        char* str = new char[count];
        
        os.getline(str, count);
       
        if (os)
        {
            delete [] obj.s;
            obj.s = str;
        }
        else
        {
            delete [] str;
            os.clear();
            os.ignore(os.rdbuf() ->in_avail());
        }
    }
    return os;
}

@TheTangro · 16.10.2016, 22:01 **[ТС]**

Ураа!!!! Работает!!!!! Простите за наглость, а не могли бы вы,если вас это не затруднит, немного пояснить код?

Добавлено через 10 минут
Для будущих читателей темы. Вот немного подправленный, правильный код. (str - все же просто указатель, а строки нельзя присваивать друг другу простым "=" )

C++

{
    char ch;
    os.get(ch);
    if (cin)
    {
        os.putback(ch);
        int count = os.rdbuf()->in_avail();
        char* str = new char[count];
 
        os.getline(str, count);
 
        if (os)
        {
            delete[] obj.s;
            obj.s = new char[strlen(str)+1];
            strcpy(obj.s, str);
        }
        else
        {
            delete[] str;
            os.clear();
            os.ignore(os.rdbuf()->in_avail());
        }
    }
    return os;
}

nd2 · 16.10.2016, 22:05

Сообщение от nd2

if(cin)
* * {

Тут я не заменил. Так нужно:

C++

1
2

if(os)
    {

Сообщение от TheTangro

Вот немного подправленный, правильный код.

Не порти код.
Добавлено через 1 минуту

Сообщение от TheTangro

а строки нельзя присваивать друг другу простым "="

Там строки никто и не присваивает, указатели присваиваются.

@TheTangro · 16.10.2016, 22:08 **[ТС]**

А нельзя ведь указатели присваивать, он же удаляться должен. А получается, что создать-то его мы создали, а удалить не удалили.

Добавлено через 1 минуту
В итоге получим:

C++

{
    char ch;
    os.get(ch);
    if (os)
    {
 
        os.putback(ch);
        int count = os.rdbuf()->in_avail();
        char* str = new char[count];
 
        os.getline(str, count);
 
        if (os)
        {
            delete[] obj.s;
            obj.s = new char[strlen(str)+1];
            strcpy(obj.s, str);
        }
        else
        {
            os.clear();
            os.ignore(os.rdbuf()->in_avail());
        }
        delete[] str;
    }
    return os;
}

Ведь str мы удаляли только в одной ветке условия

nd2 · 16.10.2016, 22:11

Сообщение от TheTangro

А нельзя ведь указатели присваивать, он же удаляться должен. А получается, что создать-то его мы создали, а удалить не удалили.

Не путай указатели, и то, на что они указывают. Ничего в моём коде добавлять и исправлять не нужно, там всё правильно сделано.

Добавлено через 1 минуту

Сообщение от TheTangro

В итоге получим:

В итоге получили, что два раза одно и то же делаем.

@TheTangro · 16.10.2016, 22:13 **[ТС]**

Я не путаю. В коде, который указали вы в ветке if строка не удалялась. Она висела в динамической памяти и все работало.
Но:
1. Выделенную память всегда нужно очищать.
2. Строки так присваивать нельзя, точнее можно, но тогда мы будем обращаться к одному и тому же участку памяти. И изменив одну строку - изменится и вторая.

nd2 · 16.10.2016, 22:13

Сообщение от TheTangro

char* str = new char[count];
os.getline(str, count);

Один раз выделили память, прочитали туда строку.

Сообщение от TheTangro

obj.s = new char[strlen(str)+1];
* * * * * * strcpy(obj.s, str);

Второй раз выделили память, скопировали туда строку. И смысл?

@TheTangro · 16.10.2016, 22:15 **[ТС]**

Эм, но ведь в первом случае выделяется память под str, а в другом под строку в объекте obj.

nd2 · 16.10.2016, 22:20

Сообщение от TheTangro

Я не путаю. В коде, который указали вы в ветке if строка не удалялась. Она висела в динамической памяти и все работало.
Но:
1. Выделенную память всегда нужно очищать.
2. Строки так присваивать нельзя, точнее можно, но тогда мы будем обращаться к одному и тому же участку памяти.

TheTangro, ты если не понимаешь, как код работает, то не спорь, а сначала разберись.

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

Сообщение от TheTangro

Эм, но ведь в первом случае выделяется память под str, а в другом под строку в объекте obj.

Это я вижу. Вопрос: зачем второй раз выделяется?

Добавлено через 4 минуты
Смотри: память выделена, строка прочитана. Теперь просто достаточно, указателю в объекте, присвоить адрес памяти, куда была прочитана строка(предварительно освободив старую память). Можно было сразу выделить память под указатель в объекте, но тут обрабатываются возможные ошибки при чтении.
Никаких утечек памяти в моём коде нет.

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	16.10.2016, 22:13	18
	Сообщение от TheTangro char* str = new char[count]; os.getline(str, count); Один раз выделили память, прочитали туда строку. Сообщение от TheTangro obj.s = new char[strlen(str)+1]; * * * * * * strcpy(obj.s, str); Второй раз выделили память, скопировали туда строку. И смысл? 0

@TheTangro -3 / 2 / 4 Регистрация: 24.09.2015 Сообщений: 98
	16.10.2016, 22:15 [ТС]	19
	Эм, но ведь в первом случае выделяется память под str, а в другом под строку в объекте obj. 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	16.10.2016, 21:09	2
	Класс покажи. 0

@TheTangro -3 / 2 / 4 Регистрация: 24.09.2015 Сообщений: 98
	16.10.2016, 21:15 [ТС]	5
	А как это? Простите за невеждество 0

@TheTangro -3 / 2 / 4 Регистрация: 24.09.2015 Сообщений: 98
	16.10.2016, 21:22 [ТС]	8
	Понятно, что можно прописать по умолчанию new char[100500]. Но разве это рациональное использование оперативной памяти? Пока на ум приходит только на вводе считывать каждый символ и забивать его в FIFO очередь. Потом же из этого формировать строку, точно зная сколько символов считалось 0

@GbaLog- Любитель чаепитий 3742 / 1798 / 566 Регистрация: 24.08.2014 Сообщений: 6,016 Записей в блоге: 1
	16.10.2016, 21:25	9
	Сообщение от TheTangro Но разве это рациональное использование оперативной памяти? А `std::string` что делает? 0

@TheTangro -3 / 2 / 4 Регистрация: 24.09.2015 Сообщений: 98
	16.10.2016, 21:27 [ТС]	10
	Мне не нужно использовать встроенный класс. Мне нужно написать свой. А что он делает? 0

@GbaLog- Любитель чаепитий 3742 / 1798 / 566 Регистрация: 24.08.2014 Сообщений: 6,016 Записей в блоге: 1
	16.10.2016, 21:29	11
	Хотя нет, `std::string` изначально 0 символов хранит. Ресайзится при надобности. 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	16.10.2016, 22:11	16
	Сообщение от TheTangro А нельзя ведь указатели присваивать, он же удаляться должен. А получается, что создать-то его мы создали, а удалить не удалили. Не путай указатели, и то, на что они указывают. Ничего в моём коде добавлять и исправлять не нужно, там всё правильно сделано. Добавлено через 1 минуту Сообщение от TheTangro В итоге получим: В итоге получили, что два раза одно и то же делаем. 0

@TheTangro -3 / 2 / 4 Регистрация: 24.09.2015 Сообщений: 98
	16.10.2016, 22:13 [ТС]	17
	Я не путаю. В коде, который указали вы в ветке if строка не удалялась. Она висела в динамической памяти и все работало. Но: 1. Выделенную память всегда нужно очищать. 2. Строки так присваивать нельзя, точнее можно, но тогда мы будем обращаться к одному и тому же участку памяти. И изменив одну строку - изменится и вторая. 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	16.10.2016, 22:20	20
	Сообщение от TheTangro Я не путаю. В коде, который указали вы в ветке if строка не удалялась. Она висела в динамической памяти и все работало. Но: 1. Выделенную память всегда нужно очищать. 2. Строки так присваивать нельзя, точнее можно, но тогда мы будем обращаться к одному и тому же участку памяти. TheTangro, ты если не понимаешь, как код работает, то не спорь, а сначала разберись. Добавлено через 55 секунд Сообщение от TheTangro Эм, но ведь в первом случае выделяется память под str, а в другом под строку в объекте obj. Это я вижу. Вопрос: зачем второй раз выделяется? Добавлено через 4 минуты Смотри: память выделена, строка прочитана. Теперь просто достаточно, указателю в объекте, присвоить адрес памяти, куда была прочитана строка(предварительно освободив старую память). Можно было сразу выделить память под указатель в объекте, но тут обрабатываются возможные ошибки при чтении. Никаких утечек памяти в моём коде нет. 0

Организация ввода собственного типа

Решение