如何声明指向二维浮点矩阵的指针？

Question

我试图声明一个指向二维浮点矩阵的指针，以便我的图像数据具有动态行为，但我遇到了编译错误 C2057：预期的常量表达式。我认为必须以这种方式投射指针，但显然不是......请任何人都可以帮助我吗？谢谢！！

    //Image size input

int imheight;
int imwidth;

cout << "Please, enter image height: \n>";
scanf ("%d",&imheight);
cout << "Please, enter image width: \n>";
scanf ("%d",&imheight);

const int imheight2 = imheight;
const int imwidth2 = imwidth;

float *zArray[imheight2][imwidth2];

这是我尝试访问 zArray 的其他功能之一。我没有正确读取数据：

void LoadRIS( char* inputFileName , float** zArray, int imageHeight , int  imageWidth){

    // Load input RIS file
FILE* lRis = fopen ( inputFileName, "rb" );

// Jump to data position
for (int i = 0; i < 88; i++){       
    uchar a = getc (lRis);   
}   

// Read z array
size_t counter = fread ( *zArray , 1 , imageHeight * imageWidth * sizeof(zArray) , lRis );

//Get max value of RIS
float RISmax = zArray [0][0];
float RISmin = zArray [0][0];
for (int i=0; i<imageHeight; i++) 
{
    for (int j=0; j<imageWidth; j++)
        {
            if (zArray[i][j] > RISmax)
            RISmax = zArray [i][j];
            if (zArray[i][j] < RISmin)
            RISmin = zArray [i][j];
        }
}
std::cout<<"The max value of the RIS file is: "<<RISmax<<"\n";
std::cout<<"The min value of the RIS file is: "<<RISmin<<"\n";
Beep(0,5000);


// Close input file
fclose (lRis);

}

Answer 1

const int imheight2 = imheight;
const int imwidth2 = imwidth;

它不做常量表达式。您不能创建具有此类边界的数组。您应该使用dynamic-allocation 或vector。

Answer 2

问题是您声明了 2 个 const int 变量，但您没有为它们分配 const 值。 imheight 和 imwidth 不是常量。

如果你对 STL 没问题：

std::vector<std::valarray<float> > floatMatrix;

编辑：仅供参考，我在上述代码行中> 之间放置的空格与我的编码风格无关。您的编译器可能会假定 >> 是右移运算符，而不是 2 个模板参数列表终止符。 Angew 的评论总结了这一点。

Answer 3

而不是float *zArray[imheight2][imwidth2]; 应该是：

float **zArray = new float*[imheight2];

for(int i=0; i<imheight2; i++)
{
    zArray[i] = new float[imwidth2];
}

Answer 4

试试这个（动态分配）

//Image size input

int imheight;
int imwidth;

cout << "Please, enter image height: \n>";
scanf ("%d",&imheight);
cout << "Please, enter image width: \n>";
scanf ("%d",&imwidth);

float** zArray = new float*[imheight];
for(int i=0;i<imheight;i++){
    zArray[i] = new float[imwidth];
}

当然，您需要通过以下方式释放分配：

for(int i=0;i<imheight;i++){
    delete[] zArray[i];
}
delete[] zArray;

希望这会有所帮助:)

附：正如@FrankH 所说，这调用了太多news 和deletes，浪费了很多时间。更好的主意应该是一起分配 imwidth*imheight 空间。

Answer 5

如果您有这样做，那么至少将其编码为：

float **zArray = new float*[imheight];
float *tmp = new float[imheight*imwidth];

for(int i=0; i<imheight; i++, tmp += imwidth)
    zArray[i] = tmp;

...
delete[] *zArray;
delete[] zArray;

这至少避免了两次以上的new / delete[] 调用。如果内存不连续，它会保留您的fread(*zArray, ...) 的功能中断（如果您通过许多new 调用来初始化它，通常不会这样）。

一个合适的包装类只会做一个new / malloc，比如：

template <class T> class Array2D {
private:
    size_t m_x;
    T* val;
public:
    Array2D(size_t x, size_t y) :
        m_x(x)),
        val(new T[x*y]) {}
    ~Array2D() { delete[] val; }
    T* operator[](size_t y) { return val + y*m_x; }
}

您仍然无法将其实例分配给float**。它仍然在堆上分配，普通的常量维数组可以在堆栈上。 float** 的额外分配的唯一优点是您不必使用乘法运算 - 而是使用单独的内存访问；这种类型的行为可以被模板化/特征化到包装类中。

一般来说，我更支持multidimensional arrays are evil（另见https://stackoverflow.com/a/14276070/512360或C++ FAQ, 16.16），但口味各不相同......

Answer 6

您不能使用具有动态大小的数组（您的宽度和高度变量不是编译时间常数）。

您可以使用 malloc() 或 new Operator 以动态方式分配内存。

Answer 7

float *pMatrix = new float[imheight2*imwidth2];

然后像这样访问元素

float f = pMatrix[x + imwidth2 * y];