Quote Originally Posted by jcnewton View Post
I think i fixed it a little bit?

Code:
	cin>>numPlayers[20];
No. You're not understanding arrays. Declaring an array such as
int numPlayers[20];
means you have 20 integers. You generally access them one at a time through an index. Valid indexes are 0 through 19. numPlayers[20] is invalid memory. I'm not even sure why you're making numPlayers an array as it appears you only need that number once.

I know you're all gung ho on functions, but you're not there yet. Go reread the chapter on arrays. The program as written would most likely crash.