mirrors/PocketMine-MP
1
0
Fork 0
mirror of https://github.com/pmmp/PocketMine-MP.git synced 2025-09-07 10:22:56 +00:00
Code Issues Projects Releases Wiki Activity

Added crafting data packet cache, reduces on-join lag spikes, close #248 (#269)

Browse Source
This commit is contained in:
Dylan K. Taylor
2017-01-23 11:03:46 +00:00
committed by GitHub
parent 21f5be27b6
commit 9c36e0cd1c
3 changed files with 55 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

50
src/pocketmine/inventory/CraftingManager.php
View File

@ -21,7 +21,9 @@
namespace pocketmine\inventory;
use pocketmine\event\Timings;
use pocketmine\item\Item;
use pocketmine\network\protocol\CraftingDataPacket;
use pocketmine\Server;
use pocketmine\utils\Config;
use pocketmine\utils\MainLogger;
@ -40,6 +42,9 @@ class CraftingManager{
private static $RECIPE_COUNT = 0;
/** @var CraftingDataPacket */
private $craftingDataCache;
public function __construct(){
// load recipes from src/pocketmine/resources/recipes.json
$recipes = new Config(Server::getInstance()->getFilePath() . "src/pocketmine/resources/recipes.json", Config::JSON, []);
@ -84,6 +89,48 @@ class CraftingManager{
break;
}
}
$this->buildCraftingDataCache();
}
/**
* Rebuilds the cached CraftingDataPacket.
*/
public function buildCraftingDataCache(){
Timings::$craftingDataCacheRebuildTimer->startTiming();
$pk = new CraftingDataPacket();
$pk->cleanRecipes = true;
foreach($this->recipes as $recipe){
if($recipe instanceof ShapedRecipe){
$pk->addShapedRecipe($recipe);
}elseif($recipe instanceof ShapelessRecipe){
$pk->addShapelessRecipe($recipe);
}
}
foreach($this->furnaceRecipes as $recipe){
$pk->addFurnaceRecipe($recipe);
}
$pk->encode();
$pk->isEncoded = true;
$this->craftingDataCache = $pk;
Timings::$craftingDataCacheRebuildTimer->stopTiming();
}
/**
* Returns a CraftingDataPacket for sending to players. Rebuilds the cache if it is outdated.
*
* @return CraftingDataPacket
*/
public function getCraftingDataPacket() : CraftingDataPacket{
if($this->craftingDataCache === null){
$this->buildCraftingDataCache();
}
return $this->craftingDataCache;
}
public function sort(Item $i1, Item $i2){
@ -162,6 +209,7 @@ class CraftingManager{
}
$this->recipeLookup[$result->getId() . ":" . $result->getDamage()][$hash] = $recipe;
$this->craftingDataCache = null;
}
/**
@ -177,6 +225,7 @@ class CraftingManager{
$hash .= $item->getId() . ":" . ($item->hasAnyDamageValue() ? "?" : $item->getDamage()) . "x" . $item->getCount() . ",";
}
$this->recipeLookup[$result->getId() . ":" . $result->getDamage()][$hash] = $recipe;
$this->craftingDataCache = null;
}
/**
@ -185,6 +234,7 @@ class CraftingManager{
public function registerFurnaceRecipe(FurnaceRecipe $recipe){
$input = $recipe->getInput();
$this->furnaceRecipes[$input->getId() . ":" . ($input->hasAnyDamageValue() ? "?" : $input->getDamage())] = $recipe;
$this->craftingDataCache = null;
}
/**