C++ Rectangle

[Lilcrew]

It doesnt show up as a rectangle and I need to use for loop and a nested for loop for this.
So far the code I've got is:

#include<iostream>
using namespace std;

int main()
{
	int row, i, width, spaces;

	cout <<"Enter the number of rows: ";
	cin >> row;
	cout <<"Enter the rectangle width: ";
	cin >> width;

	i=0;
	for(i=0; i<row; i++)
		cout << "O";
	cout << endl;	
	for(i=0; i<width; i++)
		cout << "O"<< endl;

	system("pause");
	return 0;
}

[hamsterman]

And what would you want it to be?

[Lilcrew]

It doesnt show up as a rectangle

[navarromoral]

That's because you have to nest the for statements as follows:

for(i=0; i<row; i++)
{
    for(j=0; j<width; j++)
        cout << "0";
    cout << endl;
}

[Lilcrew]

still dont get the rectangle

[navarromoral]

I don't know what you are doing... the following code should defenitely work (tell me if it doesn't):

#include<iostream>
using namespace std;

int main()
{
	int row, width, spaces;

	cout <<"Enter the number of rows: ";
	cin >> row;
	cout <<"Enter the rectangle width: ";
	cin >> width;

        for(int i=0; i<row; i++)
        {
            for(int j=0; j<width; j++)
                cout << "0";
            cout << endl;
        }

	system("pause");
	return 0;
}

[mcrist]

Ok. I'll dissect the code for you. When I code, I pseudo in English first and then transform that into C++.

print a row of "0"'s
each row contains a column.
we iterate through each column.
once we are done with all the columns in this row
we proceed to the next row.

I think you may have kept the "i" variable in your nested "for" statement - this is why your square didn't work properly and some of your formatting is off.

#include<iostream>
using namespace std;

int main()
{
	int row, i, j, width, spaces;

	cout <<"Enter the number of rows: ";
	cin >> row;
	cout <<"Enter the rectangle width: ";
	cin >> width;

        // Print a row of O's
	for(i=0; i<row; i++)
        {
                // Each row has columns iterate them
	        for(j=0; j<width; j++)
                {
                      // Print an "O"
		      cout << "O";
                }
                
                // Done with this row, proceed to next row
                cout << endl;
         }

	system("pause");
	return 0;
}

This code works just fine.

[Moschops]

Post deleter:
http://www.cplusplus.com/forum/general/45061/

And why not to:
http://www.cplusplus.com/forum/articles/40179/