Проблема с объектами С++

[bobby2008]

Здравствуйте, нужна ваша помощь. Самое простое задание - создать объект "товар" с полями "название" и "производитель". В общем вот что сделал я:

//

#include "stdafx.h"
#include <conio.h>
#include <iostream>
#include <string>

using namespace std;


class T_Product
{char* Name;
 char* Producer;
public:
    T_Product();
    T_Product(char*,char*);
    ~T_Product(){cout<<"Bye!";};
    void Show();
};

T_Product::T_Product()
{
    strcpy(Name,"");
    strcpy(Producer,"");
}
T_Product::T_Product(char* a,char* b)
{
    char* Name=new char;
    char* Product=new char;
    strcpy(Name,a);
    strcpy(Producer,b);
}

void T_Product::Show()
{
    cout<<"Name: "<<Name<<endl;
    cout<<"Producer: "<<Producer<<endl;
}

int main()
{
    char* a=new char;
    char* b=new char;
    cout<<"Input Name of the pruduct:"<<endl;
    cin>>a;
    cout<<"Input Producer of the product:"<<endl;
    cin>>b;
    cout<<a<<"\n"<<b;
    getch();
    T_Product tov1(a,b);
    tov1.Show();
    getch();
    return 0;
}

Вот что в результате получаю на строчке

Т_Product tov1(a,b);

Unhandled exception at 0x5bc0d337 (msvcr100d.dll) in Sasha.exe: 0xC0000005: Access violation writing location 0xcccccccc.

Что я неправильно сделал, подскажите, пожалуйста? Спасибо за ответы.

[pproger]

в конструкторе по умолчанию не выделяешь память. в конструкторе с параметрами и функции main выделяешь память для 1 символа, когда функция strcpy ожидает c-строку.

не используй в плюсах char без особой надобности. используй std::string

более менее правильный c++ код

#include <iostream>
#include <string>

using namespace std;

//-------------------------------------------------------------------------------------
class T_Product {
public:
    T_Product();
    T_Product(string &, string &);
	~T_Product();
	void Show();

private:
	string name;
	string producer;
};

//-------------------------------------------------------------------------------------
T_Product::T_Product()
	: name(""), producer("")
{}

//-------------------------------------------------------------------------------------
T_Product::T_Product(string &a, string &b)
	: name(a), producer(b)
{}

//-------------------------------------------------------------------------------------
T_Product::~T_Product()
{
	cout<<"Bye!";
}

//-------------------------------------------------------------------------------------
void T_Product::Show()
{
    cout << "Name: "<< name << endl;
    cout << "Producer: "<< producer << endl;
}

//-------------------------------------------------------------------------------------
int main()
{
    string a, b;
    cout << "Input Name of the pruduct:" << endl;
    cin >> a;
    cout << "Input Producer of the product:" << endl;
    cin >> b;
    T_Product tov1(a, b);
    tov1.Show();
    return 0;
}

[bobby2008]

pproger, большое спасибо. Все работает. Но я не очень понимаю запись

T_Product::T_Product(string &a, string &b)
    : name(a), producer(b)
{}

Здесь наверное выделяется память для строки а и b и записывается туда значение, но можно это как-то написать через new и cin? Заранее спасибо за пояснения.
ЗЫ Извините за нубство, просто на лекции толком ничего не рассказали, а разобраться хочется.

[Руслантус]

a и b - это аргументы конструктора. В конструктор, по ссылке, передаются заранее созданные объекты string.

: name(a), producer(b) - данная запись, равносильна следующей записи:

T_Product::T_Product(string &a, string &b)
{
    name = a;
    producer = b;
}

То есть, это укороченная запись инициализации членов объекта.

string a, b;
cout << "Input Name of the pruduct:" << endl;
cin >> a;
cout << "Input Producer of the product:" << endl;
cin >> b;

Здесь считываются введённые строки из консоли и записываются в a и b.

Класс string сам отвечает за управление и выделение памяти, поэтому, тебе не нужно об этом беспокоится.

[bobby2008]

Спасибо, все понял. Тему можно закрывать.