在我正在处理的程序中,我有一个模板函数,该函数包含在一个单独的.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;
}
答案 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()
。)