delete[]

@PakistanCode · Регистрация: 05.02.2013

Author24 — интернет-сервис помощи студентам

подскажите пожалуйста как заставить это работать?
Вылетает на строчке delete[].
Выделяю ведь память с помощью оператора new.
А в идеале нужно сделать так чтобы была отдельная ф-я типа:

C++

void deleteStr()
{
delete[] string;
}

подскажите пожалуйста и я наконец-то разберусь с этой темой)
Вот исходный код:

C++

# include <iostream>
using namespace std;
 
class mString
{
private:
    char* string;
public:
    mString(char* string_)
    {
        int lenght_ = strlen(string_);
        string = new char[lenght_];
        string[lenght_] = '\0';
 
        for (int i = 0; i < lenght_; i++)
            string[i] = string_[i];
 
        delete [] string;
    }
};
 
int main(void)
{
    mString("12.34");
    system("pause");
}

ValeryS · 18.07.2013, 16:55

Сообщение от PakistanCode

Только проблема в том, что мне постоянно нужно будет удалять и выделять память.

если тебе постоянно нужно выделять и удалять память тем более под строку
может проще воспользоваться типом string?

@PakistanCode · 18.07.2013, 16:58 **[ТС]**

ValeryS, проблема в том что:

ValeryS · 18.07.2013, 16:58

Сообщение от PakistanCode

~mString()
{
if (string) delete [] string;
}

лишнее это
можно так

C++

1	delete [] string;

для нулевого указателя delete ничего не делает

Сообщение от PakistanCode

delete [] string;
string = 0;

для указателей лучше использовать NULL и самому понятнее и другим

@nexen · 18.07.2013, 17:11

ValeryS,

C++

1 2	string = 0; delete [] string;

Сработает нормально, серьезно? Никаких исключений?

Croessmah · 18.07.2013, 17:15

Сообщение от nexen

Сработает нормально, серьезно? Никаких исключений?

Серьезно

Сообщение от 5.3.5

In the second alternative (delete array), the value of the operand of delete may be a null pointer value or a pointer value that resulted from a previous array new-expression.If not, the behavior is undefined.

ValeryS · 18.07.2013, 17:20

Сообщение от nexen

Сработает нормально, серьезно?

не ненормально

утечка будет
а вот такое сработает

C++

1
2
3

delete [] string;
string = NULL;
delete [] string;

ты с free, то не путай

Tulosba · 18.07.2013, 18:44

Сообщение от ValeryS

ты с free, то не путай

а что free ведет себя как-то иначе?

ValeryS · 18.07.2013, 19:09

Сообщение от Tulosba

а что free ведет себя как-то иначе?

та ххто его знает
я с нем толком не работал, но в книжках говорят проверять указатель

@gray_fox · 18.07.2013, 19:15

Сообщение от ValeryS

та ххто его знает
я с нем толком не работал, но в книжках говорят проверять указатель

Не надо проверять, free(NULL) ничего не делает.

ValeryS Модератор 8909 / 6678 / 918 Регистрация: 14.02.2011 Сообщений: 23,523
	18.07.2013, 16:55	21
	Сообщение от PakistanCode Только проблема в том, что мне постоянно нужно будет удалять и выделять память. если тебе постоянно нужно выделять и удалять память тем более под строку может проще воспользоваться типом string? 1

@PakistanCode 5 / 5 / 5 Регистрация: 05.02.2013 Сообщений: 197
	18.07.2013, 16:58 [ТС]	22
	ValeryS, проблема в том что: Миниатюры 0

Croessmah Неэпический 17870 / 10635 / 2054 Регистрация: 27.09.2012 Сообщений: 26,737 Записей в блоге: 1
	18.07.2013, 17:15	25
	Сообщение от nexen Сработает нормально, серьезно? Никаких исключений? Серьезно Сообщение от 5.3.5 In the second alternative (delete array), the value of the operand of delete may be a null pointer value or a pointer value that resulted from a previous array new-expression.If not, the behavior is undefined. 1

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	18.07.2013, 18:44	27
	Сообщение от ValeryS ты с free, то не путай а что free ведет себя как-то иначе? 0

ValeryS Модератор 8909 / 6678 / 918 Регистрация: 14.02.2011 Сообщений: 23,523
	18.07.2013, 19:09	28
	Сообщение от Tulosba а что free ведет себя как-то иначе? та ххто его знает я с нем толком не работал, но в книжках говорят проверять указатель 0

@gray_fox What a waste! 1608 / 1300 / 180 Регистрация: 21.04.2012 Сообщений: 2,729
	18.07.2013, 19:15	29
	Сообщение от ValeryS та ххто его знает я с нем толком не работал, но в книжках говорят проверять указатель Не надо проверять, free(NULL) ничего не делает. 0