С++, классы

[yankel]

#include <iostream>
#include <stdlib.h>
#include <conio.h>
#include <string.h>
#include <stdio.h>
#include <iomanip>
#include <windows.h>
#include <malloc.h>
#define N 3
using namespace std;
class office {
 char *name;
 char *creater;
 unsigned int count;
 float price;
public:
 void set(char *a,char *b,unsigned int c, float d);
 void get(char *a, char *b, unsigned int &c, float &d);
 void show(void);
};

void office::set(char *a,char *b,unsigned int c,float d) {
 a=new char [strlen(name)+1];
 strcpy(name,a);
 b=new char [strlen(creater)+1];
 strcpy(creater,b);
 count=c;
 price=d;
}

void office::get( char * a, char *b, unsigned int &c, float &d)
{
 a=new char [strlen(name)+1];
 strcpy(a,name);
 b=new char [strlen(creater)+1];
 strcpy(b,creater);
 c=count;
 d=price;
}

void office::show(void) {
 cout<<name<<" ";
 cout<<creater<<" ";
 cout<<count<<" ";
 cout<<price<<" ";
}


int main(void) {
  char *n;
  n=(char *)malloc(80*sizeof(char));
  char *t;
  t=(char *)malloc(80*sizeof(char));
  unsigned int s;
  float h;
  short i;
SetConsoleCP(1251);
SetConsoleOutputCP(1251); 
 
 office obj[N];
 system("cls");
 cout<<"Работа функции SET!\n";
 for (i=0; i<N; i++)
 {           
   cout<<"Название, Производитель, Количество, Цена: ";
   cin>>n;
   cin>>t;
   cin>>s;
   cin>>h;
   obj[i].set(n,t,s,h);
 }
 cout<<"Работа функции SHOW!\n";
 cout<<"Название, Производитель, Количество, Цена: \n";
 for (i=0; i<N; i++)
 {
   obj[i].show();
   cout<<"\n";
  }
 cout<<"Работа функции GET и SHOW!\n";
 cout<<"Название, Производитель, Количество, Цена: \n";
 for(i=0; i<N; i++)
 {
   obj[i].get(n,t,s,h);
   obj[i].show();
   cout<<"\n";
  }
getch();
delete[] n;
delete [] t;
return 0;
}

При отладке выдает следующее: Необработанное исключение в "0x53c8f8e0" в "45.exe": 0xC0000005: Нарушение прав доступа при чтении "0xcccccccc".

Заранее спасибо за помощь

[Alex11223]

Странный код. Сделайте отдельные функции get/set работающие с одним свойством, а не всеми сразу. И если C++, то используйте std::string для строк. И либо new/delete (C++), либо malloc/free (C), а не вперемешку.

[yankel]

Спасибо,

И если C++, то используйте std::string для строк. И либо new/delete (C++), либо malloc/free (C), а не вперемешку.

- принято к сведению:

#include <iostream>
#include <stdlib.h>
#include <conio.h>
#include <string.h>
#include <stdio.h>
#include <iomanip>
#include <windows.h>
#define N 3
using namespace std;
class office {
 char *name;
 char *creater;
 unsigned int count;
 float price;
public:
 void set(char *a,char *b,unsigned int c, float d);
 void get(char *a, char *b, unsigned int &c, float &d);
 void show(void);
};

void office::set(char *a,char *b,unsigned int c,float d) {
 a=new char [strlen(name)+1];
 strcpy(name,a);
 b=new char [strlen(creater)+1];
 strcpy(creater,b);
 count=c;
 price=d;
}

void office::get( char * a, char *b, unsigned int &c, float &d)
{
 a=new char [strlen(name)+1];
 strcpy(a,name);
 b=new char [strlen(creater)+1];
 strcpy(b,creater);
 c=count;
 d=price;
}

void office::show(void) {
 cout<<name<<" ";
 cout<<creater<<" ";
 cout<<count<<" ";
 cout<<price<<" ";
}


int main(void) {
  char *n;
  char *t;
  unsigned int s;
  float h;
  short i;
SetConsoleCP(1251);
SetConsoleOutputCP(1251); 
 
 office obj[N];
 system("cls");
 cout<<"Работа функции SET!\n";
 for (i=0; i<N; i++)
 {           
   cout<<"Название, Производитель, Количество, Цена: ";
   cin>>n;
   cin>>t;
   cin>>s;
   cin>>h;
   obj[i].set(n,t,s,h);
 }
 cout<<"Работа функции SHOW!\n";
 cout<<"Название, Производитель, Количество, Цена: \n";
 for (i=0; i<N; i++)
 {
   obj[i].show();
   cout<<"\n";
  }
 cout<<"Работа функции GET и SHOW!\n";
 cout<<"Название, Школа, Количество монахов, Площадь земли: \n";
 for(i=0; i<N; i++)
 {
   obj[i].get(n,t,s,h);
   obj[i].show();
   cout<<"\n";
  }
getch();
delete[] n;
delete [] t;
return 0;
}

Сделайте отдельные функции get/set работающие с одним свойством, а не всеми сразу.

- поясните, потому что на мой ничему не обученный взгляд, так и происходит

[Alex11223]

string getName();
void setName(string newName);

float getPriice();
void setPrice(float newPrice);

[yankel]

Да но задача поставлена так, что эти функции должны быть разработаны

[p51x]

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