почему изменяется и первый экземпляр класса

@Adept88 · Регистрация: 13.09.2011

Почему значение a.m становится равным b.m? Как сделать так, чтобы массив word можно было использовать несколько раз?

C++

#include <iostream>
using namespace std;
 
class word
{
    public:
    char *m;
};
word a, b;
 
int main()
{
    char word[20];
    cout<<"Введите слово1: ";
    cin>>word;
    a.m=word;
    cout<<"Введите слово2: ";
    cin>>word;
    b.m=word;
    cout<<a.m<<" "<<b.m;
    system("pause");
}

@solar_wind · 05.10.2011, 09:43

Ну правильно, ты смотри что делаешь:
Создаешь статический массив:

C++

1	char word[20];

В первый экземпляр засовываешь указатель на массив (но никак не сам массив)

C++

a.m=word;

Во второй экземпляр засовываешь указатель на тот же массив (но никак не сам массив)

C++

b.m=word;

Похорошему память под массив должна выделяться не снаружи, а внутри конструктора класса, тогда и получишь для каждого экземпляра свой массив.

@Adept88 · 05.10.2011, 09:51 **[ТС]**

но если написать

C++

class word
{
    public:
    char m[20];
};
word a, b;
 
int main()
{
    char word[20];
    cin>>word;
    a.m=word;
}

то компилятор ругается. Как выделить память внутри класса?

@ValeryLaptev · 05.10.2011, 09:58

Adept88, ругается он потому, что массивы-то присваивать нельзя. А про выделение памяти - читай про new - там много страниц написано, в пост на форуме не влезет...

@solar_wind · 05.10.2011, 09:58

Если нигде не ошибся, то так должно работать

C++

#include <iostream>
using namespace std;
 
class word
{
    public:
    char *m;
word::word()
{
m=new char[20];
}
word::~word()
{
delete [] m;
}
};
 
 
int main()
{
    word a, b;
    cout<<"Введите слово1: ";
    cin>>a.m;
    cout<<"Введите слово2: ";
    cin>>b.m;
    cout<<a.m<<" "<<b.m;
    system("pause");
}

@Adept88 · 05.10.2011, 10:44 **[ТС]**

Спасибо. А как реализовать, запихнув массив в private?

C++

#include <iostream>
using namespace std;
 
class word
{
    public:
        word()
        {
            m=new char[20];
        }
        void add(char x[20]) { m = x;}
        char* get() { return m;}
    private:
        char *m;
};
 
 
int main()
{
    word a, b;
    char x[20];
    cout<<"Введите слово1: ";
    cin>>x;
    a.add(x);
    cout<<"Введите слово2: ";
    cin>>x;
    b.add(x);
    cout<<a.get()<<" "<<b.get();
    system("pause");
    return 0;
}

@Deviaphan · 05.10.2011, 10:53

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

то компилятор ругается. Как выделить память внутри класса?

Компилятор ругается на присваивание, а не на выделение памяти. Читай ошибки внимательнее.

@solar_wind · 05.10.2011, 10:56

Не не....так нельзя....функцию add вообще убери.
Когда ты создал экземпляр класса, у тебя уже есть указатель и память под данные.
Используя get() ты получаешь указатель и теперь можешь работать с данными напрямую.
Если ты присвоишь указателю ссылку на статический массив, то ты потеряешь ту область памяти, которую выделял. Короче читай про использование указателей и про классы!

@Adept88 · 05.10.2011, 11:49 **[ТС]**

Спасибо всем, разобрался

C++

#include <iostream>
using namespace std;
 
class word
{
    public:
        void add(char *x) { m = new char(strlen(x)-1); strcpy (m, x);}
        char* get() { return m;}
    private:
        char *m;
};
 
 
int main()
{
    word a, b;
    char x[20];
    cout<<"Введите слово1: ";
    cin>>x;
    a.add(x);
    cout<<"Введите слово2: ";
    cin>>x;
    b.add(x);
    cout<<a.get()<<" "<<b.get();
    system("pause");
    return 0;
}

@solar_wind · 05.10.2011, 11:54

Adept88, Это намного лучше
однако что тебе вернет strlen(x) это вопрос.....под дебагером это будет действительно длина строки, а вот в релизе совсем не факт....так что длину лучше тоже передавай в функцию add.

@Deviaphan · 05.10.2011, 11:54

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

void add(char *x) { m = new char(strlen(x)-1); strcpy (m, x);}

Замени на +

@ValeryLaptev 1067 / 846 / 60 Регистрация: 30.04.2011 Сообщений: 1,659
	05.10.2011, 09:58	4
	Adept88, ругается он потому, что массивы-то присваивать нельзя. А про выделение памяти - читай про new - там много страниц написано, в пост на форуме не влезет... 0

@Deviaphan Делаю внезапно и красиво 1312 / 1227 / 72 Регистрация: 22.03.2011 Сообщений: 3,744
	05.10.2011, 10:53	7
	Сообщение от Adept88 то компилятор ругается. Как выделить память внутри класса? Компилятор ругается на присваивание, а не на выделение памяти. Читай ошибки внимательнее. 0

@solar_wind 770 / 760 / 59 Регистрация: 06.07.2009 Сообщений: 3,021
	05.10.2011, 10:56	8
	Не не....так нельзя....функцию add вообще убери. Когда ты создал экземпляр класса, у тебя уже есть указатель и память под данные. Используя get() ты получаешь указатель и теперь можешь работать с данными напрямую. Если ты присвоишь указателю ссылку на статический массив, то ты потеряешь ту область памяти, которую выделял. Короче читай про использование указателей и про классы! 0

@solar_wind 770 / 760 / 59 Регистрация: 06.07.2009 Сообщений: 3,021
	05.10.2011, 11:54	10
	Adept88, Это намного лучше однако что тебе вернет strlen(x) это вопрос.....под дебагером это будет действительно длина строки, а вот в релизе совсем не факт....так что длину лучше тоже передавай в функцию add. 0

@Deviaphan Делаю внезапно и красиво 1312 / 1227 / 72 Регистрация: 22.03.2011 Сообщений: 3,744
	05.10.2011, 11:54	11
	Сообщение от Adept88 void add(char *x) { m = new char(strlen(x)-1); strcpy (m, x);} Замени на + 1