Ошибка при возвращении указателя из функции, которая указывает на массив - PullRequest
Новая
       8

Ошибка при возвращении указателя из функции, которая указывает на массив

1 голос
/ 19 марта 2009

Я немного расстроен тем, что не знаю, почему мой код вызывает следующую ошибку при компиляции: 

1>..\SA.cpp(81) : error C2664: 'CFE' : cannot convert parameter 1 from 'int' to 'int []'
1>        Conversion from integral type to pointer type requires reinterpret_cast, C-style cast or function-style cast

По сути, я пытаюсь:

Step1: преобразовать вектор в массив, используя: 

int* VecToArray(vector<int> Vec)
{
    int ary[Vec.size()];

    for(int i = 0; i < Vec.size(); i++)
        ary[i] = Vec[i];

    return ary;
}

Step2: вызов функции, в которой массив является параметром, и он возвращает указатель из вновь созданного массива: 

int* CFE(int density[])
{
   ...do stuff to generate 'double Energy[]'....

    return Energy;
}

Шаг 3: Использование этого указателя в третьей функции для вычисления суммы энергии []: 

double ObjFunction (double *E_Array) 
{
    double SumEnergy = 0;

    int n = 10; // Is irrelivant

    for (int i = 0; i < n; i++)
    {
        SumEnergy += E_Array[i];
    }

    return SumEnergy;
}

Чтобы упростить кодирование, я использовал такие функции, где VectorName - целочисленный вектор: 

double TotalEnergy = ObjFunction ( CFE ( VecToArray ( VectorName ) ) );

Я явно где-то ошибаюсь в типах параметров, хотя сам не понимаю, почему. Может ли кто-нибудь с более опытным зрением помочь обнаружить его / их?

Ответы [ 5 ]

4 голосов
/ 19 марта 2009

Откуда берется Energy? Если это double[], то вы не можете просто привести его к int*.

std::vector<int> гарантированно будет смежным, поэтому, если вы хотите преобразовать std::vector<int> VectorName в const int*, используйте &VectorName[0]. Если, с другой стороны, ваша функция CFE изменяет передаваемый массив, вероятно, лучше создать его локально.

3 голосов
/ 19 марта 2009

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

2 голосов
/ 19 марта 2009

Вы должны это исправить: 

int ary[Vec.size()];

Кому: 

int *ary = new int(Vec.size());

Компилятор не знает Vec.size () во время компиляции, поэтому он не может создать массив.

Убедитесь, что вы освободите память позже.

Это чтобы решить проблему. Но я думаю, что ваш подход не очень хорош. Vector имеет почти такую ​​же производительность, как и обычный массив, но гораздо безопаснее и проще в использовании. Почему бы вам просто не использовать векторы?

Даже если вы решите использовать массивы, передача их по значению не очень эффективна. Тем более, что вы сразу перенаправляете вывод в другую функцию. Я бы предпочел использовать ссылки для такого рода проблем.

2 голосов
/ 19 марта 2009

Шаг 1 имеет несколько проблем:

  • Вы пытаетесь создать массив с переменным размером. Вы не можете сделать это в C89 или C ++ (я думаю, что C99 добавляет это).
  • Вы возвращаете указатель на локальную переменную, вышедшую из области видимости.
1 голос
/ 19 марта 2009

Вы не можете создать массив с динамически оцениваемым размером. Вы не можете вернуть локально определенный массив.

Однако вы можете рассматривать & myVector [0] как массив.

...