summaryrefslogtreecommitdiffstats
path: root/src/BrewingRecipes.cpp
diff options
context:
space:
mode:
authorpeterbell10 <peterbell10@live.co.uk>2017-05-28 20:08:23 +0200
committerLukas Pioch <lukas@zgow.de>2017-05-28 20:08:23 +0200
commitad4172d21c291dd1ab791900df45dfce5368bbd2 (patch)
tree02304790e5f7ad5ea199adb67eb88e9886369b02 /src/BrewingRecipes.cpp
parentDouble chest window fix (#3735) (diff)
downloadcuberite-ad4172d21c291dd1ab791900df45dfce5368bbd2.tar
cuberite-ad4172d21c291dd1ab791900df45dfce5368bbd2.tar.gz
cuberite-ad4172d21c291dd1ab791900df45dfce5368bbd2.tar.bz2
cuberite-ad4172d21c291dd1ab791900df45dfce5368bbd2.tar.lz
cuberite-ad4172d21c291dd1ab791900df45dfce5368bbd2.tar.xz
cuberite-ad4172d21c291dd1ab791900df45dfce5368bbd2.tar.zst
cuberite-ad4172d21c291dd1ab791900df45dfce5368bbd2.zip
Diffstat (limited to 'src/BrewingRecipes.cpp')
-rw-r--r--src/BrewingRecipes.cpp136
1 files changed, 40 insertions, 96 deletions
diff --git a/src/BrewingRecipes.cpp b/src/BrewingRecipes.cpp
index 4c127649a..15e9826de 100644
--- a/src/BrewingRecipes.cpp
+++ b/src/BrewingRecipes.cpp
@@ -2,7 +2,6 @@
#include "Globals.h" // NOTE: MSVC stupidness requires this to be the same across all modules
#include "BrewingRecipes.h"
-#include "Item.h"
#include <fstream>
@@ -12,23 +11,10 @@
-typedef std::vector<std::unique_ptr<cBrewingRecipes::cRecipe>> RecipeList;
-
-
-
-
-
-struct cBrewingRecipes::sBrewingRecipeState
-{
- RecipeList Recipes;
-};
-
-
cBrewingRecipes::cBrewingRecipes()
- : m_pState(new sBrewingRecipeState)
{
ReloadRecipes();
}
@@ -37,15 +23,6 @@ cBrewingRecipes::cBrewingRecipes()
-cBrewingRecipes::~cBrewingRecipes()
-{
- ClearRecipes();
-}
-
-
-
-
-
void cBrewingRecipes::ReloadRecipes(void)
{
ClearRecipes();
@@ -68,7 +45,7 @@ void cBrewingRecipes::ReloadRecipes(void)
size_t FirstCommentSymbol = ParsingLine.find('#');
if (FirstCommentSymbol != AString::npos)
{
- ParsingLine.erase(ParsingLine.begin() += static_cast<long>(FirstCommentSymbol), ParsingLine.end());
+ ParsingLine.erase(FirstCommentSymbol);
}
if (ParsingLine.empty())
@@ -78,26 +55,20 @@ void cBrewingRecipes::ReloadRecipes(void)
AddRecipeFromLine(ParsingLine, LineNum);
} // while (getline(ParsingLine))
- LOG("Loaded " SIZE_T_FMT " brewing recipes", m_pState->Recipes.size());
+ LOG("Loaded " SIZE_T_FMT " brewing recipes", m_Recipes.size());
}
-void cBrewingRecipes::AddRecipeFromLine(const AString & a_Line, unsigned int a_LineNum)
+void cBrewingRecipes::AddRecipeFromLine(AString a_Line, unsigned int a_LineNum)
{
- AString Line(a_Line);
- Line.erase(std::remove_if(Line.begin(), Line.end(), isspace), Line.end());
+ a_Line.erase(std::remove_if(a_Line.begin(), a_Line.end(), isspace), a_Line.end());
- short InputDamage;
- short OutputDamage;
+ auto Recipe = cpp14::make_unique<cRecipe>();
- std::unique_ptr<cItem> InputItem = cpp14::make_unique<cItem>();
- std::unique_ptr<cItem> IngredientItem = cpp14::make_unique<cItem>();
- std::unique_ptr<cItem> OutputItem = cpp14::make_unique<cItem>();
-
- const AStringVector & InputAndIngredient = StringSplit(Line, "+");
+ const AStringVector & InputAndIngredient = StringSplit(a_Line, "+");
if (InputAndIngredient.size() != 2)
{
@@ -114,39 +85,28 @@ void cBrewingRecipes::AddRecipeFromLine(const AString & a_Line, unsigned int a_L
return;
}
- if (!ParseItem(IngredientAndOutput[0], *IngredientItem))
+ if (!ParseItem(IngredientAndOutput[0], Recipe->Ingredient))
{
LOGWARNING("brewing.txt: Parsing of the item didn't worked.");
LOGINFO("Offending line: \"%s\"", a_Line.c_str());
return;
}
- if (!StringToInteger<short>(InputAndIngredient[0], InputDamage))
+ if (!StringToInteger<short>(InputAndIngredient[0], Recipe->Input.m_ItemDamage))
{
LOGWARNING("brewing.txt: line %d: Cannot parse the damage value for the input item\"%s\".", a_LineNum, InputAndIngredient[0].c_str());
LOGINFO("Offending line: \"%s\"", a_Line.c_str());
return;
}
- if (!StringToInteger<short>(IngredientAndOutput[1], OutputDamage))
+ if (!StringToInteger<short>(IngredientAndOutput[1], Recipe->Output.m_ItemDamage))
{
LOGWARNING("brewing.txt: line %d: Cannot parse the damage value for the output item\"%s\".", a_LineNum, IngredientAndOutput[1].c_str());
LOGINFO("Offending line: \"%s\"", a_Line.c_str());
return;
}
- // The items has always the same type
- InputItem->m_ItemType = E_ITEM_POTION;
- InputItem->m_ItemDamage = InputDamage;
-
- OutputItem->m_ItemType = E_ITEM_POTION;
- OutputItem->m_ItemDamage = OutputDamage;
-
- std::unique_ptr<cRecipe> Recipe = cpp14::make_unique<cRecipe>();
- Recipe->Input = std::move(InputItem);
- Recipe->Output = std::move(OutputItem);
- Recipe->Ingredient = std::move(IngredientItem);
- m_pState->Recipes.push_back(std::move(Recipe));
+ m_Recipes.push_back(std::move(Recipe));
}
@@ -164,7 +124,7 @@ bool cBrewingRecipes::ParseItem(const AString & a_String, cItem & a_Item)
void cBrewingRecipes::ClearRecipes(void)
{
- m_pState->Recipes.clear();
+ m_Recipes.clear();
}
@@ -173,9 +133,9 @@ void cBrewingRecipes::ClearRecipes(void)
const cBrewingRecipes::cRecipe * cBrewingRecipes::GetRecipeFrom(const cItem & a_Input, const cItem & a_Ingredient) const
{
- for (auto & Recipe : m_pState->Recipes)
+ for (const auto & Recipe : m_Recipes)
{
- if ((Recipe->Input->IsEqual(a_Input)) && (Recipe->Ingredient->IsEqual(a_Ingredient)))
+ if ((Recipe->Input.IsEqual(a_Input)) && (Recipe->Ingredient.IsEqual(a_Ingredient)))
{
return Recipe.get();
}
@@ -187,22 +147,15 @@ const cBrewingRecipes::cRecipe * cBrewingRecipes::GetRecipeFrom(const cItem & a_
if (a_Input.m_ItemDamage & 0x2000)
{
// Create new recipe and add it to list
- std::unique_ptr<cItem> InputItem = cpp14::make_unique<cItem>();
- std::unique_ptr<cItem> IngredientItem = cpp14::make_unique<cItem>();
- std::unique_ptr<cItem> OutputItem = cpp14::make_unique<cItem>();
-
- InputItem->m_ItemType = E_ITEM_POTION;
- InputItem->m_ItemDamage = a_Input.m_ItemDamage;
- OutputItem->m_ItemType = E_ITEM_POTION;
- OutputItem->m_ItemDamage = a_Input.m_ItemDamage + 8192;
- IngredientItem->m_ItemType = E_ITEM_GUNPOWDER;
-
- std::unique_ptr<cRecipe> Recipe = cpp14::make_unique<cRecipe>();
- Recipe->Input = std::move(InputItem);
- Recipe->Output = std::move(OutputItem);
- Recipe->Ingredient = std::move(IngredientItem);
- m_pState->Recipes.push_back(std::move(Recipe));
- return Recipe.get();
+ auto Recipe = cpp14::make_unique<cRecipe>();
+
+ Recipe->Input.m_ItemType = a_Input.m_ItemDamage;
+ Recipe->Output.m_ItemDamage = a_Input.m_ItemDamage + 8192;
+ Recipe->Ingredient.m_ItemType = E_ITEM_GUNPOWDER;
+
+ auto RecipePtr = Recipe.get();
+ m_Recipes.push_back(std::move(Recipe));
+ return RecipePtr;
}
return nullptr;
}
@@ -210,41 +163,32 @@ const cBrewingRecipes::cRecipe * cBrewingRecipes::GetRecipeFrom(const cItem & a_
// Check for splash potion
if (a_Input.m_ItemDamage & 0x4000)
{
- const std::unique_ptr<cRecipe> * FoundRecipe = nullptr;
// Search for the drinkable potion, the ingredients are the same
short SplashItemDamage = a_Input.m_ItemDamage - 8192;
- for (auto & Recipe : m_pState->Recipes)
+ auto FoundRecipe = std::find_if(m_Recipes.cbegin(), m_Recipes.cend(), [&](const std::unique_ptr<cRecipe>& a_Recipe)
{
- if ((Recipe->Input->m_ItemDamage == SplashItemDamage) && (Recipe->Ingredient->IsEqual(a_Ingredient)))
- {
- FoundRecipe = &Recipe;
- break;
- }
- }
+ return (
+ (a_Recipe->Input.m_ItemDamage == SplashItemDamage) &&
+ (a_Recipe->Ingredient.IsEqual(a_Ingredient))
+ );
+ });
- if (FoundRecipe == nullptr)
+ if (FoundRecipe == m_Recipes.cend())
{
return nullptr;
}
// Create new recipe and add it to list
- std::unique_ptr<cItem> InputItem = cpp14::make_unique<cItem>();
- std::unique_ptr<cItem> IngredientItem = cpp14::make_unique<cItem>();
- std::unique_ptr<cItem> OutputItem = cpp14::make_unique<cItem>();
-
- InputItem->m_ItemType = E_ITEM_POTION;
- InputItem->m_ItemDamage = a_Input.m_ItemDamage;
- OutputItem->m_ItemType = E_ITEM_POTION;
- OutputItem->m_ItemDamage = (*FoundRecipe)->Output->m_ItemDamage + 8192;
- IngredientItem->m_ItemType = (*FoundRecipe)->Ingredient->m_ItemType;
-
- std::unique_ptr<cRecipe> Recipe = cpp14::make_unique<cRecipe>();
- Recipe->Input = std::move(InputItem);
- Recipe->Output = std::move(OutputItem);
- Recipe->Ingredient = std::move(IngredientItem);
- m_pState->Recipes.push_back(std::move(Recipe));
- return Recipe.get();
+ auto Recipe = cpp14::make_unique<cRecipe>();
+
+ Recipe->Input.m_ItemDamage = a_Input.m_ItemDamage;
+ Recipe->Output.m_ItemDamage = (*FoundRecipe)->Output.m_ItemDamage + 8192;
+ Recipe->Ingredient.m_ItemType = (*FoundRecipe)->Ingredient.m_ItemType;
+
+ auto RecipePtr = Recipe.get();
+ m_Recipes.push_back(std::move(Recipe));
+ return RecipePtr;
}
return nullptr;
}
@@ -262,9 +206,9 @@ bool cBrewingRecipes::IsIngredient(const cItem & a_Ingredient) const
return true;
}
- for (auto & Recipe : m_pState->Recipes)
+ for (const auto & Recipe : m_Recipes)
{
- if (Recipe->Ingredient->IsEqual(a_Ingredient))
+ if (Recipe->Ingredient.IsEqual(a_Ingredient))
{
return true;
}