Good work! Nice to see that you are taking the time to actually put thought into the design and really learn OOP!

Also, I am not sure but on this line

cout << "\n\nWeapon: " << PlayerOne.PlayerSetWeapon();

Shouldnt you call PlayerOne.PlayerGetWeapon()? Just thought I would point that out just in case. (also since its in the player class, it doesnt really need to be called PlayerGetWeapon etc, it could just be getWeapon())...

Oh, and for your message system where you say things like weapon sold, not enough gold, etc...

You could just have one function where you pass a string and print it out, that way in a situation where the player does not have enough gold you can just call
PrintMessage("Not enough gold");, rather then having a whole bunch of small functions.

Just some suggestions for things that I saw, I am no way a pro or anything so yea..haha, keep up the good work though i enjoy keeping up with this journal.

Thanks for giving me feedback on my code. :)

Not sure if I'm answering this right, but here I go. The reason why I have small functions to display messages would be to prevent any typos so I don't have to look at each part where I type cout << "Message" to see what I did wrong. The other reason would be, if I need to change that message in the future from "Not Enough Gold" to "You're Low On Gold" all I have to do is change that message in the function to change the display of my message anywhere it's called. It just makes it easier for me to keep track of typos and if I want to change my message to something else at a later time.

The part where I call PlayerOne.SetWeapon() was because it stores the current weapon equipped and also keeps track of change in weapons for showing the name. I'm going to be adding in a better Weapon class (Which isn't in use yet) to replace all those "weapon name" you see in the shop for buy and sell with functions that keep track of names, info and how much to give in damage bonus, ect... This will also prevent typos and misinformation from spreading around. It's not going to be a big issue now, but when I get some graphic games done you wouldn't want to have this really powerful weapon that didn't give any +Damage all because of a typo like Posion Axe instead of Poison Axe.

Thanks again for the feedback! :) Now I'm off to finish off the Battle System, fix up the new Weapon System, add new items and pretty much finish this game off. Once I'm done this, other then studying some general programming concepts I didn't go over yet I'm going to start looking at HGE to get started then SDL.

Ahh I see! One thing though, you could have constant strings put in place..for example

string LOW_GOLD = "You are low on gold"

that way you can just pass those and if you need to change them, then its right there for you to change.

And also I see that you released it, great work!!!

Ahhh I forgot about doing that. Every time I read up on C++ it's always been said to use const for anything that never changes even if you plan on keeping it the same anyhow. I guess it's just to make it more clear and make it impossible to modify it by mistake. I better keep that in mind for next time.