Lesson 18.1 – Future enhancements for the game

Do all locations have a value for their ImageName property?
What values are you putting into the ImageName property? Do they include the namespace, like “/Engine;component/Images/Locations/{imageName}” (including the image file’s extension, like “png”)?
Are the images included in the project, and are their “Build Action”s set to “Resource”?

Hi Jennifer,

In the constructor of the Item class, there is an “int” in front of the “Value = value;” line. The code thinks you want a new “Value” variable, and it assigns the parameter to that new variable, instead of the Value property. So, all the Item objects currently have a value of “0”. If you remove the “int”, the rest of your code should work.

Hello Qin,

Yes, you would want to use a List. I’ll give you some suggestions for the RewardItems, and let you try making the change for multiple quests on your own. If you have trouble making the changes, let me know. But, I want you to try first. Figuring out how to solve (or fix) problems is an important part of programming.

1. In the Quest class, delete the RewardItem property and create a new RewardItems property with a datatype of “List”
2. In the Quest constructor, initialize the RewardItems property to “new List()”
3. In the World class, you’ll need to change the lines that set the RewardItem property to a value. Instead, have them use RewardItems.Add(whateverItemYouWant)
4. In the SuperAdventure MoveTo() function, when add the reward item to the player’s inventory, you’ll need to loop through the RewardItems property (probably using a “foreach”) and add each individual item to the player’s inventory.

The multiple quests would be similar, except there are several places in the MoveTo function that expect QuestAvailableHere to be one object (or null). A quick solution is to surround any place that is looking at QuestAvailableHere with a “foreach” loop. Then, instead of using newLocation.QuestAvailableHere, you’ll use the current Quest object from the loop. If you need an example of the looping, check line 163-166 of SuperAdventure.cs, where items are added to the monster’s loot table.

Let me know if you try that and have any questions.

Hi Hasan,

I sent a reply to your email, but I’ll post the answer here too, in case someone else wants to do the same thing.

To make the change you want, I would add a new property to the Location class – “public Item GiftItem { get; set; }”. You would set the value for this property when you create the Location object (in World.PopulateLocations). Then, inside the MoveTo() function, add some new code. If you only want to give the item to the player once, you could put it inside the “if(!LocationsVisited.Contains(CurrentLocation.ID))” – if you’ve done all the lessons up to 26.2. The code might look like this (I haven’t tested it, so there might be some mistakes):

If(!LocationsVisited.Contains(CurrentLocation.ID))
{
LocationsVisited.Add(CurrentLocation.ID);
If(CurrentLocation.GiftItem != null)
{
AddItemToInventory(location.GiftItem);
RaiseMessage(“You receive a ” + location.GiftItem.Name);
}
}

Or, you might want to give the item to the player if they don’t have it inside their inventory. Then you would make a separate if statement, like:

If(CurrentLocation.GiftItem != null)
{
If(!Inventory.Any(ii => ii.Details.ID == location.GiftItem.ID))
{
AddItemToInventory(location.GiftItem);
RaiseMessage(“You receive a ” + location.GiftItem.Name);
}
}

Let me know if you try it and have any questions.

Hello,

You’re welcome! You are correct. When you write more programs, you will start to remember things and understand them better.

Here is a suggestion for some changes: https://gist.github.com/ScottLilly/646fc4a9b39746b5031d018ecffd3367. It is not tested, and there might be some problems or typos. But, I think you might see what it is doing.

There is a new function SetCurrentPlayer(), that accepts a Player object, stores it in the “_player” variable, gives it the default items (rusty sword and potion), moves the player to their home, and updates the UI controls. Call this function from the SuperAdventure constructor, with the beginning Player object. You can also call this function from the ResetByDeath function, passing in the “newPlayer” object.

I think you could also change ResetByDeath() to only delete items from the “_player” variable’s inventory that are not needed to enter a location. You would not need to create the new “newPlayer” object.

Let me know if you try it and have any questions. If you leave another comment, they do not appear automatically. I have this website setup so I need to approve all comments before they are displayed, because I have received over 15,000 comments that needed to be blocked or deleted.

You’re welcome, Evan! It’s great to hear the guides have been helpful for you.