diff --git a/package-lock.json b/package-lock.json new file mode 100644 index 00000000..91962693 --- /dev/null +++ b/package-lock.json @@ -0,0 +1,6 @@ +{ + "name": "5500-Refactoring_Practice", + "lockfileVersion": 3, + "requires": true, + "packages": {} +} diff --git a/python/UML.pdf b/python/UML.pdf new file mode 100644 index 00000000..46553a3a Binary files /dev/null and b/python/UML.pdf differ diff --git a/python/gilded_rose.py b/python/gilded_rose.py index 4f21ea64..536d790c 100755 --- a/python/gilded_rose.py +++ b/python/gilded_rose.py @@ -44,3 +44,8 @@ class Item: def __repr__(self): return "%s, %s, %s" % (self.name, self.sell_in, self.quality) + + +#The original code relied on a monolithic update_quality method with deeply nested conditional logic +#I implemented the Strategy Pattern, decomposing the complex conditional logic into separate classes +#Created a new strategy class rather than risking regression bugs by modifying the existing, messy if/else block. \ No newline at end of file diff --git a/python/package-lock.json b/python/package-lock.json new file mode 100644 index 00000000..9fcf0f2b --- /dev/null +++ b/python/package-lock.json @@ -0,0 +1,6 @@ +{ + "name": "python", + "lockfileVersion": 3, + "requires": true, + "packages": {} +} diff --git a/texttests/config.gr b/texttests/config.gr index 54f8f719..83664e6b 100755 --- a/texttests/config.gr +++ b/texttests/config.gr @@ -7,8 +7,8 @@ diff_program:meld # Settings for the Python version -#executable:${TEXTTEST_HOME}/python/texttest_fixture.py -#interpreter:python +executable:${TEXTTEST_HOME}/python/texttest_fixture.py +interpreter:python # Settings for the cpp version #executable:${TEXTTEST_HOME}/cpp/cmake-build-debug/test/cpp_texttest/GildedRoseTextTests