mirror of
https://github.com/donnemartin/interactive-coding-challenges.git
synced 2024-03-22 13:11:13 +08:00
Reworked notebook: Added more detail to constraints and test cases. Reworked code and unit test.
This commit is contained in:
parent
6f90d597eb
commit
b32affa79e
|
@ -13,21 +13,28 @@
|
||||||
"source": [
|
"source": [
|
||||||
"## Problem: Partition a linked list around a value x, such that all nodes less than x come before all nodes greater than or equal to x.\n",
|
"## Problem: Partition a linked list around a value x, such that all nodes less than x come before all nodes greater than or equal to x.\n",
|
||||||
"\n",
|
"\n",
|
||||||
"* [Clarifying Questions](#Clarifying-Questions)\n",
|
"* [Constraints and Assumptions](#Constraints-and-Assumptions)\n",
|
||||||
"* [Test Cases](#Test-Cases)\n",
|
"* [Test Cases](#Test-Cases)\n",
|
||||||
"* [Algorithm](#Algorithm)\n",
|
"* [Algorithm](#Algorithm)\n",
|
||||||
"* [Code](#Code)"
|
"* [Code](#Code)\n",
|
||||||
|
"* [Unit Test](#Unit-Test)"
|
||||||
]
|
]
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
"cell_type": "markdown",
|
"cell_type": "markdown",
|
||||||
"metadata": {},
|
"metadata": {},
|
||||||
"source": [
|
"source": [
|
||||||
"## Clarifying Questions\n",
|
"## Constraints and Assumptions\n",
|
||||||
|
"\n",
|
||||||
|
"*Problem statements are often intentionally ambiguous. Identifying constraints and stating assumptions can help to ensure you code the intended solution.*\n",
|
||||||
"\n",
|
"\n",
|
||||||
"* Can we create additional data structures?\n",
|
"* Can we create additional data structures?\n",
|
||||||
" * Yes\n",
|
" * Yes\n",
|
||||||
"* Do you expect the function to return a new list?\n",
|
"* Do you expect the function to return a new list?\n",
|
||||||
|
" * Yes\n",
|
||||||
|
"* Can we assume the input x is valid?\n",
|
||||||
|
" * Yes\n",
|
||||||
|
"* Can we assume we already have a linked list class that can be used for this problem?\n",
|
||||||
" * Yes"
|
" * Yes"
|
||||||
]
|
]
|
||||||
},
|
},
|
||||||
|
@ -41,7 +48,6 @@
|
||||||
"* One element list\n",
|
"* One element list\n",
|
||||||
"* Left linked list is empty\n",
|
"* Left linked list is empty\n",
|
||||||
"* Right linked list is empty\n",
|
"* Right linked list is empty\n",
|
||||||
"* X invalid\n",
|
|
||||||
"* General case\n",
|
"* General case\n",
|
||||||
" * Partition = 10\n",
|
" * Partition = 10\n",
|
||||||
" * Input: 4, 3, 7, 8, 10, 1, 10, 12\n",
|
" * Input: 4, 3, 7, 8, 10, 1, 10, 12\n",
|
||||||
|
@ -54,7 +60,6 @@
|
||||||
"source": [
|
"source": [
|
||||||
"## Algorithm\n",
|
"## Algorithm\n",
|
||||||
"\n",
|
"\n",
|
||||||
"* Test error cases\n",
|
|
||||||
"* Create left and right linked lists\n",
|
"* Create left and right linked lists\n",
|
||||||
"* For each element in the list\n",
|
"* For each element in the list\n",
|
||||||
" * If elem < x, append to the left list\n",
|
" * If elem < x, append to the left list\n",
|
||||||
|
@ -75,7 +80,7 @@
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
"cell_type": "code",
|
"cell_type": "code",
|
||||||
"execution_count": null,
|
"execution_count": 1,
|
||||||
"metadata": {
|
"metadata": {
|
||||||
"collapsed": true
|
"collapsed": true
|
||||||
},
|
},
|
||||||
|
@ -86,9 +91,9 @@
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
"cell_type": "code",
|
"cell_type": "code",
|
||||||
"execution_count": null,
|
"execution_count": 2,
|
||||||
"metadata": {
|
"metadata": {
|
||||||
"collapsed": true
|
"collapsed": false
|
||||||
},
|
},
|
||||||
"outputs": [],
|
"outputs": [],
|
||||||
"source": [
|
"source": [
|
||||||
|
@ -96,11 +101,11 @@
|
||||||
" def partition(self, data):\n",
|
" def partition(self, data):\n",
|
||||||
" if self.head is None:\n",
|
" if self.head is None:\n",
|
||||||
" return\n",
|
" return\n",
|
||||||
" if type(data) != int:\n",
|
|
||||||
" raise ValueError('Invalid int argument: data')\n",
|
|
||||||
" left = MyLinkedList(None)\n",
|
" left = MyLinkedList(None)\n",
|
||||||
" right = MyLinkedList(None)\n",
|
" right = MyLinkedList(None)\n",
|
||||||
" curr = self.head\n",
|
" curr = self.head\n",
|
||||||
|
" \n",
|
||||||
|
" # Build the left and right lists\n",
|
||||||
" while curr is not None:\n",
|
" while curr is not None:\n",
|
||||||
" if curr.data < data:\n",
|
" if curr.data < data:\n",
|
||||||
" left.append(curr.data)\n",
|
" left.append(curr.data)\n",
|
||||||
|
@ -111,52 +116,87 @@
|
||||||
" if curr_left is None:\n",
|
" if curr_left is None:\n",
|
||||||
" return right\n",
|
" return right\n",
|
||||||
" else:\n",
|
" else:\n",
|
||||||
|
" # Merge the two lists\n",
|
||||||
" while curr_left.next is not None:\n",
|
" while curr_left.next is not None:\n",
|
||||||
" curr_left = curr_left.next\n",
|
" curr_left = curr_left.next\n",
|
||||||
" curr_left.next = right.head\n",
|
" curr_left.next = right.head\n",
|
||||||
" return left"
|
" return left"
|
||||||
]
|
]
|
||||||
},
|
},
|
||||||
|
{
|
||||||
|
"cell_type": "markdown",
|
||||||
|
"metadata": {},
|
||||||
|
"source": [
|
||||||
|
"## Unit Test"
|
||||||
|
]
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"cell_type": "markdown",
|
||||||
|
"metadata": {},
|
||||||
|
"source": [
|
||||||
|
"*It is important to identify and run through general and edge cases from the [Test Cases](#Test-Cases) section by hand. You generally will not be asked to write a unit test like what is shown below.*"
|
||||||
|
]
|
||||||
|
},
|
||||||
{
|
{
|
||||||
"cell_type": "code",
|
"cell_type": "code",
|
||||||
"execution_count": null,
|
"execution_count": 3,
|
||||||
"metadata": {
|
"metadata": {
|
||||||
"collapsed": false
|
"collapsed": false
|
||||||
},
|
},
|
||||||
"outputs": [],
|
"outputs": [
|
||||||
|
{
|
||||||
|
"name": "stdout",
|
||||||
|
"output_type": "stream",
|
||||||
|
"text": [
|
||||||
|
"Test: Empty list\n",
|
||||||
|
"Test: One element list, left list empty\n",
|
||||||
|
"Test: Right list is empty\n",
|
||||||
|
"Test: General case\n",
|
||||||
|
"Success: test_partition\n"
|
||||||
|
]
|
||||||
|
}
|
||||||
|
],
|
||||||
"source": [
|
"source": [
|
||||||
"print('Empty list')\n",
|
"from nose.tools import assert_equal\n",
|
||||||
"linked_list = MyLinkedList(None)\n",
|
"\n",
|
||||||
"linked_list.partition(10)\n",
|
"class Test(object):\n",
|
||||||
"linked_list.print_list()\n",
|
" def test_partition(self):\n",
|
||||||
"print('X invalid')\n",
|
" print('Test: Empty list')\n",
|
||||||
"head = Node(12)\n",
|
" linked_list = MyLinkedList(None)\n",
|
||||||
"linked_list = MyLinkedList(head)\n",
|
" linked_list.partition(10)\n",
|
||||||
"try:\n",
|
" assert_equal(linked_list.get_all_data(), [])\n",
|
||||||
" linked_list.partition('a')\n",
|
"\n",
|
||||||
"except ValueError as err:\n",
|
" print('Test: One element list, left list empty')\n",
|
||||||
" print(err)\n",
|
" linked_list = MyLinkedList(Node(5))\n",
|
||||||
"linked_list.print_list()\n",
|
" linked_list.partition(0)\n",
|
||||||
"print('One element list')\n",
|
" assert_equal(linked_list.get_all_data(), [5])\n",
|
||||||
"print('Left linked list is empty')\n",
|
"\n",
|
||||||
"linked_list.partition(1)\n",
|
" print('Test: Right list is empty')\n",
|
||||||
"linked_list.print_list()\n",
|
" linked_list = MyLinkedList(Node(5))\n",
|
||||||
"print('Right linked list is empty')\n",
|
" linked_list.partition(10)\n",
|
||||||
"linked_list.partition(20)\n",
|
" assert_equal(linked_list.get_all_data(), [5])\n",
|
||||||
"linked_list.print_list()\n",
|
"\n",
|
||||||
"print('General case')\n",
|
" print('Test: General case')\n",
|
||||||
"# Partition = 10\n",
|
" # Partition = 10\n",
|
||||||
"# Input 4, 3, 7, 8, 10, 1, 10, 12\n",
|
" # Input: 4, 3, 7, 8, 10, 1, 10, 12\n",
|
||||||
"# Output 4, 3, 7, 8, 1, 10, 10, 12\n",
|
" # Output: 4, 3, 7, 8, 1, 10, 10, 12\n",
|
||||||
"linked_list.insert_to_front(10)\n",
|
" linked_list = MyLinkedList(Node(12))\n",
|
||||||
"linked_list.insert_to_front(1)\n",
|
" linked_list.insert_to_front(10)\n",
|
||||||
"linked_list.insert_to_front(10)\n",
|
" linked_list.insert_to_front(1)\n",
|
||||||
"linked_list.insert_to_front(8)\n",
|
" linked_list.insert_to_front(10)\n",
|
||||||
"linked_list.insert_to_front(7)\n",
|
" linked_list.insert_to_front(8)\n",
|
||||||
"linked_list.insert_to_front(3)\n",
|
" linked_list.insert_to_front(7)\n",
|
||||||
"linked_list.insert_to_front(4)\n",
|
" linked_list.insert_to_front(3)\n",
|
||||||
"partitioned_list = linked_list.partition(10)\n",
|
" linked_list.insert_to_front(4)\n",
|
||||||
"partitioned_list.print_list()"
|
" partitioned_list = linked_list.partition(10)\n",
|
||||||
|
" assert_equal(partitioned_list.get_all_data(), \n",
|
||||||
|
" [4, 3, 7, 8, 1, 10, 10, 12])\n",
|
||||||
|
" \n",
|
||||||
|
" print('Success: test_partition')\n",
|
||||||
|
"\n",
|
||||||
|
"if __name__ == '__main__':\n",
|
||||||
|
" test = Test()\n",
|
||||||
|
" test.test_partition()"
|
||||||
]
|
]
|
||||||
}
|
}
|
||||||
],
|
],
|
||||||
|
|
Loading…
Reference in New Issue
Block a user