Printable View
Hi there. I'm currently a student in a CS degree and I'm looking for someone to mentor/tutor me. I'm looking for someone that can help me with challenging assignments/topics, have in-depth conversations regarding the topics I'm learning about and just all around help me out.
We are coding in C++ and using Visual Studio 2013.
Thanks!
Jenn
What is your C++ question?
I'm creating a program that will read in a data file regarding vehicle registration per household, then calculates the registration tax for each vehicle, the household total and the overall total. Then writes the household totals into an external file as well as on the screen.
I think I've figured out how to read in data, write out the data and calculate the fees. But I don't know how to create the loop that will read a line, do the calculation and writing, store the information for screen display, then moves onto the next line.
It's much better to team up with class mates. Be part of an informal study group. That's how you get through college (learning and having fun together).Quote:
Originally Posted by spideyflygirl90
I'm doing an online program so no one in the area. I've put the same feelers out there asking for some help but no one else seems to get it either. Haha
Well, since it's an online course it can't be considered cheating to search the internet. :)
http://learnedstuffs.wordpress.com/2...-reading-in-c/
Possibly the easiest way to have a loop that reads some data from a file until the end and processes that data is something likeQuote:
Originally Posted by spideyflygirl90
Code:ifstream infile("infilename");
while (infile >> input1 >> input 2 >> etc etc etc) {
//process data
}
haha. Which is why I'm here hoping for some discussion/help :-) I'll check that out for sure!Quote:
Originally Posted by razzle
My assignment has to be modular so have at least 3 differently defined codes. So I'm trying to write aQuote:
Originally Posted by 2kaud
int readCode()
int writeCode()
int calculateFees()
int main()
maybe an int displayScreen() but I think it would probably be easier to put this in main.
so I wrote readCode to do just one line, same with writeCode and calculateFees. then I'm trying to put my loop within main to call each of the three functions and add to displayscreen before iterating the loop.....
So I have
int getCode()
{....}
within my int main() while loop, can I use a boolean statement like: while (getCode == true) {...} ??
If you posted some of the code you're written and an example of the data file we'll be able to advise further. It could be that you just need to have a loop that just calls these functions in turn. The slight issue with having a function for readCode() is dealing with errors and end-of-file condition.
Basically the program then becomes in pseudocode something like
Code:while (file-no-error) {
ReadNextRecordFromFile;
CalculateFees;
WriteTotalsToFile;
DisplayInformation;
}
Okay. This is going to be kind of a long post ;-) I'm still pretty new to programming. This is only like my 3rd class.
So my assignment is to read in a data file of vehicles registered per household.
Input ex:
111111 A ABC123 2012 36666 M XYZ7892005 15555
222222 A DDD333 1999 22222
which follows the form of:
HouseholdID VehicleClass (Motorcycle or automobile) LicensePlate VehicleYear Original Cost
So I have to figure out registration fees based upon the age and original cost (I have the formulas for this given to me).
Then I have to output 1 file VEHICLES.txt which lists the vehicles and their registration costs
Ex:
A ABC123 1160.65
M XYZ789 341.32
A DDD333 355.11 (these may not be accurate calculations just an example)
Also an output file HOUSEHOLDS.txt which lists the house id and total registration fees for that house hold (adding all vehicle fees)
Ex:
111111 1501.97
222222 355.11
Then display it on screen in the format:
Household ID Number Vehicles Registered Total Charges
---------------------------------------------------------------------
111111 2 1501.97
222222 1 355.11
All the decimals need to line up in my outputs as well (ie, look professional). I can't use global variables. And I have to be modular - using different functions with the main basically calling the other functions.
Okay so for my code, here's what I have so far....I know I still have a lot to do. Like I said, I'm new to this :-D
/*The purpose of this program is to read in data from an existing file containing information regarding households and registered vehicles.
The program will then calculate how much the registration fees will be based upon original cost and age of the vehicle.
The program will display this information by vehicle, household, and finally a total for the county.*/
#include <iostream>
#include <fstream>
#include <iomanip>
#include <string>
using namespace std;
int getData(int houseID, int vehicleYear, int originalCost, string vehicleClass, string licensePlate)
{
ifstream myFileIn;
myFileIn.open("REGISTER.txt");
if (myFileIn.is_open() == false)
{
cout << "I'm sorry. The file " << myFileIn << " cannot be opened. Please press enter to exit the program and try again. Thank you." << endl;
return 0;
}
else
{
myFileIn >> houseID >> vehicleClass >> licensePlate >> vehicleYear >> originalCost;
}
}
int writeDataVehicles(string vehicleClass, string licensePlate, double registrationFees)
{
ofstream vehiclesOut;
vehiclesOut.open("VEHICLES.txt");
vehiclesOut << vehicleClass << setw(8) << licensePlate << setprecision(2) << setw(8) << right << registrationFees << endl;
}
int writeDataHouseholds(int houseID, double totalHouseholdFees)
{
ofstream householdsOut;
householdsOut.open("HOUSEHOLDS.txt");
householdsOut << houseID << setw(8) << totalHouseholdFees << endl;
}
int calculateFees(int originalCost, int vehicleYear, double licenseFee, double ownershipTaxA, double ownershipTaxM, string vehicleClass)
{
const double roadFeeA = 24;
const double roadFeeM = 12;
licenseFee = 0.0025 * originalCost;
double vehicleAge = 2014 - vehicleYear;
if (vehicleClass = A)
{
if (vehicleAge <= 5)
{
ownershipTaxA = (0.03 * originalCost) - ((0.015 * originalCost) * vehicleAge);
}
else if (5 < vehicleAge <= 10)
{
ownershipTaxA = (0.0275 * originalCost) - ((0.012 * originalCost) * vehicleAge);
}
else if (10 < vehicleAge <= 15)
{
ownershipTaxA = (0.025 * originalCost) - ((0.009 * originalCost) * vehicleAge);
}
else
{
ownershipTaxA = 20.00;
}
}
else if (vehicleClass = M)
{
if (vehicleAge <= 5)
{
ownershipTaxM = (0.025 * originalCost) - ((0.009 * originalCost) * vehicleAge);
}
else if (5 < vehicleAge <= 10)
{
ownershipTaxM = (0.0225 * originalCost) - ((0.009 * originalCost) * vehicleAge);
}
else if (10 < vehicleAge <= 15)
{
ownershipTaxM = (0.02 * originalCost) - ((0.009 * originalCost) * vehicleAge);
}
else
{
ownershipTaxM = 10.00;
}
}
}
int main()
{
int houseID, vehicleYear, originalCost;
string vehicleClass, licensePlate;
double registrationFees, totalHouseholdFees, licenseFee, ownershipTaxA, ownershipTaxM;
while (getData()
{
}
}
When posting code please format your code properly first and use code tags. Go Advanced, select the code and click '#'.
From this, there seems that there can be more than one vehicle per household stated on the same line in the data file? Is that right? Also should there be a space before 2005 (ie should the data actually be XYZ789 2005)?Code:111111 A ABC123 2012 36666 M XYZ7892005 15555
Also, in getdata(), you'll need to open the file in main() (or have another routine to do this) so that you don't keep opening the file everytime you read a line. The same goes for the various write functions as well. Often the names of the files would be defined as a const string at the top of the program so that it can easily be changed without having to search all of the program for them.
Have you come across map yet in c++ STL? If you have then using it is going to make the issue of getting the totals per household easier.
Yes. The number of vehicles per household is unknown. (As in it could vary based on data input file. Yes XYZ789 2005. Sorry.
I didn't know about the code box thing. I will do that next time. My apologies.
So I could write an openFile() that opens both the input and output files...correct? then have getdata() and writedata()? Would that be the easier/proper route or would it be better to write open into main()?
Writing the files as const string count as a global variable? Or because it's a constant, it's not a variable?
I have not learned map yet. I'm still very new to programming. Struggling without someone to discuss it with too! Haha. But I'm trying :-)
If the input data can have lines of varying length that conatin 1 or more sets of data relating to a specified household, then you have a very different input acquisition problem from the way you are doing it. You don't know how many sets of vechicle data there are on a line for the household so your getData() routine won't work properly.
Basically, there are two ways to approach this.
1) Use GetLine() to read a whole line of data from the file at once into a string and then parse the string which has a format of household <vechicle> {<vechicle>} where <vechicle> has the format VehicleClass (Motorcycle or automobile) LicensePlate VehicleYear Original Cost
2) Use the stream extraction method you are currently using, but if you get a single letter for the house ID then you have new vechicle data for the previous household.
Neither is exactly trivial and which way you opt to go will depend largely upon what you feel most comfortable with.
Well I did really awful on this assignment *hangs head* Now we've been given assignment 2 and it seems equally as difficult!
As you have now been assessed on your assignment 1, have you been given a 'model answer' by the course tutors so that you can see how it could have been done?
Nope. Just the next assignment. This week we are learning arrays. I've already gotten my assignment and am ready to start working on it. *deep breath* More time to work on it this week! That way I can get help easier!!
It's pretty similar to my last assignment actually.
Taking a quick look through the code, there are some very basic errors.
This routine is to obtain data from a file. However you are passing these parameters by value so any change you make to them in the function will not be reflected in the calling routine. If you want changed values to be available in the calling routine then you need to pass by reference likeCode:int getData(int houseID, int vehicleYear, int originalCost, string vehicleClass, string licensePlate)
Code:int getData(int& houseID, int& vehicleYear, int& originalCost, string& vehicleClass, string& licensePlate)
There are two issues with this. Firstly == is used as the equality comparision and secondly character constants are placed within ' '. What this statement does is assigns the value of variable A to the variable vechicleClass and tests whether this value is non-zero. You useCode:if (vehicleClass = A)
orCode:if (vechicleClass == 'A')
if comparing against a string instead of a char.Code:if (vechicleClass == "A")
whilst syntactically correct, it doesn't do what you think it will. It first compares 5 with vehicleAge which gives either a 0 or a 1. It then compares this 1 or 0 with 10. As it is always less than 10, this statement is always true. You need to writeCode:else if (5 < vehicleAge <= 10)
Code:else if (vechicleAge > 5 && vechicleAge <= 10)
Those make a lot of sense!
I had figured out the vehicleClass == 'A' one. But I didn't know about the else if or the int&.
My new assignment that I'll be starting on tonight (when I'm not on a work computer) is
1. Read in a file of vehicle type and registration cost.
2. Sort that into an array. 1 for Autos and 1 for Motorcycles,
3. Calculate the highest registration cost, the lowest, and the average for each array.
4. Array size of 500 - produce an error if more than 500 lines.
5. Output to screen number of vehicles registered, high, low and average for each array.
So I'm thinking that instead of getData, I need to use getline() so that it reads a line at a time. Then have a loop to read the next line. Test something like, if (char == '\n'). Would that work?
Then for the array, I should just be able to make one of size 499 (to store 500 vehicles) and then I'm not sure how but search it for the high and low.
Am I even in the right direction?? o_O
To help your studies, the code below is one possible version for your first assignment. I'm not sure about the calculation formulaes though.
For the registration.txt file containingCode:#include <iostream>
#include <fstream>
#include <iomanip>
#include <string>
#include <map>
using namespace std;
const string dataname = "REGISTER.txt";
const string vehname = "VECHICLES.txt";
const string housename = "HOUSEHOLDS.txt";
struct datarec {
string houseid;
int vehyear;
int orgcost;
string vehclass;
string license;
double tax;
double licfee;
};
struct houserec {
double fee;
int no;
};
typedef map<string, houserec> dmap;
typedef dmap::const_iterator cidmap;
bool getData(ifstream& ifs, datarec& data);
void calculateFees(datarec& data);
void showData(const datarec& data);
void writeDataVehicles(ofstream& ofs, const datarec& data);
void writeDataHouseholds(ofstream& ofs, const string& house, double fee);
bool getData(ifstream& ifs, datarec& data)
{
static string oldid;
ifs >> data.houseid;
if (data.houseid == "A" || data.houseid == "M") {
//Got multiple vehicle for 1 houseid
data.vehclass = data.houseid;
data.houseid = oldid;
ifs >> data.license >> data.vehyear >> data.orgcost;
} else
ifs >> data.vehclass >> data.license >> data.vehyear >> data.orgcost;
oldid = data.houseid;
calculateFees(data);
return ifs.good();
}
void writeDataVehicles(ofstream& ofs, const datarec& data)
{
ofs << data.vehclass << setw(8) << data.license << setprecision(2) << fixed << setw(8) << right << data.licfee << endl;
}
void writeDataHouseholds(ofstream& ofs, const string& house, double fee)
{
ofs << house << setprecision(2) << fixed << setw(8) << right << fee << endl;
}
void calculateFees(datarec& data)
{
const int vehicleAge = 2014 - data.vehyear;
data.licfee = 0.0025 * data.orgcost;
if (data.vehclass == "A") {
if (vehicleAge <= 5) {
data.tax = (0.03 * data.orgcost) - ((0.015 * data.orgcost) / vehicleAge);
} else {
if (vehicleAge > 5 && vehicleAge <= 10) {
data.tax = (0.0275 * data.orgcost) - ((0.012 * data.orgcost) / vehicleAge);
} else {
if (vehicleAge > 10 && vehicleAge <= 15) {
data.tax = (0.025 * data.orgcost) - ((0.009 * data.orgcost) / vehicleAge);
} else {
data.tax = 20.00;
}
}
}
} else {
if (data.vehclass == "M") {
if (vehicleAge <= 5) {
data.tax = (0.025 * data.orgcost) - ((0.009 * data.orgcost) / vehicleAge);
} else {
if (vehicleAge > 5 && vehicleAge <= 10) {
data.tax = (0.0225 * data.orgcost) - ((0.009 * data.orgcost) / vehicleAge);
} else {
if (vehicleAge > 10 && vehicleAge <= 15) {
data.tax = (0.02 * data.orgcost) - ((0.009 * data.orgcost) / vehicleAge);
} else {
data.tax = 10.00;
}
}
}
} else {
data.tax = 0;
}
}
}
void showData(const datarec& data)
{
cout << data.houseid << " " << data.vehclass << " " << data.license << " " << data.vehyear << " " << data.orgcost << " " << data.tax << " " << data.licfee << endl;
}
int main()
{
ifstream dataIn(dataname.c_str());
if (dataIn.is_open() == false) {
cout << "The file " << dataname << " cannot be opened.\n";
return 1;
}
ofstream vehOut(vehname.c_str());
if (vehOut.is_open() == false) {
cout << "The file " << vehname << " cannot be opened.\n";
return 2;
}
ofstream houseOut(housename.c_str());
if (houseOut.is_open() == false) {
cout << "The file " << housename << " cannot be opened.\n";
return 2;
}
datarec data;
dmap house;
while (getData(dataIn, data)) {
//showData(data);
writeDataVehicles(vehOut, data);
house[data.houseid].fee += data.licfee;
house[data.houseid].no++;
}
for (cidmap it = house.begin(); it != house.end(); ++it) {
writeDataHouseholds(houseOut, it->first, it->second.fee);
cout << it->first << setw(4) << it->second.no << setprecision(2) << fixed << setw(8) << right << it->second.fee << endl;
}
return 0;
}
This produces for file vehicles.txtCode:111111 A ABC123 2012 36666 M XYZ789 2005 15555
222222 A DDD333 1999 22222
and for households.txtCode:A ABC123 91.67
M XYZ789 38.89
A DDD333 55.56
and displays on the screenCode:111111 130.55
222222 55.56
which is think is what was required.Code:111111 2 130.55
222222 1 55.56
For reading data from a file, look at the getData() function in the sample code I posted and how I used that function in main().Quote:
Originally Posted by spideyflygirl90
There is an STL function sort() that will do the sort for you. See http://www.cplusplus.com/reference/algorithm/sort/
If you just want to calculate the highest, lowest and average for different vechicle types, there is no need to read the data into arrays. You want a struct containing high, low, average, total, numreg and then a 2 element array of this struct (for types A and M ) or even better use a map like in my example program. As you read in each line from the file, just update high, low, total, numreg for the appropriate array and then at the end calculate the average. Then just output the required info. Simples!:D
The assignment is teaching us to use arrays. Haha. :eek: Hahaha.Quote:
Originally Posted by 2kaud
I'm trying to get an idea of where to start before I get home from work and actually start. Haha
Well, 2kaud's suggestion is more efficient than reading into an array and then repeatedly looping over the array to obtain the results that you want. This is a good. On the other hand, it couples together the distinct operations to obtain each result, which can be a bad since you cannot reuse the code to say, only find the highest.Quote:
Originally Posted by spideyflygirl90
I would start by producing a simple program that just loops reading the data from the file and displaying it on the screen (see my sample program for an idea of one way of doing it). That was the first version of the example I posted above. Once that worked OK I then extended the program in stages. Each stage was tested first before the next stage was added. This way you always have a working program - one that doesn't do all it's supposed to, but working nevertheless. Once you have a program that you know reads the data OK, then you can add code that puts it in an array. Test that. Then sort the array and test that. Then add code to find highest etc etc etc. The important point is that at every stage you have a working program that does what's it's supposed to at that stage in the design.Quote:
I'm trying to get an idea of where to start before I get home from work and actually start
If you have a go and post some code here, we'll provide guidance but won't produce the code for you as this is a marked assessment. See http://forums.codeguru.com/showthrea...ork-assignmentCode:Do {
Design
code
test
debug
} while program-not-correct
Good luck!:thumb:
Oh, I don't want you guys to write it for me! Then I'll never learn and I'll forever be stuck!!! I'm working on it now and will post some code when I get stuck...hahaha. As in, if I finish writing the getLine part and it doesn't work! ;-PQuote:
Originally Posted by 2kaud
So I'm trying to do it piece by piece, starting with opening the file. However, it's giving me an error that says "Cannot find or open the PDB file".
Here's the very basic code I have so far. I'm wondering I guess, does the VEHICLES.txt have to be saved somewhere specific? Or linked to in some special way?
Code:#include<iostream>
#include<iomanip>
#include<fstream>
#include<string>
using namespace std;
void main()
{
char vehicleClass;
string licensePlate;
float registrationFee;
ifstream infile;
infile.open("VEHICLES.txt");
while (infile >> vehicleClass >> licensePlate >> registrationFee)
{
cout << "The file was successfully opened." << endl;
}
}
The PDB file (Program Database) holds debugging and project state information that allows incremental linking of a Debug configuration of your program. A PDB file is created when you build with /ZI or /Zi (for C/C++). See http://msdn.microsoft.com/en-us/libr...=vs.90%29.aspxQuote:
However, it's giving me an error that says "Cannot find or open the PDB file".
This message is an issue with the way the project has been set up. Have a look at
http://social.msdn.microsoft.com/For...?forum=vsdebug
Re your program. main() should be defined as
VEHICLES.txt needs to exist in the same folder as the program if you don't specify a folder location. A way to open a file and test if was opened properly is like this from my previous exampleCode:int main()
When you define the file variable, you can specifiy the file name as wellCode:ifstream infile("VEHICLES.txt");
if (infile.is_open() == false) {
cout << "The file VEHICLES.txt cannot be opened.\n";
return 1;
}
cout << "The file was successfully opened." << endl;
while (infile >> vehicleClass >> licensePlate >> registrationFee)
{
//process data
}
http://www.cplusplus.com/reference/f...ream/ifstream/
Using .open() is OK but as you know the name of the file when you define the file variable IMO specifying the name then is easier than having a separate .open() statement.
The class function .is_open() returns true if there is a valid file opened to the file variable. If the open failed it returns false. As the program is concerned with file processing, the rest of the program is invalid if the file hasn't been opened OK. Another way of coding this could be
This way there is only 1 entry and 1 exit point to the function - rather than 2 exits as in the previous example.Code:ifstream infile("VEHICLES.txt");
int retval = 0;
if (infile.is_open() == true)
{
cout << "The file was successfully opened." << endl;
while (infile >> vehicleClass >> licensePlate >> registrationFee)
{
//process data
}
} else {
cout << "The file VEHICLES.txt cannot be opened.\n";
retval = 1;
}
return retval;
}
stream extraction (infile >> vehicleClass ...) returns a NULL if there is a problem performing the extraction (see http://www.cplusplus.com/reference/i...perator%3E%3E/). Hence the while condition is true if the extraction is successfull and is false if there is an error with the extraction and the while loop then terminates. The extraction can fail for three reasons 1) end-of-file has been reached 2) the characters extracted could not be interpreted as a valid value of the appropriate type (trying to read an int and finds an 'A' 3) an error occured during the extraction and data couldn't be read. See http://www.cplusplus.com/reference/ios/ios/good/
Update:
So I figured out the major reason why I was struggling so much! My computer has a bug! Ahh!! I've been needing a new laptop for school, picked it up yesterday, ran the codes I have finished so far for my assignment this week and they worked!
What a relief.
So I have my openfile and getdata functions figured out! working on sort data and displayresults!
I'm stuck again, maybe someone can help ;-P
I wrote my openFile function and tested it, worked just fine! I put a cout statement to verify that it worked. It displayed perfectly. Then I moved it into it's own function instead of main, called it from main, and now it won't display the cout statement. It just gives me a return 0 value. Here's my code: I have extra #include statements for my following code but I haven't gotten that far yet.
Code:#include<iostream>
#include<iomanip>
#include<fstream>
#include<string>
#include<sstream>
using namespace std;
int main()
{
// declare the needed variables
char vehicleClass;
string licensePlate;
float registrationFee;
ifstream inFile;
int openFile();
}
int openFile(ifstream& inFile)
{
// open the file
inFile.open("VEHICLES.txt");
if (!inFile) // checks for error while opening
{
cout << "Cannot open the input file. "
<< "The program terminates." <<endl;
}
else
{
cout << "The file was opened successfully." << endl;
}
}
I think my issue is with how I'm passing the variable from main to openFile but I'm not sure.
This is a function declaration for a function that takes no arguments and returns an int. It's not a function call. As openFile is defined after it is going to be used, openFile needs to have a forward declaration before it is used so the compiler knows what arguments it takes and the return type if any. This function declaration would beCode:int openFile();
and usually function declarations are placed near the beginning of the program.Code:int openFile(ifstream& inFile);
To call openFile you would use
openFile is defined as returning an int but the function definition doesn't return any value. What value do you want the function to return if the open succeeds and what value if the open fails? Then when you call the function you should test its return value to check whether the function succeeded or failed.Code:openFile(inFile);
I changed it to void openFile() since it doesn't return a value really, just a statement. Does that work? I could add inside the if(!inFile) someway to close the program?
Also, I put openfile() before main() already. Haha. Noticed that. I changed it to openfile(ifstream& inFile). Figured those out on my own! Yay. I'm learning.
I didn't get the openFile(inFile) though. I'll try that right now :-)
How do you break a string into different variables?
I got openFile to work. And I've gotten getLine figured out. And my loop! Yay.
However, my getLine is putting the whole line into a string. How do I parse that string into the 3 variables I need?
openfile() really should return some value to indicate that the file has been opened properly oe not which is checked for in the calling program. One possibility is have bool openfile() which returns true if the file opened or false if it did not.Quote:
I changed it to void openFile() since it doesn't return a value really, just a statement
Rather than getLine putting the whole line into a string, why not just extract the 3 parts direct via stream extraction (eg as I did in my example program is post #21). Something likeQuote:
However, my getLine is putting the whole line into a string. How do I parse that string into the 3 variables I need?
You can parse the getLIne string into the 3 parts yourself, but its much simpler/easier just to extract them direct from the file.Code:infile >> first >> second >> third;
Thanks! I keep posting questions and then figuring out how to get past them. Hahaha. I've gotten as far as writing data to my array but for some reason the loop is running 1 too many times and giving an extra value in each array that is something like 3.49e-59.
The most common cause of looping 1 too many times is using <= in a condition rather than <. Without seeing the code I can't really offer more guidance.
I figured that out! Hahahaha. Told you. I seem to be asking, and then figuring it out!! Makes me feel so good!!!
However, can you possibly explain enum types to me? I'm supposed to use them in my program to store LOW, HIGH and AVERAGE.
I've gotten my program to work completely including cout statements that display the low high and average values but I don't understand enum at all.
You'll be answering questions next! :cool:Quote:
I figured that out! Hahahaha. Told you. I seem to be asking, and then figuring it out!! Makes me feel so good!!!
Here's my code (just the relevant parts - the rest is checked) but I'm obviously doing it wrong since it isn't working. Suggestions on how to fix it?
Code:void displayData (const int& vehicleCount, const string& vehicleType, float stats[])
{
cout << "For " << vehicleCount << " " << vehicleType << " registrations in the county: " << endl;
cout << "Lowest Registration Charge = $" << stats[LOW] << endl;
cout << "Highest Registration Charge = $" << stats[HIGH] << endl;
cout << "Average Registration Charge = $" << stats[AVERAGE] << endl;
}
int main()
{
// declare the needed variables
ifstream inFile;
const int MAX = 500;
float autoArray[MAX], motoArray[MAX];
int autoCount = 0, motoCount = 0;
enum charges {LOW, HIGH, AVERAGE};
openFile (inFile);
sortData (inFile, autoArray, autoCount, motoArray, motoCount);
charges autoStats[2], motoStats[2];
autoStats[LOW] == minCharges (autoArray, autoCount);
motoStats[LOW] == minCharges (motoArray, motoCount);
autoStats[HIGH] == maxCharges (autoArray, autoCount);
motoStats[HIGH] == maxCharges (motoArray, motoCount);
autoStats[AVERAGE] == avgCharges (autoArray, autoCount);
motoStats[AVERAGE] == avgCharges (motoArray, motoCount);
displayData (autoCount, "automobile", autoStats[charges]);
displayData (motoCount, "motorcycle", motoStats[charges]);
You are using the == conditon where I think you want the = assignment. What does minCharges() function return?
When you create an enum type, the elements are assigned consecutive numbers starting at 0 (unless specified otherwise). So LOW is 0, HIGH is 1 and AVERAGE is 2. You are defining the arrays as having 2 elements (access 0 and 1) but then access using 2 (AVERAGE) so you have a memory buffer overflow as you are accessing memory outside that reserved for the arrays.
You'll also need to change the definiton of autoStats etc from charges to double. Having autoStats an array of charges means that the array elements can only take the values of the type charges (ie LOW HIGH or AVERAGE) which doesn't look like is what you want.
Why is vehicleCount of type const int& ? Parameters of type string are often passed this way but why are you passing an int this way?Code:void displayData (const int& vehicleCount, const string& vehicleType, float stats[])
minCharges() returns the variable MIN. I got the whole assignment to work though!!!