Question

我无法弄清楚为什么这不会运行。我应该在工资数组中存储hours * payrate，然后showResults获取此信息并cout。我运行它时得到的错误是

error LNK2019: unresolved external symbol "void __cdecl showResults(int * const,int,double * const,double * const,double * const)" (?showResults@@YAXQAHHQAN11@Z) referenced in function
error LNK2019: unresolved external symbol "void __cdecl getEmployeeData(int * const,int,double * const,double * const,double * const)" (?getEmployeeData@@YAXQAHHQAN11@Z) referenced in function `_main`

以下代码：

#include <iomanip>
#include <iostream>
using namespace std;

void getEmployeeData(int[], int, double[], double[], double[]);
void showResults(int[], int, double[], double[], double[]);


int main()
{
const int ARRAY_SIZE = 7;
int empId[ARRAY_SIZE] = {565, 845, 452, 130, 789, 758, 877};

double hoursWorked[ARRAY_SIZE]; // Holds hours worked
double payrate[ARRAY_SIZE]; // Holds pay rate
double wages[ARRAY_SIZE]; // Holds wages

getEmployeeData(empId, ARRAY_SIZE, payrate, hoursWorked, wages);

showResults(empId, ARRAY_SIZE, payrate, hoursWorked, wages);

system("pause");
    return 0;
}

void getEmployeedata(int nums[],int size,double pay[],double hours[],
double wages[])
{

//Hours worked and Pay rate
for(int index = 0; index < size; index++)
{
    cout << "Enter number of hours worked by employee number "
         << nums[index] << ": ";
    cin >> hours[index]; 
    cout << "\nEnter hourly pay rate ";
    cin >> pay[index];
    wages[index] = hours[index] * pay[index]; 
}
}

void showResults(int nums[], int size, double pay, double hours, double wages[])
{
for(int index = 0; index < size; index++)
{
    cout << "Employee Number Gross Wage " << endl
         << nums[index] << " " << wages[index];

}

}

Answer 1

showResults的声明和定义不同意参数类型。特别是，pay和hours参数在声明中为double[]，但在定义中仅为double。

void showResults(int[], int, double[], double[], double[]) // Declaration
//                                 ↕↕        ↕↕
void showResults(int[], int, double  , double  , double[]) // Definition

声明似乎是正确的，而且定义不是，因为你要将数组传递给它们。

getEmployeeData的声明和定义不同意该名称。

void getEmployeeData(int[], int, double[], double[], double[]) // Declaration
/               ↕
void getEmployeedata(int[], int, double[], double[], double[]) // Definition

您可能希望getEmployeeData与其他命名保持一致。

Answer 2

您的函数名称有拼写错误：

getEmployeeData(empId, ARRAY_SIZE, payrate, hoursWorked, wages);

然而，在定义中你有：

getEmployeedata(...);

这里也不匹配：

 void showResults(int nums[], int size, double pay, double hours, double wages[])
                                            //^^^^^^^^^^^^^^^^^

Answer 3

如果要将数组传递给函数，也可以尝试将函数写为

void getEmployeeData(int*, int, double*, double*, double*);
void showResults(int*, int, double*, double*, double*);

因为当您传递数组时，您将指针传递给该数组的第0个索引。另外，正如所述，您的代码中存在拼写错误，将getEmployeedata替换为getEmployeeData

以下是更正后的代码：

#include <iomanip>
#include <iostream>
using namespace std;

void getEmployeeData(int*, int, double*, double*, double*);
void showResults(int*, int, double*, double*, double*);


int main()
{
const int ARRAY_SIZE = 7;
int empId[7] = {565, 845, 452, 130, 789, 758, 877};

double hoursWorked[ARRAY_SIZE]; // Holds hours worked
double payrate[ARRAY_SIZE]; // Holds pay rate
double wages[ARRAY_SIZE]; // Holds wages

getEmployeeData(empId, ARRAY_SIZE, payrate, hoursWorked, wages);

showResults(empId, ARRAY_SIZE, payrate, hoursWorked, wages);

system("pause");
    return 0;
}

