c++ - 将二维数组传递给 double function_name 时出错

Question

我正在开发一个程序，该程序将文件中的高度值读取到二维数组(矩阵)中，并且我试图将该数组传递给另一个找到最大值的函数。我知道，默认情况下，数组是通过引用传递的，但我并没有试图在函数中更改数组的值，所以这应该没什么大不了的。我已经浏览了几页关于调用数组的内容，但是我没有找到任何关于我在编译代码时遇到的错误类型的提及。问题似乎在于调用的参数数量或调用方式，但我看不出函数的各种外观有任何差异。我的猜测是关于传递二维数组，我在类里面没有被告知，而且我自己也没有学会。任何帮助将不胜感激。代码是:

#include <fstream>
#include <iostream>
#include <string>
#include <cstdlib>

using namespace std;

// First instance of function declaration
double find_max(double elevations[][3600], double ilat, double ilon, int nlat, int nlon);

int main(int argc, char *argv[]) {

// Declare program variables
double lat_init, lon_init;
double lat_res, lon_res;
double peak, valley;
int lon_col, lat_row;
string indat, inpoints;
.
.
.
double elevations[lat_row][lon_col];

// Open and read topographic data file
ifstream topo_points;
topo_points.open(inpoints.c_str());

for (int i=0; i<lat_row; i++) {
    for (int j=0; j<lon_col; j++)
        topo_points >> elevations[i][j];
}

// Call function to find peak in the data
peak = find_max(elevations, lat_init, lon_init, lat_row, lon_col);

return 0;

}


// ***** Here lie the functions *****

// This function reads in the array of elevations, initial latitude and longitude
// of the data, and the number of data points and uses this information to find
// the latidude and longitude of the highest point on earth
double find_max(double elev[][3600], double ilat, double ilon, int nlat, int nlon) {

double num, max;
double latpos, lonpos;

max = 0;

for (int i=0; i<nlat; i++) {
    for (int j=0; j<nlon; j++) {
    num = elev[i][j];
    if (num > max) {
        max=num;
        latpos= ilat - i;
        lonpos= ilon + j;
    }
    }
}

cout << "The tallest peak on earth has an altitude of " << max;
cout << " and is located at " << latpos << "deg latitude and ";
cout << lonpos << "deg longitude";

return max;
}

但是，当我调用该函数时出现以下错误:

错误:无法将参数“double (*)[(((long unsigned int)(((long int)lon_col) - 1)) + 1u)]”转换为“double (*)[3600]” 1' 到 'double find_max(double (*)[3600], double, double, int, int)'

Answer 1

从我在代码中看到的情况来看，存在一些小问题。

• 您已将数组高程定义为

  双高程[lat_row][lon_col];

这是行不通的，因为 c 风格数组的大小必须在编译时确定。由于 lat_row 和 lon_col 是变量，这是一个错误。

因此，您可以使用具有动态内存分配的数组或 std::vector，后者在大多数情况下更可取。所以，在你的情况下你可以有类似的东西:

typedef std::vector< std::vector<double> > ElevationsType;
ElevationsType elevations;

然后只使用该数组或 double 组。然后，您的 find_max 函数可以声明为:

double find_max(const ElevationsType &elevations, double ilat, double ilon);

请注意，在这种情况下，您不需要传递 nlat 和 nlon，因为您可以这样做:

ElevationsType::size_type nlat, nlon, i, j;
nlat = elevations.size();

for (i = 0; i != nlat; ++i) {
    nlon = elevations[i].size();
    for (j = 0; j != nlon; ++j) {
        const double element = elevations[i][j];
        // do whatever you need to do with the element
    }
}

当然，如果您的数组具有固定大小，您可以在创建 ElevationsType 类型的对象后设置它 (std::vector::resize)，或者只分配足够的空间 (std::vector::reserve) 和然后初始化它。如果它很大，可能会提高性能。

但是，如果您选择使用 C 风格的数组，它会是这样的:

double **elevations = (double **)malloc(sizeof(double*) * lat_row);
for (size_t i = 0; i != lat_row; ++i) {
    elevations[i] = (double*)malloc(sizeof(double) * lat_col);

    // initialize the elements
    for (size_t j = 0; j != lat_col; ++j) {
        elevations[i][j] = 100.0; /* your value */
        std::cout << "elevations[" << i << "][" << j << "] = " << elevations[i][j] << std::endl;
    }
}

这对很多人来说更乏味..可以这么说。如果您走这条路，请不要忘记使用 free() 释放所有分配的内存。

你也可以使用c++的new运算符来分配内存，但是原理是差不多的。

所以我建议您使用 std::vector。它更容易使用，至少在您经验有限的情况下。它还会处理内存分配/取消分配，这会导致许多不好的事情、溢出、泄漏等，如果您使用 vector 就可以避免这些问题。