From 65f7a1f0db1887ee49f510abd2e4732f5b403f06 Mon Sep 17 00:00:00 2001
From: Dan Holmes <dh@danielholmes.com>
Date: Thu, 3 Dec 2020 14:53:07 +0000
Subject: [PATCH] Extract logic for aged brie

---
 js-jasmine/src/shop.js | 28 +++++++++++++++-------------
 1 file changed, 15 insertions(+), 13 deletions(-)

diff --git a/js-jasmine/src/shop.js b/js-jasmine/src/shop.js
index 4a8e39ee..579a8059 100644
--- a/js-jasmine/src/shop.js
+++ b/js-jasmine/src/shop.js
@@ -13,23 +13,17 @@ class Shop {
 
   _updateItemQuality(item) {
     if (item.name == 'Backstage passes to a TAFKAL80ETC concert') {
-      this._updateQualityBackstagePass(item)
+      this._updateQualityBackstagePass(item);
+    } else if (item.name == 'Aged Brie') {
+      this._updateQualityAgedBrie(item);
     } else {
-      if (item.name != 'Aged Brie') {
-        if (item.name != 'Sulfuras, Hand of Ragnaros') {
-          item.quality = item.quality - 1;
-        }
-      } else {
-        item.quality = item.quality + 1;
+      if (item.name != 'Sulfuras, Hand of Ragnaros') {
+        item.quality = item.quality - 1;
       }
       this._updateSellIn(item)
       if (item.sellIn < 0) {
-        if (item.name != 'Aged Brie') {
-          if (item.name != 'Sulfuras, Hand of Ragnaros') {
-            item.quality = item.quality - 1;
-          }
-        } else {
-          item.quality = item.quality + 1;
+        if (item.name != 'Sulfuras, Hand of Ragnaros') {
+          item.quality = item.quality - 1;
         }
       }
     }
@@ -49,6 +43,14 @@ class Shop {
     }
   }
 
+  _updateQualityAgedBrie(item) {
+    if (item.sellIn < 0) {
+      item.quality += 2;
+    } else {
+      item.quality += 1;
+    }
+  }
+
   _updateSellIn(item) {
     if (item.name != 'Sulfuras, Hand of Ragnaros') {
       item.sellIn = item.sellIn - 1;