麻烦将指针数组传递给函数

时间:2014-11-10 17:18:56

标签: c++ arrays function pointers vector

在我正在处理的程序中,我有一个模板函数,该函数包含在一个单独的.h文件中,该文件从.txt文件中读取五列数据。数据传递给主程序,在这种情况下,我只关心数组标题“MISC_DATA”。我试图确定数组“MISC_DATA”中的最大值,并编写了另一个必须传递数据的函数,以确定这一点。但是,编译器告诉我它无法识别函数调用“Maximum_Value”。我很确定它在例程调用中包含的变量MISC_DATA存在问题,而不是函数本身。要么它不能将MISC_DATA识别为数组,要么我的语法错误。我只包含重要的代码片段,以使其更具可读性。 Read_Five_Columns函数工作正常,它是函数“Maximum_Value”,由于指针数组MISC_DATA在主程序中的写入方式,编译器无法识别该函数。为了澄清,函数调用中的变量MISC_DATA是包含数组的float,变量“size_Mis”是包含数组大小的整数。任何想法都将不胜感激。

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

#include "Use_RNG.h"
#include "Read_Columnar_File.h"
#include <fstream>
#include <iostream>

 std::vector<std::string> str3;
 std::vector<int> str4;
 std::vector<char> str5;
 std::vector<int> str6;

    unsigned long size_Mis;
    std::vector<float> MISC_DATA;  // Reads in Misc. spending data
    char File1[8];
    strcpy(File1, "Misc.txt");
    Read_Five_Columns(File1,MISC_DATA,str3,str4,str5,str6);
    str3.clear(); str4.clear(); str5.clear(); str6.clear();
    size_Mis = MISC_DATA.size();

float value;
value = Maximum_Value(MISC_DATA,size_Mis);

end_time = clock();
std::cout << std::endl << "Total Time: " << (end_time-start_time)/CLOCKS_PER_SEC << std::endl;
return 0;
}

int Maximum_Value(float *array,int array_size)
{
    float max = 0;
   for(int i =10; i < array_size-1; i++)
   {
        if(array[i] > max) max = array[i];
   }
    return max;
}

1 个答案:

答案 0 :(得分:6)

我在这里看到了四个问题。

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

#include "Use_RNG.h"
#include "Read_Columnar_File.h"
#include <fstream>
#include <iostream>

所有这些东西都是错误的顺序。您不应该将系统头文件包含在函数体中,并且通常在其他内容之前包含标准库。将其修改为如下所示:

#include <fstream>
#include <iostream>

#include "Use_RNG.h"
#include "Read_Columnar_File.h"

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

其次,在使用之前,您不会声明Maximum_Value。您需要在定义main()之前移动此函数的定义,或者需要在main()之前添加原型:

int Maximum_Value(float *array,int array_size);

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

然后,您尝试将std::vector<float>作为float*传递,但不起作用:

value = Maximum_Value(MISC_DATA,size_Mis);

但是,因为向量的存储保证是连续的并且像数组一样布局,所以可以安全地将指针传递给第一个成员:

value = Maximum_Value(&MISC_DATA[0],size_Mis);

最后,当您应该返回int时,从[{1}}返回Maximum_Value


如果可能,我建议利用float,这是标准std::max_element标题的一部分:

<algorithm>

现在// If you don't have C++11 then use std::vector<float>::iterator instead of auto. auto max = std::max_element(MISC_DATA.begin(), MISC_DATA.end()); 是最大元素的迭代器,因此max本身就是最大的*max

(如果输入范围为空,则float将等于max,因此与您的函数相当的是MISC_DATA.end()。)