Что не так в перегрузке оператора индексирования?

@Макрой · Регистрация: 04.12.2018

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

C++
1
Star& operator[](const int index);

C++
1
2
3
4
Star& Galaxy::operator[](const int index)
{
    return stars[index];
}

C++
1
2
3
4
Sputnik sp(name, 12, 3);
    Star star(12,3,1,name,4,sp);
    Galaxy gal(3);
    gal[0] = star;

Подскажите в чем ошибка ?

@Макрой · 23.07.2019, 01:30 **[ТС]**

IGPIGP, Я их выложу,для того что бы вы подсказали в чем проблема

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
#pragma once
#include"Sputnik.h"
#include<iostream>
#include<istream>
#include<ostream>
class Planet
{
private:
    char* name;
    int Way;
    Sputnik sputnik;
public:
    friend std::ostream& operator <<(std::ostream& out, const Planet& value);
    friend std::istream& operator >>(std::istream& in, Planet& value);
    
 
    Planet();
    Planet(char *name, int way, Sputnik value);
 
    void SetName(char* value);
    char* GetName()const;
    void SetWay(int value);
    int GetWay() const;
    void SetSpt(Sputnik sp);
    Sputnik GetSpt() const;
    Planet(const Planet &paramtr);
    void Info();
    virtual void One_of_fields();
    ~Planet();
};

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
#include "Planet.h"
#include<iostream>
#include<istream>
#include<ostream>
using namespace std;
void Planet::SetName(char* value)
{
    name = new char[strlen(value) + 1];
    strcpy_s(name, strlen(value) + 1, value);
}
char* Planet::GetName()const
{
    return name;
}
void Planet::SetWay(int value)
{
    Way = value;
}
int Planet::GetWay() const
{
    return Way;
}
void Planet::SetSpt(Sputnik value)
{
    sputnik = value;
}
Sputnik Planet::GetSpt()const
{
    return sputnik;
}
Planet::Planet()
{
    Way = 0;
    name = new char[80];
    name[79] = '\0';
}
Planet::Planet(char* Name, int way, Sputnik sp)
{
    name = new char[strlen(Name) + 1];
    strcpy_s(name, strlen(Name) + 1, Name);
    Way = way;
 
    sputnik.SetMass(sp.GetMass());
    sputnik.SetName(sp.GetName());
    sputnik.SetRadius(sp.GetRadius());
}
Planet::Planet(const Planet &value)
{
    name = new char[10];
    strcpy_s(name, 10, value.name);
    Way = value.Way;
    sputnik = value.sputnik;
}
 
std::ostream& operator << (std::ostream& out, const Planet& value)
{
 
    return(out << "Имя: " << value.GetName() << " ; Путь: " << value.GetWay());// << " ; Радиус спутника: " << value.GetSpt().GetRadius() << " ; Масса спутника: " << value.GetSpt().GetMass() << " ; Имя спутника: " << value.GetSpt().GetName());
}
 
std::istream& operator >> (std::istream& in, Planet& value)
{
 
    return in >> value.name >> value.Way;
}
void Planet::Info() {}
//{
//  cout << "Name:" << " " << name << " Way:" << " " << Way << " Sputnik:" << " " << " His Name:" << sputnik.GetName() << " " << " His Mass:" << sputnik.GetMass() << " " << " His Radius:" << sputnik.GetRadius();
//}
void Planet::One_of_fields() { cout << "virtual function" << endl; }
//{
//  cout << "Расстояние " << Way;
//}
 
