Наследование классов

@InSpectr · Регистрация: 09.09.2013

Author24 — интернет-сервис помощи студентам

Здравствуйте, начал изучать наследование классов и не могу понять как ведут себя конструкторы с параметром.
Вот код, но он печатает на экран не то что нужно. Как я понимаю я не правильно оформил конструкторы с параметром, подскажите как надо.

C++

#include "stdafx.h"
#include <iostream>
#include <Windows.h>
using namespace std;
class Fluid
{
protected:
    float density;
    char *name;
 
public:
    Fluid()
    {
        name="0";
        density=0.0;
    }
    Fluid(char name[30], float temp)
    {
        this->name=new char[30];
        strcpy(this->name, name);
        density=temp;
    }
    ~Fluid()
    {
        cout<<endl<<"the end";
    }
 
};
class Alcohol:public Fluid
{
private:
    int strength;
public:
    
    Alcohol()
    {
        strength=0;
    }
    Alcohol(char name[30], float temp,int pro)
    {
        strength=pro;
    }
    ~Alcohol()
    {
        cout<<endl<<"the end";
    }
    void print()
    {
        cout<<name<<" "<<density<<" "<<strength<<endl;
    }
};
    
 
int main()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    Alcohol x("Водка",3.45,40);
    x.print();
    return 0;
}

@ninja2 · 04.11.2013, 01:36

Попробуй явно записать:

C++

Alcohol(char name[30], float temp,int pro)
:Fluid(name,temp)
    {
        strength=pro;
    }

Как то так вызвать конструктор базового класса, похоже у тебя вызывается конструктор базового класса который по умолчанию Fluid()

@InSpectr · 04.11.2013, 02:39 **[ТС]**

спасибо, работает.

MrGluck · 04.11.2013, 02:56

В деструкторе не освобождается память. И, кстати, сразу вытечет ошибка, связанная с тем, что в конструкторе по умолчанию у вас name становится указателем на литерал. Надо приравнивать к NULL/nullptr.
Нет оператора присваиваний.
Что значит 30? Почему не 26, не 78, не 256? Каждый раз надо будет писать этот magic number по памяти или все же стоит оформить его в константу?

@InSpectr 10 / 10 / 1 Регистрация: 09.09.2013 Сообщений: 53
	04.11.2013, 02:39 [ТС]	3
	спасибо, работает. 0

MrGluck Форумчанин 8216 / 5046 / 1437 Регистрация: 29.11.2010 Сообщений: 13,453
	04.11.2013, 02:56	4
	В деструкторе не освобождается память. И, кстати, сразу вытечет ошибка, связанная с тем, что в конструкторе по умолчанию у вас name становится указателем на литерал. Надо приравнивать к NULL/nullptr. Нет оператора присваиваний. Что значит 30? Почему не 26, не 78, не 256? Каждый раз надо будет писать этот magic number по памяти или все же стоит оформить его в константу? 0