task-5-magic-strings

gildedRose/src/gilded-rose.js

@@ -5,61 +5,65 @@ export class GildedRose {
     this.items = items
   }
 
-  
-
   update_quality() {
     for (const item of this.items) {
-      if (
-        item.name != 'Aged Brie' &&
-        item.name != 'Backstage passes to a TAFKAL80ETC concert'
-      ) {
-        if (item.quality > 0) {
-          if (item.name != 'Sulfuras, Hand of Ragnaros') {
-            item.quality = item.quality - 1
-          }
-        }
-      } else {
-        if (item.quality < 50) {
-          item.quality = item.quality + 1
-          if (
-            item.name == 'Backstage passes to a TAFKAL80ETC concert'
-          ) {
-            if (item.sell_in < 11) {
-              if (item.quality < 50) {
-                item.quality = item.quality + 1
-              }
-            }
-            if (item.sell_in < 6) {
-              if (item.quality < 50) {
-                item.quality = item.quality + 1
-              }
-            }
-          }
-        }
-      }
-      if (item.name != 'Sulfuras, Hand of Ragnaros') {
-        item.sell_in = item.sell_in - 1
-      }
-      if (item.sell_in < 0) {
-        if (item.name != 'Aged Brie') {
-          if (
-            item.name != 'Backstage passes to a TAFKAL80ETC concert'
-          ) {
-            if (item.quality > 0) {
-              if (item.name != 'Sulfuras, Hand of Ragnaros') {
-                item.quality = item.quality - 1
-              }
-            }
-          } else {
-            item.quality =
-              item.quality - item.quality
-          }
-        } else {
-          if (item.quality < 50) {
-            item.quality = item.quality + 1
-          }
-        }
-      }
+      item._updateItem(item);
     }
   }
+
+  // update_quality() {
+  //   for (const item of this.items) {
+  //     if (
+  //       item.name != 'Aged Brie' &&
+  //       item.name != 'Backstage passes to a TAFKAL80ETC concert'
+  //     ) {
+  //       if (item.quality > 0) {
+  //         if (item.name != 'Sulfuras, Hand of Ragnaros') {
+  //           item.quality = item.quality - 1
+  //         }
+  //       }
+  //     } else {
+  //       if (item.quality < 50) {
+  //         item.quality = item.quality + 1
+  //         if (
+  //           item.name == 'Backstage passes to a TAFKAL80ETC concert'
+  //         ) {
+  //           if (item.sell_in < 11) {
+  //             if (item.quality < 50) {
+  //               item.quality = item.quality + 1
+  //             }
+  //           }
+  //           if (item.sell_in < 6) {
+  //             if (item.quality < 50) {
+  //               item.quality = item.quality + 1
+  //             }
+  //           }
+  //         }
+  //       }
+  //     }
+  //     if (item.name != 'Sulfuras, Hand of Ragnaros') {
+  //       item.sell_in = item.sell_in - 1
+  //     }
+  //     if (item.sell_in < 0) {
+  //       if (item.name != 'Aged Brie') {
+  //         if (
+  //           item.name != 'Backstage passes to a TAFKAL80ETC concert'
+  //         ) {
+  //           if (item.quality > 0) {
+  //             if (item.name != 'Sulfuras, Hand of Ragnaros') {
+  //               item.quality = item.quality - 1
+  //             }
+  //           }
+  //         } else {
+  //           item.quality =
+  //             item.quality - item.quality
+  //         }
+  //       } else {
+  //         if (item.quality < 50) {
+  //           item.quality = item.quality + 1
+  //         }
+  //       }
+  //     }
+  //   }
+  // }
 }

gildedRose/src/item.js

+const backStage = 'Backstage passes to a TAFKAL80ETC concert'
+const Sulfuras = 'Sulfuras, Hand of Ragnaros'
+const AgedBrie = 'Aged Brie'
 export class Item {
   name
   sell_in
@@ -12,4 +15,51 @@ export class Item {
   toString() {
     return `${this.name}, ${this.sell_in}, ${this.quality}`
   }
+  
+  _updateItem(item) {
+    if (item.name != AgedBrie && item.name != backStage) {
+      if (item.quality > 0) {
+        if (item.name != Sulfuras) {
+          item.quality = item.quality - 1
+        }
+      }
+    } else {
+      if (item.quality < 50) {
+        item.quality = item.quality + 1
+        if (
+          item.name == backStage) {
+          if (item.sell_in < 11) {
+            if (item.quality < 50) {
+              item.quality = item.quality + 1
+            }
+          }
+          if (item.sell_in < 6) {
+            if (item.quality < 50) {
+              item.quality = item.quality + 1
+            }
+          }
+        }
+      }
+    }
+  if (item.name != Sulfuras) {
+    item.sell_in = item.sell_in - 1
+  }
+  if (item.sell_in < 0) {
+    if (item.name != AgedBrie) {
+      if (item.name != backStage) {
+        if (item.quality > 0) {
+          if(item.name !=Sulfuras) {
+            item.quality = item.quality - 1
+          }
+        } else {
+          item.quality = item.quality - item.quality
+        }
+      } else {
+        if (item.quality < 50) {
+          item.quality = item.quality + 1
+        }
+      }
+    }
+  }  
+ }
 }