Issue with the print of this program

For once I actually have the full program working. The only issue I am having is this. The last number of the row that is printed needs to not have a , after it. The program works if a number that is divisible by 10 is enter but if it is not there is a , after it. Can anyone help lead me down the correct way to format the function to print the way I need it.

Here are the exact instructions for how it should work : print the values in the array. The values must be displayed with 10 integers per line and a comma after each value. The last value on each line does not have a comma after it. The last line may have fewer than10 values if the total numbers of values printed is not evenly divisible by 10. For example, if the user asked for 15 array elements to be printed, then the last line would only have 5 values; or if the user asked for 92 values, then the output would be 9 lines containing 10 values and the last output line would have only 2 values.

Here is what I have currently for the function:

Code:

---

void printArray (int array[], int n)
{
  for (int y = 0; y < n; y++)
 {
    cout<< array[y];


    if( (y % 10) == 9)
    {
        cout << endl;
    }


    else
        cout << ", ";
 }
    cout << endl;


  }

---

Terminate the loop one iteration earlier, and output the last value separately.

That will also address the concern (which you haven't mentioned) of an extraneous blank line if the number of values is a multiple of 10.

well that does not fix the issue I am having. So here is the example of what is happening. The user inputs 5 for how big they want the array to be, the numbers are generated randomly and stored and then output. The output at 5 currently would look like this: 55, 24, 88, 56, 12,

I need the output to look like this: 55, 24, 88, 56, 12

currently it only works with 10 I am not sure how to make it if they number is less than 10 no matter what row it is the last number does not have the comma after it.

Quote:

---

Originally Posted by Jesse Bettcher

well that does not fix the issue I am having.

---

That means you did something other than follow the advice.

So I fixed it going a different route I do know if this is exactly if you were meaning but it is what I came up with

Code:

---

void printArray (int array[], int n)
{
  for (int y = 0; y < n; y++)
  {
    if( y + 1 == 10  || y + 1 == n || ((y + 1 ) % 10 ) == 0)
    {
        cout<<array[y]<<endl;
    }
    else
    {
        cout<<array[y]<< " , ";
    }


  }




}

---

While your solution is one way to address the problem, it is certainly not what I suggested. No matter.

so what would the code have looked like doing it the way you suggested. I was struggling to come up with what you suggested so I went with something that addressed all the possible conditions and would output correctly.

There were two parts of my one-sentence reply "Terminate the loop one iteration earlier, and output the last value separately."

What do you think the part before the comma means?

What do you think the part after the comma means?


Think though those questions - they are hints that, if you think through them, are designed to point you in the direction of answering your own question.