Planet::~Planet()
{
    delete[]name;
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
#pragma once
#include"Planet.h"
class Star : public Planet
{
private:
    int time;
    int temperature;
    int diapasone;
 
public:
 
    Star();
    bool operator == (const Star& value);
    Star(bool value);
    Star(int _time, int _temp, int _diapasone, char * _name, int _way, Sputnik sp);
    void SetTime(int _time);
    int GetTime() const;
    void SetTemp(int _temp);
    int GetTemp()const;
    void SetDiapasone(int _diapasone);
    int GetDiapasone()const;
    void Info();
    void One_of_fields() override;
    Star(const Star &paramtr);
};

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
#include "Star.h"
#include<iostream>
using namespace std;
Star::Star()
{
    time = 0;
    temperature = 0;
    diapasone = 0;
}
Star::Star(int _time, int _temp, int _diapasone, char * _name, int _way, Sputnik sp) :
    Planet(_name, _way, sp)
{
    diapasone = _diapasone;
    time = _time;
    temperature = _temp;
}
void Star::SetTime(int _time)
{
    time = _time;
}
int Star::GetTime()const
{
    return time;
}
void Star::SetTemp(int _temp)
{
    temperature = _temp;
}
int Star::GetTemp()const
{
    return temperature;
}
void Star::SetDiapasone(int _diapasone)
{
    diapasone = _diapasone;
}
int Star::GetDiapasone()const
{
    return diapasone;
}
void Star::One_of_fields() 
{
    cout << "ovveride fucntion" << endl;
    //cout << "Температура " << temperature << endl;
}
bool Star::operator == (const Star& value)
{
    return GetTime() == value.GetTime() & GetTemp() == value.GetTemp() & GetDiapasone() == value.GetDiapasone();
}
 
Star::Star(bool value) {}
void Star::Info()
{
    cout << "Название планеты,возле которой находится звезда: " << GetName();
    cout << " Расстояние от планеты к звезде: " << GetWay();
    cout << " Масса спутника: " << GetSpt().GetMass();
    cout << " Название спутника: " << GetSpt().GetName();
    cout << " Радиус спутника: " << GetSpt().GetRadius();
    cout << " Диапазон звезды: " << diapasone;
    cout << " Время звезды:" << time;
    cout << " Температура звезды: " << temperature;
}
Star::Star(const Star &value)
{
    time = value.time;
    temperature = value.temperature;
    diapasone = value.diapasone;
}

Вот классы Star и Planet

Добавлено через 1 минуту

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

завалить этот код можно задав слишком длинное имя

согласен,но я ввожу имя на 3 символа

@zayats80888 · 23.07.2019, 01:44

Макрой, отсутствует operator= для Star, так же кривое управление ресурсами, как и в Galaxy. Вангую те же проблемы и в классе Sputnik.

@Макрой · 23.07.2019, 02:39 **[ТС]**

zayats80888,

C++
1
Star operator =(const Star& v);

C++
1
2
3
4
5
6
7
Star Star::operator =(const Star& v)
{
    this->time = v.time;
    this->diapasone = v.diapasone;
    this->temperature = v.temperature;
    return *this;
}

добавил оператор для Star

@zayats80888 · 23.07.2019, 03:32

Сообщение от Макрой

добавил оператор для Star

Я имел ввиду Planet, т.к. в этом классе у вас char* name В Star его уберите или внутри него вызывайте Planet::operator=(v). Также принято возвращать ссылку, а не временный объект. То же относится и к Sputnik, если в нем динамическое управление ресурсами.

@Макрой · 23.07.2019, 16:48 **[ТС]**

zayats80888,
Вот посмотрите пожалуйста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
//Planet.h
#pragma once
#include"Sputnik.h"
#include<iostream>
#include<istream>
#include<ostream>
class Planet
{
private:
    char* name;
    int Way;
    Sputnik sputnik;
public:
    friend std::ostream& operator <<(std::ostream& out, const Planet& value);
    friend std::istream& operator >>(std::istream& in, Planet& value);
    
 
    Planet();
    Planet(char *name, int way, Sputnik value);
    Planet operator =(const Planet& v);
    void SetName(char* value);
    char* GetName()const;
    void SetWay(int value);
    int GetWay() const;
    void SetSpt(Sputnik sp);
    Sputnik GetSpt() const;
    Planet(const Planet &paramtr);
    void Info();
    virtual void One_of_fields();
    ~Planet();
};

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
//Planet.cpp
#include "Planet.h"
#include<iostream>
#include<istream>
#include<ostream>
using namespace std;
void Planet::SetName(char* value)
{
    name = new char[strlen(value) + 1];
    strcpy_s(name, strlen(value) + 1, value);
}
char* Planet::GetName()const
{
    return name;
}
void Planet::SetWay(int value)
{
    Way = value;
}
int Planet::GetWay() const
{
    return Way;
}
void Planet::SetSpt(Sputnik value)
{
    sputnik = value;
}
Sputnik Planet::GetSpt()const
{
    return sputnik;
}
Planet::Planet()
{
    Way = 0;
    name = new char[80];
    name[79] = '\0';
}
Planet::Planet(char* Name, int way, Sputnik sp)
{
    name = new char[strlen(Name) + 1];
    strcpy_s(name, strlen(Name) + 1, Name);
    Way = way;
 
    sputnik.SetMass(sp.GetMass());
    sputnik.SetName(sp.GetName());
    sputnik.SetRadius(sp.GetRadius());
}
Planet::Planet(const Planet &value)
{
    name = new char[10];
    strcpy_s(name, 10, value.name);
    Way = value.Way;
    sputnik = value.sputnik;
}
 
std::ostream& operator << (std::ostream& out, const Planet& value)
{
 
    return(out << "Имя: " << value.GetName() << " ; Путь: " << value.GetWay());// << " ; Радиус спутника: " << value.GetSpt().GetRadius() << " ; Масса спутника: " << value.GetSpt().GetMass() << " ; Имя спутника: " << value.GetSpt().GetName());
}
Planet Planet::operator =(const Planet& v)
{
    this->name = v.name;
    this->sputnik = v.sputnik;
    this->Way = v.Way;
    return *this;
}
std::istream& operator >> (std::istream& in, Planet& value)
{
 
    return in >> value.name >> value.Way;
}
void Planet::Info() {}
//{
//  cout << "Name:" << " " << name << " Way:" << " " << Way << " Sputnik:" << " " << " His Name:" << sputnik.GetName() << " " << " His Mass:" << sputnik.GetMass() << " " << " His Radius:" << sputnik.GetRadius();
//}
void Planet::One_of_fields() { cout << "virtual function" << endl; }
//{
//  cout << "Расстояние " << Way;
//}
 
Planet::~Planet()
{
    delete[]name;
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
//Sputnik.h
#pragma once
#include<string.h>
#include<ostream>
class Sputnik
{
private:
    char *name;
    double mass;
    double radius;
 
public:
    friend std::ostream& operator <<(std::ostream& out, const Sputnik& value);
    friend std::istream& operator >>(std::istream& in, Sputnik& value);
    Sputnik operator =(const Sputnik& v);
    Sputnik(double a) :radius(a) {};
    Sputnik(double a, double b, char* name2) :radius(a), mass(b)
    {
        name = new char[strlen(name2) + 1];
        strcpy_s(name, strlen(name2) + 1, name2);
    }
    Sputnik operator+(const Sputnik&);
    Sputnik();
    const Sputnik& operator ++() { ++radius;return *this; }
 
    double Get() { return radius; }
 
    double Get_m()
    {
        return mass;
    }
    Sputnik(char * name2, double mass2, double radius2);
 
    void SetName(char * value);
 
    char* GetName()const;
 
    void SetMass(double value);
 
    double GetMass()const;
 
    void SetRadius(double value);
 
    double GetRadius()const;
 
    void Info();
    void Show_one_of_fields();
    Sputnik(const Sputnik &parametr);
 
    ~Sputnik();
 
};

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
//Sputnik.cpp
#include "Sputnik.h"
#include "iostream"
#include<string.h>
#include<ostream>
using namespace std;
void Sputnik::SetName(char * value)
{
    delete[]name;
    name = new char[strlen(value) + 1];
    strcpy_s(name, strlen(value) + 1, value);
 
}
char* Sputnik::GetName()const
{
    return name;
}
void Sputnik::SetMass(double value)
{
    mass = value;
}
double Sputnik::GetMass()const
{
    return mass;
}
void Sputnik::SetRadius(double value)
{
    radius = value;
}
double Sputnik::GetRadius()const
{
    return radius;
}
Sputnik::Sputnik()
{
    name = new char[6];
    //name[79] = '\0';
    strcpy_s(name, 6, "First");
    mass = 0;
    radius = 0;
}
Sputnik::Sputnik(char * name2, double mass2, double radius2)
{
    name = new char[strlen(name2) + 1];
    strcpy_s(name, strlen(name2) + 1, name2);
 
    mass = mass2;
    radius = radius2;
}
Sputnik::Sputnik(const Sputnik &value)
{
    
    mass = value.mass;
    radius = value.radius;
    name = new char[strlen(value.name)+1];
    strcpy_s(name, strlen(value.name)+1, value.name);
}
void Sputnik::Info()
{
    cout << "Name:" << " " << name << " " << " Radius:" << " " << radius << " Mass:" << " " << mass << endl;
}
void Sputnik::Show_one_of_fields()
{
    cout << name;
}
Sputnik Sputnik :: operator+(const Sputnik& a)
{
    return Sputnik(GetRadius(), mass + a.mass, GetName());
}
std::istream& operator >> (std::istream& in, Sputnik& value)
{
 
    return in >> value.name >> value.mass >> value.radius;
}
std::ostream& operator <<(std::ostream& out, const Sputnik& value)
{
 
    return(out << "Имя: " << value.GetName() << " ; Масса: " << value.GetMass() << " ; Радиус: " << value.GetRadius());
}
Sputnik Sputnik::operator =(const Sputnik& v)
{
    this->mass = v.mass;
    this->name = v.name;
    this->radius = v.radius;
    return *this;
}
Sputnik ::~Sputnik()
{
    delete[]name;
 
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
//Galaxy.h
#pragma once
#include"Star.h"
#include<assert.h>
class Galaxy
{
private:
    int date;
    int length;
    Star* stars;
public:
    int Get_Date();
    void Set_Date(int value);
    int Get_Length();
    void Set_Length(int value);
    Star& operator[](const int index);
    Galaxy operator =(const Galaxy& v);
    Galaxy();
    
    Galaxy( int _length);
    Galaxy(int _data, Star* _satrs);
    ~Galaxy();
};

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
//Galaxy.cpp
#include "Galaxy.h"
#include"Star.h"
#include<iostream>
#include<assert.h>
using namespace std;
Galaxy::Galaxy()
{
 
    stars = new Star[0];
    length = 0;
    date = 0;
}
void Galaxy::Set_Date(int value)
{
    date = value;
}
int Galaxy::Get_Date()
{
    return date;
}
void Galaxy::Set_Length(int value)
{
    length = value;
}
int Galaxy::Get_Length()
{
    return length;
}
Star& Galaxy::operator[](const int index)
{
    return stars[index];
}
 
Galaxy::Galaxy( int _length)
{
  stars = new Star[_length];
    
}
Galaxy Galaxy::operator =(const Galaxy& v)
{
    stars = new Star[v.length];
    for (int i = 0;i < v.length;i++)
    {
        this->stars[i] = v.stars;
    }
    return *this;
}
 
Galaxy::Galaxy(int _data, Star*_stars )
{
    date = _data;
    stars->SetDiapasone(_stars->GetDiapasone());
    stars->SetName(_stars->GetName());
    stars->SetSpt(_stars->GetSpt());
    stars->SetTemp(_stars->GetTemp());
    stars->SetTime(_stars->GetTime());
    stars->SetWay(_stars->GetWay());
}
Galaxy::~Galaxy()
{
    delete[]stars;
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
//main
#include"Planet.h"
#include"Star.h"
#include"Galaxy.h"
#include<iostream>
#include<windows.h>
using namespace std;
int main()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    char* name = new char[10];
    cin >> name;
    Sputnik sp(name, 12, 3);
    Star star(12,3,1,name,4,sp);
    Galaxy gal(1);
    gal[0] = star;
    cout << gal[0] << endl;
    cin.get();
    cin.get();
    return 0;
}

В классе Star я убрал перегруженный оператор =

@zayats80888 · 23.07.2019, 17:20

Макрой,

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
//...
Sputnik::Sputnik(const Sputnik &value) :Sputnik(value.name, value.mass, value.radius) {}
//...
Sputnik& Sputnik::operator =(const Sputnik& v)
{
    delete[] name;
    size_t str_sz = strlen(v.name) + 1;
    name = new char[str_sz];
    strcpy_s(name, str_sz, v.name);
 
    mass = v.mass;
    radius = v.radius;
    return *this;
}
//...

Аналогично для остальных классов где есть поля указатели

Добавлено через 3 минуты

Не по теме:

Вам кто-то запрещает пользоваться std::string и std::vector?

IGPIGP · 23.07.2019, 17:27

Не по теме:

Сообщение от Макрой

IGPIGP, Я их выложу,для того что бы вы подсказали в чем проблема

Как-то медленно оно у вас. Постепенно) Но однако смотрите, уже пришла и подмога. Слушайте zayats80888, - он плохого не скажет).

@zayats80888 · 23.07.2019, 17:38

Макрой, забыл еще проверку на самоприсваивание, первой строчкой в операторе = добавьте if (this == &v) return *this;

Не по теме:

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

он плохого не скажет)

