[Jgiles21]

#include<iostream>
#include<string>
#include<fstream>
#include<iomanip>

using namespace std;

int const SIZE = 10;

int getValidInput(string driverName[], double judgesScores[][SIZE], double degreeDifficulty[])
{
	int length = 0;
	
	cout<<"Enter Driver Name, or ctrl z to exit: ";
	getline(cin, driverName[length]);

	while(!cin.eof())
	{
			for(int ctr = 0; ctr <= 4; ctr++)
			{
				cout<<"Enter the score given by the jodges: ";
				cin>>judgesScores[length][ctr];

				while(judgesScores[length][ctr] <= 0 || judgesScores[length][ctr] > 10.0 || cin.fail())
				{	
					cin.clear();
					cout<<"Scores mus be between 1.0 and 10.0: ";
					cin>>judgesScores[length][ctr];
				}
			}	
		
		cout<<"What was the degree of difficulty: ";
		cin>>degreeDifficulty[length];
		
		while(degreeDifficulty[length] <= 0 || degreeDifficulty[length] > 1.67 || cin.fail())
		{
			cin.clear();
			cout<<"Degree of difficulty must be between 1.0 and 1.67: ";
			cin>>degreeDifficulty[length];
		}
		
		length ++;
		
		cin.ignore(80, '\n');
		cin.clear();
		cout<<"Enter Driver name, or ctrl z to exit: ";
		getline(cin, driverName[length]);

	}
	return length;
}

void addScores(double driverScore[], double judgesScores[][SIZE], int length)
{
	double totalScore = 0;
	
		for(int ctr = 0; ctr <= 4; ctr++)
		{
			totalScore += judgesScores[length][ctr];
		}
	
		driverScore[length] = totalScore;
		cout<<driverScore[length];
}

int main()
{
	cout<<fixed;

	string driverName[SIZE];
	double driverScore[SIZE];
	double judgesScores[5][SIZE];
	double degreeDifficulty[SIZE];
	int length = 0;

	length = getValidInput(driverName, judgesScores, degreeDifficulty);

	//while(length != 0)
	//{
		addScores(driverScore, judgesScores, length);
	//}

	system("Pause");
}

This post has been edited by ButchDean: 31 May 2011 - 02:43 PM
Reason for edit:: Please use code tags!

[anonymous26]

Could you be more descriptive?

[Jgiles21]

when i print out the contents of the array, in the addScores function, there is a huge jumble of negative numbers, or garbage inside the array

[anonymous26]

That is because you haven't initialized your arrays, hence the garbage. Also you are passing your arrays around by value meaning that once you return from the function all data is lost.

[JackOfAllTrades]

HINT: What is the value of length when you pass it into addScores?

[#define]

Also, you declare

double judgesScores[5][SIZE];

and pass it

int getValidInput( string driverName[], 
                   double judgesScores[][SIZE], 
                   double degreeDifficulty[])

and then use the subscripts the other way round.

cin>>judgesScores[length][ctr];

[Xupicor]

ButchDean, on 31 May 2011 - 02:49 PM, said:

That is because you haven't initialized your arrays, hence the garbage. Also you are passing your arrays around by value meaning that once you return from the function all data is lost.

That's not true at all. If you pass an array to a function, you don't really pass it by value, since it will be implicitly converted to pointer. If you want to pass it by value - you'll have to pack it into a struct - then pass the struct by value. Consider the code below:

//c++0x
#include <iostream>

void modify(int array[], size_t len) {
// will behave the same as: 
//void modify(int* array, size_t len) {
    for (size_t i = 0; i < len; ++i) {
        array[i] = -1;
    }
}

struct Foo {
    int array[5];
};

void try_to_modify(Foo foo) {
    for (int& i : foo.array) {
        i = -1;
    }
}

int main() {
    using std::cout;
    
    int array[5] = { 1, 2, 3, 4, 5 };
    modify(array, 5);
    for (int i : array) {
        cout << i << ' ';
    }
    cout << '\n';
    
    
    Foo foo = { { 1, 2, 3, 4, 5 } };
    try_to_modify(foo);
    for (int i : foo.array) {
        cout << i << ' ';
    }
    cout << '\n';
    
    return 0;
}

[JackOfAllTrades]

for (int& i : foo.array) {

WHOA. New syntax? Neato!

[Xupicor]

Well, it's C++0x ranged for, once you start to use it you forget to stop when typing out a simple example on the boards.

You can pretty much use the first for loop and copy paste it with minimal changes to make the example working in C++03 compiler.

[anonymous26]

Xupicor, on 31 May 2011 - 11:22 PM, said:

ButchDean, on 31 May 2011 - 02:49 PM, said:

That is because you haven't initialized your arrays, hence the garbage. Also you are passing your arrays around by value meaning that once you return from the function all data is lost.

That's not true at all. If you pass an array to a function, you don't really pass it by value, since it will be implicitly converted to pointer. If you want to pass it by value - you'll have to pack it into a struct - then pass the struct by value. Consider the code below:

//c++0x
#include <iostream>

void modify(int array[], size_t len) {
// will behave the same as: 
//void modify(int* array, size_t len) {
    for (size_t i = 0; i < len; ++i) {
        array[i] = -1;
    }
}

struct Foo {
    int array[5];
};

void try_to_modify(Foo foo) {
    for (int& i : foo.array) {
        i = -1;
    }
}

int main() {
    using std::cout;
    
    int array[5] = { 1, 2, 3, 4, 5 };
    modify(array, 5);
    for (int i : array) {
        cout << i << ' ';
    }
    cout << '\n';
    
    
    Foo foo = { { 1, 2, 3, 4, 5 } };
    try_to_modify(foo);
    for (int i : foo.array) {
        cout << i << ' ';
    }
    cout << '\n';
    
    return 0;
}

I stand corrected. I knew that but my brain shorted out.