Program has stopped working

[gr8schism]

I need to create a program that generates 20 random values and stores them in an array. Then adds those values in a separate function. My program stops working when it gets to the function that is supposed to add all of the values together and return a value. Having an issue finding where the error is in the function.

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;



void randNums(int randomArray[], int i);
float gettotal(int (&randomArray)[20], int i );


int main(){
	srand(time(0));
	int i = 0;
	const int size = 20; 
	int randomArray[size];
	
	
	randNums( randomArray, i);
	
	 
	// just to test i am getting random numbers
	cout << randomArray[1] << endl;
	cout << randomArray[2] << endl;
	cout << randomArray[20] << endl;
	
	
	float total = gettotal( (&randomArray)[20], i );
	//this is where my program stops
	// test to see the same random numbers are being added
	cout << randomArray[1] + randomArray[2] << endl;






return 0;
}
void randNums(int randomArray[], int i ){
			
  
		for(i= 0; i < 20; i++){
		
			randomArray[i] = (rand() % 100);
                  
              }
       }
       
float gettotal(int (&randomArray)[20], int i ){
	
	float total = 0;
	
	for(i= 0; i < 20; i++){
		
		total = total + randomArray[i];
		
		
		}
	return total;
}

[Moschops]

In what way does it not work?

Your function randNums writes to randomArray[20], which is really bad because the last element is randomArray[19].

[gr8schism]

It gives me a program.exe has stopped working error message. A problem caused the program to stop working correctly. And only outputs up until the second function. I fixed the randNums function I believe and am still getting the error.

[DeathLeap]

Your program works fine with me.

The output:

87
53
-858993460
140
Press any key to continue . . .

EDIT:

I fixed your code.

First, please don't pass 'i' to the functions, just do the for loop this way:

for(int i = 0; i < ......; i++)

There is really no point in passing i to a function.

And, you did a horrible mistake in line 28:

float total = gettotal( (&randomArray)[20], i );

Here, you're passing the 20th element of the array to the function getTotal.

You want to pass the WHOLE array to the function, so just pass its name.

It will be like this: float total = gettotal( (randomArray), i );

Here is the working code:

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;

void randNums(int randomArray[20])
{
	for (int i = 0; i < 19; i++)
	{

		randomArray[i] = (rand() % 100);
	}
}

float gettotal(int randomArray[20])
{

	float total = 0;

	for (int i = 0; i < 19; i++) 
	{

		total = total + randomArray[i];
	}
	return total;
}


int main()
{
	srand(time(0));
	const int size = 20;
	int randomArray[size];

	randNums(randomArray);

	cout << "Random numbers are: ";
	for (int i = 0; i < 19; i++)
	{
		cout << randomArray[i] << ", ";
	}

	float total = gettotal((randomArray));

	cout << "Your total is: " << total << endl; 

	system("PAUSE");
	return 0;
}