From fbc21a54c660952ff0cd1563d0d842e0225d696f Mon Sep 17 00:00:00 2001 From: Remus Decea Date: Fri, 10 Nov 2023 18:00:16 +0200 Subject: [PATCH] [COSMETIC] private at the end --- .../main/java/com/gildedrose/GildedRose.java | 38 +++++++++---------- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/Java/src/main/java/com/gildedrose/GildedRose.java b/Java/src/main/java/com/gildedrose/GildedRose.java index 5b20a63e..7231cb31 100644 --- a/Java/src/main/java/com/gildedrose/GildedRose.java +++ b/Java/src/main/java/com/gildedrose/GildedRose.java @@ -13,6 +13,25 @@ class GildedRose { this.items = items; } + public void updateQuality() { + Item[] agingItems = Arrays.stream(items).filter(i -> !i.name.equals(sulfuras)).toArray(Item[]::new); + Item[] bries = Arrays.stream(agingItems).filter(i -> i.name.equals(agedBrie)).toArray(Item[]::new); + Item[] backstagePasses = Arrays.stream(agingItems).filter(i -> i.name.equals(backStagePasses)).toArray(Item[]::new); + Item[] otherItems = Arrays.stream(agingItems).filter(i -> !i.name.equals(backStagePasses) && !i.name.equals(agedBrie)).toArray(Item[]::new); + for (Item item : otherItems) { + item.sellIn = item.sellIn - 1; + decreaseQualityOfNormalItems(item); + } + for (Item brie : bries) { + brie.sellIn = brie.sellIn - 1; + increaseQualityOfBries(brie); + } + for (Item item : backstagePasses) { + item.sellIn = item.sellIn - 1; + changeQualityOfPasses(item); + } + } + private int getDecreasedQuality(int oldQuality, int factor){ return Math.max(oldQuality - factor, 0); } @@ -41,23 +60,4 @@ class GildedRose { } else item.quality = getIncreasedQuality(item, 1); } - - public void updateQuality() { - Item[] agingItems = Arrays.stream(items).filter(i -> !i.name.equals(sulfuras)).toArray(Item[]::new); - Item[] bries = Arrays.stream(agingItems).filter(i -> i.name.equals(agedBrie)).toArray(Item[]::new); - Item[] backstagePasses = Arrays.stream(agingItems).filter(i -> i.name.equals(backStagePasses)).toArray(Item[]::new); - Item[] otherItems = Arrays.stream(agingItems).filter(i -> !i.name.equals(backStagePasses) && !i.name.equals(agedBrie)).toArray(Item[]::new); - for (Item item : otherItems) { - item.sellIn = item.sellIn - 1; - decreaseQualityOfNormalItems(item); - } - for (Item brie : bries) { - brie.sellIn = brie.sellIn - 1; - increaseQualityOfBries(brie); - } - for (Item item : backstagePasses) { - item.sellIn = item.sellIn - 1; - changeQualityOfPasses(item); - } - } }