Форум программистов, компьютерный форум 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() подробнее

Показать сообщение отдельно
Croessmah
Модератор
Эксперт С++
 Аватар для Croessmah
11847 / 6826 / 771
Регистрация: 27.09.2012
Сообщений: 16,924
Записей в блоге: 2
Завершенные тесты: 1
29.01.2016, 11:47     Написал свой string. Проверьте код на наличие подводных камней, утечек памяти и других ошибок
C++
1
2
3
4
vstring(vstring& str);
vstring& operator=(vstring& str);
vstring(char* str = 0);
vstring& operator=(char* s);
Почему параметры не const?
Разве нельзя копировать константную строку?
C++
1
friend std::ostream& operator<<(std::ostream& out, vstring& str);
а константные строки выводу не подлежат?
C++
1
int length;
почему знаковый тип?
Думаете, длина строки когда-то будет меньше нуля?
C++
1
char operator[](int index);
опять же, почему параметр имеет знаковый тип?
Почему возвращается не ссылка на символ, а копия?
C++
1
2
        vstring operator+(vstring& str);
        vstring operator+(char symbol);
Почему функции-члены не const?
Ну и ссылка в параметре тоже не const почему?
C++
1
operator char*();
хреновая идея. Возможны неявные касты,
а значит могут быть ошибки потом, которые черт поймаешь.
Лучше сделайте функцию-член c_str
C++
1
vchar vstring::operator[](int index)
что за vchar?
если это псевдоним,
то почему в объявлении char, а не vchar?
C++
1
for(length = 0; str[length] != '\0'; length++);
strlen и strcpy уже не в ходу?
C++
1
2
3
4
5
6
7
8
9
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;
}
Возвращаем копию строки, созданную динамически,
при этом заботу об освобождении возлагаем на клиента?
Фигня сразу. И спровоцирует чертову тучу утечек, т.к. неявный каст, см. выше.
C++
1
2
3
4
5
    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;
strcpy всё еще не в ходу.
Ну тогда хоть в свою функцию всё это дело оберните,
а то наплодили один и тот же код в тысячи местах. Не хорошо это

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