Звезда jQuery оценка плагина - отключает после представления голосования

Список инициализаторов

Чего вам не хватает, так это списка инициализаторов.

Type::Type(Parameters)
    : member1(init)       // This is the initializer list
    , member2(init)
{
    Your code
}

Если вы явно не предоставите его, компилятор сделает это за вас, используя конструктор по умолчанию для родительского класса, а затем вызов конструктора по умолчанию для каждого члена.

Итак, давайте посмотрим на ваш класс.

Student::Student(const std::string & name, int age, const std::string & course)
{
     // Code
}

Это то, что вы написали. Но это то, что реализовал компилятор.

Student::Student(const std::string & name, int age, const std::string & course)
    : Person()
    , course()
{
     // Code
}

Так как вы ничего не сделали, компилятор добавил, что он вызывает конструктор по умолчанию Person и конструктор по умолчанию course (std::string).

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

Но есть и проблема в том, что то, как вы пишете, очень неэффективно, поскольку вы в основном инициализируете всех членов дважды. Вы вызываете конструктор по умолчанию, а затем в разделе Code вы повторно инициализируете элементы с другим значением.

Student::Student(const std::string & name, int age, const std::string & course)
    : Person(name, age)
    , course()                       // Initialize it to empty here.
{
    course = "[unassigned course]";  // Re-Initialize it with info.
}

Вы можете просто сделать это один раз:

Student::Student()
    : Person()                        // Note person must have a default constructor for this to work.
    , course("[unassigned course]")
{}

Временный объект

Student::Student(const std::string & name, int age, const std::string & course)
{
    Person(name, age);
    // CODE.
}

Это не то, что вы думаете.
Добавим список инициализаторов.

Student::Student(const std::string & name, int age, const std::string & course)
    : Person()
    , course()
{
    Person(name, age);   // This is creating a person object localy.
                         // the person object has no name so is a
                         // temporary variable and goes out of scope
                         // at the ';'. So it is created and destroyed
                         // in place before other code is executed.
                         //
                         // This does not help initialize the class.
                         // It is creating a completely different
                         // and independent object.
    // CODE.
}

Вы можете увидеть выполнение здесь:

Hello from Person::Person()                       // Init parent
Hello from Person::Person(string, int)            // Init the temp object.
                                                  // If you put a print in the destructor
                                                  // You will also see that executed
                                                  // Before the Student
                                                  // This is also why the object has "unknown name" and 0 age. 

Hello from Student::Student(string, int, string)  // Now we init the student
[unknown name], 0
Physics

Совет

Есть ли допустимый сценарий, в котором вы хотите инициализированные версии объекта? Лично я думаю, что нет (если есть тогда игнорировать это), поэтому избавиться от конструкторов по умолчанию Person и Student. Тогда вы не можете создавать неинициализированные Students или `People.