The Location and Monster model classes currently call into MonsterFactory (to instantiate a Monster at a Location) and the ItemFactory (to instantiate items in the Monster’s inventory for loot).
The factory classes will be moved to SOSCSRPG.Services, which won’t be visible to Model classes, so we need to remove these dependencies.
We’ll do this by moving the code that gets the monster at a location and the code that populates the Monster’s inventory into the MonsterFactory class.
Step 1: Modify Engine\Models\Location.cs
Delete the GetMonster() function from this class (lines 55-86).
Remove the unnecessary “using” directives: Engine.Factories and SOSCSRPG.Core.
Location.cs
using System.Collections.Generic;
using System.Linq;
using Newtonsoft.Json;
namespace Engine.Models
{
public class Location
{
public int XCoordinate { get; }
public int YCoordinate { get; }
[JsonIgnore]
public string Name { get; }
[JsonIgnore]
public string Description { get; }
[JsonIgnore]
public string ImageName { get; }
[JsonIgnore]
public List<Quest> QuestsAvailableHere { get; } = new List<Quest>();
[JsonIgnore]
public List<MonsterEncounter> MonstersHere { get; } =
new List<MonsterEncounter>();
[JsonIgnore]
public Trader TraderHere { get; set; }
public Location(int xCoordinate, int yCoordinate, string name, string description, string imageName)
{
XCoordinate = xCoordinate;
YCoordinate = yCoordinate;
Name = name;
Description = description;
ImageName = imageName;
}
public void AddMonster(int monsterID, int chanceOfEncountering)
{
if(MonstersHere.Exists(m => m.MonsterID == monsterID))
{
// This monster has already been added to this location.
// So, overwrite the ChanceOfEncountering with the new number.
MonstersHere.First(m => m.MonsterID == monsterID)
.ChanceOfEncountering = chanceOfEncountering;
}
else
{
// This monster is not already at this location, so add it.
MonstersHere.Add(new MonsterEncounter(monsterID, chanceOfEncountering));
}
}
}
}
Step 2: Modify Engine\Models\Monster.cs
Replace the private “_lootTable” class-level variable with a public “LootTable” property, since we’ll need access to the loot table in MonsterFactory, when we read it to populate each Monster object with its inventory.
In the AddItemToLootTable function, change the use of “_lootTable” to “LootTable”.
Delete the GetNewInstance function (lines 37-57).
Add a public Clone function that instantiates a new Monster object with the same values in the properties. This will be used by the MonsterFactory.
Remove the unnecessary “using” directives: Engine.Factories and SOSCSRPG.Core.
Monster.cs
using System.Collections.Generic;
namespace Engine.Models
{
public class Monster : LivingEntity
{
public int ID { get; }
public string ImageName { get; }
public int RewardExperiencePoints { get; }
public List<ItemPercentage> LootTable { get; } =
new List<ItemPercentage>();
public Monster(int id, string name, string imageName,
int maximumHitPoints, IEnumerable<PlayerAttribute> attributes,
GameItem currentWeapon,
int rewardExperiencePoints, int gold) :
base(name, maximumHitPoints, maximumHitPoints, attributes, gold)
{
ID = id;
ImageName = imageName;
CurrentWeapon = currentWeapon;
RewardExperiencePoints = rewardExperiencePoints;
}
public void AddItemToLootTable(int id, int percentage)
{
// Remove the entry from the loot table,
// if it already contains an entry with this ID
LootTable.RemoveAll(ip => ip.ID == id);
LootTable.Add(new ItemPercentage(id, percentage));
}
public Monster Clone()
{
Monster newMonster =
new Monster(ID, Name, ImageName, MaximumHitPoints, Attributes,
CurrentWeapon, RewardExperiencePoints, Gold);
newMonster.LootTable.AddRange(LootTable);
return newMonster;
}
}
}
Step 3: Modify Engine\Factories\MonsterFactory.cs
Add a “using SOSCSRPG.Core;” directive.
Rename the static class-level variable “_baseMonsters” to “s_baseMonsters” – to follow the variable-naming rules.
Add a new GetMonsterFromLocation function (lines 41-72). This is basically the code that was previously in the Location.GetMonster function.
Change the GetMonster function (now on lines 115-129) to call the new Monster.Clone function and populate the Monster’s inventory from the LootTable. This is basically the code that was previously in Monster.GetNewInstance().
MonsterFactory.cs
using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Xml;
using Engine.Models;
using Engine.Services;
using Engine.Shared;
using SOSCSRPG.Core;
namespace Engine.Factories
{
public static class MonsterFactory
{
private const string GAME_DATA_FILENAME = ".\\GameData\\Monsters.xml";
private static readonly GameDetails s_gameDetails;
private static readonly List<Monster> s_baseMonsters = new List<Monster>();
static MonsterFactory()
{
if(File.Exists(GAME_DATA_FILENAME))
{
s_gameDetails = GameDetailsService.ReadGameDetails();
XmlDocument data = new XmlDocument();
data.LoadXml(File.ReadAllText(GAME_DATA_FILENAME));
string rootImagePath =
data.SelectSingleNode("/Monsters")
.AttributeAsString("RootImagePath");
LoadMonstersFromNodes(data.SelectNodes("/Monsters/Monster"), rootImagePath);
}
else
{
throw new FileNotFoundException($"Missing data file: {GAME_DATA_FILENAME}");
}
}
public static Monster GetMonsterFromLocation(Location location)
{
if (!location.MonstersHere.Any())
{
return null;
}
// Total the percentages of all monsters at this location.
int totalChances = location.MonstersHere.Sum(m => m.ChanceOfEncountering);
// Select a random number between 1 and the total (in case the total chances is not 100).
int randomNumber = DiceService.Instance.Roll(totalChances, 1).Value;
// Loop through the monster list,
// adding the monster's percentage chance of appearing to the runningTotal variable.
// When the random number is lower than the runningTotal,
// that is the monster to return.
int runningTotal = 0;
foreach (MonsterEncounter monsterEncounter in location.MonstersHere)
{
runningTotal += monsterEncounter.ChanceOfEncountering;
if (randomNumber <= runningTotal)
{
return GetMonster(monsterEncounter.MonsterID);
}
}
// If there was a problem, return the last monster in the list.
return GetMonster(location.MonstersHere.Last().MonsterID);
}
private static void LoadMonstersFromNodes(XmlNodeList nodes, string rootImagePath)
{
if(nodes == null)
{
return;
}
foreach(XmlNode node in nodes)
{
var attributes = s_gameDetails.PlayerAttributes;
attributes.First(a => a.Key.Equals("DEX")).BaseValue =
Convert.ToInt32(node.SelectSingleNode("./Dexterity").InnerText);
attributes.First(a => a.Key.Equals("DEX")).ModifiedValue =
Convert.ToInt32(node.SelectSingleNode("./Dexterity").InnerText);
Monster monster =
new Monster(node.AttributeAsInt("ID"),
node.AttributeAsString("Name"),
$".{rootImagePath}{node.AttributeAsString("ImageName")}",
node.AttributeAsInt("MaximumHitPoints"),
attributes,
ItemFactory.CreateGameItem(node.AttributeAsInt("WeaponID")),
node.AttributeAsInt("RewardXP"),
node.AttributeAsInt("Gold"));
XmlNodeList lootItemNodes = node.SelectNodes("./LootItems/LootItem");
if(lootItemNodes != null)
{
foreach(XmlNode lootItemNode in lootItemNodes)
{
monster.AddItemToLootTable(lootItemNode.AttributeAsInt("ID"),
lootItemNode.AttributeAsInt("Percentage"));
}
}
s_baseMonsters.Add(monster);
}
}
private static Monster GetMonster(int id)
{
Monster newMonster = s_baseMonsters.FirstOrDefault(m => m.ID == id).Clone();
foreach (ItemPercentage itemPercentage in newMonster.LootTable)
{
// Populate the new monster's inventory, using the loot table
if (DiceService.Instance.Roll(100).Value <= itemPercentage.Percentage)
{
newMonster.AddItemToInventory(ItemFactory.CreateGameItem(itemPercentage.ID));
}
}
return newMonster;
}
}
}
Step 4: Modify Engine\ViewModels\GameSession.cs
Update the places that were populating the CurrentMonster property with calls to the new MonsterFactory.GetMonsterAtLocation function.
There’s one in the CurrentLocation property setter (line 59) and another in the OnMonsterKilled function (line 299).
GameSession.cs
using System.ComponentModel;
using System.Linq;
using Engine.Factories;
using Engine.Models;
using Engine.Services;
using Newtonsoft.Json;
namespace Engine.ViewModels
{
public class GameSession : INotifyPropertyChanged
{
private readonly MessageBroker _messageBroker = MessageBroker.GetInstance();
#region Properties
private Player _currentPlayer;
private Location _currentLocation;
private Battle _currentBattle;
private Monster _currentMonster;
public event PropertyChangedEventHandler PropertyChanged;
[JsonIgnore]
public GameDetails GameDetails { get; private set; }
[JsonIgnore]
public World CurrentWorld { get; }
public Player CurrentPlayer
{
get => _currentPlayer;
set
{
if(_currentPlayer != null)
{
_currentPlayer.OnLeveledUp -= OnCurrentPlayerLeveledUp;
_currentPlayer.OnKilled -= OnPlayerKilled;
}
_currentPlayer = value;
if(_currentPlayer != null)
{
_currentPlayer.OnLeveledUp += OnCurrentPlayerLeveledUp;
_currentPlayer.OnKilled += OnPlayerKilled;
}
}
}
public Location CurrentLocation
{
get => _currentLocation;
set
{
_currentLocation = value;
CompleteQuestsAtLocation();
GivePlayerQuestsAtLocation();
CurrentMonster = MonsterFactory.GetMonsterFromLocation(CurrentLocation);
CurrentTrader = CurrentLocation.TraderHere;
}
}
[JsonIgnore]
public Monster CurrentMonster
{
get => _currentMonster;
set
{
if(_currentBattle != null)
{
_currentBattle.OnCombatVictory -= OnCurrentMonsterKilled;
_currentBattle.Dispose();
_currentBattle = null;
}
_currentMonster = value;
if(_currentMonster != null)
{
_currentBattle = new Battle(CurrentPlayer, CurrentMonster);
_currentBattle.OnCombatVictory += OnCurrentMonsterKilled;
}
}
}
[JsonIgnore]
public Trader CurrentTrader { get; private set; }
[JsonIgnore]
public bool HasLocationToNorth =>
CurrentWorld.LocationAt(CurrentLocation.XCoordinate, CurrentLocation.YCoordinate + 1) != null;
[JsonIgnore]
public bool HasLocationToEast =>
CurrentWorld.LocationAt(CurrentLocation.XCoordinate + 1, CurrentLocation.YCoordinate) != null;
[JsonIgnore]
public bool HasLocationToSouth =>
CurrentWorld.LocationAt(CurrentLocation.XCoordinate, CurrentLocation.YCoordinate - 1) != null;
[JsonIgnore]
public bool HasLocationToWest =>
CurrentWorld.LocationAt(CurrentLocation.XCoordinate - 1, CurrentLocation.YCoordinate) != null;
[JsonIgnore]
public bool HasMonster => CurrentMonster != null;
[JsonIgnore]
public bool HasTrader => CurrentTrader != null;
#endregion
public GameSession(Player player, int xCoordinate, int yCoordinate)
{
PopulateGameDetails();
CurrentWorld = WorldFactory.CreateWorld();
CurrentPlayer = player;
CurrentLocation = CurrentWorld.LocationAt(xCoordinate, yCoordinate);
}
public void MoveNorth()
{
if(HasLocationToNorth)
{
CurrentLocation = CurrentWorld.LocationAt(CurrentLocation.XCoordinate, CurrentLocation.YCoordinate + 1);
}
}
public void MoveEast()
{
if(HasLocationToEast)
{
CurrentLocation = CurrentWorld.LocationAt(CurrentLocation.XCoordinate + 1, CurrentLocation.YCoordinate);
}
}
public void MoveSouth()
{
if(HasLocationToSouth)
{
CurrentLocation = CurrentWorld.LocationAt(CurrentLocation.XCoordinate, CurrentLocation.YCoordinate - 1);
}
}
public void MoveWest()
{
if(HasLocationToWest)
{
CurrentLocation = CurrentWorld.LocationAt(CurrentLocation.XCoordinate - 1, CurrentLocation.YCoordinate);
}
}
private void PopulateGameDetails()
{
GameDetails = GameDetailsService.ReadGameDetails();
}
private void CompleteQuestsAtLocation()
{
foreach(Quest quest in CurrentLocation.QuestsAvailableHere)
{
QuestStatus questToComplete =
CurrentPlayer.Quests.FirstOrDefault(q => q.PlayerQuest.ID == quest.ID &&
!q.IsCompleted);
if(questToComplete != null)
{
if(CurrentPlayer.Inventory.HasAllTheseItems(quest.ItemsToComplete))
{
CurrentPlayer.RemoveItemsFromInventory(quest.ItemsToComplete);
_messageBroker.RaiseMessage("");
_messageBroker.RaiseMessage($"You completed the '{quest.Name}' quest");
// Give the player the quest rewards
_messageBroker.RaiseMessage($"You receive {quest.RewardExperiencePoints} experience points");
CurrentPlayer.AddExperience(quest.RewardExperiencePoints);
_messageBroker.RaiseMessage($"You receive {quest.RewardGold} gold");
CurrentPlayer.ReceiveGold(quest.RewardGold);
foreach(ItemQuantity itemQuantity in quest.RewardItems)
{
GameItem rewardItem = ItemFactory.CreateGameItem(itemQuantity.ItemID);
_messageBroker.RaiseMessage($"You receive a {rewardItem.Name}");
CurrentPlayer.AddItemToInventory(rewardItem);
}
// Mark the Quest as completed
questToComplete.IsCompleted = true;
}
}
}
}
private void GivePlayerQuestsAtLocation()
{
foreach(Quest quest in CurrentLocation.QuestsAvailableHere)
{
if(!CurrentPlayer.Quests.Any(q => q.PlayerQuest.ID == quest.ID))
{
CurrentPlayer.Quests.Add(new QuestStatus(quest));
_messageBroker.RaiseMessage("");
_messageBroker.RaiseMessage($"You receive the '{quest.Name}' quest");
_messageBroker.RaiseMessage(quest.Description);
_messageBroker.RaiseMessage("Return with:");
foreach(ItemQuantity itemQuantity in quest.ItemsToComplete)
{
_messageBroker
.RaiseMessage($" {itemQuantity.Quantity} {ItemFactory.CreateGameItem(itemQuantity.ItemID).Name}");
}
_messageBroker.RaiseMessage("And you will receive:");
_messageBroker.RaiseMessage($" {quest.RewardExperiencePoints} experience points");
_messageBroker.RaiseMessage($" {quest.RewardGold} gold");
foreach(ItemQuantity itemQuantity in quest.RewardItems)
{
_messageBroker
.RaiseMessage($" {itemQuantity.Quantity} {ItemFactory.CreateGameItem(itemQuantity.ItemID).Name}");
}
}
}
}
public void AttackCurrentMonster()
{
_currentBattle?.AttackOpponent();
}
public void UseCurrentConsumable()
{
if(CurrentPlayer.CurrentConsumable != null)
{
if (_currentBattle == null)
{
CurrentPlayer.OnActionPerformed += OnConsumableActionPerformed;
}
CurrentPlayer.UseCurrentConsumable();
if (_currentBattle == null)
{
CurrentPlayer.OnActionPerformed -= OnConsumableActionPerformed;
}
}
}
private void OnConsumableActionPerformed(object sender, string result)
{
_messageBroker.RaiseMessage(result);
}
public void CraftItemUsing(Recipe recipe)
{
if(CurrentPlayer.Inventory.HasAllTheseItems(recipe.Ingredients))
{
CurrentPlayer.RemoveItemsFromInventory(recipe.Ingredients);
foreach(ItemQuantity itemQuantity in recipe.OutputItems)
{
for(int i = 0; i < itemQuantity.Quantity; i++)
{
GameItem outputItem = ItemFactory.CreateGameItem(itemQuantity.ItemID);
CurrentPlayer.AddItemToInventory(outputItem);
_messageBroker.RaiseMessage($"You craft 1 {outputItem.Name}");
}
}
}
else
{
_messageBroker.RaiseMessage("You do not have the required ingredients:");
foreach(ItemQuantity itemQuantity in recipe.Ingredients)
{
_messageBroker
.RaiseMessage($" {itemQuantity.QuantityItemDescription}");
}
}
}
private void OnPlayerKilled(object sender, System.EventArgs e)
{
_messageBroker.RaiseMessage("");
_messageBroker.RaiseMessage("You have been killed.");
CurrentLocation = CurrentWorld.LocationAt(0, -1);
CurrentPlayer.CompletelyHeal();
}
private void OnCurrentMonsterKilled(object sender, System.EventArgs eventArgs)
{
// Get another monster to fight
CurrentMonster = MonsterFactory.GetMonsterFromLocation(CurrentLocation);
}
private void OnCurrentPlayerLeveledUp(object sender, System.EventArgs eventArgs)
{
_messageBroker.RaiseMessage($"You are now level {CurrentPlayer.Level}!");
}
}
}
Step 5: Test the game
NEXT LESSON: Lesson 19.10: Create a GameState model for SaveGameService
PREVIOUS LESSON: Lesson 19.8: Create SOSCSRPG.Core project for native language extensions
Unlike your previous lessons, you have posted the ‘before’ code rather than the ‘after’.
Phil
Hi Phil,
Thanks for letting me know. It should have the changed code now.
There are way too many manual steps to posting a lesson. 🙁