Question

Can anyone help me with this one? I really appreciate it!

Find and fix errors and Simplify the createGroups() function using only one loop instead of two

#include

using namespace std;

const double NO_STU = 40;

void createGroups(const string myClass[], int no_stu, string group1, string group2);
void printList(const string [], int);

int main()
{
int no_stu = 8;
string myClass[NO_STU] = { "Linda", "Bob", "Mary", "Jo", "Tim", "Jamie", "Ann", "Tom" };
string group1[NO_STU % 2];
string group2[NO_STU % 2];

createGroups(no_stu, myClass, group1, group2);
printList(myClass[], no_stu); // print the original list
printList(group1[], no_stu ); // print the first group
printList(group2[], no_stu ); // print the second group
  
return 0;
}

/*~*~*~*~*~*~
This function takes an array of strings and
its number of elements and displays its contents
on the same line
*~*/
void printList(const string list[], int size)
{
for( int i = 0; i <= size; i++ )
{
cout << list[i] << " ";
}
cout << endl;
}

/*~*~*~*~*~*~
This function takes an array of strings and
its number of elements, and populates two arrays
group1 - contains the first half
group2 - contains the second half
*~*/
void createGroups(const string myClass[], int no_stu, string group1[], string group2[])
{
int half = no_stu / 2;
  
for ( int i = 0; i < half; i++)
{
group1[i] = myClass[i];
}
  
for ( int i = half, k = 0; i < no_stu; i++, k++)
{
group2[k] = myClass[i];
}
}

Answer 1

#include <iostream>
#include <string>

using namespace std;

const int NO_STU = 40;

void createGroups(const string myClass[], int no_stu, string group1[], string group2[]);

void printList(const string [], int);

int main() {
    int no_stu = 8;
    string myClass[NO_STU] = {"Linda", "Bob", "Mary", "Jo", "Tim", "Jamie", "Ann", "Tom"};
    string group1[NO_STU / 2];
    string group2[NO_STU / 2];

    createGroups(myClass, no_stu, group1, group2);
    printList(myClass, no_stu); // print the original list
    printList(group1, no_stu); // print the first group
    printList(group2, no_stu); // print the second group

    return 0;
}

/*~*~*~*~*~*~
This function takes an array of strings and
its number of elements and displays its contents
on the same line
*~*/
void printList(const string list[], int size) {
    for (int i = 0; i <= size; i++) {
        cout << list[i] << " ";
    }
    cout << endl;
}

/*~*~*~*~*~*~
This function takes an array of strings and
its number of elements, and populates two arrays
group1 - contains the first half
group2 - contains the second half
*~*/
void createGroups(const string myClass[], int no_stu, string group1[], string group2[]) {
    int half = no_stu / 2, k = 0;

    for (int i = 0; i < no_stu; i++) {
        if (i < half)
            group1[i] = myClass[i];
        else
            group2[k++] = myClass[i];
    }
}