Hi all, so i am having trouble with pointers. Our assignment is to fill in the places where the comments are to make it work and i did some areas but the rest i still can't figure out how to make it work. Any help and all help will be greatly appreciated.
#include <iostream>
#include <iomanip>
usingnamespace std;
float * GetSales(int numOfSales);
int main()
{
float *monthSales; // a pointer used to point to an array holding monthly sales float total = 0;
float average; // average of monthly sales
int numOfSales; // number of sales to be processed
float total = 0; // total of all sales
cout << fixed << showpoint << setprecision(2);
float GetSales(numOfSales);
//***** write code to call function to read sales data from stdin into the dynamically allocated array
for (int count = 0; count < numOfSales; count++)
{
total = total + monthSales[count];
}
average = total / numOfSales;//**** Fill in code to find the average;
cout << "Average Monthly sale is $" << average << endl;
delete [] monthSales;
monthSales = 0;
//****** Fill in the code to deallocate memory assigned to the array.
return 0;
}
float * GetSales(int numOfSales)
{
float *dataPtr;
cout << "How many monthly sales will be processed? ";
cin >> numOfSales;
dataPtr = newfloat [numOfSales];
//******** Fill in the code to allocate memory for the array pointed to by monthSales.
if ( dataPtr == NULL )
{
cout << "Error allocating memory!\n";
return NULL;
}
cout << "Enter the sales below\n";
for (int count = 0; count < numOfSales; count++)
{
cout << "Sales for Month number " << (count + 1)//******* Fill in code to show the number of the month
<< ":";
cin >> dataPtr[count];
//****** Fill in code to read and store sales into an element of the array
}
return dataPtr;
}
Line 31 isn't quite right. If you take a closer look at GetSales(), you'll see that it takes a number and returns dynamically allocated array of floats. But you aren't saving that pointer to floats thing anywhere.
I'm not sure I follow the logic here, but a couple of things stand out:
You haven't actually initialised numOfSales, because it gets passed by value to GetSales(int). The numOfSales in main() remains unaffected by the changes you make to the numOfSales in GetSales(int). Do your cin >> numOfSales line in main() instead. Also, your line 31 has done nothing for monthSales, you need to assign the return value - maybe try "monthSales = GetSales(numOfSales)" instead.
You might also want to try some input validation, and write some tests, to impress your lecturer.
To summarize:
Move the numOfSales initialisation to main()
Properly initialise monthSales
#include <iostream>
#include <iomanip>
usingnamespace std;
float * GetSales(int numOfSales);
int main()
{
float *monthSales; // a pointer used to point to an array holding monthly sales float total = 0;
float average; // average of monthly sales
int numOfSales; // number of sales to be processed
float total = 0; // total of all sales
cout << fixed << showpoint << setprecision(2);
cout << "How many monthly sales will be processed? ";
cin >> numOfSales;
monthSales = GetSales(numOfSales);
//***** write code to call function to read sales data from stdin into the dynamically allocated array
for (int count = 0; count < numOfSales; count++)
{
total = total + monthSales[count];
}
average = total / numOfSales;//**** Fill in code to find the average;
cout << "Average Monthly sale is $" << average << endl;
delete [] monthSales;
monthSales = 0;
//****** Fill in the code to deallocate memory assigned to the array.
return 0;
}
float * GetSales(int numOfSales)
{
float *dataPtr;
dataPtr = newfloat [numOfSales];
//******** Fill in the code to allocate memory for the array pointed to by monthSales.
if ( dataPtr == NULL )
{
cout << "Error allocating memory!\n";
return NULL;
}
cout << "Enter the sales below\n";
for (int count = 0; count < numOfSales; count++)
{
cout << "Sales for Month number " << (count + 1)//******* Fill in code to show the number of the month
<< ":";
cin >> dataPtr[count];
//****** Fill in code to read and store sales into an element of the array
}
return dataPtr;
}
You're not supposed to actually do someone's homework. You should try to help without actually doing it for them, well not without working thru it with the op.