void getEmployeeData(int nums[],int size,double pay[],double hours[],double wages[])
{

//Hours worked and Pay rate
for(int index = 0; index < size; index++)
{
    cout << "Enter number of hours worked by employee number "
         << nums[index] << ": ";
    cin >> hours[index];
    cout << "\nEnter hourly pay rate ";
    cin >> pay[index];
    wages[index] = hours[index] * pay[index];
}
}

void showResults(int *nums, int size, double *pay, double *hours, double *wages)
{
for(int index = 0; index < size; index++)
{
    cout << "Employee Number Gross Wage " << endl
         << nums[index] << " " << wages[index];

}

}

Answer 4

由于您确切知道每个数组有多少个元素，因此您可以尝试将ARRAY_SIZE设为全局常量（因此它与函数原型的范围相同），允许您执行以下操作：

void getEmployeeData(int (&)[ARRAY_SIZE], int, double (&)[ARRAY_SIZE], double (&)[ARRAY_SIZE], double (&)[ARRAY_SIZE]);
void showResults(int (&)[ARRAY_SIZE], int, double (&)[ARRAY_SIZE], double (&)[ARRAY_SIZE], double (&)[ARRAY_SIZE]);

void getEmployeeData(int (&nums)[ARRAY_SIZE], int size, double (&pay)[ARRAY_SIZE], double (&hours)[ARRAY_SIZE], double (&wages)[ARRAY_SIZE]) {
// Function body
}
void showResults(int (&nums)[ARRAY_SIZE], int size, double (&pay)[ARRAY_SIZE], double (&hours)[ARRAY_SIZE], double (&wages)[ARRAY_SIZE]) {
// Function body
}

现在，这看起来很尴尬，但它的作用是告诉编译器该函数引用了具有完全 ARRAY_SIZE元素的数组。（具体来说，[]的优先级高于&，因此您需要使用括号告诉编译器该参数是对数组的引用而不是引用数组。）它还允许您使用ol'C sizeof(nums)/sizeof(nums[0])来确定数组的大小;不像让数组衰减成指针（在这种情况下sizeof会给你无用的信息，因为它将nums视为int*而不是int[ARRAY_SIZE]），传递给它参考保留了这些信息。

或者，如果您可以自由地重写代码并且您的编译器支持C ++ 11，则可以使用std::array而不是C风格的数组。

#include <array>
using namespace std;

void getEmployeeData(array<int, ARRAY_SIZE>, int, array<double, ARRAY_SIZE>, array<double, ARRAY_SIZE>, array<double, ARRAY_SIZE>);
void showResults(array<int, ARRAY_SIZE>, int, array<double, ARRAY_SIZE>, array<double, ARRAY_SIZE>, array<double, ARRAY_SIZE>);


array<int, ARRAY_SIZE> empId = {565, 845, 452, 130, 789, 758, 877};

array<double, ARRAY_SIZE> hoursWorked; // Holds hours worked
array<double, ARRAY_SIZE> payrate; // Holds pay rate
array<double, ARRAY_SIZE> wages; // Holds wages

作为容器，std::array可以使用与C样式数组相同的语法（即operator[]），因此您不需要重写任何实际使用该代码的代码阵列。它还允许getEmployeeData()和showResults()直接检查数组的大小，例如nums.size()。

另请注意，如果您使用其中任何一项，则实际上不需要传递int size，因为您可以使用sizeof或std::array.size()。尽管如此，这确实有一点污染全局命名空间的缺点，但我个人认为这是一个公平的权衡，保证只有在函数给出正确大小的数组时才能编译代码。

[请注意，如果您需要能够在不同时间传递不同大小的数组，则这些方法都不会对您有用。]

我为可能错过的任何错字道歉。

c ++通过函数传递数组

4 个答案: