• FEATURED

View more

View more

View more

### Image of the Day Submit

IOTD | Top Screenshots

### The latest, straight to your Inbox.

Subscribe to GameDev.net Direct to receive the latest updates and exclusive content.

# C++ help

Old topic!

Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.

12 replies to this topic

### #1jelani435  Members

Posted 28 February 2013 - 03:47 PM

Hey guys i'm 14 and i'm just starting to learn c++. I have an rpg intro i'm working on but it wont compile or run. would any of you be able to point out any issues?:

________________________________________________________________________________________________________________

#include <iostream>
#include <cstring>

using namespace std;

int main()
{
char name[50];
char weapon[100];
char weapon_name[100];
cout<<"What do you want your hero's name to be?";
cin.getline (name, 50);
cout<<"I like that name!";
cout<<"What would you like your weapon to be?";
cin.getline (weapon, 100);
cout<<"That's my favourite weapon!";
cout<<"What would you like to name your weapon?";
cin.getline (weapon_name, 100);
cout<<"cool!";
cout<<"You shall be crowned: "<<name<<"the wielder of the"<<weapon<<"named"<<weapon_name";
cin.get();
}

Posted 28 February 2013 - 03:57 PM

Compile/link errors, we need them? Also what compiler you are using might be handy ;)

Just a guess... you aren't building a console application or you have a unicode console build (replace main with wmain then).

"Most people think, great God will come from the sky, take away everything, and make everybody feel high" - Bob Marley

### #3jelani435  Members

Posted 28 February 2013 - 04:06 PM

I am using code::blocks compiler.

The error is:

Checking for existence: C:\Users\user\Desktop\Rpg intro.exe

Executing: C:\Program Files (x86)\CodeBlocks/cb_console_runner.exe "C:\Users\user\Desktop\Rpg intro.exe" (in C:\Users\user\Desktop)
Process terminated with status 1972040841 (0 minutes, 33 seconds)

And im still doing tutorials and all that so i decided to try my hand at my own sample program. I really have no idea what these errors mean lol!

Posted 28 February 2013 - 04:13 PM

Looks like it built, since it ran.

33 seconds execution time... did you wait 33 seconds before pressing enter several times? None of the text will probably be printed to the console since you don't flush the output (using cout << endl) perhaps? Put that after you write each line to cout, see if that works.

Looks like you are returning from main with a non-zero value... which sounds like a bug in code::blocks since if you don't return a value from main it should implicitly return 0.

"Most people think, great God will come from the sky, take away everything, and make everybody feel high" - Bob Marley

### #5jelani435  Members

Posted 28 February 2013 - 04:21 PM

Is there any way i can write \n instead of std:

Posted 28 February 2013 - 04:24 PM

Just try

cout<<"What do you want your hero's name to be?" << endl;

etc.

I think using "\n" doesn't flush the output stream. Using endl definitely does. You don't need std:: before endl since you already had

using namespace std;

Edited by Paradigm Shifter, 28 February 2013 - 04:24 PM.

"Most people think, great God will come from the sky, take away everything, and make everybody feel high" - Bob Marley

### #7jelani435  Members

Posted 28 February 2013 - 06:13 PM

Thanks

### #8jelani435  Members

Posted 28 February 2013 - 06:16 PM

Also for the last two couts should i use \n on that or samething?

### #9Cornstalks  Members

Posted 28 February 2013 - 06:51 PM

Also for the last two couts should i use \n on that or samething?

Meh, it's up to you. Whether you use \n or std::endl, both will print out a new line. The only difference between \n and std::endl is that \n will not flush the buffer, but std::endl will flush the buffer. For little programs like these, it make almost zero difference what you choose. For large scale applications that print out a lot of information, they may prefer using \n because constantly flushing the buffer can slow things down a good amount.

I'm going to guess you aren't quite sure what "flushing the buffer" means, though. When you perform input/output operations, like you're doing here, things are commonly buffered. That is, there's some memory that let's things "build up" before finally dumping them out, kind of like a temporary damn. Let me walk you through it:

std::cout << "Hello world!\n";

"Hello world!\n" is only a few characters (13 to be exact). It can be more efficient if, instead of instantly dumping those 13 characters out to the console, you put those 13 characters in a memory buffer, temporarily storing them until you get more data. Once this memory buffer is full (i.e. you try printing out more data and eventually the memory buffer is full), it dumps all of the stored characters out to the console in one go. If you use std::endl instead of \n, it will force the buffer to dump out its data, meaning it won't wait until it's full.

Like I said, for small applications, this isn't something you should really worry about. But if you've got a large application that outputs a lot of data, you might want to not use std::endl so you can let things build up in the buffer (dumping 1000 characters to the console in one go is more efficient than dumping 10 characters to the console 100 times).

Also, I know you're learning, but I hope you're learning that you should pretty much never be getting input into raw char arrays like that. It's okay for a learning exercise, but don't let it become something that you think is the "right way." There are things like std::string and std::getline, which should be your best friends (someday).

Oh yeah. This site has [ code ][ /code ] (use them without spaces) tags that you can put your code in to get nice, pretty formatting

Edited by Cornstalks, 28 February 2013 - 06:54 PM.

[ I was ninja'd 71 times before I stopped counting a long time ago ] [ f.k.a. MikeTacular ] [ My Blog ] [ SWFer: Gaplessly looped MP3s in your Flash games ]

### #10jellyfishchris  Members

Posted 28 February 2013 - 07:04 PM

This line...

cout<<"You shall be crowned: "<<name<<"the wielder of
the"<<weapon<<"named"<<weapon_name";


Lets zoom in to the problem

weapon_name";


Your complier seems to not be working I used the following:

http://ideone.com/L2j2hL

And it showed me straight away.

### #11Indifferent  Members

Posted 28 February 2013 - 07:09 PM

He got it to compile so I'd assume he fixed that.

### #12Khatharr  Members

Posted 28 February 2013 - 08:41 PM

Has C::B improved lately? I tried it some years ago and was really disappointed with it.
void hurrrrrrrr() {__asm sub [ebp+4],5;}

There are ten kinds of people in this world: those who understand binary and those who don't.

### #13DevLiquidKnight  Members

Posted 28 February 2013 - 10:25 PM

Do not use:

char name[50];

char weapon[100];
char weapon_name[100];

Use std::string.

string name;
string weapon;
string weapon_name;

Old topic!

Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.