From c4f6bc4d2a31a4ceedb041c2fdbfa6212b86e05e Mon Sep 17 00:00:00 2001 From: "jiajun.chou" Date: Thu, 13 Aug 2020 21:26:05 +0800 Subject: [PATCH] Inverse if statement --- .../main/java/com/gildedrose/GildedRose.java | 40 +++++++++++-------- 1 file changed, 23 insertions(+), 17 deletions(-) diff --git a/Java/src/main/java/com/gildedrose/GildedRose.java b/Java/src/main/java/com/gildedrose/GildedRose.java index 2afaa306..81353091 100644 --- a/Java/src/main/java/com/gildedrose/GildedRose.java +++ b/Java/src/main/java/com/gildedrose/GildedRose.java @@ -9,13 +9,7 @@ class GildedRose { public void updateQuality() { for (int i = 0; i < items.length; i++) { - if (!items[i].name.equals("Aged Brie") && !items[i].name.equals("Backstage passes to a TAFKAL80ETC concert")) { - if (items[i].quality > 0) { - if (!items[i].name.equals("Sulfuras, Hand of Ragnaros")) { - items[i].quality = items[i].quality - 1; - } - } - } else { + if (items[i].name.equals("Aged Brie") || items[i].name.equals("Backstage passes to a TAFKAL80ETC concert")) { if (items[i].quality < 50) { items[i].quality = items[i].quality + 1; @@ -33,26 +27,38 @@ class GildedRose { } } } + } else { + if (items[i].quality > 0) { + if (items[i].name.equals("Sulfuras, Hand of Ragnaros")) { + + } else { + items[i].quality = items[i].quality - 1; + } + } } - if (!items[i].name.equals("Sulfuras, Hand of Ragnaros")) { + if (items[i].name.equals("Sulfuras, Hand of Ragnaros")) { + + } else { items[i].sellIn = items[i].sellIn - 1; } if (items[i].sellIn < 0) { - if (!items[i].name.equals("Aged Brie")) { - if (!items[i].name.equals("Backstage passes to a TAFKAL80ETC concert")) { + if (items[i].name.equals("Aged Brie")) { + if (items[i].quality < 50) { + items[i].quality = items[i].quality + 1; + } + } else { + if (items[i].name.equals("Backstage passes to a TAFKAL80ETC concert")) { + items[i].quality = items[i].quality - items[i].quality; + } else { if (items[i].quality > 0) { - if (!items[i].name.equals("Sulfuras, Hand of Ragnaros")) { + if (items[i].name.equals("Sulfuras, Hand of Ragnaros")) { + + } else { items[i].quality = items[i].quality - 1; } } - } else { - items[i].quality = items[i].quality - items[i].quality; - } - } else { - if (items[i].quality < 50) { - items[i].quality = items[i].quality + 1; } } }