C++: как передать объект другому конструктору объекта?

Question

Я знаю, что этот вопрос задан очень много, но я пришел из Java, и я не делал никаких C/C++ в течение очень долгого времени.

Не могли бы вы напомнить мне, как правильно передать объект другому конструктору объекта в C++?

Например, мне нужно, чтобы передать новый свет объекта в конструкторе Button:

// LED Light 
class Light { 
    int pin; 

    public: 
     Light(int p) { 
      pin = p; 
     } 
}; 

// BUTTON 
class Button { 
    Light ledLight; 

    public: 
     Button(Light l) { 
      ledLight = l; 
     } 

}; 

Light my_led(0); 

Button my_button(my_led); 

Это, как я хотел бы сделать это в Java-как фьюжн. Однако это приводит следующее сообщение об ошибке:

:: In constructor ‘Button::Button(Light)’: 
:: 16:19: error: no matching function for call to ‘Light::Light()’ 
:: 16:19: note: candidates are: 
:: 6:3: note: Light::Light(int) 
:: 6:3: note: candidate expects 1 argument, 0 provided 
:: 2:7: note: Light::Light(const Light&) 
:: 2:7: note: candidate expects 1 argument, 0 provided 

ли объект, передаваемый по ссылке или он пытается создать новую, когда я создаю новую кнопку?

Или мне нужно объявить свет как указатель в конструкторе Button?

Любая помощь очень высоко ценится!

Answer 1

Вам не нужен конструктор по умолчанию. Попробуйте изменить свой класс, как это:

class Light { 
    int pin; 

    public: 
     Light()=default; //<-- put this in 
     Light(int p) { 
      pin = p; 
     } 
}; 

Относительно того, почему вы не автоматически генерировать конструктор по умолчанию, это потому, что вы уже объявить конструктор некоторого типа, так что компилятор не будет генерировать по умолчанию один в (от http://en.cppreference.com/w/cpp/language/default_constructor, см. разделы о неявно объявленных/определенных конструкторах).

---

В ответ на комментарий:

Wouldn't it attempt to call the default constructor instead of the one which passes the arguments?

Когда выполнение кода дойдет до последней строки:

Button my_button(my_led); 

Вы инстанцировании ButtonLight и как часть Button процесс создания экземпляра. Не указывая конструктор по умолчанию Light::Light(), вы оставляете инструкции о том, как создать экземпляр объекта Light без значения, которое необходимо выполнить перед копированием значения my_led в my_button. Вы получите результаты, которые вы намеревались, в настоящее время вы оставляете инструкции для компилятора о том, как выполнить промежуточный шаг, то есть создать экземпляр Light без каких-либо аргументов.

---

Некоторые вещи, чтобы думать:

Что вы действительно пытается сделать?Просто глядя на ваш код инструкции:

Light my_led(0); 

Button my_button(my_led); 

Мне кажется, что вы пытаетесь сказать, «свет по имени my_led существует» и «кнопка под названием my_button существует и нуждается в my_led». Вы хотите, чтобы кнопка ссылалась на определенный светодиод, который уже существует, или только светодиод, который имеет одинаковые значения?

Если вы думаете, вы хотите сослаться на фактический LED от этой кнопки, то вы можете рассмотреть возможность сделать следующее:

class Button { 
    const Light *ledLight; 

    public: 
     Button(const Light& l) : 
      ledLight{&l} 
     {} 
}; 

Таким образом, ваш Button имеет указатель на Light вы создали , и все, что вы делаете с Light внутри вашего кода Button, будет выполнено на объекте Light, который вы уже создали, вместо отдельной копии Light, которую вы создаете путем копирования-построения Light с вашим текущим конструктором Button.

Если вы хотите, чтобы кнопка на самом деле изменить свет, пересмотреть Button так:

class Button { 
    // constant reference to changeable object 
    Light * const ledLight; 

    public: 
     /** 
     * You're providing a reference to a light that can be changed 
     * (i.e. button code can toggle lit status or something) 
     */ 
     Button(Light & l) : 
      ledLight{&l} 
     {} 
}; 

выше вы могли бы думать, как имеющие неизменный провод от кнопки на LED. Если вы думаете, что будете переназначать светодиод, который горит от кнопки, вы также вытащите ключевое слово const из элемента указателя в Button.

Answer 2

Программа пытается инициировать ledLight с использованием конструктора по умолчанию, а затем назначает l - ledLight. Компилятор выдает сообщение об ошибке, потому что нет конструктора по умолчанию для Light. Попробуйте это вместо:

Button(Light l): ledLight(l) { }