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

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

Восстановить пароль Регистрация
Другие темы раздела
C++ Ошибка в вычислениях синуса http://www.cyberforum.ru/cpp-beginners/thread1650365.html
дана формула z=(2^(-x))*sqrt{x+\sqrt{abs(y)}}*\sqrt{exp^((x-1)/sin(x))}, по ней надо вычислить. Но вот ошибка возникает в синусе, не могу понять как исправить уже. Помогите пожалуста. #include <iostream> #include <math.h> #include <ctime> #include <cstdlib> #include<locale.h> using namespace std; main() {setlocale(LC_CTYPE,"Russian"); double z, q, w, e, r, t;
C++ Стоит ли избегать указателя this? Слышал от знакомого, что обращение к полям класса через указатель this трудоёмкая операция и лучше её избегать в программах со сложными вычислениями. Насколько это правдоподобно и почему? http://www.cyberforum.ru/cpp-beginners/thread1650354.html
Исправьте,пожалуйста, ошибки в коде :) C++
Не пойму почему всё горит красным. Я не подключил библиотеку какую-то?Если нет,то в чём ошибке? Исправьте,пожалуйста. Спасибо! //ejhkfjhkj #include"stdafx.h" #include<iostream> #include<conio.h> int main(){
Определить минимальное количество монет, которое должно находиться в автомате, чтобы всем хватило сдачи C++
Здравствуйте. Не первый раз создаю тему об олимпиадных задачах , думаю, и не последнюю)) Возникла проблема со следующей задачей: Фирма bookface, созданная в Ужляндии, в которой работает Степан, решила установить в своих офисах автоматы по продаже чая и кофе, чтобы программисты во время перерыва могли с толком провести время. Стоимость стакана чая и кофе в автомате предполагается установить...
C++ Почему на консоль выводится "1" и дальше ничего не происходит? (цикл while) http://www.cyberforum.ru/cpp-beginners/thread1650323.html
Вот код: #include <iostream> #include <Windows.h> #include <conio.h> using namespace std; int main() { int i = 0; while (true) { cout << ++i; Sleep(500);
C++ Запись и чтение вектора из файла Здравствуйте! Вот небольшая прога которая пишет в вектор структуру... при повторном запуске она читает с файла в вектор и по идее можно еще добавлять записи.. но при повторном запуске если попытаться добавить не работает. #include "stdafx.h" struct Administrator { std::string login_admin; std::string pass_admin; }; int main() подробнее

Показать сообщение отдельно
vaziliybober
0 / 0 / 0
Регистрация: 02.01.2016
Сообщений: 25
29.01.2016, 23:06  [ТС]     Написал свой string. Проверьте код на наличие подводных камней, утечек памяти и других ошибок
Цитата Сообщение от Croessmah Посмотреть сообщение
Как исправите, приходите
ну вот вроде все.

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
26
27
28
29
30
31
#pragma once
#include <iostream>
 
class vstring
    {
    private:
        char* Str;
        unsigned int length;
 
        char* string_copy(const char* str);
        unsigned int string_length(const char* str);
    public:
        vstring(const char* str = 0);
        vstring(char s);
        vstring(const vstring& str);
 
        ~vstring(){delete [] Str;}
 
        char& operator[](unsigned int index) const;
 
        friend std::ostream& operator<<(std::ostream& out, const vstring& str);
 
        vstring operator+(const vstring& str) const;
        vstring operator+(char symbol) const;
        friend vstring operator+(char symbol, vstring& str);
 
        vstring& operator=(const char* s);
        vstring& operator=(const vstring& str);
 
        char* c_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
#include "vstring.h"
 
unsigned int vstring::string_length(const char* str)
{
    if(str == 0) return 0;
 
    unsigned int size;
    for(size = 0; str[size] != '\0'; size++);
    return size;
}
 
char* vstring::string_copy(const char* str)
{
    if(str == 0)
    {
        char* ret = new char[1];
        ret[0] = '\0';
        return ret;
    }
 
    char* ret = new char[string_length(str)+1];
    ret[string_length(str)] = '\0';
    for(int i(0); i < string_length(str); i++)
        ret[i] = str[i];
    return ret;
}
 
vstring::vstring(const char* str)
{
    Str = string_copy(str);
    length = string_length(str);
}
vstring::vstring(char s)
{
    Str = new char[2];
    length = 1;
    Str[1] = '\0';
    Str[0] = s;
}
 
char& vstring::operator[](unsigned int index) const
{
    return Str[index];
}
 
vstring::vstring(const vstring& str)
{
    Str = string_copy(str.Str);
    length = str.length;
}
 
std::ostream& operator<<(std::ostream& out, const 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+(const vstring& str) const
{
    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) const
{
    return *this + vstring(symbol);
}
 
vstring& vstring::operator=(const char* s)
{
    delete [] Str;
    Str = string_copy(s);
    length = string_length(s);
 
    return *this;
}
 
vstring& vstring::operator=(const vstring& str)
{
    delete [] Str;
 
    Str = string_copy(str.Str);
    length = str.length;
 
    return *this;
}
 
char* vstring::c_str()
{
    return string_copy(Str);
}
Цитата Сообщение от Croessmah Посмотреть сообщение
Также нет main'а, чтобы посмотреть как Вы всё это используете
Да пока никак не использую. Как раз и обратился, чтобы потом использовать без сомнений в работе класса.

Добавлено через 4 минуты
Цитата Сообщение от Croessmah Посмотреть сообщение
тогда можем попасть впросак,
если нет include guard'ов, т.к. прагмы не стандартны.
ну пусть подключит стандартные include guardы вместо прагмы.
 
Текущее время: 15:47. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru