From 2d71b0e34a7b99c36bcbcd462c37a90c2029a0d4 Mon Sep 17 00:00:00 2001 From: thatsIch Date: Fri, 25 Sep 2015 23:18:27 +0200 Subject: [PATCH] prefer isEmpty over length() == 0 --- .../appeng/client/gui/implementations/GuiCraftAmount.java | 4 ++-- .../client/gui/implementations/GuiInterfaceTerminal.java | 2 +- .../appeng/client/gui/implementations/GuiLevelEmitter.java | 4 ++-- .../appeng/client/gui/implementations/GuiMEMonitorable.java | 2 +- .../java/appeng/client/gui/implementations/GuiPriority.java | 4 ++-- .../container/implementations/ContainerCraftConfirm.java | 2 +- .../container/implementations/ContainerCraftingStatus.java | 2 +- src/main/java/appeng/core/api/ApiPart.java | 2 +- .../appeng/core/features/registries/InscriberRegistry.java | 2 +- .../appeng/core/features/registries/WirelessRegistry.java | 2 +- src/main/java/appeng/recipes/GroupIngredient.java | 2 +- src/main/java/appeng/recipes/IngredientSet.java | 2 +- 12 files changed, 15 insertions(+), 15 deletions(-) diff --git a/src/main/java/appeng/client/gui/implementations/GuiCraftAmount.java b/src/main/java/appeng/client/gui/implementations/GuiCraftAmount.java index 2876b8ea..12375e8d 100644 --- a/src/main/java/appeng/client/gui/implementations/GuiCraftAmount.java +++ b/src/main/java/appeng/client/gui/implementations/GuiCraftAmount.java @@ -202,7 +202,7 @@ public class GuiCraftAmount extends AEBaseGui this.amountToCraft.setText( out ); } - if( out.length() == 0 ) + if( out.isEmpty() ) { out = "0"; } @@ -276,7 +276,7 @@ public class GuiCraftAmount extends AEBaseGui this.amountToCraft.setText( out ); } - if( out.length() == 0 ) + if( out.isEmpty() ) { out = "0"; } diff --git a/src/main/java/appeng/client/gui/implementations/GuiInterfaceTerminal.java b/src/main/java/appeng/client/gui/implementations/GuiInterfaceTerminal.java index 50c3480e..0ab6ac5c 100644 --- a/src/main/java/appeng/client/gui/implementations/GuiInterfaceTerminal.java +++ b/src/main/java/appeng/client/gui/implementations/GuiInterfaceTerminal.java @@ -190,7 +190,7 @@ public class GuiInterfaceTerminal extends AEBaseGui { if( !this.checkHotbarKeys( key ) ) { - if( character == ' ' && this.searchField.getText().length() == 0 ) + if( character == ' ' && this.searchField.getText().isEmpty() ) { return; } diff --git a/src/main/java/appeng/client/gui/implementations/GuiLevelEmitter.java b/src/main/java/appeng/client/gui/implementations/GuiLevelEmitter.java index a1eac918..c8380a0f 100644 --- a/src/main/java/appeng/client/gui/implementations/GuiLevelEmitter.java +++ b/src/main/java/appeng/client/gui/implementations/GuiLevelEmitter.java @@ -210,7 +210,7 @@ public class GuiLevelEmitter extends GuiUpgradeable this.level.setText( Out ); } - if( Out.length() == 0 ) + if( Out.isEmpty() ) { Out = "0"; } @@ -260,7 +260,7 @@ public class GuiLevelEmitter extends GuiUpgradeable this.level.setText( Out ); } - if( Out.length() == 0 ) + if( Out.isEmpty() ) { Out = "0"; } diff --git a/src/main/java/appeng/client/gui/implementations/GuiMEMonitorable.java b/src/main/java/appeng/client/gui/implementations/GuiMEMonitorable.java index 3a84c0a6..873eb138 100644 --- a/src/main/java/appeng/client/gui/implementations/GuiMEMonitorable.java +++ b/src/main/java/appeng/client/gui/implementations/GuiMEMonitorable.java @@ -461,7 +461,7 @@ public class GuiMEMonitorable extends AEBaseMEGui implements ISortSource, IConfi { if( !this.checkHotbarKeys( key ) ) { - if( character == ' ' && this.searchField.getText().length() == 0 ) + if( character == ' ' && this.searchField.getText().isEmpty() ) { return; } diff --git a/src/main/java/appeng/client/gui/implementations/GuiPriority.java b/src/main/java/appeng/client/gui/implementations/GuiPriority.java index 01af445a..c25a9092 100644 --- a/src/main/java/appeng/client/gui/implementations/GuiPriority.java +++ b/src/main/java/appeng/client/gui/implementations/GuiPriority.java @@ -220,7 +220,7 @@ public class GuiPriority extends AEBaseGui this.priority.setText( out ); } - if( out.length() == 0 ) + if( out.isEmpty() ) { out = "0"; } @@ -266,7 +266,7 @@ public class GuiPriority extends AEBaseGui this.priority.setText( out ); } - if( out.length() == 0 ) + if( out.isEmpty() ) { out = "0"; } diff --git a/src/main/java/appeng/container/implementations/ContainerCraftConfirm.java b/src/main/java/appeng/container/implementations/ContainerCraftConfirm.java index 298258d7..fea27f6b 100644 --- a/src/main/java/appeng/container/implementations/ContainerCraftConfirm.java +++ b/src/main/java/appeng/container/implementations/ContainerCraftConfirm.java @@ -179,7 +179,7 @@ public class ContainerCraftConfirm extends AEBaseContainer this.sendCPUs(); } - this.noCPU = this.cpus.size() == 0; + this.noCPU = this.cpus.isEmpty(); super.detectAndSendChanges(); diff --git a/src/main/java/appeng/container/implementations/ContainerCraftingStatus.java b/src/main/java/appeng/container/implementations/ContainerCraftingStatus.java index f4cc55d6..afac23e7 100644 --- a/src/main/java/appeng/container/implementations/ContainerCraftingStatus.java +++ b/src/main/java/appeng/container/implementations/ContainerCraftingStatus.java @@ -93,7 +93,7 @@ public class ContainerCraftingStatus extends ContainerCraftingCPU this.sendCPUs(); } - this.noCPU = this.cpus.size() == 0; + this.noCPU = this.cpus.isEmpty(); super.detectAndSendChanges(); } diff --git a/src/main/java/appeng/core/api/ApiPart.java b/src/main/java/appeng/core/api/ApiPart.java index d3619ed8..ac5a4ced 100644 --- a/src/main/java/appeng/core/api/ApiPart.java +++ b/src/main/java/appeng/core/api/ApiPart.java @@ -82,7 +82,7 @@ public class ApiPart implements IPartHelper public Class getCombinedInstance( String base ) { - if( this.desc.size() == 0 ) + if( this.desc.isEmpty() ) { try { diff --git a/src/main/java/appeng/core/features/registries/InscriberRegistry.java b/src/main/java/appeng/core/features/registries/InscriberRegistry.java index 9c985370..0b788fad 100644 --- a/src/main/java/appeng/core/features/registries/InscriberRegistry.java +++ b/src/main/java/appeng/core/features/registries/InscriberRegistry.java @@ -178,7 +178,7 @@ public final class InscriberRegistry implements IInscriberRegistry { throw new IllegalStateException( "Input must be defined." ); } - if( this.inputs.size() == 0 ) + if( this.inputs.isEmpty() ) { throw new IllegalStateException( "Input must have a size." ); } diff --git a/src/main/java/appeng/core/features/registries/WirelessRegistry.java b/src/main/java/appeng/core/features/registries/WirelessRegistry.java index 3cdd60c8..28a922c5 100644 --- a/src/main/java/appeng/core/features/registries/WirelessRegistry.java +++ b/src/main/java/appeng/core/features/registries/WirelessRegistry.java @@ -94,7 +94,7 @@ public final class WirelessRegistry implements IWirelessTermRegistry final IWirelessTermHandler handler = this.getWirelessTerminalHandler( item ); final String unparsedKey = handler.getEncryptionKey( item ); - if( unparsedKey.length() == 0 ) + if( unparsedKey.isEmpty() ) { player.addChatMessage( PlayerMessages.DeviceNotLinked.get() ); return; diff --git a/src/main/java/appeng/recipes/GroupIngredient.java b/src/main/java/appeng/recipes/GroupIngredient.java index f70abc16..8cacd8a4 100644 --- a/src/main/java/appeng/recipes/GroupIngredient.java +++ b/src/main/java/appeng/recipes/GroupIngredient.java @@ -110,7 +110,7 @@ public class GroupIngredient implements IIngredient this.isInside = false; } - if( out.size() == 0 ) + if( out.isEmpty() ) { throw new MissingIngredientError( this.toString() + " - group could not be resolved to any items." ); } diff --git a/src/main/java/appeng/recipes/IngredientSet.java b/src/main/java/appeng/recipes/IngredientSet.java index 2ac140b4..914ad371 100644 --- a/src/main/java/appeng/recipes/IngredientSet.java +++ b/src/main/java/appeng/recipes/IngredientSet.java @@ -75,7 +75,7 @@ public class IngredientSet implements IIngredient List out = new LinkedList(); out.addAll( this.items ); - if( out.size() == 0 ) + if( out.isEmpty() ) { throw new MissingIngredientError( this.toString() + " - group could not be resolved to any items." ); }