C++ array issues

If you're in a C class, don't use new/delete, use malloc/free.It is also possible to pass the size of the array as an additional parameter to the function. For example:bDayParadox(people, numPeople);And in the function definition:int bDayParadox(int people[], int numPeople) { // code here}
  • #1
27
0

Homework Statement


Write a program that approximates the probability that at least two people in a room of 50 people have the same birthday. Run 5,000 trials, count up the number at least two people have the same birthday, and divide by the number of trials to get a probability.


Homework Equations





The Attempt at a Solution



Code:
#include <iostream>
#include <ctime>
#include <cstdlib>

using namespace std;

int bDayParadox (int people[]);

///////////////////////////////////////////////////

int main ()

{
  
  int probability;
  int sum = 0;
  const int trials = 5000;
  int people[49];
 
 for (int j = 0; j < trials; j++)
   {
     sum = sum + bDayParadox(people); 
   }
 
 probability = (sum / trials);

 cout <<  bDayParadox(people) << endl;
 cout << sum << endl;
 cout << trials << endl;
 cout << probability << endl;
   
  


}

/////////////////////////////////////////////////

int bDayParadox (int people[])
{

  int coincidence = 0;
  srand(time(NULL)); //generates seed for random numbers

  for (int a = 0; a < 50; a++)
      {
       people[a] = rand() % 365 + 1;
      }

  for (int b = 0; b < 50; b++)
      {
        for (int c = b+1; c < 50; c++)
	  { 
	   if (people[c] == people[b])
	     {     
	       coincidence = 1;
	     }


	   }
      }
  return coincidence;
 
}

The function I made, bDayParadox, puts random numbers (from 1 - 365) into an array of 50. The function then checks to see if at least two of the values are similar, and if so, counts it as 1 (coincidence variable). Up top, I'm running the function 5000 times and adding the values together to get sum. Sum divided by trials should give me the probability.

My output looks like this:

1 (bDayParadox run once)
5000 (sum)
5000 (trials, the constant)
353 (probability)

The value that is spit out for probability is unique each time.

My main problem with this is that probability should not be what it says. Also, my sum shouldn't always be 5000.
 
Physics news on Phys.org
  • #2
1. Correct your array size.
2. Probability is not an integer.
3. Initialize the random number generator once. You are getting the same sequence over and over again.
 
  • #3
Thank you very much. That cleared up my issues perfectly.
 
  • #4
I've come apon another small issue.

I want to make arrays of different sizes, from 2 - 50. In int main(), there is something like this:

Code:
for (numPeople = 2; numPeople <= 50, numPeople++)
              bDayParadox(people)

In the function definition of bDayParadox, I have:

Code:
for (int a = 0; a < numPeople; a++)
      {
       people[a] = rand() % 365 + 1;
      }

How do I go about getting the numPeople number from the int main() for loop into my function definition? Apparently I'm doing something wrong.
 
  • #5
Look up pointers, dynamical allocation and operators new/delete.
 

Suggested for: C++ array issues

Replies
3
Views
589
Replies
7
Views
796
Replies
3
Views
762
Replies
3
Views
806
Replies
4
Views
756
Replies
21
Views
2K
Replies
6
Views
868
Replies
2
Views
681
Replies
8
Views
658
Back
Top