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

Написал свой string. Проверьте код на наличие подводных камней, утечек памяти и других ошибок - C++

Восстановить пароль Регистрация
 
 
vaziliybober
0 / 0 / 0
Регистрация: 02.01.2016
Сообщений: 25
29.01.2016, 11:30     Написал свой string. Проверьте код на наличие подводных камней, утечек памяти и других ошибок #1
Сделал класс, который ведет себя аналогично char*, но с возможностью сложения строк. Вроде все работает как надо, но есть ли какие-то подводные камни, утечки памяти? Может что-то сделано криво и можно поумнее? Дело в том, что на этот класс мне надо будет потом полностью полагаться и быть уверенным, что в нем нет ошибок. Вот код:

vstring.h
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
class vstring
    {
    private:
        char* Str;
        int length;
    public:
        vstring(char* str = 0);
        vstring(char s);
        vstring(vstring& str);
 
        ~vstring(){delete [] Str;}
 
        char operator[](int index);
 
        friend std::ostream& operator<<(std::ostream& out, vstring& str);
 
        vstring operator+(vstring& str);
        vstring operator+(char symbol);
        friend vstring operator+(char symbol, vstring& str);
 
        operator char*();
 
        vstring& operator=(char* s);
        vstring& operator=(vstring& str);
         };
vstring.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
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
vstring::vstring(char* str)
{
    if(str == 0)
    {
        Str = new char[1];
        length = 0;
        Str[0] = '\0';
        return;
    }
    for(length = 0; str[length] != '\0'; length++);
    Str = new char[length+1];
    Str[length] = '\0';
    for(int i(0); i < length; i++)
        Str[i] = str[i];
}
vstring::vstring(char s)
{
    Str = new char[2];
    length = 1;
    Str[1] = '\0';
    Str[0] = s;
}
 
vchar vstring::operator[](int index)
{
    return Str[index];
}
 
vstring::vstring(vstring& str)
{
    Str = new char [str.length+1];
 
    for(int i(0); i < str.length; i++)
        Str[i] = str[i];
    Str[str.length] = '\0';
    length = str.length;
}
 
std::ostream& operator<<(std::ostream& out, vstring& str)
{
    for(int i(0); i < str.length; i++)
        out << str[i];
    return out;
}
 
vstring operator+(char symbol, vstring& str)
{
    return vstring(symbol) + str;
}
 
vstring vstring::operator+(vstring& str)
{
    char* resStr = new char[length + str.length + 1];
 
    int i(0);
    for(; i < length; i++)
        resStr[i] = Str[i];
    for(; i < length + str.length; i++)
        resStr[i] = str[i-length]; 
    resStr[i] = '\0';
        
        vstring ret(resStr);
        delete [] resStr;
    return ret;
}
 
vstring vstring::operator+(char symbol)
{
    return *this + vstring(symbol);
}
 
vstring& vstring::operator=(char* s)
{
    delete [] Str;
 
    vstring str(s);
 
    Str = new char[str.length+1];
    for(int i(0); i < str.length; i++)
        Str[i] = str[i];
    Str[str.length] = '\0';
    length = str.length;
 
    return *this;
}
 
vstring& vstring::operator=(vstring& str)
{
    delete [] Str;
 
    Str = new char[str.length+1];
    for(int i(0); i < str.length; i++)
        Str[i] = str[i];
    Str[str.length] = '\0';
    length = str.length;
 
    return *this;
}
 
vstring::operator char*()
{
    char* str = new char[length+1];
    for(int i(0); i < length; i++)
        str[i] = Str[i];
    str[length] = '\0';
 
    return str;
}
Лучшие ответы (1)
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
29.01.2016, 11:30     Написал свой string. Проверьте код на наличие подводных камней, утечек памяти и других ошибок
Посмотрите здесь:

C++ Проверьте код на наличие ошибок.
как проверить наличие утечек памяти в шаблоне? C++
Чтобы не было утечек памяти C++
Подключении механизма обнаружение утечек памяти C++
Подскажите программу для диагностики утечек памяти C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
30.01.2016, 00:07     Написал свой string. Проверьте код на наличие подводных камней, утечек памяти и других ошибок
Еще ссылки по теме:

Проверьте, на наличие ошибок C++
C++ Visual Studio 2015 Обнаружение утечек памяти
C++ Проверить код на наличие ошибок

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

Или воспользуйтесь поиском по форуму:
hoggy
5113 / 2114 / 403
Регистрация: 15.11.2014
Сообщений: 4,799
Завершенные тесты: 1
30.01.2016, 00:07     Написал свой string. Проверьте код на наличие подводных камней, утечек памяти и других ошибок #21
Цитата Сообщение от vaziliybober Посмотреть сообщение
ну а в конструкторах копирования мы ж без сомнений дублируем данные. Почему тогда в них нет таких заморочек?
мы создаем копию объекта-строки.
класс строки знает собственные данные,
и как с ними правильно обращаться.
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Yandex
Объявления
30.01.2016, 00:07     Написал свой string. Проверьте код на наличие подводных камней, утечек памяти и других ошибок
Ответ Создать тему
Опции темы

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