копирование статического массива в динамический массив

Question

копирование статического массива в динамический массив

0

Я определяю динамический массив в c++:

double *values;
int size = 5;
values = new (nothrow) double[size];

Я знаю, что это работает, потому что оно компилируется, но я вижу некоторые потенциальные проблемы.

Скажем, я хочу присвоить значения этому массиву:

double samples = [1,2,3,4,5,6];
values = samples; //runtime error: free or corruption

Что именно происходит для генерации этой ошибки?

knk 21 июнь 2014, в 00:43

Источник

1

Может быть, вы имеете в виду double samples[] = {1,2,3,4,5,6};
101010 20 июнь 2014, в 21:58
9

«Я знаю, что это работает, потому что компилируется». Интересный подход.
Baum mit Augen♦ 20 июнь 2014, в 21:58
0

Вы не присваиваете значения values , вы просто указываете на первый элемент samples .
chris 20 июнь 2014, в 21:59
0

values = samples; на самом деле не вызывает никаких ошибок. Это заставляет values указателя указывать на этот массив. Тем не менее, будет ошибка, если вы позже попытаетесь delete[] values; потому что вы пытаетесь удалить что-то new
M.M 21 июнь 2014, в 01:42

Показать ещё 2 комментария

Теги:

c++

arrays

3 ответа

Ещё вопросы

Может быть, вы имеете в виду double samples[] = {1,2,3,4,5,6};
«Я знаю, что это работает, потому что компилируется». Интересный подход.
Вы не присваиваете значения values , вы просто указываете на первый элемент samples .
values = samples; на самом деле не вызывает никаких ошибок. Это заставляет values указателя указывать на этот массив. Тем не менее, будет ошибка, если вы позже попытаетесь delete[] values; потому что вы пытаетесь удалить что-то new

101010 · Answer 1 · 2014-06-20T19-31-00.000Z

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

#include <iostream>
#include <algorithm>


int main() {

   int *a = new int[5];
   int  b[] = {1, 2, 3, 4, 5};
   std::copy(b, b + 5, a);
   for(std::size_t i(0); i < 5; ++i) std::cout << a[i] << " ";
   std::cout << std::endl;

    return 0;
}

LIVE DEMO

Альтернативно, если вы хотите удобство назначений вместо элементарного копирования и при условии, что вы знаете размер массивов во время компиляции, а ваш компилятор поддерживает функции С++ 11, используйте std::array как показано ниже:

#include <iostream>
#include <array>

int main() {

   std::array<int, 5> a;
   std::array<int, 5> b {{1, 2, 3, 4, 5}};
   a = b;
   for(auto i : a) std::cout << i << " ";
   std::cout << std::endl;

    return 0;
}

LIVE DEMO

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

#include <iostream>
#include <vector>
#include <algorithm>

int main() {
   std::vector<int> a(5);
   int b[] = {1, 2, 3, 4, 5};
   std::copy(b, b + 5, a.begin());
   for(auto i : a) std::cout << i << " ";
   std::cout << std::endl;

    return 0;
}

LIVE DEMO

Вы не должны публиковать какие-либо решения, которые связаны с новым [].

John D · Answer 2 · 2014-06-20T19-30-00.000Z

Это не работает, потому что вы назначаете статический массив указателю.

double *values;
double samples[] = {1,2,3,4,5,6};

Это два разных типа данных в отношении компилятора.

Когда ты говоришь:

values = new double[size];

Вы создаете блок кучи (динамической) памяти, а "значения" содержат адрес памяти первого элемента в массиве. Чтобы заполнить значения из вашего статического массива, вам необходимо отдельно назначить каждый элемент следующим образом:

values[0] = samples[0];
values[1] = samples[1];
// or better yet
for (int i = 0; i < size; i++)
    values[i] = samples[i]

Когда вы говорите new double[size] , вы создаете утечки памяти и переполнения буфера.

Puppy · Answer 3 · 2014-06-20T20-38-00.000Z

Вы можете использовать std::vector который имеет конструктор итератора, это решит проблему для вас.

std::vector<double> values(std::begin(samples), std::end(samples));

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