Есть динамический массив с размерностью 5, необходимо его сделать с размерностью 6, заполнить цифрами и вывести на экран

@CheburatorUA · Регистрация: 27.03.2014

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

День добрый.
Не хочется никому морочить голову дурацкими вопросами, поэтому стараюсь находить на все свои вопросы ответы сам. Но тут я столкнулся с совершенно непонятной для меня проблемой и решение ее нигде не могу найти. Когда я запускаю выполнение программы на экран выводится полная ерунда , но если запустить отладчик все нормально и на экран выводятся данные динамического массива. Из-за чего такое может происходить? Голову ломаю второй день. Задача следующая - есть динамический массив с размерностью 5, необходимо его сделать с размерностью 6, заполнить цифрами и вывести на экран. Код, который внизу, при запуске отладчика все отлично выводит на экран, а при запуске программы выводить ерунду. Буду очень благодарен если кто-то натолкнет на решение проблемы. Пользуюсь стандартным инструментарием microsoft visual studio 2010

C++

#include <iostream>
#include <stdlib.h>
using namespace std;
void masplus(float*, int*);
void main ()
{   
    int size=5;
        float* mas1;
    mas1 = new float[size];
        
    masplus(mas1, &size);
    
    for(int i=0;i<size; i++)
         cout<<mas1[i];
}
 
void masplus(float* mas, int *n)
{
    delete [] mas;
    *n+=1;
    mas = new float[*n];
    for(int i=0;i<*n; i++)
        mas[i]=i;
 
}

P.S.
Если просто заполнить массив данными , то все ОК и в отладчике и в запуске программы, если сделать как в коде вверху, то в отладчике ОК , а при запуске программы нет.

ValeryS · 27.03.2014, 18:41

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

*n+=1;

не уверен
но по моему нарушен приоритет, не увеличиваешь значение а увеличиваешь адрес
по аналогии

C++

*n++

попробуй перепиши
вот так

C++

1	n=(n)+1;

или так

C++

(*n)+=1;

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

C++

void masplus(float* mas, int &n)
{
    delete [] mas;
    n+=1;
    mas = new float[n];
    for(int i=0;i<n; i++)
        mas[i]=i;
 
}
 
masplus(mas1, size);

@CheburatorUA · 27.03.2014, 21:49 **[ТС]**

Нет , не помогает.
И если я иду именно по шагам в отладчике, то все работает и указателем и ссылкой. Но только я запускаю программу на экран выводится как будто массив не заполнен.

Есть динамический массив с размерностью 5, необходимо его сделать с размерностью 6, заполнить цифрами и вывести на экран

@CheburatorUA · 27.03.2014, 22:09 **[ТС]**

А вот если отладчиком идти по шагам все получается, как такое возможно?

alsav22 · 27.03.2014, 22:10

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

Код, который внизу, при запуске отладчика все отлично выводит на экран, а при запуске программы выводить ерунду.

У меня всё нормально выводит.

@CheburatorUA · 27.03.2014, 22:20 **[ТС]**

Это наверно проблема в моем microsoft visual studio 2010?

@DrOffset · 27.03.2014, 22:27

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

но по моему нарушен приоритет, не увеличиваешь значение а увеличиваешь адрес

Нет, там все в порядке.
CheburatorUA,
Обратив внимание, что ты скопировал указатель в функцию. Снаружи осталось старое значение. А новый массив выделился в совершенно новом месте. Попробуй вот такой код, он должен работать правильно:

C++

#include <iostream>
#include <stdlib.h>
using namespace std;
void masplus(float* &, int*);
int main ()
{
    int size=5;
    float* mas1;
    mas1 = new float[size];
 
    masplus(mas1, &size);
 
    for(int i=0;i<size; i++)
         cout<<mas1[i];
}
 
void masplus(float* & mas, int *n)
{
    delete [] mas;
    *n+=1;
    mas = new float[*n];
    for(int i=0;i<*n; i++)
        mas[i]=i;
}

Добавлено через 17 секунд

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

Это наверно проблема в моем microsoft visual studio 2010?

