Upstream/ShopChest
1
0
Fork 0
mirror of https://github.com/EpicEricEE/ShopChest.git synced 2024-11-23 19:16:41 +01:00
Code Issues Projects Releases Wiki Activity

Only compare a world's name to improve performace (#40)

Browse Source 
When using .equals() on classes with many variables and methods, means that every single thing on the class would be checked to be equals to the one from the another class, and its taking too much CPU.
This commit is contained in:
gonzalociocca 2016-12-08 16:11:48 -03:00 committed by Eric
parent f4b7df8c92
commit 253e40e8b9
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/main/java/de/epiceric/shopchest/listeners/HologramUpdateListener.java
View File

@ -52,8 +52,8 @@ public class HologramUpdateListener implements Listener {
if (shop.getHologram() == null) continue; if (shop.getHologram() == null) continue;
Location shopLocation = shop.getLocation(); Location shopLocation = shop.getLocation();
if (playerLocation.getWorld().equals(shopLocation.getWorld())) { if (playerLocation.getWorld().getName().equals(shopLocation.getWorld().getName())) {
if (playerLocation.distanceSquared(shop.getHologram().getLocation()) <= hologramDistanceSquared) { if (playerLocation.distanceSquared(shop.getHologram().getLocation()) <= hologramDistanceSquared) {
if (!shop.getHologram().isVisible(p)) { if (!shop.getHologram().isVisible(p)) {
shop.getHologram().showPlayer(p); shop.getHologram().showPlayer(p);