From 3b9bd1188e932e6bb8041f7bb9afbf3ce75e26d3 Mon Sep 17 00:00:00 2001 From: Alexdoru <57050655+Alexdoru@users.noreply.github.com> Date: Wed, 2 Oct 2024 07:31:08 +0200 Subject: Cleanup the codebase (#3311) Co-authored-by: boubou19 --- .../gtPlusPlus/core/block/base/BasicBlock.java | 2 +- .../block/general/antigrief/BlockWitherProof.java | 14 +- .../core/block/machine/BlockCircuitProgrammer.java | 3 +- .../core/block/machine/BlockFlaskSetter.java | 3 +- .../core/block/machine/BlockPooCollector.java | 12 +- .../core/block/machine/BlockSuperJukebox.java | 10 +- .../gtPlusPlus/core/client/CustomTextureSet.java | 4 +- .../core/client/model/ModelSickBlaze.java | 4 +- .../client/renderer/CustomOreBlockRenderer.java | 18 ++- .../core/client/renderer/RenderDecayChest.java | 2 +- .../core/client/renderer/RenderToxinball.java | 2 +- .../gtPlusPlus/core/common/compat/CompatIC2.java | 4 +- .../core/container/ContainerCircuitProgrammer.java | 4 +- .../core/container/ContainerFishTrap.java | 4 +- .../core/container/ContainerPestKiller.java | 11 +- .../core/container/ContainerProjectTable.java | 8 +- .../core/container/ContainerSuperJukebox.java | 4 +- .../container/ContainerVolumetricFlaskSetter.java | 8 +- .../core/entity/projectile/EntityToxinball.java | 6 +- .../gtPlusPlus/core/gui/beta/GUIIDRegistry.java | 9 +- .../java/gtPlusPlus/core/gui/beta/GUITypes.java | 2 +- .../gtPlusPlus/core/gui/machine/GUIPestKiller.java | 8 +- .../core/gui/machine/GUIVolumetricFlaskSetter.java | 46 +++--- .../core/handler/Recipes/DecayableRecipe.java | 6 +- .../handler/events/EnderDragonDeathHandler.java | 6 +- .../core/handler/events/SneakManager.java | 6 +- .../workbench/WorkbenchCraftingHandler.java | 32 ++--- .../gtPlusPlus/core/interfaces/IGuiManager.java | 10 +- .../core/interfaces/IGuiManagerMiscUtils.java | 2 +- .../gtPlusPlus/core/interfaces/IItemBlueprint.java | 22 +-- .../inventories/InventoryCircuitProgrammer.java | 2 +- .../core/inventories/InventoryDecayablesChest.java | 2 +- .../core/inventories/InventoryEggBox.java | 2 +- .../core/inventories/InventoryFishTrap.java | 2 +- .../core/inventories/InventoryPestKiller.java | 2 +- .../core/inventories/InventorySuperJukebox.java | 2 +- .../InventoryVolumetricFlaskSetter.java | 2 +- .../projecttable/InventoryProjectMain.java | 2 +- .../projecttable/InventoryProjectOutput.java | 2 +- .../inventories/tradetable/InventoryTradeMain.java | 2 +- .../tradetable/InventoryTradeOutput.java | 2 +- src/main/java/gtPlusPlus/core/item/ModItems.java | 2 +- .../core/item/base/BaseItemBurnable.java | 2 +- .../core/item/base/BaseItemComponent.java | 20 +-- .../core/item/base/BaseItemDamageable.java | 16 +-- .../core/item/base/BaseItemTickable.java | 3 +- .../gtPlusPlus/core/item/base/BasicSpawnEgg.java | 9 +- .../core/item/base/cell/BaseItemCell.java | 2 +- .../core/item/base/cell/BaseItemPlasmaCell.java | 2 +- .../core/item/base/dusts/BaseItemDust.java | 2 +- .../core/item/base/dusts/BaseItemDustUnique.java | 6 +- .../core/item/base/ingots/BaseItemIngotHot.java | 10 +- .../core/item/base/ingots/BaseItemIngotOld.java | 4 +- .../core/item/base/itemblock/ItemBlockMeta.java | 9 +- .../core/item/base/misc/BaseItemMisc.java | 4 +- .../core/item/base/ore/BaseOreComponent.java | 17 +-- .../core/item/base/plates/BaseItemPlate_OLD.java | 4 +- .../gtPlusPlus/core/item/bauble/BaseBauble.java | 4 +- .../core/item/bauble/BatteryPackBaseBauble.java | 37 +---- .../core/item/chemistry/AgriculturalChem.java | 9 +- .../core/item/chemistry/MilledOreProcessing.java | 2 +- .../chemistry/general/ItemGenericChemBase.java | 10 +- .../item/circuit/GTPPIntegratedCircuitItem.java | 11 +- .../core/item/crafting/ItemDummyResearch.java | 6 +- .../core/item/food/BaseItemMetaFood.java | 9 +- .../item/general/ItemBasicScrubberTurbine.java | 7 +- .../core/item/general/ItemBlueprint.java | 15 +- .../core/item/general/ItemGenericToken.java | 3 +- .../core/item/general/ItemHealingDevice.java | 9 +- .../core/item/general/ItemLavaFilter.java | 2 +- .../core/item/general/ItemMagicFeather.java | 2 +- .../item/general/spawn/ItemCustomSpawnEgg.java | 111 +++++++------- .../core/item/materials/DustDecayable.java | 12 +- .../core/item/tool/misc/ItemGregtechPump.java | 34 ++--- .../java/gtPlusPlus/core/lib/VanillaColours.java | 2 +- .../java/gtPlusPlus/core/material/Material.java | 63 ++++---- .../core/material/MaterialGenerator.java | 19 +-- .../gtPlusPlus/core/material/MaterialsAlloy.java | 2 +- .../core/material/MaterialsElements.java | 41 +++--- .../java/gtPlusPlus/core/material/Particle.java | 4 +- .../core/material/nuclear/MaterialsFluorides.java | 70 ++++----- .../core/material/nuclear/MaterialsNuclides.java | 110 +++++++------- .../core/material/state/MaterialState.java | 2 +- .../core/network/packet/AbstractPacket.java | 2 +- .../gtPlusPlus/core/recipe/RecipesGeneral.java | 56 ++++---- .../gtPlusPlus/core/recipe/RecipesMachines.java | 4 +- .../java/gtPlusPlus/core/recipe/common/CI.java | 14 +- .../core/slots/SlotCraftingNoCollect.java | 2 +- .../core/tileentities/base/TileEntityBase.java | 96 +++++++------ .../general/TileEntityCircuitProgrammer.java | 14 +- .../general/TileEntityDecayablesChest.java | 2 +- .../general/TileEntityInfiniteFluid.java | 23 ++- .../general/TileEntityVolumetricFlaskSetter.java | 5 +- .../machines/TileEntityAdvPooCollector.java | 2 +- .../machines/TileEntityBaseFluidCollector.java | 20 ++- .../machines/TileEntityPestKiller.java | 46 +++--- .../machines/TileEntityPooCollector.java | 2 +- src/main/java/gtPlusPlus/core/util/Utils.java | 2 +- .../java/gtPlusPlus/core/util/data/ArrayUtils.java | 14 +- .../java/gtPlusPlus/core/util/data/FileUtils.java | 5 +- .../gtPlusPlus/core/util/data/StringUtils.java | 6 +- .../java/gtPlusPlus/core/util/math/MathUtils.java | 31 ++-- .../gtPlusPlus/core/util/minecraft/FluidUtils.java | 15 +- .../core/util/minecraft/InventoryUtils.java | 3 +- .../gtPlusPlus/core/util/minecraft/ItemUtils.java | 159 +++++++-------------- .../core/util/minecraft/MaterialUtils.java | 5 +- .../gtPlusPlus/core/util/minecraft/NBTUtils.java | 11 +- .../core/util/minecraft/PlayerUtils.java | 3 +- .../core/util/minecraft/RecipeUtils.java | 42 ++---- .../core/util/recipe/RecipeHashStrat.java | 5 +- .../core/util/reflect/ReflectionUtils.java | 25 ++-- .../core/world/explosions/MiningExplosion.java | 1 - 112 files changed, 681 insertions(+), 936 deletions(-) (limited to 'src/main/java/gtPlusPlus/core') diff --git a/src/main/java/gtPlusPlus/core/block/base/BasicBlock.java b/src/main/java/gtPlusPlus/core/block/base/BasicBlock.java index d39c194dfd..7747deb5da 100644 --- a/src/main/java/gtPlusPlus/core/block/base/BasicBlock.java +++ b/src/main/java/gtPlusPlus/core/block/base/BasicBlock.java @@ -31,7 +31,7 @@ public class BasicBlock extends BlockContainer { this.setStepSound(soundTypeMetal); } - public static enum BlockTypes { + public enum BlockTypes { STANDARD("blockBlock", "pickaxe", soundTypeMetal), FRAME("blockFrameGt", "wrench", soundTypeMetal), diff --git a/src/main/java/gtPlusPlus/core/block/general/antigrief/BlockWitherProof.java b/src/main/java/gtPlusPlus/core/block/general/antigrief/BlockWitherProof.java index 7e64f19bf3..a85e700db2 100644 --- a/src/main/java/gtPlusPlus/core/block/general/antigrief/BlockWitherProof.java +++ b/src/main/java/gtPlusPlus/core/block/general/antigrief/BlockWitherProof.java @@ -7,8 +7,6 @@ import net.minecraft.block.material.Material; import net.minecraft.client.renderer.texture.IIconRegister; import net.minecraft.entity.Entity; import net.minecraft.entity.EnumCreatureType; -import net.minecraft.entity.boss.EntityDragon; -import net.minecraft.entity.boss.EntityWither; import net.minecraft.entity.boss.IBossDisplayData; import net.minecraft.entity.player.EntityPlayer; import net.minecraft.entity.player.EntityPlayerMP; @@ -73,8 +71,7 @@ public class BlockWitherProof extends Block { if ((entity == null) || !entity.isEntityAlive()) { return false; } - if ((entity instanceof EntityWither) || (entity instanceof EntityDragon) - || (entity instanceof IBossDisplayData)) { + if (entity instanceof IBossDisplayData) { return false; } else { return super.canEntityDestroy(world, x, y, z, entity); @@ -109,7 +106,7 @@ public class BlockWitherProof extends Block { @Override public float getPlayerRelativeBlockHardness(EntityPlayer aPlayer, World p_149737_2_, int p_149737_3_, int p_149737_4_, int p_149737_5_) { - if (aPlayer != null && aPlayer instanceof EntityPlayerMP) { + if (aPlayer instanceof EntityPlayerMP) { return 1f; } return -1f; @@ -131,10 +128,7 @@ public class BlockWitherProof extends Block { if ((entity == null) || !entity.isEntityAlive()) { return; } - if ((entity instanceof EntityWither) || (entity instanceof EntityDragon) - || (entity instanceof IBossDisplayData)) { - return; - } else { + if (!(entity instanceof IBossDisplayData)) { super.onEntityCollidedWithBlock(world, x, y, z, entity); } } @@ -147,7 +141,7 @@ public class BlockWitherProof extends Block { @Override public boolean canHarvestBlock(EntityPlayer player, int meta) { - if (player != null && player instanceof EntityPlayerMP) { + if (player instanceof EntityPlayerMP) { return true; } return super.canHarvestBlock(player, meta); diff --git a/src/main/java/gtPlusPlus/core/block/machine/BlockCircuitProgrammer.java b/src/main/java/gtPlusPlus/core/block/machine/BlockCircuitProgrammer.java index dcbe41f150..568df08922 100644 --- a/src/main/java/gtPlusPlus/core/block/machine/BlockCircuitProgrammer.java +++ b/src/main/java/gtPlusPlus/core/block/machine/BlockCircuitProgrammer.java @@ -147,7 +147,6 @@ public class BlockCircuitProgrammer extends BasicTileBlockWithTooltip { GTPlusPlus.ID + ":metro/TEXTURE_TECH_PANEL_B", GTPlusPlus.ID + ":metro/TEXTURE_METAL_PANEL_I", GTPlusPlus.ID + ":metro/TEXTURE_METAL_PANEL_I", GTPlusPlus.ID + ":metro/TEXTURE_METAL_PANEL_I", GTPlusPlus.ID + ":metro/TEXTURE_METAL_PANEL_I" }; - CubicObject[] aTextureData = new CubicObject[] { new CubicObject<>(aTexData) }; - return aTextureData; + return (CubicObject[]) new CubicObject[] { new CubicObject<>(aTexData) }; } } diff --git a/src/main/java/gtPlusPlus/core/block/machine/BlockFlaskSetter.java b/src/main/java/gtPlusPlus/core/block/machine/BlockFlaskSetter.java index 5ec07bb9e3..134bbafd6b 100644 --- a/src/main/java/gtPlusPlus/core/block/machine/BlockFlaskSetter.java +++ b/src/main/java/gtPlusPlus/core/block/machine/BlockFlaskSetter.java @@ -151,7 +151,6 @@ public class BlockFlaskSetter extends BasicTileBlockWithTooltip { GTPlusPlus.ID + ":metro/TEXTURE_TECH_PANEL_C", GTPlusPlus.ID + ":metro/TEXTURE_METAL_PANEL_H", GTPlusPlus.ID + ":metro/TEXTURE_METAL_PANEL_H", GTPlusPlus.ID + ":metro/TEXTURE_METAL_PANEL_H", GTPlusPlus.ID + ":metro/TEXTURE_METAL_PANEL_H" }; - CubicObject[] aTextureData = new CubicObject[] { new CubicObject<>(aTexData) }; - return aTextureData; + return (CubicObject[]) new CubicObject[] { new CubicObject<>(aTexData) }; } } diff --git a/src/main/java/gtPlusPlus/core/block/machine/BlockPooCollector.java b/src/main/java/gtPlusPlus/core/block/machine/BlockPooCollector.java index 6a371a3284..b7c84ddc8c 100644 --- a/src/main/java/gtPlusPlus/core/block/machine/BlockPooCollector.java +++ b/src/main/java/gtPlusPlus/core/block/machine/BlockPooCollector.java @@ -85,13 +85,11 @@ public class BlockPooCollector extends BlockContainer { } else { TileEntityBaseFluidCollector tank = (TileEntityBaseFluidCollector) world.getTileEntity(x, y, z); if (tank != null) { - Item handItem; - try { - handItem = player.getHeldItem() - .getItem(); - } catch (Throwable t) { - handItem = null; - } + // Item handItem; + // try { + // handItem = player.getHeldItem() + // .getItem(); + // } catch (Throwable t) {} // Fluid container code /* diff --git a/src/main/java/gtPlusPlus/core/block/machine/BlockSuperJukebox.java b/src/main/java/gtPlusPlus/core/block/machine/BlockSuperJukebox.java index be55db80a0..9d6f76ca2c 100644 --- a/src/main/java/gtPlusPlus/core/block/machine/BlockSuperJukebox.java +++ b/src/main/java/gtPlusPlus/core/block/machine/BlockSuperJukebox.java @@ -77,7 +77,7 @@ public class BlockSuperJukebox extends BlockJukebox { } final TileEntity te = world.getTileEntity(x, y, z); - if ((te != null) && (te instanceof TileEntitySuperJukebox)) { + if (te instanceof TileEntitySuperJukebox) { player.openGui(GTplusplus.instance, GuiHandler.GUI14, world, x, y, z); return true; } @@ -328,7 +328,7 @@ public class BlockSuperJukebox extends BlockJukebox { Logger.INFO("b1"); // Select First Record ItemStack aRecordToPlay; - if (mValidRecords.size() == 0) { + if (mValidRecords.isEmpty()) { Logger.INFO("bX"); return false; } else { @@ -399,8 +399,8 @@ public class BlockSuperJukebox extends BlockJukebox { ItemStack aRecordToPlay = tileentityjukebox.func_145856_a(); if (aRecordToPlay != null) { aWorld.playAuxSFX(1005, aX, aY, aZ, 0); - aWorld.playRecord((String) null, aX, aY, aZ); - tileentityjukebox.func_145857_a((ItemStack) null); + aWorld.playRecord(null, aX, aY, aZ); + tileentityjukebox.func_145857_a(null); this.markDirty(); } } @@ -568,7 +568,7 @@ public class BlockSuperJukebox extends BlockJukebox { @Override public boolean hasCustomInventoryName() { - return (this.customName != null) && !this.customName.equals(""); + return (this.customName != null) && !this.customName.isEmpty(); } } } diff --git a/src/main/java/gtPlusPlus/core/client/CustomTextureSet.java b/src/main/java/gtPlusPlus/core/client/CustomTextureSet.java index ce4d9a0320..ca9328dba0 100644 --- a/src/main/java/gtPlusPlus/core/client/CustomTextureSet.java +++ b/src/main/java/gtPlusPlus/core/client/CustomTextureSet.java @@ -4,7 +4,7 @@ import gregtech.api.enums.TextureSet; public class CustomTextureSet extends TextureSet { - public static enum TextureSets { + public enum TextureSets { REFINED(), GEM_A(), @@ -13,7 +13,7 @@ public class CustomTextureSet extends TextureSet { private final CustomTextureSet A; - private TextureSets() { + TextureSets() { A = new CustomTextureSet( this.name() .toUpperCase()); diff --git a/src/main/java/gtPlusPlus/core/client/model/ModelSickBlaze.java b/src/main/java/gtPlusPlus/core/client/model/ModelSickBlaze.java index 3b5922444d..877fe3886c 100644 --- a/src/main/java/gtPlusPlus/core/client/model/ModelSickBlaze.java +++ b/src/main/java/gtPlusPlus/core/client/model/ModelSickBlaze.java @@ -12,9 +12,9 @@ import cpw.mods.fml.relauncher.SideOnly; public class ModelSickBlaze extends ModelBlaze { /** The sticks that fly around the Blaze. */ - private ModelRenderer[] blazeSticks = new ModelRenderer[24]; + private final ModelRenderer[] blazeSticks = new ModelRenderer[24]; - private ModelRenderer blazeHead; + private final ModelRenderer blazeHead; public ModelSickBlaze() { for (int i = 0; i < this.blazeSticks.length; ++i) { diff --git a/src/main/java/gtPlusPlus/core/client/renderer/CustomOreBlockRenderer.java b/src/main/java/gtPlusPlus/core/client/renderer/CustomOreBlockRenderer.java index 689f075703..a32dd2da8f 100644 --- a/src/main/java/gtPlusPlus/core/client/renderer/CustomOreBlockRenderer.java +++ b/src/main/java/gtPlusPlus/core/client/renderer/CustomOreBlockRenderer.java @@ -31,8 +31,7 @@ public class CustomOreBlockRenderer implements ISimpleBlockRenderingHandler { public boolean renderStandardBlock(IBlockAccess aWorld, int aX, int aY, int aZ, Block aBlock, RenderBlocks aRenderer) { - Block tTileEntity = aBlock; - if ((tTileEntity instanceof ITexturedBlock)) { + if ((aBlock instanceof ITexturedBlock)) { return renderStandardBlock( aWorld, aX, @@ -40,12 +39,12 @@ public class CustomOreBlockRenderer implements ISimpleBlockRenderingHandler { aZ, aBlock, aRenderer, - new ITexture[][] { ((ITexturedBlock) tTileEntity).getTexture(ForgeDirection.DOWN), - ((ITexturedBlock) tTileEntity).getTexture(ForgeDirection.UP), - ((ITexturedBlock) tTileEntity).getTexture(ForgeDirection.NORTH), - ((ITexturedBlock) tTileEntity).getTexture(ForgeDirection.SOUTH), - ((ITexturedBlock) tTileEntity).getTexture(ForgeDirection.WEST), - ((ITexturedBlock) tTileEntity).getTexture(ForgeDirection.EAST) }); + new ITexture[][] { ((ITexturedBlock) aBlock).getTexture(ForgeDirection.DOWN), + ((ITexturedBlock) aBlock).getTexture(ForgeDirection.UP), + ((ITexturedBlock) aBlock).getTexture(ForgeDirection.NORTH), + ((ITexturedBlock) aBlock).getTexture(ForgeDirection.SOUTH), + ((ITexturedBlock) aBlock).getTexture(ForgeDirection.WEST), + ((ITexturedBlock) aBlock).getTexture(ForgeDirection.EAST) }); } return false; } @@ -556,8 +555,7 @@ public class CustomOreBlockRenderer implements ISimpleBlockRenderingHandler { .getTextureManager() .getTexture(TextureMap.locationBlocksTexture)).getAtlasSprite("missingno"); } - - return (IIcon) iicon; + return iicon; } IBlockAccess blockAccess = RenderBlocks.getInstance().blockAccess; diff --git a/src/main/java/gtPlusPlus/core/client/renderer/RenderDecayChest.java b/src/main/java/gtPlusPlus/core/client/renderer/RenderDecayChest.java index 78d4f367eb..a5ab58ef59 100644 --- a/src/main/java/gtPlusPlus/core/client/renderer/RenderDecayChest.java +++ b/src/main/java/gtPlusPlus/core/client/renderer/RenderDecayChest.java @@ -23,7 +23,7 @@ public class RenderDecayChest extends TileEntitySpecialRenderer { private static final ResourceLocation mChestTexture = new ResourceLocation( GTPlusPlus.ID, "textures/blocks/TileEntities/DecayablesChest_full.png"); - private ModelDecayChest mChestModel = new ModelDecayChest(); + private final ModelDecayChest mChestModel = new ModelDecayChest(); public static RenderDecayChest INSTANCE; public final int mRenderID; diff --git a/src/main/java/gtPlusPlus/core/client/renderer/RenderToxinball.java b/src/main/java/gtPlusPlus/core/client/renderer/RenderToxinball.java index 2c0ed7f562..bc8c3b7bbc 100644 --- a/src/main/java/gtPlusPlus/core/client/renderer/RenderToxinball.java +++ b/src/main/java/gtPlusPlus/core/client/renderer/RenderToxinball.java @@ -18,7 +18,7 @@ import gtPlusPlus.core.entity.projectile.EntityToxinball; @SideOnly(Side.CLIENT) public class RenderToxinball extends Render { - private float mSize; + private final float mSize; public RenderToxinball(float scale) { this.mSize = scale; diff --git a/src/main/java/gtPlusPlus/core/common/compat/CompatIC2.java b/src/main/java/gtPlusPlus/core/common/compat/CompatIC2.java index 7e60f95d37..905c1a870b 100644 --- a/src/main/java/gtPlusPlus/core/common/compat/CompatIC2.java +++ b/src/main/java/gtPlusPlus/core/common/compat/CompatIC2.java @@ -10,8 +10,8 @@ import gtPlusPlus.core.util.minecraft.ItemUtils; public class CompatIC2 { - private static ItemStack itemCropnalyzer = ItemUtils.simpleMetaStack("IC2:itemCropnalyzer", 0, 1); - private static ItemStack itemSolarHelmet = ItemUtils.simpleMetaStack("IC2:itemSolarHelmet", 0, 1); + private static final ItemStack itemCropnalyzer = ItemUtils.simpleMetaStack("IC2:itemCropnalyzer", 0, 1); + private static final ItemStack itemSolarHelmet = ItemUtils.simpleMetaStack("IC2:itemSolarHelmet", 0, 1); public static ShapedRecipeObject Cropnalyzer = new ShapedRecipeObject( "ore:cableGt02Copper", diff --git a/src/main/java/gtPlusPlus/core/container/ContainerCircuitProgrammer.java b/src/main/java/gtPlusPlus/core/container/ContainerCircuitProgrammer.java index 9afccfb0f4..6e1da7bc61 100644 --- a/src/main/java/gtPlusPlus/core/container/ContainerCircuitProgrammer.java +++ b/src/main/java/gtPlusPlus/core/container/ContainerCircuitProgrammer.java @@ -137,7 +137,7 @@ public class ContainerCircuitProgrammer extends Container { @Override public ItemStack transferStackInSlot(final EntityPlayer par1EntityPlayer, final int par2) { ItemStack var3 = null; - final Slot var4 = (Slot) this.inventorySlots.get(par2); + final Slot var4 = this.inventorySlots.get(par2); if ((var4 != null) && var4.getHasStack()) { final ItemStack var5 = var4.getStack(); @@ -153,7 +153,7 @@ public class ContainerCircuitProgrammer extends Container { */ if (var5.stackSize == 0) { - var4.putStack((ItemStack) null); + var4.putStack(null); } else { var4.onSlotChanged(); } diff --git a/src/main/java/gtPlusPlus/core/container/ContainerFishTrap.java b/src/main/java/gtPlusPlus/core/container/ContainerFishTrap.java index 1a9344c08c..08b6ee2ebb 100644 --- a/src/main/java/gtPlusPlus/core/container/ContainerFishTrap.java +++ b/src/main/java/gtPlusPlus/core/container/ContainerFishTrap.java @@ -95,7 +95,7 @@ public class ContainerFishTrap extends Container { @Override public ItemStack transferStackInSlot(final EntityPlayer par1EntityPlayer, final int par2) { ItemStack var3 = null; - final Slot var4 = (Slot) this.inventorySlots.get(par2); + final Slot var4 = this.inventorySlots.get(par2); if ((var4 != null) && var4.getHasStack()) { final ItemStack var5 = var4.getStack(); @@ -111,7 +111,7 @@ public class ContainerFishTrap extends Container { */ if (var5.stackSize == 0) { - var4.putStack((ItemStack) null); + var4.putStack(null); } else { var4.onSlotChanged(); } diff --git a/src/main/java/gtPlusPlus/core/container/ContainerPestKiller.java b/src/main/java/gtPlusPlus/core/container/ContainerPestKiller.java index 6f63a1d04e..546c367401 100644 --- a/src/main/java/gtPlusPlus/core/container/ContainerPestKiller.java +++ b/src/main/java/gtPlusPlus/core/container/ContainerPestKiller.java @@ -83,10 +83,7 @@ public class ContainerPestKiller extends Container { @Override public ItemStack slotClick(final int aSlotIndex, final int aMouseclick, final int aShifthold, final EntityPlayer aPlayer) { - boolean fluid = false; - if (aSlotIndex == 2) { - fluid = true; - } + boolean fluid = aSlotIndex == 2; if (!fluid) { return super.slotClick(aSlotIndex, aMouseclick, aShifthold, aPlayer); } else { @@ -111,7 +108,7 @@ public class ContainerPestKiller extends Container { @Override public ItemStack transferStackInSlot(final EntityPlayer par1EntityPlayer, final int par2) { ItemStack var3 = null; - final Slot var4 = (Slot) this.inventorySlots.get(par2); + final Slot var4 = this.inventorySlots.get(par2); if ((var4 != null) && var4.getHasStack()) { final ItemStack var5 = var4.getStack(); @@ -127,7 +124,7 @@ public class ContainerPestKiller extends Container { */ if (var5.stackSize == 0) { - var4.putStack((ItemStack) null); + var4.putStack(null); } else { var4.onSlotChanged(); } @@ -148,7 +145,7 @@ public class ContainerPestKiller extends Container { return super.func_94530_a(p_94530_1_, p_94530_2_); } - private class GT_Slot_Render extends Slot { + private static class GT_Slot_Render extends Slot { public final int mSlotIndex; public boolean mEnabled = true; diff --git a/src/main/java/gtPlusPlus/core/container/ContainerProjectTable.java b/src/main/java/gtPlusPlus/core/container/ContainerProjectTable.java index 306c6f0026..05f4182833 100644 --- a/src/main/java/gtPlusPlus/core/container/ContainerProjectTable.java +++ b/src/main/java/gtPlusPlus/core/container/ContainerProjectTable.java @@ -178,15 +178,11 @@ public class ContainerProjectTable extends Container { } public ItemStack getOutputContent() { - ItemStack output = this.craftResult.getStackInSlot(0); - if (output != null) { - return output; - } - return null; + return this.craftResult.getStackInSlot(0); } public ItemStack[] getInputComponents() { - ItemStack inputs[] = new ItemStack[9]; + ItemStack[] inputs = new ItemStack[9]; for (int r = 0; r < this.craftMatrix.getSizeInventory(); r++) { ItemStack temp = this.craftMatrix.getStackInSlot(r); inputs[r] = temp; diff --git a/src/main/java/gtPlusPlus/core/container/ContainerSuperJukebox.java b/src/main/java/gtPlusPlus/core/container/ContainerSuperJukebox.java index 45782f1384..04ea296e35 100644 --- a/src/main/java/gtPlusPlus/core/container/ContainerSuperJukebox.java +++ b/src/main/java/gtPlusPlus/core/container/ContainerSuperJukebox.java @@ -134,7 +134,7 @@ public class ContainerSuperJukebox extends Container { @Override public ItemStack transferStackInSlot(final EntityPlayer par1EntityPlayer, final int par2) { ItemStack var3 = null; - final Slot var4 = (Slot) this.inventorySlots.get(par2); + final Slot var4 = this.inventorySlots.get(par2); if ((var4 != null) && var4.getHasStack()) { final ItemStack var5 = var4.getStack(); @@ -150,7 +150,7 @@ public class ContainerSuperJukebox extends Container { */ if (var5.stackSize == 0) { - var4.putStack((ItemStack) null); + var4.putStack(null); } else { var4.onSlotChanged(); } diff --git a/src/main/java/gtPlusPlus/core/container/ContainerVolumetricFlaskSetter.java b/src/main/java/gtPlusPlus/core/container/ContainerVolumetricFlaskSetter.java index bf1f8c4133..1a517e6761 100644 --- a/src/main/java/gtPlusPlus/core/container/ContainerVolumetricFlaskSetter.java +++ b/src/main/java/gtPlusPlus/core/container/ContainerVolumetricFlaskSetter.java @@ -104,7 +104,7 @@ public class ContainerVolumetricFlaskSetter extends Container { @Override public ItemStack transferStackInSlot(final EntityPlayer par1EntityPlayer, final int par2) { ItemStack var3 = null; - final Slot var4 = (Slot) this.inventorySlots.get(par2); + final Slot var4 = this.inventorySlots.get(par2); if ((var4 != null) && var4.getHasStack()) { final ItemStack var5 = var4.getStack(); @@ -120,7 +120,7 @@ public class ContainerVolumetricFlaskSetter extends Container { */ if (var5.stackSize == 0) { - var4.putStack((ItemStack) null); + var4.putStack(null); } else { var4.onSlotChanged(); } @@ -159,8 +159,8 @@ public class ContainerVolumetricFlaskSetter extends Container { @Override public void updateProgressBar(int par1, int par2) { super.updateProgressBar(par1, par2); - switch (par1) { - case 0 -> mCustomValue = (short) par2; + if (par1 == 0) { + mCustomValue = (short) par2; } } diff --git a/src/main/java/gtPlusPlus/core/entity/projectile/EntityToxinball.java b/src/main/java/gtPlusPlus/core/entity/projectile/EntityToxinball.java index a3b2f6f28b..65e2385003 100644 --- a/src/main/java/gtPlusPlus/core/entity/projectile/EntityToxinball.java +++ b/src/main/java/gtPlusPlus/core/entity/projectile/EntityToxinball.java @@ -170,9 +170,7 @@ public abstract class EntityToxinball extends EntityFireball { this.rotationYaw = (float) (Math.atan2(this.motionZ, this.motionX) * 180.0D / Math.PI) + 90.0F; for (this.rotationPitch = (float) (Math.atan2(f1, this.motionY) * 180.0D / Math.PI) - - 90.0F; this.rotationPitch - this.prevRotationPitch < -180.0F; this.prevRotationPitch -= 360.0F) { - ; - } + - 90.0F; this.rotationPitch - this.prevRotationPitch < -180.0F; this.prevRotationPitch -= 360.0F) {} while (this.rotationPitch - this.prevRotationPitch >= 180.0F) { this.prevRotationPitch += 360.0F; @@ -241,7 +239,7 @@ public abstract class EntityToxinball extends EntityFireball { aTag.setShort("zTile", (short) this.entityZ); aTag.setByte("inTile", (byte) Block.getIdFromBlock(this.block)); aTag.setByte("inGround", (byte) (this.inGround ? 1 : 0)); - aTag.setTag("direction", this.newDoubleNBTList(new double[] { this.motionX, this.motionY, this.motionZ })); + aTag.setTag("direction", this.newDoubleNBTList(this.motionX, this.motionY, this.motionZ)); } /** diff --git a/src/main/java/gtPlusPlus/core/gui/beta/GUIIDRegistry.java b/src/main/java/gtPlusPlus/core/gui/beta/GUIIDRegistry.java index f6d899d247..fbb34ee5b7 100644 --- a/src/main/java/gtPlusPlus/core/gui/beta/GUIIDRegistry.java +++ b/src/main/java/gtPlusPlus/core/gui/beta/GUIIDRegistry.java @@ -12,14 +12,12 @@ public class GUIIDRegistry { private static final Map idMap = new HashMap<>(); private static int nextId = 0; - static {} - private static void registerGuiHandlers(final GUITypes MU_GuiType, final List> guiHandlerClasses) { for (final Class tileGuiHandlerClass : guiHandlerClasses) { final MUGuild guiId = new MUGuild(nextId++, MU_GuiType, tileGuiHandlerClass); classMap.put(tileGuiHandlerClass, guiId); - idMap.put(Integer.valueOf(guiId.getId()), guiId); + idMap.put(guiId.getId(), guiId); } } @@ -29,7 +27,8 @@ public class GUIIDRegistry { if (guiId == null) { for (final Map.Entry, MUGuild> classGuiIdEntry : classMap .entrySet()) { - if (((Class) classGuiIdEntry.getKey()).isAssignableFrom(guiHandlerClass)) { + if (classGuiIdEntry.getKey() + .isAssignableFrom(guiHandlerClass)) { guiId = classGuiIdEntry.getValue(); break; } @@ -42,6 +41,6 @@ public class GUIIDRegistry { } public static MUGuild getGuiId(final int id) { - return idMap.get(Integer.valueOf(id)); + return idMap.get(id); } } diff --git a/src/main/java/gtPlusPlus/core/gui/beta/GUITypes.java b/src/main/java/gtPlusPlus/core/gui/beta/GUITypes.java index 124e2fc276..40d0442b59 100644 --- a/src/main/java/gtPlusPlus/core/gui/beta/GUITypes.java +++ b/src/main/java/gtPlusPlus/core/gui/beta/GUITypes.java @@ -6,5 +6,5 @@ public enum GUITypes { Tile, Entity; - private GUITypes() {} + GUITypes() {} } diff --git a/src/main/java/gtPlusPlus/core/gui/machine/GUIPestKiller.java b/src/main/java/gtPlusPlus/core/gui/machine/GUIPestKiller.java index 0f3e1fc832..e42f6a9da9 100644 --- a/src/main/java/gtPlusPlus/core/gui/machine/GUIPestKiller.java +++ b/src/main/java/gtPlusPlus/core/gui/machine/GUIPestKiller.java @@ -37,7 +37,7 @@ public class GUIPestKiller extends GuiContainer { @Override protected void drawGuiContainerForegroundLayer(final int i, final int j) { if (mTileKiller != null) { - this.fontRendererObj.drawString(I18n.format(mTileKiller.getInventoryName(), new Object[0]), 4, 6, 4210752); + this.fontRendererObj.drawString(I18n.format(mTileKiller.getInventoryName()), 4, 6, 4210752); drawFluidTank(mTileKiller.getTank(), 134, 35); } } @@ -77,7 +77,7 @@ public class GUIPestKiller extends GuiContainer { if (aTank != null && aTank.getFluidAmount() > 0) { aPercentage = MathUtils.findPercentage(aTank.getFluidAmount(), aTank.getCapacity()); // Logger.INFO("Percent = "+aPercentage); - aFrameHeight = (int) (aPercentage / aDivisor); + // aFrameHeight = (int) (aPercentage / aDivisor); // Logger.INFO("Frame Height = "+aFrameHeight); } this.fontRendererObj.drawString("Tier: 0", 4, 18, 4210752); @@ -99,7 +99,7 @@ public class GUIPestKiller extends GuiContainer { this.fontRendererObj.drawString("Range: 5x5", 4, 30, 4210752); this.fontRendererObj.drawString("Poison: ", 4, 42, 4210752); this.fontRendererObj.drawString( - "" + aTile.getTank() + aTile.getTank() .getFluid() .getLocalizedName(), 4, @@ -127,7 +127,7 @@ public class GUIPestKiller extends GuiContainer { this.fontRendererObj.drawString("Range: 9x9", 4, 30, 4210752); this.fontRendererObj.drawString("Poison: ", 4, 42, 4210752); this.fontRendererObj.drawString( - "" + aTile.getTank() + aTile.getTank() .getFluid() .getLocalizedName(), 4, diff --git a/src/main/java/gtPlusPlus/core/gui/machine/GUIVolumetricFlaskSetter.java b/src/main/java/gtPlusPlus/core/gui/machine/GUIVolumetricFlaskSetter.java index 3e1a735f7e..8a57b3987f 100644 --- a/src/main/java/gtPlusPlus/core/gui/machine/GUIVolumetricFlaskSetter.java +++ b/src/main/java/gtPlusPlus/core/gui/machine/GUIVolumetricFlaskSetter.java @@ -24,10 +24,10 @@ public class GUIVolumetricFlaskSetter extends GuiContainer { private static final ResourceLocation mGuiTextures = new ResourceLocation( GTPlusPlus.ID, "textures/gui/VolumetricFlaskSetter.png"); - private ContainerVolumetricFlaskSetter mContainer; + private final ContainerVolumetricFlaskSetter mContainer; private boolean mIsOpen = false; private GuiValueField mText; - private TileEntityVolumetricFlaskSetter mTile; + private final TileEntityVolumetricFlaskSetter mTile; public GUIVolumetricFlaskSetter(ContainerVolumetricFlaskSetter aContainer) { super(aContainer); @@ -48,16 +48,16 @@ public class GUIVolumetricFlaskSetter extends GuiContainer { protected void drawGuiContainerForegroundLayer(final int i, final int j) { super.drawGuiContainerForegroundLayer(i, j); this.mText.drawTextBox(); - this.fontRendererObj.drawString(I18n.format("container.VolumetricFlaskSetter", new Object[0]), 4, 3, 4210752); + this.fontRendererObj.drawString(I18n.format("container.VolumetricFlaskSetter"), 4, 3, 4210752); int aYVal = 49; - this.fontRendererObj.drawString(I18n.format("0 = 16l", new Object[0]), 8, aYVal, 4210752); - this.fontRendererObj.drawString(I18n.format("4 = 576l", new Object[0]), 64, aYVal, 4210752); - this.fontRendererObj.drawString(I18n.format("1 = 36l", new Object[0]), 8, aYVal += 8, 4210752); - this.fontRendererObj.drawString(I18n.format("5 = 720l", new Object[0]), 64, aYVal, 4210752); - this.fontRendererObj.drawString(I18n.format("2 = 144l", new Object[0]), 8, aYVal += 8, 4210752); - this.fontRendererObj.drawString(I18n.format("6 = 864l", new Object[0]), 64, aYVal, 4210752); - this.fontRendererObj.drawString(I18n.format("3 = 432l", new Object[0]), 8, aYVal += 8, 4210752); - this.fontRendererObj.drawString(I18n.format("-> = Custom", new Object[0]), 59, aYVal, 4210752); + this.fontRendererObj.drawString(I18n.format("0 = 16l"), 8, aYVal, 4210752); + this.fontRendererObj.drawString(I18n.format("4 = 576l"), 64, aYVal, 4210752); + this.fontRendererObj.drawString(I18n.format("1 = 36l"), 8, aYVal += 8, 4210752); + this.fontRendererObj.drawString(I18n.format("5 = 720l"), 64, aYVal, 4210752); + this.fontRendererObj.drawString(I18n.format("2 = 144l"), 8, aYVal += 8, 4210752); + this.fontRendererObj.drawString(I18n.format("6 = 864l"), 64, aYVal, 4210752); + this.fontRendererObj.drawString(I18n.format("3 = 432l"), 8, aYVal += 8, 4210752); + this.fontRendererObj.drawString(I18n.format("-> = Custom"), 59, aYVal, 4210752); } @Override @@ -127,7 +127,7 @@ public class GUIVolumetricFlaskSetter extends GuiContainer { } else if (par2 == Keyboard.KEY_BACK) { log("Pressed Backspace."); String aCurrentText = getText(); - if (aCurrentText.length() > 0) { + if (!aCurrentText.isEmpty()) { this.mText.setText(aCurrentText.substring(0, aCurrentText.length() - 1)); if (getText().length() <= 0) { setText(0); @@ -137,14 +137,8 @@ public class GUIVolumetricFlaskSetter extends GuiContainer { } else { if (isNumber(par2) || isNumber(par1)) { log("Pressed number."); - if (this.mText.getText() - .equals("0")) { - this.mText.textboxKeyTyped(par1, par2); - sendUpdateToServer(); - } else { - this.mText.textboxKeyTyped(par1, par2); - sendUpdateToServer(); - } + this.mText.textboxKeyTyped(par1, par2); + sendUpdateToServer(); } else { log("Pressed unused key."); super.keyTyped(par1, par2); @@ -191,7 +185,7 @@ public class GUIVolumetricFlaskSetter extends GuiContainer { } public void sendUpdateToServer() { - if (getText().length() > 0) { + if (!getText().isEmpty()) { PacketHandler.sendToServer(new PacketVolumetricFlaskGui(mTile, parse(getText()))); } } @@ -211,13 +205,11 @@ public class GUIVolumetricFlaskSetter extends GuiContainer { this.mText.updateCursorCounter(); // Check TextBox Value is correct - if (getText().length() > 0) { + if (!getText().isEmpty()) { int aCustomValue = parse(getText()); - int aTileValue = ((ContainerVolumetricFlaskSetter) mContainer).mCustomValue; - if (mContainer != null) { - if (aTileValue != aCustomValue) { - setText(aTileValue); - } + int aTileValue = mContainer.mCustomValue; + if (aTileValue != aCustomValue) { + setText(aTileValue); } } } diff --git a/src/main/java/gtPlusPlus/core/handler/Recipes/DecayableRecipe.java b/src/main/java/gtPlusPlus/core/handler/Recipes/DecayableRecipe.java index 36e3b6edd8..7004b3517c 100644 --- a/src/main/java/gtPlusPlus/core/handler/Recipes/DecayableRecipe.java +++ b/src/main/java/gtPlusPlus/core/handler/Recipes/DecayableRecipe.java @@ -25,10 +25,8 @@ public class DecayableRecipe { @Override public boolean equals(Object o) { if (o instanceof DecayableRecipe i) { - if (i.mTime == this.mTime && GTUtility.areStacksEqual(mInput, i.mInput) - && GTUtility.areStacksEqual(mOutput, i.mOutput)) { - return true; - } + return i.mTime == this.mTime && GTUtility.areStacksEqual(mInput, i.mInput) + && GTUtility.areStacksEqual(mOutput, i.mOutput); } return false; } diff --git a/src/main/java/gtPlusPlus/core/handler/events/EnderDragonDeathHandler.java b/src/main/java/gtPlusPlus/core/handler/events/EnderDragonDeathHandler.java index 51780fd030..e6133e22db 100644 --- a/src/main/java/gtPlusPlus/core/handler/events/EnderDragonDeathHandler.java +++ b/src/main/java/gtPlusPlus/core/handler/events/EnderDragonDeathHandler.java @@ -35,7 +35,7 @@ public class EnderDragonDeathHandler implements IMobExtraInfoProvider { event.entityLiving.entityDropItem( MaterialsElements.STANDALONE.DRAGON_METAL.getNugget(aAmount), MathUtils.randFloat(0, 1)); - aCountTotal = +aAmount; + aCountTotal = aAmount; } } else if (mChaoseDragonClass != null && mChaoseDragonClass.isInstance(event.entityLiving)) { for (int y = 0; y < MathUtils.randInt(100, 200); y++) { @@ -43,7 +43,7 @@ public class EnderDragonDeathHandler implements IMobExtraInfoProvider { event.entityLiving.entityDropItem( MaterialsElements.STANDALONE.DRAGON_METAL.getIngot(aAmount), MathUtils.randFloat(0, 1)); - aCountTotal = +aAmount; + aCountTotal = aAmount; } } else if (event.entityLiving instanceof EntityDragon) { for (int y = 0; y < MathUtils.randInt(25, 50); y++) { @@ -51,7 +51,7 @@ public class EnderDragonDeathHandler implements IMobExtraInfoProvider { event.entityLiving.entityDropItem( MaterialsElements.STANDALONE.DRAGON_METAL.getNugget(aAmount), MathUtils.randFloat(0, 1)); - aCountTotal = +aAmount; + aCountTotal = aAmount; } } if (aCountTotal > 0) { diff --git a/src/main/java/gtPlusPlus/core/handler/events/SneakManager.java b/src/main/java/gtPlusPlus/core/handler/events/SneakManager.java index 7af831c675..258bca3bff 100644 --- a/src/main/java/gtPlusPlus/core/handler/events/SneakManager.java +++ b/src/main/java/gtPlusPlus/core/handler/events/SneakManager.java @@ -29,7 +29,7 @@ public class SneakManager { } private static String getKey(EntityPlayer aPlayer) { - return "" + aPlayer.getGameProfile() + return aPlayer.getGameProfile() .getId() .toString(); } @@ -113,14 +113,14 @@ public class SneakManager { return mIsWearingRing; } - public static enum State { + public enum State { ON(true), OFF(false); private final boolean STATE; - private State(final boolean State) { + State(final boolean State) { this.STATE = State; } diff --git a/src/main/java/gtPlusPlus/core/handler/workbench/WorkbenchCraftingHandler.java b/src/main/java/gtPlusPlus/core/handler/workbench/WorkbenchCraftingHandler.java index 1d8b2ed68d..4f0023ccb3 100644 --- a/src/main/java/gtPlusPlus/core/handler/workbench/WorkbenchCraftingHandler.java +++ b/src/main/java/gtPlusPlus/core/handler/workbench/WorkbenchCraftingHandler.java @@ -20,45 +20,39 @@ public class WorkbenchCraftingHandler { private static final WorkbenchCraftingHandler instance = new WorkbenchCraftingHandler(); private final List recipes = new ArrayList<>(); - public static final WorkbenchCraftingHandler getInstance() { + public static WorkbenchCraftingHandler getInstance() { return instance; } public WorkbenchCraftingHandler() { // just a example recipe so you know how to add them - this.addRecipe( - new ItemStack(Blocks.iron_block), - new Object[] { "###", "###", "###", Character.valueOf('#'), Items.iron_ingot }); + this.addRecipe(new ItemStack(Blocks.iron_block), new Object[] { "###", "###", "###", '#', Items.iron_ingot }); // another example Recipe, but shapeless this.addShapelessRecipe(new ItemStack(Items.cake), new Object[] { Items.stick }); } - void addRecipe(final ItemStack par1ItemStack, final Object par2ArrayOfObj[]) { - String s = ""; + void addRecipe(final ItemStack par1ItemStack, final Object[] par2ArrayOfObj) { + StringBuilder s = new StringBuilder(); int i = 0; int j = 0; int k = 0; if (par2ArrayOfObj[i] instanceof String[]) { - final String as[] = (String[]) par2ArrayOfObj[i++]; + final String[] as = (String[]) par2ArrayOfObj[i++]; for (final String s2 : as) { k++; j = s2.length(); - s = (new StringBuilder()).append(s) - .append(s2) - .toString(); + s.append(s2); } } else { while (par2ArrayOfObj[i] instanceof String) { final String s1 = (String) par2ArrayOfObj[i++]; k++; j = s1.length(); - s = (new StringBuilder()).append(s) - .append(s1) - .toString(); + s.append(s1); } } @@ -79,13 +73,13 @@ public class WorkbenchCraftingHandler { hashmap.put(character, itemstack); } - final ItemStack aitemstack[] = new ItemStack[j * k]; + final ItemStack[] aitemstack = new ItemStack[j * k]; for (int i1 = 0; i1 < (j * k); i1++) { final char c = s.charAt(i1); - if (hashmap.containsKey(Character.valueOf(c))) { - aitemstack[i1] = hashmap.get(Character.valueOf(c)) + if (hashmap.containsKey(c)) { + aitemstack[i1] = hashmap.get(c) .copy(); } else { aitemstack[i1] = null; @@ -95,12 +89,10 @@ public class WorkbenchCraftingHandler { this.recipes.add(new ShapedRecipes(j, k, aitemstack, par1ItemStack)); } - public void addShapelessRecipe(final ItemStack par1ItemStack, final Object par2ArrayOfObj[]) { + public void addShapelessRecipe(final ItemStack par1ItemStack, final Object[] par2ArrayOfObj) { final ArrayList arraylist = new ArrayList<>(); - final Object aobj[] = par2ArrayOfObj; - final int i = aobj.length; - for (final Object obj : aobj) { + for (final Object obj : par2ArrayOfObj) { if (obj instanceof ItemStack) { arraylist.add(((ItemStack) obj).copy()); continue; diff --git a/src/main/java/gtPlusPlus/core/interfaces/IGuiManager.java b/src/main/java/gtPlusPlus/core/interfaces/IGuiManager.java index 599ed6b59b..7ac453fc49 100644 --- a/src/main/java/gtPlusPlus/core/interfaces/IGuiManager.java +++ b/src/main/java/gtPlusPlus/core/interfaces/IGuiManager.java @@ -4,13 +4,13 @@ import net.minecraft.entity.player.EntityPlayer; import net.minecraft.util.ChunkCoordinates; import net.minecraft.world.World; -public abstract interface IGuiManager extends IGuiManagerMiscUtils { +public interface IGuiManager extends IGuiManagerMiscUtils { - public abstract ChunkCoordinates getCoordinates(); + ChunkCoordinates getCoordinates(); - public abstract World getWorld(); + World getWorld(); - public abstract Object getGui(EntityPlayer paramEntityPlayer, int paramInt); + Object getGui(EntityPlayer paramEntityPlayer, int paramInt); - public abstract Object getContainer(EntityPlayer paramEntityPlayer, int paramInt); + Object getContainer(EntityPlayer paramEntityPlayer, int paramInt); } diff --git a/src/main/java/gtPlusPlus/core/interfaces/IGuiManagerMiscUtils.java b/src/main/java/gtPlusPlus/core/interfaces/IGuiManagerMiscUtils.java index f7f1194b9a..634a71388b 100644 --- a/src/main/java/gtPlusPlus/core/interfaces/IGuiManagerMiscUtils.java +++ b/src/main/java/gtPlusPlus/core/interfaces/IGuiManagerMiscUtils.java @@ -1,4 +1,4 @@ package gtPlusPlus.core.interfaces; -public abstract interface IGuiManagerMiscUtils { +public interface IGuiManagerMiscUtils { } diff --git a/src/main/java/gtPlusPlus/core/interfaces/IItemBlueprint.java b/src/main/java/gtPlusPlus/core/interfaces/IItemBlueprint.java index b0fa752f3f..df205891ac 100644 --- a/src/main/java/gtPlusPlus/core/interfaces/IItemBlueprint.java +++ b/src/main/java/gtPlusPlus/core/interfaces/IItemBlueprint.java @@ -8,45 +8,45 @@ public interface IItemBlueprint { /** * The inventory size for the blueprint~ */ - public int INV_SIZE = 9; + int INV_SIZE = 9; /** * Meta Compatible function to allow meta items to be blueprints - * + * * @param stack yourMetaItem * @return true if it is a Blueprint */ - public boolean isBlueprint(ItemStack stack); + boolean isBlueprint(ItemStack stack); /** * Sets the blueprint for this itemstack. - * + * * @param stack yourMetaItem * @return true if blueprint is set successfully */ - public boolean setBlueprint(ItemStack stack, IInventory craftingTable, ItemStack output); + boolean setBlueprint(ItemStack stack, IInventory craftingTable, ItemStack output); /** * Sets the name of the recipe/blueprint - * + * * @param String Blueprint Name * @return N/A */ - public void setBlueprintName(ItemStack stack, String name); + void setBlueprintName(ItemStack stack, String name); /** * Does this itemstack hold a blueprint? - * + * * @param stack yourMetaItem * @return true if is holding a Blueprint */ - public boolean hasBlueprint(ItemStack stack); + boolean hasBlueprint(ItemStack stack); /** * Gets the recipe held by the item - * + * * @param stack yourMetaItem * @return the blueprints contents */ - public ItemStack[] getBlueprint(ItemStack stack); + ItemStack[] getBlueprint(ItemStack stack); } diff --git a/src/main/java/gtPlusPlus/core/inventories/InventoryCircuitProgrammer.java b/src/main/java/gtPlusPlus/core/inventories/InventoryCircuitProgrammer.java index 180df60e81..ea65338559 100644 --- a/src/main/java/gtPlusPlus/core/inventories/InventoryCircuitProgrammer.java +++ b/src/main/java/gtPlusPlus/core/inventories/InventoryCircuitProgrammer.java @@ -104,7 +104,7 @@ public class InventoryCircuitProgrammer implements IInventory { // 1.7.2+ renamed to hasCustomInventoryName @Override public boolean hasCustomInventoryName() { - return this.name.length() > 0; + return !this.name.isEmpty(); } @Override diff --git a/src/main/java/gtPlusPlus/core/inventories/InventoryDecayablesChest.java b/src/main/java/gtPlusPlus/core/inventories/InventoryDecayablesChest.java index 6466996cc9..6f6859c536 100644 --- a/src/main/java/gtPlusPlus/core/inventories/InventoryDecayablesChest.java +++ b/src/main/java/gtPlusPlus/core/inventories/InventoryDecayablesChest.java @@ -102,7 +102,7 @@ public class InventoryDecayablesChest implements IInventory { // 1.7.2+ renamed to hasCustomInventoryName @Override public boolean hasCustomInventoryName() { - return this.name.length() > 0; + return !this.name.isEmpty(); } @Override diff --git a/src/main/java/gtPlusPlus/core/inventories/InventoryEggBox.java b/src/main/java/gtPlusPlus/core/inventories/InventoryEggBox.java index 545988347e..7407115614 100644 --- a/src/main/java/gtPlusPlus/core/inventories/InventoryEggBox.java +++ b/src/main/java/gtPlusPlus/core/inventories/InventoryEggBox.java @@ -102,7 +102,7 @@ public class InventoryEggBox implements IInventory { // 1.7.2+ renamed to hasCustomInventoryName @Override public boolean hasCustomInventoryName() { - return this.name.length() > 0; + return !this.name.isEmpty(); } @Override diff --git a/src/main/java/gtPlusPlus/core/inventories/InventoryFishTrap.java b/src/main/java/gtPlusPlus/core/inventories/InventoryFishTrap.java index e4797d67bc..5d059f1883 100644 --- a/src/main/java/gtPlusPlus/core/inventories/InventoryFishTrap.java +++ b/src/main/java/gtPlusPlus/core/inventories/InventoryFishTrap.java @@ -102,7 +102,7 @@ public class InventoryFishTrap implements IInventory { // 1.7.2+ renamed to hasCustomInventoryName @Override public boolean hasCustomInventoryName() { - return this.name.length() > 0; + return !this.name.isEmpty(); } @Override diff --git a/src/main/java/gtPlusPlus/core/inventories/InventoryPestKiller.java b/src/main/java/gtPlusPlus/core/inventories/InventoryPestKiller.java index b508de1d6d..87d6dd8f49 100644 --- a/src/main/java/gtPlusPlus/core/inventories/InventoryPestKiller.java +++ b/src/main/java/gtPlusPlus/core/inventories/InventoryPestKiller.java @@ -102,7 +102,7 @@ public class InventoryPestKiller implements IInventory { // 1.7.2+ renamed to hasCustomInventoryName @Override public boolean hasCustomInventoryName() { - return this.name.length() > 0; + return !this.name.isEmpty(); } @Override diff --git a/src/main/java/gtPlusPlus/core/inventories/InventorySuperJukebox.java b/src/main/java/gtPlusPlus/core/inventories/InventorySuperJukebox.java index 89f733f5ac..d76323fea8 100644 --- a/src/main/java/gtPlusPlus/core/inventories/InventorySuperJukebox.java +++ b/src/main/java/gtPlusPlus/core/inventories/InventorySuperJukebox.java @@ -102,7 +102,7 @@ public class InventorySuperJukebox implements IInventory { // 1.7.2+ renamed to hasCustomInventoryName @Override public boolean hasCustomInventoryName() { - return this.name.length() > 0; + return !this.name.isEmpty(); } @Override diff --git a/src/main/java/gtPlusPlus/core/inventories/InventoryVolumetricFlaskSetter.java b/src/main/java/gtPlusPlus/core/inventories/InventoryVolumetricFlaskSetter.java index 0b16011181..fc5cdd1b92 100644 --- a/src/main/java/gtPlusPlus/core/inventories/InventoryVolumetricFlaskSetter.java +++ b/src/main/java/gtPlusPlus/core/inventories/InventoryVolumetricFlaskSetter.java @@ -104,7 +104,7 @@ public class InventoryVolumetricFlaskSetter implements IInventory { // 1.7.2+ renamed to hasCustomInventoryName @Override public boolean hasCustomInventoryName() { - return this.name.length() > 0; + return !this.name.isEmpty(); } @Override diff --git a/src/main/java/gtPlusPlus/core/inventories/projecttable/InventoryProjectMain.java b/src/main/java/gtPlusPlus/core/inventories/projecttable/InventoryProjectMain.java index 40e848dc3c..56e14d309f 100644 --- a/src/main/java/gtPlusPlus/core/inventories/projecttable/InventoryProjectMain.java +++ b/src/main/java/gtPlusPlus/core/inventories/projecttable/InventoryProjectMain.java @@ -118,7 +118,7 @@ public class InventoryProjectMain implements IInventory { // 1.7.2+ renamed to hasCustomInventoryName @Override public boolean hasCustomInventoryName() { - return this.name.length() > 0; + return !this.name.isEmpty(); } @Override diff --git a/src/main/java/gtPlusPlus/core/inventories/projecttable/InventoryProjectOutput.java b/src/main/java/gtPlusPlus/core/inventories/projecttable/InventoryProjectOutput.java index 77bcbd1e35..955ff4d30f 100644 --- a/src/main/java/gtPlusPlus/core/inventories/projecttable/InventoryProjectOutput.java +++ b/src/main/java/gtPlusPlus/core/inventories/projecttable/InventoryProjectOutput.java @@ -105,7 +105,7 @@ public class InventoryProjectOutput implements IInventory { // 1.7.2+ renamed to hasCustomInventoryName @Override public boolean hasCustomInventoryName() { - return this.name.length() > 0; + return !this.name.isEmpty(); } @Override diff --git a/src/main/java/gtPlusPlus/core/inventories/tradetable/InventoryTradeMain.java b/src/main/java/gtPlusPlus/core/inventories/tradetable/InventoryTradeMain.java index 24dd053e31..d8dc073d9c 100644 --- a/src/main/java/gtPlusPlus/core/inventories/tradetable/InventoryTradeMain.java +++ b/src/main/java/gtPlusPlus/core/inventories/tradetable/InventoryTradeMain.java @@ -118,7 +118,7 @@ public class InventoryTradeMain implements IInventory { // 1.7.2+ renamed to hasCustomInventoryName @Override public boolean hasCustomInventoryName() { - return this.name.length() > 0; + return !this.name.isEmpty(); } @Override diff --git a/src/main/java/gtPlusPlus/core/inventories/tradetable/InventoryTradeOutput.java b/src/main/java/gtPlusPlus/core/inventories/tradetable/InventoryTradeOutput.java index edfbe7602e..767d4ccb1f 100644 --- a/src/main/java/gtPlusPlus/core/inventories/tradetable/InventoryTradeOutput.java +++ b/src/main/java/gtPlusPlus/core/inventories/tradetable/InventoryTradeOutput.java @@ -105,7 +105,7 @@ public class InventoryTradeOutput implements IInventory { // 1.7.2+ renamed to hasCustomInventoryName @Override public boolean hasCustomInventoryName() { - return this.name.length() > 0; + return !this.name.isEmpty(); } @Override diff --git a/src/main/java/gtPlusPlus/core/item/ModItems.java b/src/main/java/gtPlusPlus/core/item/ModItems.java index a7fe8ced30..49a22bc8a8 100644 --- a/src/main/java/gtPlusPlus/core/item/ModItems.java +++ b/src/main/java/gtPlusPlus/core/item/ModItems.java @@ -582,7 +582,7 @@ public final class ModItems { // LFTR Control Circuit itemCircuitLFTR = new CoreItem( "itemCircuitLFTR", - "" + EnumChatFormatting.GREEN + "Control Circuit", + EnumChatFormatting.GREEN + "Control Circuit", AddToCreativeTab.tabMisc, 1, 0, diff --git a/src/main/java/gtPlusPlus/core/item/base/BaseItemBurnable.java b/src/main/java/gtPlusPlus/core/item/base/BaseItemBurnable.java index 3ce461ffab..f79b20ccb9 100644 --- a/src/main/java/gtPlusPlus/core/item/base/BaseItemBurnable.java +++ b/src/main/java/gtPlusPlus/core/item/base/BaseItemBurnable.java @@ -16,7 +16,7 @@ public class BaseItemBurnable extends CoreItem { super(unlocalizedName, creativeTab, stackSize, maxDmg, description); this.itemName = displayName; this.meta = meta; - if (oredictName != null && !oredictName.equals("")) { + if (oredictName != null && !oredictName.isEmpty()) { registerOrdictionary(oredictName); } registerFuel(burnTime); diff --git a/src/main/java/gtPlusPlus/core/item/base/BaseItemComponent.java b/src/main/java/gtPlusPlus/core/item/base/BaseItemComponent.java index 07744c5c2e..ba98bb25b7 100644 --- a/src/main/java/gtPlusPlus/core/item/base/BaseItemComponent.java +++ b/src/main/java/gtPlusPlus/core/item/base/BaseItemComponent.java @@ -231,9 +231,7 @@ public class BaseItemComponent extends Item { } } - if ((this.materialName != null) && (this.materialName != "") - && !this.materialName.equals("") - && (this.componentMaterial != null)) { + if (this.materialName != null && !this.materialName.isEmpty() && (this.componentMaterial != null)) { if (this.componentMaterial != null) { if (!this.componentMaterial.vChemicalFormula.contains("?")) { @@ -255,10 +253,8 @@ public class BaseItemComponent extends Item { } if (this.componentType == ComponentTypes.INGOT || this.componentType == ComponentTypes.HOTINGOT) { - if ((this.materialName != null) && (this.materialName != "") - && !this.materialName.equals("") - && this.unlocalName.toLowerCase() - .contains("hot")) { + if (this.unlocalName.toLowerCase() + .contains("hot")) { list.add( EnumChatFormatting.GRAY + "Warning: " + EnumChatFormatting.RED @@ -269,7 +265,7 @@ public class BaseItemComponent extends Item { } } else { String aChemicalFormula = Material.sChemicalFormula.get(materialName.toLowerCase()); - if (aChemicalFormula != null && aChemicalFormula.length() > 0) { + if (aChemicalFormula != null && !aChemicalFormula.isEmpty()) { list.add(Utils.sanitizeStringKeepBrackets(aChemicalFormula)); } } @@ -300,7 +296,6 @@ public class BaseItemComponent extends Item { super.addInformation(stack, aPlayer, list, bool); } - @SuppressWarnings("unchecked") @Override public void onUpdate(final ItemStack iStack, final World world, final Entity entityHolding, final int p_77663_4_, final boolean p_77663_5_) { @@ -325,10 +320,9 @@ public class BaseItemComponent extends Item { @Override @SideOnly(Side.CLIENT) public boolean requiresMultipleRenderPasses() { - return (Configuration.visual.useGregtechTextures ? true : false); + return Configuration.visual.useGregtechTextures; } - @SuppressWarnings("unchecked") @Override public int getColorFromItemStack(final ItemStack stack, final int renderPass) { @@ -402,7 +396,7 @@ public class BaseItemComponent extends Item { } } - public static enum ComponentTypes { + public enum ComponentTypes { DUST("Dust", "@ Dust", "dust", OrePrefixes.dust), DUSTSMALL("DustSmall", "Small Pile of @ Dust", "dustSmall", OrePrefixes.dustSmall), @@ -434,7 +428,7 @@ public class BaseItemComponent extends Item { private final String OREDICT_NAME; private final OrePrefixes a_GT_EQUAL; - private ComponentTypes(final String LocalName, final String DisplayName, final String OreDictName, + ComponentTypes(final String LocalName, final String DisplayName, final String OreDictName, final OrePrefixes aPrefix) { this.COMPONENT_NAME = LocalName; this.DISPLAY_NAME = DisplayName; diff --git a/src/main/java/gtPlusPlus/core/item/base/BaseItemDamageable.java b/src/main/java/gtPlusPlus/core/item/base/BaseItemDamageable.java index 9a4109a1de..fb24fbf2f1 100644 --- a/src/main/java/gtPlusPlus/core/item/base/BaseItemDamageable.java +++ b/src/main/java/gtPlusPlus/core/item/base/BaseItemDamageable.java @@ -86,7 +86,7 @@ public class BaseItemDamageable extends Item { @Override public String getItemStackDisplayName(final ItemStack tItem) { - if ((this.itemName == null) || this.itemName.equals("")) { + if ((this.itemName == null) || this.itemName.isEmpty()) { return super.getItemStackDisplayName(tItem); } return this.itemName; @@ -101,7 +101,7 @@ public class BaseItemDamageable extends Item { return true; } - public static final long getItemDamage(final ItemStack aStack) { + public static long getItemDamage(final ItemStack aStack) { NBTTagCompound aNBT = aStack.getTagCompound(); if (aNBT != null) { aNBT = aNBT.getCompoundTag("Damage"); @@ -114,7 +114,7 @@ public class BaseItemDamageable