копирование массива изменяет поведение функции с использованием исходного массива

Question

копирование массива изменяет поведение функции с использованием исходного массива

0

Изменение: я отредактировал свой код, чтобы добавить часть "extern" C "(не уверен, что это имеет значение)

Поэтому у меня есть этот небольшой код:

extern "C"{
   void foo(int* nR,int* mR,float* x,float* out){       //&& 
       const int n=*nR,m=*mR;
       other_foo(n,m,x,out);
   }
}

это прекрасно работает. Но теперь я хочу скопировать n-array float x прежде чем передавать его функции other_foo (так как other_foo будет изменять x и я хочу сохранить копию).

если я так делаю, все работает нормально:

extern "C"{
   void foo(int* nR,int* mR,float* x,float* out){       //&& 
       const int n=*nR,m=*mR;
       float y[n];
       for(int i=0;i<n;i++) y[i]=x[i];
       other_foo(n,m,x,out);
   }
}

но если я сделаю так:

extern "C"{
   void foo(int* nR,int* mR,float* x,float* out){       //&& 
       const int n=*nR,m=*mR;
       float y[n];
       std::copy(x,x+n,y);
       other_foo(n,m,x,out);
   }
}

все адские промахи проиграют: результат other_foo уже не тот!

Мой вопрос, конечно, почему?

user189035 02 апр. 2014, в 19:38

Источник

Теги:

c++

arrays

c

1 ответ

Ещё вопросы

Vlad from Moscow · Accepted Answer · 2014-04-02T15-16-00.000Z

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

void foo(int* nR,int* mR,float* x,float* out){      //&& 
    const int n=*nR,m=*mR;
    float y[n];
    for(int i=0;i<n;i++)    y[i]=x[i];
    other_foo(n,m,x,out);
}


void foo(int* nR,int* mR,float* x,float* out){      //&& 
    const int n=*nR,m=*mR;
    float y[n];
    std::copy(x,x+n,y);
    other_foo(n,m,x,out);
}

Оба копируют n элементов из x в y.

Однако этот код не соответствует C++. Размер массива должен быть постоянным выражением, известным во время компиляции. Поэтому было бы правильнее распределить массив динамически.

Например

void foo(int* nR,int* mR,float* x,float* out){      //&& 
    int n = *nR, m = *mR;
    float *y = new float[n];
    std::copy( x, x+n, y );
    other_foo( n, m, x, out );
    // other code  maybe including delete [] y
}

Я думаю, что проблема не в этой функции. Похоже, что проблема в вызываемой функции other_foo

Большое спасибо! Так что это (ваше второе предложение) работает, и я буду его использовать.
@ user189035 Вы должны проверить функцию other_foo. Возможно это имеет неопределенное поведение.
вот блок объявления other_foo: void other_foo (const int & n, const int & m, float x [], float ox []). Как вы думаете, это может вызвать проблемы?
@ user189035 Проблема не в объявлении функции. Проблема может быть в определении функции. Также, если вы используете неквалифицированное имя копии алгоритма, например, копии (x, x + n, y), то возможно, что вы вызываете некоторую функцию C вместо алгоритма.
спасибо за подсказки. Это действительно float * y = new float [n]; линия. Когда я заменяю это 'float y [n];' (и не используйте цикл), он перестает работать (вывод other_foo становится бессмысленным). Я отредактировал свой вопрос (код деформирован во внешнем «C»), как вы думаете, это может вызвать это?
@ user189035 это может быть причиной, если у вас есть функция с тем же именем, но с языковой связью на языке C.
то же имя, что и что? (извините за медленную ловлю)
@ user189035 Я имею в виду функцию копирования.
нет, у меня нет функции с именем copy (и я всегда использую std :: copy) ...
extern "C" не означает компиляцию кода на C, это означает, что любая объявленная под ней функция будет вызываться через C ABI. Он должен окружать ваш прототип, который виден всем вызывающим функциям; не реализация функции.
@VladfromMoscow: большое спасибо, сир! Я добавляю неинициализированное значение в мой код, который затем использовался!