C++ Advanced Judge Assignment Task2 - Register of Three - Софтуерен университет

ValentinT 4 Точки

C++ Advanced Judge Assignment Task2 - Register of Three

Здравейте,

Това са ми copy конструктора и copy assignment оператора

Register& Register::operator=(const Register& other)
{
   if (this != &other)
   {
       this->companiesArray = other.companiesArray;
       this->numAdded = other.numAdded;
   }
   return *this;
}

Програмта ми се чупи и връща Exception thrown: read access violation.
this was 0xBC1ECA00. когато стигне до реда return companyRegister.get(id); в doRun метода, а това е мястото където отива

if _ITERATOR_DEBUG_LEVEL == 2
if (_Myproxy != nullptr)
{ // proxy allocated, drain it

ред 216 във файла xutility, имате ли представа откъде може да идва тази грешка ?

Поздрави

Тагове:

23/10/2018 20:41:23 C++ Programming

ValentinT 4 Точки

Чрез дебъгване установих че в get метода не намира обект с такова id и връщам празен Company обект, кеото чупи програмата

Company Register::get(int companyId) const
{
   for (size_t i = 0; i < this->numAdded; i++) {
       if (companiesArray[i].getId() == companyId) {
           return companiesArray[i];
       }
   }
   return Company();

}

това е имплементацията на метода, бихте ли ми помогнали с откриването какво не е наред

23/10/2018 21:08:47

Jovanna 186 Точки

Здравей,

мисля, че се опитваш да копираш полетата директно, което не върши работа тук. Виж тази тема:

възможно ли е директно копиране на полетата при copy-assignment operator=

Иначе, ето моя код:

Register& Register::operator=(const Register& other) {
Company* newData;
   if (other.numAdded == 0) {
       newData = nullptr;
   }
   else {
       newData = new Company[other.numAdded]{};
   }
   this->numAdded = other.numAdded;
   this->companiesArray = newData;
   for (size_t i = 0; i < other.numAdded; i++) {
       this->companiesArray[i] = other.companiesArray[i];
   }
   return *this;
}

23/10/2018 21:31:19

ValentinT 4 Точки

Здравей,

много е вероятно да си права, дори се сещам сега в презентацията пишеш за правилото на 3-те че полетата се слагат в локални променливи преди да се сетнат this Полетата. Много благодаря

Поздрави

23/10/2018 21:52:34