Освободить память и записать по новой

@Кротяка · Регистрация: 20.12.2010

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

Есть массив:

C++

1	char Name[25];

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

C++

void Country::setName(void){
    if(Name!=NULL){
        char *Name = new char[];
        delete[] Name;
    }
    else{
        Name[25] = *new char[strlen(Name)+1];
        strcpy_s(Name, Name);
    }
    cout<<"Введите новое название страны: ";
    cin>>Name;
}

скажите, я правильно делаю?

Добавлено через 6 часов 3 минуты
может хоть кто то сказать? просто "да" или "нет"?

@ViktorKozlov · 25.11.2013, 23:18

Кривовато у вас как-то все..

C++

if(Name!=NULL){
        char *Name = new char[];
        delete[] Name;
    }

Здесь, если Name указывает на строку, вы сразу выделяете память под новую строку и тут же ее удаляете. Нелогично как-то. Это во первых. Далее, указатель на старую строку при этом теряется, так как затирается указателем на новую строку, и освободить память вы при этом уже никак не сможете. В третьих,

C++

1	new char[];

так писать нельзя, да и компилятор это запретит, так как он должен знать, под какого размера массив выделять память. В четвертых, непонятно назначение этой строки

C++

1	Name[25] = *new char[strlen(Name)+1];

которая выполняется при условии, что указатель на строку пустой, т.е. равен NULL. А вы еще пытаетесь вычислить длину строки, на которую указывает этот указатель, а при этом указатель в принципе никуда не указывает.
И последнее,

C++

1	strcpy_s(Name, Name);

копирование строки в саму себя. Идея, в принципе, интересная, но смысловой нагрузки опять же никакой не несет

@Кротяка · 25.11.2013, 23:44 **[ТС]**

ViktorKozlov, скажите пожалуйста, как я должен освободить память при неправильном вводе данных чтобы заполнить массив новым значением?

alsav22 · 26.11.2013, 07:12

Сообщение от Кротяка

Есть массив: Код C++1
char Name[25];

Если массив такой (не динамически созданный), то специально освобождать ничего не нужно. При чтении новой строки в массив, старое содержимое удалится.

@Кротяка · 27.11.2013, 05:09 **[ТС]**

alsav22, препод требует, ничего не поделаешь

Добавлено через 5 часов 21 минуту
немножко переделал

C++

    char *Nazvanie = new char[*Name];
    Name[25] = *new char[strlen(Nazvanie)+1];
    if(Name!=NULL){
        cout<<"Введите новое название страны: ";
        cin>>Nazvanie;
        strcpy_s(this->Name, Nazvanie);
        delete[] Nazvanie;
    }

так правильно, скажите?

@MrCold · 27.11.2013, 05:27

Сообщение от Кротяка

нужно выполнить проверку ввода данных и при надобности

Хоть напишите что такое выполнить проверку ввода данных
Угадал-неугадал что-ли ?

alsav22 · 27.11.2013, 09:26

Сообщение от Кротяка

alsav22, препод требует, ничего не поделаешь

Что требует? Освобождать память под статически созданным массивом?

Добавлено через 2 минуты

Сообщение от Кротяка

так правильно, скажите?

Компилятор не спрашивали?

@Кротяка 7 / 7 / 1 Регистрация: 20.12.2010 Сообщений: 392
	25.11.2013, 23:44 [ТС]	3
	ViktorKozlov, скажите пожалуйста, как я должен освободить память при неправильном вводе данных чтобы заполнить массив новым значением? 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	26.11.2013, 07:12	4
	Сообщение от Кротяка Есть массив: Код C++1 char Name[25]; Если массив такой (не динамически созданный), то специально освобождать ничего не нужно. При чтении новой строки в массив, старое содержимое удалится. 0

@MrCold 873 / 771 / 173 Регистрация: 11.01.2012 Сообщений: 1,942
	27.11.2013, 05:27	6
	Сообщение от Кротяка нужно выполнить проверку ввода данных и при надобности Хоть напишите что такое выполнить проверку ввода данных Угадал-неугадал что-ли ? 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	27.11.2013, 09:26	7
	Сообщение от Кротяка alsav22, препод требует, ничего не поделаешь Что требует? Освобождать память под статически созданным массивом? Добавлено через 2 минуты Сообщение от Кротяка так правильно, скажите? Компилятор не спрашивали? 0