Привет!

1. Register::Register(size_t numCompanies){...}

numAdded = 0; тази стойност ти отговаря на колко компании си добавил в масива, а не на големината на масива. В този конструктор само заделяме памет за някакъв брой компании, но все още не сме добавили такива.

2. void Register::add(const Company& c){...}

Тук numAdded се използва като индекс на който да добавиш дадената компания.

Тъй като в Register.h не е заложено size, а от друга страна в RegisterMain.cpp се поверява дали сме достигнали до капацитета на масива, то не е нужно да правиш проверки дали не си достигнал size преди да добавиш новият елемент.

3. Company Register::get(int companyId) const{...}

Тук задължително трябва да върнеш променлива, в случая обект от тип Company. Тъй като имаш само if то ако не влезе там не се знае какво ще върне като резултат от този метод. Поради това след цъкала добави и return Company(); ако искаш може да зададеш и да ти е с недействителен, невъзможен id за компания, примерно -1, но и така ще ти свърши работа, както съм го написал. След return няма смисъл от break.

4. Register& Register::operator=(const Register& other){...}

Тук имаш memory leak. Не си освободил старата памет, ако преди това е заделена такава.

Примерен вариант как да подходиш:

if (this != &other) {
    if (this->companiesArray != nullptr) {
        delete[] this->companiesArray;
    }

    this->numAdded = other.numAdded;
    this->companiesArray = new Company[other.numAdded];

    int arrSize = (size_t) other.numAdded;
    for (int i = 0; i < arrSize; ++i) {
        this->companiesArray[i] = other.companiesArray[i];
    }
}

5. Register::Register(const Register& other){...}

Company* companiesArray - това е променлива от тип пойнтър, което ще рече, че съхнанява едно число, което само по себе си е обикновена size_t стойност(цяло число без знаково), но то отговаря номера на адреса на дадена клетка в RAM-та - клетаката от която започва заделената динамично памет на масива в other.

Така както си го направил ти даваш това число на настоящият обект, който създаваш или с други думи при дереференциране на other или на this(настоящият обект) ще достъпват една и съща памет, т.е. споделят обща референция. Проблеме е, че когато единия излезе извън неговия скоуп, то ще се извика деструктора му и ще освободи тази памет. След това и другия обект ще излезе извън скоуп и на него щу му се викне деструктора и ще каже освободи ми същата памет. Имаш недефенирано поведение тук - едно, че ще освобождаваш два пъти една и съща памет, но също така може да пробваш да достъпиш освободената памет, чрез някой от обектите ти, който все още не е излязал извън скоупа си.

Решението е подобно на точка 4, като няма памет за осбождаване, тъй като сега създаваш новия обект.

Поздрави!