Нет, проблема в UB в коде.

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

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

У меня всё нормально выводит.

Потому что нормально работающая программа - частный случай неопределенного поведения

alsav22 · 27.03.2014, 22:37

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

Обратив внимание, что ты скопировал указатель в функцию.

Да, сейчас увидел. Странно, что у меня выводит не мусор.

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

Нет, проблема в UB в коде.

А почему UB? Почему у меня выводит не мусор?

Добавлено через 3 минуты
Как в особождённой памяти оказываются данные из памяти под другим массивом?

@DrOffset · 27.03.2014, 22:43

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

А почему UB?

Доступ по указателю к которому был применен delete - UB

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

Как в особождённой памяти оказываются данные из памяти под другим массивом?

Может случиться, если компилятор встроит функцию.

@CheburatorUA · 27.03.2014, 22:46 **[ТС]**

Да, такой код заработал, спасибо.
Вот только тогда почему если делать пошагово в отладчике, то все работает, я даже выше скришнот прикрепил. А у alsav22 вообще все получилось с моим кодом, мистика).

Еще раз спасибо за работающий код.

@DrOffset · 27.03.2014, 22:48

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

Вот только тогда почему если делать пошагово в отладчике, то все работает

Неопределенное поведение, это термин для обозначения поведения некорректной программы. Оно может выражаться в том числе и в том, что все работает нормально. Ты понял почему была проблема?

ValeryS · 27.03.2014, 22:51

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

Вот только тогда почему если делать пошагово в отладчике, то все работает,

потому что UB неопределенное поведение
при релизе идет оптимизация и результирующий код совершенно не похож на исходный
попробуй отключи оптимизацию и проверь

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

Обратив внимание, что ты скопировал указатель в функцию.

как я на это не обратил внимания

на четыре раза пересмотрел и не заметил

@DrOffset · 27.03.2014, 22:52

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

Как в особождённой памяти оказываются данные из памяти под другим массивом?

alsav22, кстати, про встраивание я зря сказал: оптимизация не должна влиять на поведение программы. Скорее всего это может случиться, если new вернет тот же самый адрес, при новом выделении памяти. Оказался свободный участок в том же месте. Просто повезло.

UPD: проверил у себя, так и есть - адреса равны, в итоге все работает. При чем даже в релизе. Но это случайность.

@CheburatorUA · 27.03.2014, 23:14 **[ТС]**

Ты понял почему была проблема?

Да, вроде. Не ожидал я такого нюанса. Завтра на свежую голову еще обдумаю.

@DrOffset · 27.03.2014, 23:20

CheburatorUA, ключевое слово "копия"

Ты записал новое значение от new в копию. А исходное, которое ты передавал осталось старым (и уже невалидным, т.к. был delete[]). Так получилось что new вернул тот же самый адрес, в итоге ничего не сломалось.

@CheburatorUA · 31.03.2014, 00:23 **[ТС]**

Как-то странно все равно. В моей версии в main проинициализировали указатель, затем под этот указатель выделена память для массива. Указатель передан в функцию, затем по этому указателю память удалена и снова выделена, но на одну ячейку больше. Ну выделена эта память в копии т.е. в функции, но ведь указатель не поменялся и память выделена. Перечитав довольно много литературы я так понял , что указатели именно для таких ситуаций и существуют, а тут оказалось, что необходимо еще дополнительно и ссылку указывать.

alsav22 · 31.03.2014, 00:33

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

Перечитав довольно много литературы я так понял , что указатели именно для таких ситуаций и существуют

Всё хорошо пока действия в функции, куда передан указатель по значению (копия), осуществляются над данными, адрес которых находится в указателе:

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

затем по этому указателю память удалена

Адрес и в копии, и в оригинале (в main()) пока одинаковы, поэтому всё хорошо.

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

и снова выделена, но на одну ячейку больше.

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

но ведь указатель не поменялся и память выделена.