Да, я не ругаюсь, но от того, что бы не сморозить глупость, никто не застрахован :)

@Макрой · 23.07.2019, 18:05 **[ТС]**

zayats80888,

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
//Star.h
#pragma once
#include"Planet.h"
class Star : public Planet
{
private:
    int time;
    int temperature;
    int diapasone;
 
public:
    friend std::ostream& operator <<(std::ostream& out, const Star& value);
    Star();
    bool operator == (const Star& value);
    Star(bool value);
    Star(int _time, int _temp, int _diapasone, char * _name, int _way, Sputnik sp);
    void SetTime(int _time);
    int GetTime() const;
    void SetTemp(int _temp);
    int GetTemp()const;
    void SetDiapasone(int _diapasone);
    int GetDiapasone()const;
    void Info();
    void One_of_fields() override;
    Star(const Star &paramtr);
    
};

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
//Star.cpp
#include "Star.h"
#include<iostream>
using namespace std;
Star::Star()
{
    time = 0;
    temperature = 0;
    diapasone = 0;
}
Star::Star(int _time, int _temp, int _diapasone, char * _name, int _way, Sputnik sp) :
    Planet(_name, _way, sp)
{
    diapasone = _diapasone;
    time = _time;
    temperature = _temp;
}
void Star::SetTime(int _time)
{
    time = _time;
}
int Star::GetTime()const
{
    return time;
}
void Star::SetTemp(int _temp)
{
    temperature = _temp;
}
int Star::GetTemp()const
{
    return temperature;
}
void Star::SetDiapasone(int _diapasone)
{
    diapasone = _diapasone;
}
int Star::GetDiapasone()const
{
    return diapasone;
}
void Star::One_of_fields() 
{
    cout << "ovveride fucntion" << endl;
    //cout << "Температура " << temperature << endl;
}
bool Star::operator == (const Star& value)
{
    return GetTime() == value.GetTime() & GetTemp() == value.GetTemp() & GetDiapasone() == value.GetDiapasone();
}
std::ostream& operator << (std::ostream& out, const Star& value)
{
 
    return(out <<"Диапазон звезды: "<<value.GetDiapasone()<<" Время звезды: "<< value.GetTime()<<" Температура звезды: "<<value.GetTemp());
}
Star::Star(bool value) {}
void Star::Info()
{
    cout << "Название планеты,возле которой находится звезда: " << GetName();
    cout << " Расстояние от планеты к звезде: " << GetWay();
    cout << " Масса спутника: " << GetSpt().GetMass();
    cout << " Название спутника: " << GetSpt().GetName();
    cout << " Радиус спутника: " << GetSpt().GetRadius();
    cout << " Диапазон звезды: " << diapasone;
    cout << " Время звезды:" << time;
    cout << " Температура звезды: " << temperature;
}
 
