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

Что неправильно в этом коде - C++

Восстановить пароль Регистрация
 
KaridatMerkader
Сообщений: n/a
25.12.2010, 07:54     Что неправильно в этом коде #1
Здраствуйте. Учусь заочно. Вот, писала одну из задач к домашней контрольной и, вроде бы, всё должно быть правильно, но...
Задание состоит в том, что бы описать класс String и перегрузить для него разные операторы, в том числе и <<, >>.
Программа работает, но вот в конце, на последней строчке return 0; выскакивает окно Microsoft Visual C++ Debug Library с таким содержанием: Debug Error! Program: D:\DOCUMENTS\KR\Debug\main.exe DAMAGE: after Normal block (#45) at 0x00441EA0.

Код main.cpp такой:
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
#include "String.hpp"
 
int main ()
{
    String str1, str2;
    cout << "String1: " << endl;
    cin >> str1;
    cout << "String2: " << endl;
    cin >> str2;
    if(str1 > str2)
        cout << "String1 > String2" << endl;
    if(str1 < str2)
        cout << "String1 < String2" << endl;
    if(str1 == str2)
        cout << "String1 = String2" << endl;
    cout << "String3: " << endl;
    cin >> str1;
    cout << "String4: " << endl;
    cin >> str2;
    if(str1 != str2)
        cout << "String3 =/= String4" << endl;
    else
        cout << "String3 = String4" << endl;
    cout << str1 << endl;
    cout << str2 << endl;
    return 0;
}
Код String.hpp:
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
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
#include <iostream.h>
#include <string.h>
 
class String
{
    public:
        String();
        String(const char *const);
        String(const String &);
        ~String();
 
        char & operator[](int offset);
        char operator[](int offset) const;
        String operator+(const String&);
        void operator+=(const String&);
        String & operator=(const String &);
        bool operator==(const String &);
        bool operator!=(const String &);
        bool operator<(const String&);
        bool operator>(const String&);
        friend ostream& operator<<(ostream&, const String);
        friend istream& operator>>(istream&, String&);
 
        int GetLen()const { return itsLen; }
        const char * GetString() const { return itsString; }
 
    private:
        String(int);
        char * itsString;
        int itsLen;
};
 
String::String()
{
    itsString = new char[1];
    itsString[0] = '\0';
    itsLen=0;
}
 
String::String(int len)
{
    itsString = new char[len+1];
    int i;
    for (i = 0; i<=len; i++)
        itsString[i] = '\0';
    itsLen=len;
}
 
String::String(const char * const cString)
{
    itsLen = strlen(cString);
    itsString = new char[itsLen+1];
    int i;
    for (i = 0; i<itsLen; i++)
        itsString[i] = cString[i];
    itsString[itsLen]='\0';
}
 
String::String (const String & rhs)
{
       itsLen=rhs.GetLen();
   itsString = new char[itsLen+1];
   int i;
   for ( i = 0; i<itsLen;i++)
               itsString[i] = rhs[i];
   itsString[itsLen] = '\0';
}
 
String::~String ()
{
    delete [] itsString;
    itsLen = 0;
}
 
String& String::operator= (const String & rhs)
{
       if (this == &rhs)
               return *this;
   delete [] itsString;
   itsLen=rhs.GetLen();
   itsString = new char[itsLen+1];
   int i;
   for (i = 0; i<itsLen;i++)
               itsString[i] = rhs[i];
   itsString[itsLen] = '\0';
   return *this;
}
 
bool String::operator== (const String &rhs)
{
       if(itsLen != rhs.itsLen) return false;
       else
       {
               for(int i=0; i<=itsLen;i++)
                       if(itsString[i]!=rhs.itsString[i])
                               return false;
       }
               return true;
}
 
bool String::operator!= (const String &rhs)
{
    if(itsLen == rhs.itsLen)
        for(int i=0; i<=itsLen;i++)
            if(itsString[i]!=rhs.itsString[i])
                return true;
    return false;
}
 
bool String::operator<(const String& rhs)
{
       if(itsLen < rhs.itsLen) return true;
       if(itsLen > rhs.itsLen) return false;
       for(int i=0; i<=itsLen; i++)
       {
               if(itsString[i] < rhs.itsString[i]) return true;
               if(itsString[i] > rhs.itsString[i]) return false;
       }
       return false;
}
 
bool String::operator>(const String& rhs)
{
       if(itsLen > rhs.itsLen) return true;
       if(itsLen < rhs.itsLen) return false;
       for(int i=0; i<=itsLen; i++)
       {
               if(itsString[i] > rhs.itsString[i]) return true;
               if(itsString[i] < rhs.itsString[i]) return false;
       }
       return false;
}
 
char & String::operator[](int offset)
{
       if (offset > itsLen)
               return itsString[itsLen-1];
   else
               return itsString[offset];
}
 
char String::operator[](int offset) const
{
       if (offset > itsLen)
               return itsString[itsLen-1];
   else
               return itsString[offset];
}
 
String String::operator+(const String& rhs)
{
       int  totalLen = itsLen + rhs.GetLen();
   int i,j;
   String temp(totalLen);
   for (i = 0; i<itsLen; i++)
               temp[i] = itsString[i];
   for (j = 0; j<rhs.GetLen(); j++, i++)
               temp[i] = rhs[j];
   temp[totalLen]='\0';
   return temp;
}
 
void String::operator+=(const String& rhs)
{
       int rhsLen = rhs.GetLen();
   int totalLen = itsLen + rhsLen;
   int i,j;
   String  temp(totalLen);
   for (i = 0; i<itsLen; i++)
               temp[i] = itsString[i];
   for (j = 0; j<rhs.GetLen(); j++, i++)
               temp[i] = rhs[i-itsLen];
   temp[totalLen]='\0';
   *this = temp;
}
 
ostream& operator<<(ostream & out, const String str)
{
       out << str.itsString << endl;
       return out;
}
 
istream& operator>>(istream &in, String &str)
{
       in >> str.itsString;
       str.itsLen=strlen(str.itsString);
       return in;
}
Помогите, пожалуйста, исправить.

Добавлено через 1 час 23 минуты
исправила код перегрузки >> в String.hpp
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
istream& operator>>(istream &in, String &str)
{
    delete [] str.itsString;
    char s[100];
    in.getline(s,sizeof(s));
    str.itsLen=strlen(s);
    str.itsString = new char[str.itsLen+1];
    int i;
    for (i = 0; i<str.itsLen; i++)
        str.itsString[i] = s[i];
    str.itsString[str.itsLen]='\0'; 
    return in;
}
теперь работает хорошо, но решение какое-то нехорошее, выглядит как стена здания, которая треснула и её заклеили скотчем.
Может, есть лучшее решение?
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
25.12.2010, 07:54     Что неправильно в этом коде
Посмотрите здесь:

Что не правильно в этом коде? C++
Пишу крестики-нолики. Что не правильно в этом коде? C++
Проверьте задачку по циклам, неправильно работает. [думаю что неправильно] C++
C++ С++ напишите, что делает каждая строчка в этом коде?!
C++ С++ напишите, что делает каждая строчка в этом коде?!
C++ Посоветуйте альтернативу scanf() в этом коде
C++ Перевод числа из десятичного вида в двоичный. Что неправильно в коде?
C++ Что в этом коде не правильно?

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

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Ответ Создать тему
Опции темы

Текущее время: 02:52. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru