Need feedback on code

Hi there, just made this little application(but borrowed some code from another classmate) which prints the following:

Code:

5*5*5*5*5
4*4*4*4
3*3*3
2*2
1

Here is the code:

Code:

#include<iostream>

using namespace std;

int main()
{
	int iNum1 = 6;
	int iNum2 = 5;
	char cStar = '*';
	
	do
	{
        for (int x=1;x<iNum2;x++)
        {
        	cout << iNum2 << cStar;
        }
        cout << iNum2;
        iNum2--; iNum1--;
        cout << endl;
	} while(iNum1>1);
	
    cin.get();
    return 0;
}

Im just wondering how i could possibly optimize this code, and also would like your thoughts about the code, whatever it may be, some hints or pointers about anything of this code which i should take into consideration in fututre, thanks.

Seriously consider using better indentation. It really does help others view your code and get a real feel for what it does at a glance.

As far as optimizing it.... not sure you really could worry about optimiziation at this level. With that said, I don't think you need two different ints for this. Consider something like this:

Code:

#include <iostream>

int main()
{
	int i, j, num = 5;
	char star = '*';
	
	for(i=num;i>0;i--)
	{
		std::cout << i;
		for(j=0;j<i-1;j++)
		{
			std::cout << star << i;
		}
		std::cout << std::endl;
	}
	
	return 0;
}

Very similar, so there's not that much room to really improve it.

Yeah, there is a minimum amount of work that needs to be done to solve any problem, and it looks like you have done it. Any optimisation would just be a side-effect of refactoring, methinks.

I would write something like:

Code:

#include<iostream>

using namespace std;

int main()
{
    for (int i = 5; i > 0; --i)
    {
        for (int j = 1; j < i; ++j)
        {
            cout << i << '*';
        }
        cout << i << endl;
    }
}