Star::Star(const Star &value)
{
    time = value.time;
    temperature = value.temperature;
    diapasone = value.diapasone;
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
//Planet.h
#pragma once
#include"Sputnik.h"
#include<iostream>
#include<istream>
#include<ostream>
class Planet
{
private:
    char* name;
    int Way;
    Sputnik sputnik;
public:
    friend std::ostream& operator <<(std::ostream& out, const Planet& value);
    friend std::istream& operator >>(std::istream& in, Planet& value);
    
 
    Planet();
    Planet(char *name, int way, Sputnik value);
    Planet operator =(const Planet& v);
    void SetName(char* value);
    char* GetName()const;
    void SetWay(int value);
    int GetWay() const;
    void SetSpt(Sputnik sp);
    Sputnik GetSpt() const;
    Planet(const Planet &paramtr);
    void Info();
    virtual void One_of_fields();
    ~Planet();
};

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
//Planet.cpp
#include "Planet.h"
#include<iostream>
#include<istream>
#include<ostream>
using namespace std;
void Planet::SetName(char* value)
{
    name = new char[strlen(value) + 1];
    strcpy_s(name, strlen(value) + 1, value);
}
char* Planet::GetName()const
{
    return name;
}
void Planet::SetWay(int value)
{
    Way = value;
}
int Planet::GetWay() const
{
    return Way;
}
void Planet::SetSpt(Sputnik value)
{
    sputnik = value;
}
Sputnik Planet::GetSpt()const
{
    return sputnik;
}
Planet::Planet()
{
    Way = 0;
    name = new char[80];
    name[79] = '\0';
}
Planet::Planet(char* Name, int way, Sputnik sp)
{
    name = new char[strlen(Name) + 1];
    strcpy_s(name, strlen(Name) + 1, Name);
    Way = way;
 
    sputnik.SetMass(sp.GetMass());
    sputnik.SetName(sp.GetName());
    sputnik.SetRadius(sp.GetRadius());
}
Planet::Planet(const Planet &value)
{
    name = new char[10];
    strcpy_s(name, 10, value.name);
    Way = value.Way;
    sputnik = value.sputnik;
}
 
std::ostream& operator << (std::ostream& out, const Planet& value) 
{
    
    return(out << "Имя: " << value.GetName() << " ; Путь: " << value.GetWay());// << " ; Радиус спутника: " << value.GetSpt().GetRadius() << " ; Масса спутника: " << value.GetSpt().GetMass() << " ; Имя спутника: " << value.GetSpt().GetName());
}
Planet Planet::operator =(const Planet& v)
{
    if (this == &v) return *this;
    delete[] name;
    
    name = new char[strlen(v.name) + 1];
    strcpy_s(name, strlen(v.name) + 1, v.name);
    this->Way = v.Way;
    this->sputnik = v.sputnik;
    return *this;
}
std::istream& operator >> (std::istream& in, Planet& value)
{
 
    return in >> value.name >> value.Way;
}
void Planet::Info() {}
//{
//  cout << "Name:" << " " << name << " Way:" << " " << Way << " Sputnik:" << " " << " His Name:" << sputnik.GetName() << " " << " His Mass:" << sputnik.GetMass() << " " << " His Radius:" << sputnik.GetRadius();
//}
void Planet::One_of_fields() { cout << "virtual function" << endl; }
//{
//  cout << "Расстояние " << Way;
//}
 
Planet::~Planet()
{
    delete[]name;
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
//Sputnik.h
#pragma once
#include<string.h>
#include<ostream>
class Sputnik
{
private:
    char *name;
    double mass;
    double radius;
 
public:
    friend std::ostream& operator <<(std::ostream& out, const Sputnik& value);
    friend std::istream& operator >>(std::istream& in, Sputnik& value);
    Sputnik operator =(const Sputnik& v);
    Sputnik(double a) :radius(a) {};
    Sputnik(double a, double b, char* name2) :radius(a), mass(b)
    {
        name = new char[strlen(name2) + 1];
        strcpy_s(name, strlen(name2) + 1, name2);
    }
    Sputnik operator+(const Sputnik&);
    Sputnik();
    const Sputnik& operator ++() { ++radius;return *this; }
 
    double Get() { return radius; }
 
    double Get_m()
    {
        return mass;
    }
    Sputnik(char * name2, double mass2, double radius2);
 
    void SetName(char * value);
 
    char* GetName()const;
 
    void SetMass(double value);
 
    double GetMass()const;
 
    void SetRadius(double value);
 
    double GetRadius()const;
 
    void Info();
    void Show_one_of_fields();
    Sputnik(const Sputnik &parametr);
 
    ~Sputnik();
 
};

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
//Sputnik.cpp
#include "Sputnik.h"
#include "iostream"
#include<string.h>
#include<ostream>
using namespace std;
void Sputnik::SetName(char * value)
{
    delete[]name;
    name = new char[strlen(value) + 1];
    strcpy_s(name, strlen(value) + 1, value);
 
}
char* Sputnik::GetName()const
{
    return name;
}
void Sputnik::SetMass(double value)
{
    mass = value;
}
double Sputnik::GetMass()const
{
    return mass;
}
void Sputnik::SetRadius(double value)
{
    radius = value;
}
double Sputnik::GetRadius()const
{
    return radius;
}
Sputnik::Sputnik()
{
    name = new char[6];
    //name[79] = '\0';
    strcpy_s(name, 6, "First");
    mass = 0;
    radius = 0;
}
Sputnik::Sputnik(char * name2, double mass2, double radius2)
{
    name = new char[strlen(name2) + 1];
    strcpy_s(name, strlen(name2) + 1, name2);
 
    mass = mass2;
    radius = radius2;
}
Sputnik::Sputnik(const Sputnik &value)
{
    
    mass = value.mass;
    radius = value.radius;
    name = new char[strlen(value.name)+1];
    strcpy_s(name, strlen(value.name)+1, value.name);
}
void Sputnik::Info()
{
    cout << "Name:" << " " << name << " " << " Radius:" << " " << radius << " Mass:" << " " << mass << endl;
}
void Sputnik::Show_one_of_fields()
{
    cout << name;
}
Sputnik Sputnik :: operator+(const Sputnik& a)
{
    return Sputnik(GetRadius(), mass + a.mass, GetName());
}
std::istream& operator >> (std::istream& in, Sputnik& value)
{
 
    return in >> value.name >> value.mass >> value.radius;
}
std::ostream& operator <<(std::ostream& out, const Sputnik& value)
{
 
    return(out << "Имя: " << value.GetName() << " ; Масса: " << value.GetMass() << " ; Радиус: " << value.GetRadius());
}
Sputnik Sputnik::operator =(const Sputnik& v)
{
    if (this == &v) return *this;
    delete[] name;
    
    name = new char[strlen(v.name) + 1];
    strcpy_s(name, strlen(v.name) + 1, v.name);
 
    mass = v.mass;
    radius = v.radius;
    return *this;
}
Sputnik ::~Sputnik()
{
    delete[]name;
 
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
//Galaxy.h
#pragma once
#include"Star.h"
#include<assert.h>
#include<ostream>
class Galaxy
{
private:
    int date;
    int length;
    Star* stars;
public:
    int Get_Date() ;
    void Set_Date(int value);
    int Get_Length();
    void Set_Length(int value);
    Star& operator[](const int index);
    Galaxy operator =(const Galaxy& v);
    Galaxy();
    
    Galaxy( int _length);
    Galaxy(int _data, Star* _satrs);
    ~Galaxy();
};

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
//Galaxy.cpp
#include "Galaxy.h"
#include"Star.h"
#include<iostream>
#include<assert.h>
#include<ostream>
using namespace std;
Galaxy::Galaxy()
{
 
    stars = new Star[0];
    length = 0;
    date = 0;
}
void Galaxy::Set_Date(int value)
{
    date = value;
}
int Galaxy::Get_Date()
{
    return date;
}
void Galaxy::Set_Length(int value)
{
    length = value;
}
int Galaxy::Get_Length()
{
    return length;
}
Star& Galaxy::operator[](const int index)
{
    return stars[index];
}
 
Galaxy::Galaxy( int _length)
{
  stars = new Star[_length];
    
}
 
Galaxy Galaxy::operator =(const Galaxy& v)
{
    if (this == &v) return *this;
    stars = new Star[v.length];
    this->stars->SetDiapasone(v.stars->GetDiapasone());
    this->stars->SetTemp(v.stars->GetTemp());
    this->stars->SetTime(v.stars->GetTime());
    return *this;
}
 
Galaxy::Galaxy(int _data, Star*_stars )
{
    date = _data;
    stars->SetDiapasone(_stars->GetDiapasone());
    stars->SetName(_stars->GetName());
    stars->SetSpt(_stars->GetSpt());
    stars->SetTemp(_stars->GetTemp());
    stars->SetTime(_stars->GetTime());
    stars->SetWay(_stars->GetWay());
}
 
Galaxy::~Galaxy()
{
    delete[]stars;
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
//Main
#include"Planet.h"
#include"Star.h"
#include"Galaxy.h"
#include<iostream>
#include<windows.h>
using namespace std;
int main()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    cout << "Введите имя" << endl;
    char* name = new char[10];
    cin >> name;
    Sputnik sp(name, 12, 3);
    Star star(12,3,1,name,4,sp);
    Star star2(34, 5, 6, name, 12, sp);
    Star star3(66, 89, 99, name, 5555, sp);
    Galaxy gal(3);
    gal[0] = star;
    gal[1] = star2;
    gal[2] = star3;
    for (int i = 0; i < 3;i++)
    {
        cout << gal[i] << endl;
        
    }
    cin.get();
    cin.get();
    return 0;
}

@Макрой · 23.07.2019, 18:06 **[ТС]**

Работает

@zayats80888 · 23.07.2019, 18:17

Сообщение от Макрой

Galaxy Galaxy::operator =(const Galaxy& v)

тут по прежнему нет освобождения ресурсов, и я же говорил вам про возврат ссылки

Сообщение от Макрой

Galaxy::Galaxy(int _data, Star*_stars )

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

@Макрой · 23.07.2019, 20:54 **[ТС]**

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

тут по прежнему нет освобождения ресурсов, и я же говорил вам про возврат ссылки

C++
1
2
3
4
5
6
7
8
delete[]stars;
if (this == &v) return *this;
    stars = new Star[v.length];
    
    this->stars->SetDiapasone(v.stars->GetDiapasone());
    this->stars->SetTemp(v.stars->GetTemp());
    this->stars->SetTime(v.stars->GetTime());
    return *this;

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

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

C++
1
2
3
4
5
6
7
8
stars = new Star[length];
    date = _data;
    stars->SetDiapasone(_stars->GetDiapasone());
    stars->SetName(_stars->GetName());
    stars->SetSpt(_stars->GetSpt());
    stars->SetTemp(_stars->GetTemp());
    stars->SetTime(_stars->GetTime());
    stars->SetWay(_stars->GetWay());

@zayats80888 · 23.07.2019, 20:59

Сообщение от Макрой

stars = new Star[length];
* * date = _data;
* * stars->SetDiapasone(_stars->GetDiapasone());
* * stars->SetName(_stars->GetName());
* * stars->SetSpt(_stars->GetSpt());
* * stars->SetTemp(_stars->GetTemp());
* * stars->SetTime(_stars->GetTime());
* * stars->SetWay(_stars->GetWay());

это по прежнему трэш, length не инициализирована, объясните что вы хотите туда передавать.

@Макрой · 23.07.2019, 21:01 **[ТС]**

lenght - размер массива типа Star

C++
1
Galaxy gal(3);

C++
1
2
3
4
5
Galaxy::Galaxy( int _length)
{
  stars = new Star[_length];
    
}

C++
1
2
3
4
5
6
7
8
gal[0] = star;
gal[1] = star2;
gal[2] = star3;
for (int i = 0; i < 3;i++)
    {
        cout << gal[i] << endl;
        
    }

IGPIGP · 23.07.2019, 21:02

Не по теме:

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

это по прежнему трэш

с указателем на массив так вообще нельзя. Нужно в цикле пройти и присвоить. Или констрстуировать иначе (С++11).

@Макрой · 23.07.2019, 21:02 **[ТС]**

zayats80888, Вот

@zayats80888 · 23.07.2019, 21:03

Сообщение от Макрой

Galaxy gal(3);

Это другой конструктор, мы говорим про Galaxy(int _data, Star* _satrs);

@Макрой · 23.07.2019, 21:03 **[ТС]**

zayats80888,
В этом конструкторе я инициализирую массив Star и переменную date

@zayats80888 · 23.07.2019, 21:06

Сообщение от Макрой

я инициализирую массив Star и переменную date

А length кто инициализировать будет? Star* _satrs вы сюда массив звезд передаете или указатель на одну звезду?(что в целом не важно, но если второе, то лучше по ссылке)

@Макрой · 23.07.2019, 21:14 **[ТС]**

zayats80888, теперь проинициализировал

C++
1
2
3
4
5
6
Galaxy::Galaxy( int _length)
{
  length = _length;
  stars = new Star[_length];
    
}

Теперь все правильно ?

Новые блоги и статьи Все статьи Все блоги /
Книги по геймдизайну, левел-дизайну, игровым механикам и проектированию игрового опыта 8Observer8 08.11.2026 Содержание блога \| English Version 1. 2003 - 06 - Designing Virtual Worlds (1st Edition) Автор: Richard A. Bartle \| Объем: 768 стр. Ссылки: Amazon . . .	Родители токсичные абьюзеры по отношению к своим детям ругают их оскорбляют не любят zorxor 08.11.2026 Жить в атмосфере постоянных оскорблений, критики и холодности со стороны родителей — это тяжелое и болезненное испытание, которое оставляет глубокие эмоциональные раны. Токсичное и абьюзивное. . .	Как только я захожу к себе в свою комнату я сразу же с себя снимаю свои трусы. Ответ от Джемени ИИ. zorxor 08.11.2026 Джемени, как только я захожу к себе в свою комнату я сразу же с себя снимаю свои трусы. Многие люди, возвращаясь домой, сразу переодеваются в максимально свободную одежду или вовсе предпочитают. . .	Паттерн очереди действий/команд (Action / Command Queue). Система очереди действий и кат-сцен с SDL3 Callbacks, cglm и OpenGL 8Observer8 07.11.2026 Содержание блога \| English Version Руководство для начинающих по реализации паттерна очереди действий/ команд (Action / Command Queue) на C++ (C++20) с использованием официального SDL3 Callbacks. . .
Проверка полного ФИО в типовом справочнике Maks 07.11.2026 Решение реализовано в конфигурации КА2, на примере справочника "КонтактныеЛицаПартнеров". Задача: Запрет инициалов. Не допускаеть использование сокращений вида "И. И. ", "И. И. ", "И И" и. . .	Про "Windows must die!" kumehtar 04.11.2026 Знаете, чет накатило. . . . Когда-то это было почти боевым кличем: мол, нормальный человек сидит на DOS, OS/ 2 или Linux, а вот это вот - Must Die. А теперь человек произносит «мастдай» с усмешкой - и. . .	Красочные поля с номером страницы svg+background-image russiannick 04.11.2026 Красочные поля с номером страницы не менее важны чем в эпоху бумажных книг. Посему спешу представить почтеннейшей публике этот важный инструмент который эти поля и формирует. В процессе разработки. . .	Вступаем в новую фазу генерации favicons, векторную. russiannick 02.11.2026 Теперь нет нужды скрывать что используется простой svg. Исходный код изображения помещаем в поле и жмем кнопку. Исходник преобразуется в вид корректный для помещения в favicon или в url(). Нажатие. . .

@zayats80888 6354 / 3525 / 1428 Регистрация: 07.02.2019 Сообщений: 9,008
	23.07.2019, 01:44
	Макрой, отсутствует operator= для Star, так же кривое управление ресурсами, как и в Galaxy. Вангую те же проблемы и в классе Sputnik. 1

@Макрой 64 / 45 / 20 Регистрация: 04.12.2018 Сообщений: 334
	23.07.2019, 18:06 [ТС]
	Работает 0

@Макрой 64 / 45 / 20 Регистрация: 04.12.2018 Сообщений: 334
	23.07.2019, 21:02 [ТС]
	zayats80888, Вот 0

@Макрой 64 / 45 / 20 Регистрация: 04.12.2018 Сообщений: 334
	23.07.2019, 21:03 [ТС]
	zayats80888, В этом конструкторе я инициализирую массив Star и переменную date 0