New to C++ help plz

I want to get the out put like this .. if I input no 4 then for stars in a row and 4 colmns..

****
****
****
****
if i input 10 then 10 stars 10 colomns

I wrote the coding

Code:

 #include "stdafx.h"
#include <iostream>

#include<limits>
using namespace std;

void main()
{ int x;
cin >> x;
for(int i=1;i<=x;i++)
{
	for(int g=1;g<=x;g++)
	{
		cout<<"*"<<endl;
	}
}
system("pause");
}

butr this outputs like .. if I input 4 then the output comes as
*
*
*
*
*
*
*
*
*
*
*
*
*
*
*
*

how do I get this output??? whats wrong in my coding O.o

****
****
****
****

You should use a multi-dimensional array.

Think about what std::endl does and particularly, what it does where you have placed it.

Also, in C++ it is always int main, not void main.

without using multi dimentional array how do you do it??

thanks
I got it

Originally Posted by Shakti

Think about what std::endl does and particularly, what it does where you have placed it.

Also, in C++ it is always int main, not void main.

our teacher said its int mnain nah.. you sure.. ohh btw I placed the endl in the correct place
Thanks

C++ standard explicitly says main shall return int unless for very specialized cases. If you are unsure wether you are using c++ in such a case then it is safe to say you aren't.

Glad you got it

thanks oh and btw when you are using arrays in visual c++ 2008 what are the header files that we have to import?? and when we are using mod, factorial values n al what are the header files that you have to import in visual c++ 2008

for arrays you dont need to include anything. For std::vector though you need to include vector, for math-functions you need cmath. Mod however is a built in operator
a % b is a mod b.

Originally Posted by Shakti

for arrays you dont need to include anything. For std::vector though you need to include vector, for math-functions you need cmath. Mod however is a built in operator
a % b is a mod b.

but then why is my code not working?

Code:

#include "stdafx.h"
#include <iostream>

#include<limits>
using namespace std;
 void main()
{ 
	int array[20];
	int tot=0;
	for(int i=0;i<19;i++)
	{
		cin<<array[i];
		tot=tot+array[i];
	}
	float avg=tot/20;
	cout<<"tot"<<"="<<tot<<   <<"avg"<<"="<<avg;
	system("pause");

 }

Error 1 error C2784: 'std::basic_ostream<char,_Traits> &std:perator <<(std::basic_ostream<char,_Traits> &,unsigned char)' : could not deduce template argument for 'std::basic_ostream<char,_Traits> &' from 'std::istream'

Error 2 error C2784: 'std::basic_ostream<char,_Traits> &std:perator <<(std::basic_ostream<char,_Traits> &,unsigned char)' : could not deduce template argument for 'std::basic_ostream<char,_Traits> &' from 'std::istream'

Warning 42 warning C4244: 'initializing' : conversion from 'int' to 'float', possible loss of data c:\documents and settings\administrator\my documents\visual studio 2008\projects\oogh\oogh.cpp 16 oogh

You can't have << and then << again with nothing in between them. If you want to print some spaces, then you need to print a string with spaces in it, i.e., " ".

Originally Posted by godly 20

You should use a multi-dimensional array.

No, you should not use a multi-dimensional array. In fact, there is no need for an array or a container like std::vector at all. Sure, you could use them (though using std::string would actually make it easier), but they are unnecessary.

Your original idea is correct. The problem is that you printed a new line (through endl) in the wrong place.

Originally Posted by tabstop

You can't have << and then << again with nothing in between them. If you want to print some spaces, then you need to print a string with spaces in it, i.e., " ".

still the errors comes nah

Originally Posted by laserlight

No, you should not use a multi-dimensional array. In fact, there is no need for an array or a container like std::vector at all. Sure, you could use them (though using std::string would actually make it easier), but they are unnecessary.

Your original idea is correct. The problem is that you printed a new line (through endl) in the wrong place.

No i didnt print a new line.., this is my code

Code:

#include "stdafx.h"
#include <iostream>

#include<limits>
using namespace std;
 void main()
{ 
	int array[20];
	int tot=0;
	for(int i=0;i<19;i++)
	{
		cin<<array[i];
		tot=tot+array[i];
	}
	float avg=tot/20;
	cout<<"tot"<<"="<<tot<<"avg"<<"="<<avg;
	system("pause");

 }

Originally Posted by thili

No i didnt print a new line.., this is my code

I am talking about the code that you posted in post #1. However, I now notice that you already solved it as per post #5. You should have started a new thread concerning your new code and new problem.