Easy noob question (probobly) :)

Is there anything wrong with this code, I keep getting error messages when I try to compile it:

Code:

#include <iostream>	

using namespace std;

int main;

{
    int n
    
    cout<<"5+1= ";
    cin>> n;
    cin.ignore();
    if ( n = 6 ) {
         cout<<"Good Job! That was fun right?"
         }
    
    cin.get();
}

Update: i get an error on line 7, It says

invalad function direction

Hmmmmmm

There are several problems in your code, but they are all easy to fix.

First,
"int n" should be follow by a ";"

second,
= means to assign something, while == is to compare.
Try changing if(n = 6) to if(n==6)

third,
int main() should return something, Try adding return 0 at the end of main. Also, main is a function, and should not have ; at the end

Code:

#include <iostream>	

using namespace std;

int main
{
    int n;
    
    cout<<"5+1= ";
    cin>> n;
    cin.ignore();
    if ( n == 6 ) {
         cout<<"Good Job! That was fun right?";
         }
    
    cin.get();
return 0;
}

>> also, int main should return something. Try adding return 0 at the end of main
That's not actually necessary, there's a special rule with main that says it's ok not to. The code is fine (except for the == thing).

Three problems, highlighted in bold and red:

Code:

#include <iostream>	

using namespace std;

int main;

{
    int n;
    
    cout<<"5+1= ";
    cin>> n;
    cin.ignore();
    if ( n == 6 ) {
         cout<<"Good Job! That was fun right?";
         }
    
    cin.get();
}

Originally Posted by h3ro

There are several problems in your code, but they are all easy to fix.

First,
"int n" should be follow by a ";"

second,
= means to assign something, while == is to compare.
Try changing if(n = 6) to if(n==6)

third,
int main() should return something, Try adding return 0 at the end of main. Also, main is a function, and should not have ; at the end

Code:

#include <iostream>	

using namespace std;

int main()
{
    int n;
    
    cout<<"5+1= ";
    cin>> n;
    cin.ignore();
    if ( n == 6 ) {
         cout<<"Good Job! That was fun right?";
         }
    
    cin.get();
return 0;
}

That doesn't compile either because you forgot the part in red.

I personally feel that the standard should ensure main returns a value.

You explicity say int main(), so in essence, you are telling the function that it will return an integer value when the function terminates. If main is ok to not return a value, then why do some compilers throw an error when you use void main()? seems like it is a bit like double standards.

main() must return a value - it's just that the compiler is required to implicitly add a return 0 if you don't.

I personally feel main should be void, just like how it is in Java, for the sake of portability. But, of course, there is C compatibility to worry about, too.

C compatibility would be easily handled by allowing main to be int, but making the canonical form void.

A more interesting question is the effect that calling exit() would have on local objects with destructors.

Originally Posted by cyberfish

I personally feel main should be void, just like how it is in Java, for the sake of portability. But, of course, there is C compatibility to worry about, too.

I feel that would be stupid seeing as how it would break 99% of the programs out there written for platforms with CLI in mind or otherwise, especially since they can return a value telling of success or failure or whatever.
I feel Java is wrong. A return code can tell a thousand words.