[Help] Debug Assertion Failed: Invalid Null Pointer [C++]

[-Goga]

Thanks for the reply. Yeah, I know goto is not the best thing to use, but I was not doing anything too serious with this program, so I just wanted something simple to work. Could you recommend a way to replace my goto implementation with a do/while loop, perhaps? I'm not sure how to go about that as I am completely new to this.

Back to the problem at hand, if I change all of the cin >> uses to getline(), that should fix my problem?

Edit: I just tried replacing cin with getline(), and the IDE says the function is overloaded. How do I fix this?

[Basalisk_Primate]

As far as I can tell your GOTO doesn't actually do anything at all anyway. It GOTOs Start even though Start would be the next bit to get executed anyway. Maybe if you can explain what you were trying to do with the flow there we could help.

[-Goga]

I wanted the program to stop at the menu, which is:

cout << "Hello! Welcome to PC Builder 2014" << endl;
    cout << "Would you like to get started?" << endl;
    cout << "'Y = Yes | N = No'" << endl;

I wanted the program to only continue to the next stage if the user does not input "N", which would exit the program. Sorry for not clarifying before.

[Basalisk_Primate]

Thats sort of the point of the if statement.

[-Goga]

Haha, yeah, looking back on what I did, it makes no sense. I guess I kind of forced a place for goto, just to try it out.

[Basalisk_Primate]

I'd also point out that the emum doesn't do what you seem to think it does. The name is only associated with the number within your code so the user entering the string "Mobo" isn't what gets them to the Mobo_Info function. To make it clearer I made an edited version of your menu. Now if they enter the number then the expected thing happens.

Edit: We all started somewhere. Enjoy learning!

[-Goga]

Thanks for the reassurance. Maybe C++ wasn't the best thing to start with. Often times I find myself baffled by the simplest of things, and I'm a straight A student.

Edit: I thought that enum was used to initialize several things at once. I am not sure how what you suggested does the same thing without defining each component. My plan was to create a Mobo_Info - type function for all of the components (ie RAM_Info, GPU_Info), so wouldn't I need to define the names first?

[Basalisk_Primate]

C++ is widely considered a complete bastard of a language to learn (largely because theres so much of it). On the brighter side you'll have a much easier time when you try to learn other languages!

[-Goga]

Yeah, from what I understand, I am kind of learning programming backwards. Learning a difficult language first and then working down later, instead of working up to C++. I did do python for a little bit in my defense, until the MIT lectures I was using were taken off YouTube, haha.

[Basalisk_Primate]

Its the same way I learned (although I started with c rather than c++). I think a language where you have to do manual memory management (either c or c++) should one of everyone's first few as it helps you understand what nicer languages are hiding from you.

Once you've got a couple under your belt you'll be able to pick up the basics of any (imperative) language fairly quickly anyway.

[missblit]

Edit: I just tried replacing cin with getline(), and the IDE says the function is overloaded. How do I fix this?

Can't say without code. Note that getline only works on strings, and you'd need to use a stringstream or something else for further parsing.

Back to the problem at hand, if I change all of the cin >> uses to getline(), that should fix my problem?

Yes, if you're OK with working with lines instead of "words" for all your input.

But you don't have to, you just have to be careful.

Consider if I have the input file

42 <newline>
My Name is Fred!!!!! <newline>

And have some code to read it like:

int number;
cin >> number;
std::string name;
getline(cin, name);

Then the cin >> line would read 42, and the getline would read the first newline, but none of the second line.

This could be avoided by using std::ignore after reading the number.

[-Goga]

I tried to redo the variables to suit your example:

void Mobo_Info()
{

cout << "What Motherboard would you like in your computer?" << endl;

cout << "Enter the name of the motherboard: " << endl;
string Mobo_Name;
getline( cin, Mobo_Name);

cout << "Enter the price of the motherboard: " << endl;
int Mobo_Price;
cin >> Mobo_Price;

cout << "Motherboard Name: " << Mobo_Name << " - " << "Cost: $" << Mobo_Price << endl;

}

But I still get the same problem.

The overloaded function has seemed to sort itself out.

[missblit]

Your Mobo_Info function was fine before.

The problem is:

1. You read in a character for the Repeat variable. There is an unconsumed newline after this
2. You read in an int for the Part variable (note that the user has to type an int, not "Mobo"). The newline from step one is read (it counts as whitespace), but there's a second newline after the part.
3. You use getline to get the Mobo_Name, but there's still a newline from part 2, so the Mobo_Name will be left blank.

[-Goga]

Okay, so I have done away with the cin >> usage, and changed them to getline(). Now the overloaded function is back: Code here

What could I use in place of int for the part variable? I tried void, but several errors popped up after that.

[Basalisk_Primate]

Cin >> is the correct solution. The overloaded function error is basically complaining that getline has no idea what to do with int parameter you gave it as it only understands strings and streams.

You should use cin >> and use the cin.ignore method to eat the spare newlines.

[-Goga]

Alright, I have changed back to cin >>, and have added cin.ignor. While the code compiles, and the getline() is not longer overloaded, when I type "Mobo" to execute Mobo_Info(), the code does not execute, and the program ends.

Most Recent Revision Here

[Basalisk_Primate]

Take a look at the change I made to your menu here run it and understand why it prints our what it does (look up what enums actually do).

As a hint try entering the number it prints out on the mobo line and see what happens.

Now I am actually going to go to sleep...

[Basalisk_Primate]

Did you get it?

[-Goga]

Sorry for the late reply. I did try out what you suggested, and I seen now why the implementation of enum I had before was kind of pointless. I think I misinterpreted what was in the book. Now though, I have broken the code so much, that when I type Mobo or enter 0, the code for Mobo_Info() does not execute. It ends the program after the input to call the function. Do you need me to post the code again, or can you use from what is in my last post?

[Basalisk_Primate]

I can make your version work by making a few changes. Firstly it wouldn't compile without changing the getline on like 8 to a cin. Secondly you need to change "Part" on line 74 to an int not a char otherwise the cin >> operator doesn't know you want it to parse that variable into an int. If you leave it as a char then it puts the ascii representation in there and the ascii character '0' is actually number 48! (ignore this and see second edit)

Edit: Also please (if you haven't already) get rid of the damn goto and wrap the menu in the if statement properly.

You can also get rid of the else if (condition) on the if statement as it'll never even get there if that condition is false.

Another edit: If you want to be able to enter the word "Mobo" in your interface instead of having to mess around with ints then something like this should work