Skip to content

Commit 4a7cad6

Browse files
Increase Readability
1 parent ad818ec commit 4a7cad6

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

src/main/java/me/flame/menus/menu/fillers/BorderFiller.java

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -27,27 +27,28 @@ public void fillBorders(Material borderMaterial) {
2727
final var itemStack = new ItemStack(borderMaterial);
2828

2929
for (int i = 0; i < size; i++) {
30-
if ((i < 9 || i >= size - 9) ||
31-
(i % 9 == 0 || i % 9 == 8)) menu.setItem(i, itemStack);
30+
if (isBorderSlot(i, size)) menu.setItem(i, itemStack);
3231
}
3332
}
3433

3534
public void fillBorders(@NotNull MenuItem borderMaterial) {
3635
final int size = menu.getSize();
3736
final var itemStack = borderMaterial.getItemStack();
3837

39-
for (int i = 0; i < size; i++) {
40-
if ((i < 9 || i >= size - 9) ||
41-
(i % 9 == 0 || i % 9 == 8)) menu.setItem(i, itemStack);
38+
for (int i = 1; i < size; i++) {
39+
if (isBorderSlot(i, size)) menu.setItem(i, itemStack);
4240
}
4341
}
4442

4543
public void fillBorders(ItemStack itemStack) {
4644
final int size = menu.getSize();
4745

4846
for (int i = 0; i < size; i++) {
49-
if ((i < 9 || i >= size - 9) ||
50-
(i % 9 == 0 || i % 9 == 8)) menu.setItem(i, itemStack);
47+
if (isBorderSlot(i, size)) menu.setItem(i, itemStack);
5148
}
5249
}
50+
51+
private static boolean isBorderSlot(int i, int size) {
52+
return (i < 9 || i >= size - 9) || (i % 9 == 0 || i % 9 == 8);
53+
}
5354
}

0 commit comments

Comments
 (0)