Структура\Класс студент..Применение конструкторов.

[8Observer8]

Немного теории

Важно, чтобы вы запомнили разницу между двумя случаями: стек и куча (new)

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

#include <iostream>
#include <string.h>

#define STRSIZE 30

class Student
{
public:
    Student( const char *name )
    {
        setName( name );
    }

    void setName( const char *name )
    {
        strcpy( m_name, name );
    }

    char *name()
    {
        return m_name;
    }

private:
    char m_name[STRSIZE];
};

int main()
{
    Student ivan( "Ivan" );
    Student ivan2( ivan );

    std::cout << ivan2.name() << std::endl;

    return 0;
}

А в примере из предыдущего моего сообщения используется выделение из кучи с помощью оператора new. Там обязательно нужен деструктор, чтобы удалить то, что выделено с помощью new из кучи, чтобы возвращать операционной системе занятый участок памяти. И ещё нужен конструктор копирования. А вот зачем он нужен - почитайте, например, здесь (первое, что нагуглил) http://cppstudio.com/post/9903/ Ну или ещё где-нибудь

P.S. Ждём ваш код

[8Observer8]

Я там протупил немного с передачей размера в setName(). Этого не надо делать. Ориентируйтесь на этот код:

#include <string.h>
#include <iostream>

class Student
{
public:
    Student() :
        defaultSize( 30 )
    {
        this->setDefaultName();
    }

    void setName( const char *name )
    {
        delete m_name;
        m_name = new char[ strlen( name ) + 1 ];
        strcpy( m_name, name );
    }

    Student( const Student &student ) :
        defaultSize( 30 )
    {
        this->setDefaultName();
        this->setName( student.name(), strlen( student.name() ) );
    }

    char *name() const
    {
        return m_name;
    }

    ~Student()
    {
        delete m_name;
    }

private:
    void setDefaultName()
    {
        m_name = new char[defaultSize];
        char *defaultName = "emptyName";
        strcpy( m_name, defaultName );
    }

private:
    char *m_name;
    const int defaultSize;
};

int main()
{
    Student ivan;
    char *name = "Ivan";
    ivan.setName( name );
    std::cout << ivan.name() << std::endl;

    Student ivan2( ivan );
    std::cout << ivan2.name() << std::endl;

    return 0;
}

[ACE Valery]

А точно надо? - я не уверен (поправьте если не прав), но мне кажется, что если конструктор копирования вообще не объявлять, то он будет создан автоматически и будет вести себя именно так, как вы описали.

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

[rrrFer]

ACE Valery

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

Я про это и говорил. В коде из первого поста динамически память не выделялась. Ну вот обсервер уже описал про это с примерами ))

Так то я стараюсь конструкторы копирования вообще не писать и память динамически не выделять. Обычно это получается и конструктора по умолчанию хватает. А еще утечек не бывает, с исключениями проблем нет и т.п.

Задача некорректно стоит. Требуют от ТС писать руками то, что компилятор сам автоматически напишет. Ну если в этой задаче string вместо С-строк юзать.

[8Observer8]

Задача некорректно стоит. Требуют от ТС писать руками то, что компилятор сам автоматически напишет. Ну если в этой задаче string вместо С-строк юзать.

У меня такое предположения, что условия задачи так поставлены, чтобы познакомить с конструктором копирования, показать зачем он нужен. Задача неудачно выбрана. Здесь, конечно, надо использовать С++-строки вместо С-строк:

#include <iostream>
#include <string>

class Student
{
public:
    Student( const std::string &name = "emptyName" ) :
        m_name( name )
    {
    }

    void setName( const std::string &name )
    {
        m_name = name;
    }

    std::string name() const
    {
        return m_name;
    }

private:
    std::string m_name;
};

int main()
{
    Student ivan;

    // Пустое имя
    std::cout << "Emty name: " << ivan.name() << std::endl;

    // Установленное имя
    std::string name( "ivan" );
    ivan.setName( name );
    std::cout << "Set name: " << ivan.name() << std::endl;

    // Конструктор копирования поумолчанию
    Student ivan2( ivan );
    std::cout << "With copy-constuctor: " << ivan2.name() << std::endl;

    return 0;
}

Вывод:

Emty name: emptyName
Set name: ivan
With copy-constuctor: ivan

P.S. Но с C-строками тоже надо уметь работать. Вот цитата из книжки "Professinal C++":

One of the most common situations is where
a C++ program has to call a C-based interface
in some third-party library or as part of
interfacing to the operating system.

То есть для работы с сторонними библиотеками (которые написаны на Си) и для взаимодействия с операционной системой, например, с Windows через WinAPI или с Linux через его системные вызовы

[rrrFer]

P.S. Но с C-строками тоже надо уметь работать
...
То есть для работы с сторонними библиотеками (которые написаны на Си) и для взаимодействия с операционной системой, например, с Windows через WinAPI или с Linux через его системные вызовы

Чтобы не работать с ними и с WinAPI и прочим есть высокоуровневые библиотеки.

Добрая половина матерых программистов рекомендует вместо прямого использования какого бы то ни было АПИ в своей программе, завернуть его в красивый фантик (шаблон типа Адаптер), покрыть фантик тестами и пользоваться им не зная бед.

Ну фантик нужен если например таки приходится работать со страшным АПИ.
Всякие С-строки кнешно будут нужны пока бы будешь обертку писать, но потом про них можно будет забыть.

[Otar4ik]

Большое спасибо за помощь!

[8Observer8]

Сообщение от Otar4ik

Большое спасибо за помощь!

Cheers Otar4ik!