Форум программистов, компьютерный форум CyberForum.ru

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
Faltfromoss
0 / 0 / 0
Регистрация: 28.03.2014
Сообщений: 32
#1

Некорректно работает оператор присваивания - C++

24.06.2014, 00:44. Просмотров 175. Ответов 2
Метки нет (Все метки)

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
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
class Card              
{
private:
    int Cost;           
    char *Name, *Suit;
public:
    Card(): Cost(0), Name (0), Suit (0){}
    Card(int cost, char *name, char *suit);
    Card (const Card & obj);
    ~Card();
    friend ostream& operator<<(ostream& os, const Card& T);
    void operator () (int cost, char *name, char *suit);
    Card operator = (const Card &obj);
    int operator + (const Card &obj);
    friend int operator + (int k, const Card &obj);
    Card& operator ++ (int);
};
 
 
 
 
Card::Card (int cost, char *name, char *suit)
{
    Name = new char [strlen(name) + 1];
    strcpy (Name, name);
    Suit = new char [strlen(suit) + 1];
    strcpy (Suit, suit);
    Cost = cost;
}
 
Card::Card (const Card& obj)
{
    Name = new char [strlen (obj.Name)+1];
    Suit = new char [strlen (obj.Suit)+1];
    strcpy (Name, obj.Name);
    strcpy (Suit, obj.Suit);
    Cost = obj.Cost;
}
 
Card::~Card ()
{
    if (Name)
        delete [] Name;
    if (Suit)
        delete [] Suit;
}
 
ostream& operator<<(ostream& os, const Card& T)
{
    os<<T.Name<<" "<<T.Suit;
    return os;
}
 
void Card::operator() (int cost, char *name, char *suit)
{
    Name = new char [strlen(name) + 1];
    strcpy (Name, name);
    Suit = new char [strlen(suit) + 1];
    strcpy (Suit, suit);
    Cost = cost;
}
 
Card Card::operator = (const Card &obj)
{
    Name = new char [strlen (obj.Name) + 1];
    Suit = new char [strlen (obj.Suit) + 1];
    strcpy (Name, obj.Name);
    strcpy (Suit, obj.Suit);
    Cost = obj.Cost;
    return *this;
}
 
int Card::operator + (const Card &obj)
{
    int temp;
    temp = Cost+obj.Cost;
    return temp;
}
 
int operator + (int k, const Card &obj)
{
    int temp;
    temp = k + obj.Cost;
    return temp;
}
 
class Player
{
private:
    Card *Cards;
    int Sum, Number;
public:
    Player(): Cards(0), Sum (0), Number (0) {}
    void Set (Card card);
    Card Get (int i);
    Player (const Player& obj);
    ~Player() {if (Cards) delete [] Cards;}
    Player operator = (const Player& obj);
    Player & operator ++ (int);
 
};
 
void Player::Set (Card card)        // эта функция работает некорректно
{
    if (!Number)    
    {
        Number = 1;
        Cards = new Card [Number];
        Cards [Number] = card;  //здесь срабатывает перегруженный оператор присваивания в классе Card и всё копируется в *this и возвращается, но почему-то не копируется в Cards [Number].
    }
    else
    {               
        Card *temp = new Card [Number];
        for (int i = 0; i<Number; i++)
        {
            temp [i] = Cards[i];
        }
        delete [] Cards;
        Cards = new Card [Number+1];
        for (int i = 0; i<Number; i++)
        {
            Cards[i]=temp[i];
        }
        Cards [Number] = card;
        Number++;
        delete [] temp;
    }
 
}
 
Card Player::Get (int i)
{
    return Cards [i];
}
 
Player::Player (const Player& obj)
{
    Sum = obj.Sum;
    Number = obj.Number;
    if (Cards)
        delete [] Cards;
    Cards = new Card [Number];
    for (int i = 0; i<Number; i++)
        Cards [i] = obj.Cards [i];
}
Player Player::operator = (const Player& obj)
{
    Sum = obj.Sum;
    Number = obj. Number;
    if (Cards)
        delete [] Cards;
    Cards = new Card [Number];
    for (int i = 0; i<Number; i++)
        Cards [i] = obj.Cards [i];
    return *this;
}
Отметил комментом непонятный момент. В общем я хочу добавить объект Card в класс Player с помощью функции Set(), но по какой-то причине при присваивании переданного аргумента card созданному объекту в массиве Cards [Number] ничего не присваивается и он остается пустым. А данные остаются в *this функции operator = (). Может кто объяснить что происходит?

Добавлено через 6 минут
Аааа, баран. Дошло. Я ж за пределы массива этим Number сразу вылажу...

Тут темы удалять нельзя?
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
24.06.2014, 00:44     Некорректно работает оператор присваивания
Посмотрите здесь:

Не работает перегруженный оператор присваивания - C++
#include &lt;iostream&gt; #include &lt;string&gt; #include &lt;Windows.h&gt; class OutOfBoundsException {}; class MyString { private: unsigned...

оператор if, некорректно работает программа - C++
#include &lt;iostream.h&gt; #include &lt;math.h&gt; main () { int b; double z, x=0.63, c=1.5, d=2, a=0.37; cout &lt;&lt; &quot;Vvedite znachenie...

Оператор присваивания в c++ - C++
Такой вопрос, для каких классов следует явно перегрузить оператор присваивания??

Оператор присваивания - C++
Почему в этом случае вызывается оператор присваивания Fraction f4; f4 = f1+f2+f3; а в другом не вызывается: ...

Оператор присваивания - C++
Какая разница между оператором &quot;=&quot; и &quot;==&quot; Почему , например , в цикле нужно писать while(count==1) а не ...

Оператор присваивания и деструктор - C++
Вопрос: почему в 52 строке Access violation? Если убрать строку 50, все будет нормально. #include &lt;iostream&gt; using namespace std; ...

Оператор присваивания по умолчанию - C++
#include &lt;iostream&gt; using namespace std; class Foo { private: int var; //int var = 17; public: /*Foo&amp;...

Перегрузка операторов, оператор присваивания - C++
Перегрузка операторов, запутано-намутано #include &quot;stdafx.h&quot; #include &quot;conio.h&quot; #include &lt;iostream&gt; using namespace::std; ...

Оператор присваивания для массива - C++
Есть шаблонный класс который служит контейнером для различных типов c помощью массива. Как получить значение конкретного индекса нашел,...

Логическая переменная и оператор присваивания - C++
С помощью логической переменной и оператора присваивания определить: 1-я цифра у 3-хзначного числа больше 3-й цифры.


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
TheChosenOne
13 / 13 / 1
Регистрация: 13.09.2013
Сообщений: 113
24.06.2014, 00:47     Некорректно работает оператор присваивания #2
Цитата Сообщение от Faltfromoss Посмотреть сообщение
Number = 1;
* * * * Cards = new Card [Number];
* * * * Cards [Number] = card; *//здесь срабатывает перегруженный оператор присваивания в
Весь код я не разбирал,но тут ты обращаешься к Card[1] хотя у тебя есть только Card[0]

Не успел малех )
Faltfromoss
0 / 0 / 0
Регистрация: 28.03.2014
Сообщений: 32
24.06.2014, 01:00  [ТС]     Некорректно работает оператор присваивания #3
Цитата Сообщение от TheChosenOne Посмотреть сообщение
Не успел малех )
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru