Констсруктор и дестуктор класса

[SilverFish]

Если, к примеру, у меня есть такой класс:

Код

class TMyData : public TObject
{
public:
     int Value;
      int CoutSens;
   String Login;
        bool Aut;
       bool ProverkaSetup;
     int PeriodTime;
 String Otvet;
        __fastcall TMyData(const int &aValue, const int &aCoutSens, const String &sLogin, const bool &sAut, const bool &sProverkaSetup,
                        const int &aPeriodTime, const String &aOtvet)
   : TObject(), Value(aValue), CoutSens(aCoutSens), Login(sLogin), Aut(sAut), ProverkaSetup(sProverkaSetup),
         PeriodTime(aPeriodTime), Otvet(aOtvet) {}
};
 

Я хочу добавить в этот класс переменную:
BYTE RamdomData[16];
Как правильно добавить инициализацию этой переменной в конструктор класса? Как будет правильно выглядеть деструктор чтобы очищать эту переменную?
Спасибо.

[d7d1cd]

Наверное вы имеете ввиду "как правильно выделить память для данной переменной"? Можно так и добавить:

Код

class TMyData : public TObject
{
public:
       int Value;
      int CoutSens;
   BYTE RamdomData[16]; // Тут память выделится сразу, при объявлении переменной данного класса
     ...
 

А можно так:

Код

class TMyData : public TObject
{
public:
 int Value;
      int CoutSens;
   BYTE *RamdomData; // Тут память не будет выделена
  ...
 
// В конструкторе:
RamdomData = new BYTE[16]; // Выделяю память
 
// В деструкторе:
delete RamdomData; // Освобождаю память

[SilverFish]

Если тогда такой код:

Код

class TMyData : public TObject
{
public:
 int Value;
      int CoutSens;
   String Login;
        bool Aut;
       bool ProverkaSetup;
     int PeriodTime;
 String Otvet;
        BYTE  * RamdomData;
//не правильно
__fastcall TMyData(const int &aValue, const int &aCoutSens, const String &sLogin, const bool &sAut, const bool &sProverkaSetup,
                       const int &aPeriodTime, const String &aOtvet, const BYTE & RamdomData)
    : TObject(), Value(aValue), CoutSens(aCoutSens), Login(sLogin), Aut(sAut), ProverkaSetup(sProverkaSetup),
         PeriodTime(aPeriodTime), Otvet(aOtvet) {RamdomData = new BYTE[16];}//нельзя менять const
};
 

И как передовать в конструктор?

Код

   BYTE Data[16];
   Randomize();
   for(int i = 1; i  16; i++)
             {
                BYTE rnd = Random(256);
          Data[i] = rnd;
//*** код ***
 
//не правильно:
TMyData OurData = new TMyData(0, 0,"", false, false, 0, Otvet, Data);
 

[OVE]

У Вас совпадает имя члена класса и параметра, передаваемого в конструктор (RamdomData), поэтому компилятор и ругается.
Можно сделать так

Код:

class TMyData : public TObject
{
public:
int Value;
int CoutSens;
String Login;
bool Aut;
bool ProverkaSetup;
int PeriodTime;
String Otvet;
BYTE  * RamdomData;
__fastcall TMyData(const int &aValue, const int &aCoutSens, const String &sLogin, const bool &sAut, const bool &sProverkaSetup,
const int &aPeriodTime, const String &aOtvet, const BYTE & aRamdomData)
: TObject(), Value(aValue), CoutSens(aCoutSens), Login(sLogin), Aut(sAut), ProverkaSetup(sProverkaSetup),
PeriodTime(aPeriodTime), Otvet(aOtvet) {RamdomData = new BYTE[16]; memcpy(RamdomData,aRamdomData,16);}
};

В деструкторе

Код:

delete RamdomData;

[SilverFish]

Спасибо.
Теперь тут ругается:
memcpy(RamdomData,aRamdomData,16);
Cannot convert 'int' to 'const void *'

[OVE]

попробуйте так

Код:

memcpy((void*)RamdomData,(void*)aRamdomData,16)

[SilverFish]

Спасибо, так работает, но я не соображу как создать объект.

Код

//***
BYTE  * RamdomData;
__fastcall TMyData(const int &aValue, const int &aCoutSens, const String &sLogin, const bool &sAut, const bool &sProverkaSetup,
                       const int &aPeriodTime, const String &aOtvet, const BYTE & aRamdomData)
   : TObject(), Value(aValue), CoutSens(aCoutSens), Login(sLogin), Aut(sAut), ProverkaSetup(sProverkaSetup),
         PeriodTime(aPeriodTime), Otvet(aOtvet)
        {
        RamdomData = new BYTE[16];
     memcpy((void*)RamdomData,(void*)aRamdomData,16);
   }
__fastcall ~TMyData()
  {
if(RamdomData)   
delete [] RamdomData;
     }
 

Код

   Randomize();
   BYTE Data[16];
   for(int i = 0; i  16; i++)
           {
                BYTE rnd = Random(256);
          Data[i] = rnd;
//****
 
TMyData OurData = new TMyData(0, 0,"", false, false, 0, "", Data);
 

Could not find a match for 'TMyData::TMyData(int,int,const char *,bool,bool,int,const char *,unsigned char *)'

[OVE]

Объявите конструктор следующим образом

Код:

__fastcall TMyData(int aValue, int aCoutSens, String sLogin, bool sAut, bool sProverkaSetup,
int aPeriodTime, String aOtvet, BYTE *aRamdomData)
: TObject(), Value(aValue), CoutSens(aCoutSens), Login(sLogin), Aut(sAut), ProverkaSetup(sProverkaSetup),
PeriodTime(aPeriodTime), Otvet(aOtvet)
{
  RamdomData = new BYTE[16];
  memcpy((void*)RamdomData,(void*)aRamdomData,16);
}

[SilverFish]

Could not find a match for 'TMyData::TMyData(const TMyData&)'
 

[OVE]

При вызове оператора new создается указатель на новый объект.

Код:

TMyData * OurData = new TMyData(0, 0,"", false, false, 0, "", Data);

[SilverFish]

Спасибо. Досадная оплошность.

[SilverFish]

Объясните, пожалуйста, а почему надо везде убрать const?
Переменные const устанавливаются в списки инициализации, а RamdomData в теле конструктора. Поэтому почему это не работает?

Код

__fastcall TMyData(const int &aValue, const int &aCoutSens, const String &sLogin, const bool &sAut, const bool &sProverkaSetup,
                        const int &aPeriodTime, const String &aOtvet, BYTE & aRamdomData)
      : TObject(), Value(aValue), CoutSens(aCoutSens), Login(sLogin), Aut(sAut), ProverkaSetup(sProverkaSetup),
         PeriodTime(aPeriodTime), Otvet(aOtvet)
        {
        RamdomData = new BYTE[16];
     memcpy((void*)RamdomData,(void*)aRamdomData,16);
   }
 
__fastcall ~TMyData()
      {
        if(RamdomData)
        delete [] RamdomData;
   }
 

[OVE]

const можно оставить
главное - последний параметр должен быть указатель на char, а не ссылка.(... BYTE * aRamdomData)

[SilverFish]

Спасибо.