В том то и дело, что поменялся (поменялся адрес, который хранится в указателе).
При выделении новой памяти, адрес этой памяти записыается в сам указатель, т.е. меняются уже не данные, адрес которых находится в указателе, а сам этот адрес (значение указателя), и меняется он в копии, а не в том указателе, который был в main().

@DrOffset · 31.03.2014, 00:49

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

еще дополнительно и ссылку указывать.

Ссылку не обязательно. Есть еще вариант с указателем на указатель (float **). Тогда мы будем передавать адрес указателя, который указывает на наши данные. И через этот адрес мы всегда можем получить доступ к тому самому указателю (в нашем случае он будет находиться в main), указывающему на наши данные. Я всегда тем, кто плохо понимает всю эту кухню, рисую схемы на доске. Но здесь не получится, так что попробуй понять, что написали alsav22 и я.

@CheburatorUA · 31.03.2014, 13:37 **[ТС]**

Спасибо за пояснения. Теперь я понял. А то я не мог сам понять, меня как заклинило на этом указателе.

@DrOffset 18844 / 9843 / 2408 Регистрация: 30.01.2014 Сообщений: 17,285
	27.03.2014, 22:48	11
	Сообщение от CheburatorUA Вот только тогда почему если делать пошагово в отладчике, то все работает Неопределенное поведение, это термин для обозначения поведения некорректной программы. Оно может выражаться в том числе и в том, что все работает нормально. Ты понял почему была проблема? 0

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,521
	27.03.2014, 22:51	12
	Сообщение от CheburatorUA Вот только тогда почему если делать пошагово в отладчике, то все работает, потому что UB неопределенное поведение при релизе идет оптимизация и результирующий код совершенно не похож на исходный попробуй отключи оптимизацию и проверь Сообщение от DrOffset Обратив внимание, что ты скопировал указатель в функцию. как я на это не обратил внимания на четыре раза пересмотрел и не заметил 0

@DrOffset 18844 / 9843 / 2408 Регистрация: 30.01.2014 Сообщений: 17,285
	27.03.2014, 23:20	15
	CheburatorUA, ключевое слово "копия" Ты записал новое значение от new в копию. А исходное, которое ты передавал осталось старым (и уже невалидным, т.к. был delete[]). Так получилось что new вернул тот же самый адрес, в итоге ничего не сломалось. 0

@CheburatorUA 0 / 0 / 0 Регистрация: 27.03.2014 Сообщений: 24
	31.03.2014, 00:23 [ТС]	16
	Как-то странно все равно. В моей версии в main проинициализировали указатель, затем под этот указатель выделена память для массива. Указатель передан в функцию, затем по этому указателю память удалена и снова выделена, но на одну ячейку больше. Ну выделена эта память в копии т.е. в функции, но ведь указатель не поменялся и память выделена. Перечитав довольно много литературы я так понял , что указатели именно для таких ситуаций и существуют, а тут оказалось, что необходимо еще дополнительно и ссылку указывать. 0

@CheburatorUA 0 / 0 / 0 Регистрация: 27.03.2014 Сообщений: 24
	31.03.2014, 13:37 [ТС]	19
	Спасибо за пояснения. Теперь я понял. А то я не мог сам понять, меня как заклинило на этом указателе. 0

@CheburatorUA 0 / 0 / 0 Регистрация: 27.03.2014 Сообщений: 24
	27.03.2014, 21:49 [ТС]	3
	Нет , не помогает. И если я иду именно по шагам в отладчике, то все работает и указателем и ссылкой. Но только я запускаю программу на экран выводится как будто массив не заполнен. 0

@CheburatorUA 0 / 0 / 0 Регистрация: 27.03.2014 Сообщений: 24
	27.03.2014, 22:09 [ТС]	4
	А вот если отладчиком идти по шагам все получается, как такое возможно? 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	27.03.2014, 22:10	5
	Сообщение от CheburatorUA Код, который внизу, при запуске отладчика все отлично выводит на экран, а при запуске программы выводить ерунду. У меня всё нормально выводит. Миниатюры 0

