mirror of
https://github.com/emilybache/GildedRose-Refactoring-Kata.git
synced 2026-02-15 14:31:28 +00:00
Refactor modules
This commit is contained in:
parent
125bb22b38
commit
97a10998ae
@ -1,12 +1,8 @@
|
||||
const agedBrie = {
|
||||
regex_matcher: /aged brie/,
|
||||
qualityChange: (sellIn) => {
|
||||
if (sellIn <= 0) {
|
||||
return 2;
|
||||
} else {
|
||||
return 1;
|
||||
}
|
||||
exports.regex_matcher = /aged brie/;
|
||||
exports.qualityChange = function (sellIn, quality) {
|
||||
if (sellIn <= 0) {
|
||||
return 2;
|
||||
} else {
|
||||
return 1;
|
||||
}
|
||||
}
|
||||
|
||||
module.exports = { agedBrie };
|
||||
};
|
||||
|
||||
@ -1,16 +1,12 @@
|
||||
const backstagePass = {
|
||||
regex_matcher: /backstage pass/,
|
||||
qualityChange: (sellIn, quality) => {
|
||||
if (sellIn <= 0) {
|
||||
return -quality;
|
||||
} else if (sellIn <= 5) {
|
||||
return 3;
|
||||
} else if (sellIn <= 10) {
|
||||
return 2;
|
||||
} else {
|
||||
return 1;
|
||||
}
|
||||
exports.regex_matcher = /backstage pass/
|
||||
exports.qualityChange = function (sellIn, quality) {
|
||||
if (sellIn <= 0) {
|
||||
return -quality;
|
||||
} else if (sellIn <= 5) {
|
||||
return 3;
|
||||
} else if (sellIn <= 10) {
|
||||
return 2;
|
||||
} else {
|
||||
return 1;
|
||||
}
|
||||
}
|
||||
|
||||
module.exports = { backstagePass };
|
||||
@ -3,7 +3,7 @@ const backstagePass = require('./backstage_pass_update.js')
|
||||
const agedBrie = require('./aged_brie_update.js')
|
||||
const sulfuras = require('./sulfuras_update.js')
|
||||
|
||||
const itemTypes = [backstagePass.backstagePass, agedBrie.agedBrie, sulfuras.sulfuras]
|
||||
const itemTypes = [backstagePass, agedBrie, sulfuras]
|
||||
|
||||
class Shop {
|
||||
constructor(items = []) {
|
||||
@ -34,7 +34,7 @@ class Shop {
|
||||
return item.quality += itemType.qualityChange(item.sellIn, item.quality);
|
||||
}
|
||||
}
|
||||
return item.quality += standardItem.standardItem.qualityChange(item.sellIn);
|
||||
return item.quality += standardItem.qualityChange(item.sellIn);
|
||||
}
|
||||
|
||||
_updateItemSellIn(item) {
|
||||
|
||||
@ -1,11 +1,7 @@
|
||||
const standardItem = {
|
||||
qualityChange: (sellIn) => {
|
||||
if (sellIn <= 0) {
|
||||
return -2;
|
||||
} else {
|
||||
return -1;
|
||||
}
|
||||
exports.qualityChange = function (sellIn) {
|
||||
if (sellIn <= 0) {
|
||||
return -2;
|
||||
} else {
|
||||
return -1;
|
||||
}
|
||||
}
|
||||
|
||||
module.exports = { standardItem };
|
||||
}
|
||||
@ -1,8 +1,4 @@
|
||||
const sulfuras = {
|
||||
regex_matcher: /sulfuras/,
|
||||
qualityChange: () => {
|
||||
return 0;
|
||||
}
|
||||
}
|
||||
|
||||
module.exports = { sulfuras };
|
||||
exports.regex_matcher = /sulfuras/;
|
||||
exports.qualityChange = function () {
|
||||
return 0;
|
||||
};
|
||||
Loading…
Reference in New Issue
Block a user