|I should change|
Well, that's up to you. Is a team class a single letter, or is it a string? Is a division class a single letter, or is it a string? Is a division level a number, or is it a string? You know what these things are, and you haven't told us.
|and they should become arrays|
If you want to store more than one of them, then, yes, you''ll need something that stores more than one value, and an array looks suitable.
Although unless there's a very
good reason for using an array, you're much better off using a std::vector. Vectors do what arrays do, but they're safer, and easier to use. You'll be glad you made the effort to learn about them.
Once you've got that working, you might want to think about better ways to organize your data. If you have an array that stores one name per team, and another that stores a class per team, and another that stores a division level per team, and another, and another, it would be much more straightforward to have a structure that stores the information for one team, and then have a single array (or, better, a single vector) of structures.
|char i should be changed to int|
Well, that's what I would do. But then you may have had some particular reason for making it a char rather than an int in the first place.