@CheburatorUA 0 / 0 / 0 Регистрация: 27.03.2014 Сообщений: 24
	27.03.2014, 22:20 [ТС]	6
	Это наверно проблема в моем microsoft visual studio 2010? 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	27.03.2014, 22:37	8
	Сообщение от DrOffset Обратив внимание, что ты скопировал указатель в функцию. Да, сейчас увидел. Странно, что у меня выводит не мусор. Сообщение от DrOffset Нет, проблема в UB в коде. А почему UB? Почему у меня выводит не мусор? Добавлено через 3 минуты Как в особождённой памяти оказываются данные из памяти под другим массивом? 0

@DrOffset 18844 / 9843 / 2408 Регистрация: 30.01.2014 Сообщений: 17,285
	27.03.2014, 22:43	9
	Сообщение от alsav22 А почему UB? Доступ по указателю к которому был применен delete - UB Сообщение от alsav22 Как в особождённой памяти оказываются данные из памяти под другим массивом? Может случиться, если компилятор встроит функцию. 1

@CheburatorUA 0 / 0 / 0 Регистрация: 27.03.2014 Сообщений: 24
	27.03.2014, 22:46 [ТС]	10
	Да, такой код заработал, спасибо. Вот только тогда почему если делать пошагово в отладчике, то все работает, я даже выше скришнот прикрепил. А у alsav22 вообще все получилось с моим кодом, мистика). Еще раз спасибо за работающий код. 0

@DrOffset 18844 / 9843 / 2408 Регистрация: 30.01.2014 Сообщений: 17,285
	27.03.2014, 22:52	13
	Сообщение от DrOffset Как в особождённой памяти оказываются данные из памяти под другим массивом? alsav22, кстати, про встраивание я зря сказал: оптимизация не должна влиять на поведение программы. Скорее всего это может случиться, если new вернет тот же самый адрес, при новом выделении памяти. Оказался свободный участок в том же месте. Просто повезло. UPD: проверил у себя, так и есть - адреса равны, в итоге все работает. При чем даже в релизе. Но это случайность. 1

@CheburatorUA 0 / 0 / 0 Регистрация: 27.03.2014 Сообщений: 24
	27.03.2014, 23:14 [ТС]	14
	Ты понял почему была проблема? Да, вроде. Не ожидал я такого нюанса. Завтра на свежую голову еще обдумаю. 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	31.03.2014, 00:33	17
	Сообщение было отмечено CheburatorUA как решение Решение Сообщение от CheburatorUA Перечитав довольно много литературы я так понял , что указатели именно для таких ситуаций и существуют Всё хорошо пока действия в функции, куда передан указатель по значению (копия), осуществляются над данными, адрес которых находится в указателе: Сообщение от CheburatorUA затем по этому указателю память удалена Адрес и в копии, и в оригинале (в main()) пока одинаковы, поэтому всё хорошо. Сообщение от CheburatorUA и снова выделена, но на одну ячейку больше. Сообщение от CheburatorUA но ведь указатель не поменялся и память выделена. В том то и дело, что поменялся (поменялся адрес, который хранится в указателе). При выделении новой памяти, адрес этой памяти записыается в сам указатель, т.е. меняются уже не данные, адрес которых находится в указателе, а сам этот адрес (значение указателя), и меняется он в копии, а не в том указателе, который был в main(). 2

@DrOffset 18844 / 9843 / 2408 Регистрация: 30.01.2014 Сообщений: 17,285
	31.03.2014, 00:49	18
	Сообщение от CheburatorUA еще дополнительно и ссылку указывать. Ссылку не обязательно. Есть еще вариант с указателем на указатель (float ). Тогда мы будем передавать адрес указателя, который указывает на наши данные. И через этот адрес мы всегда можем получить доступ к тому самому указателю (в нашем случае он будет находиться в main), указывающему на наши данные. Я всегда тем, кто плохо понимает всю эту кухню, рисую схемы на доске. Но здесь не получится, так что попробуй понять, что написали alsav22** и я. 2

Есть динамический массив с размерностью 5, необходимо его сделать с размерностью 6, заполнить цифрами и вывести на экран

Решение