Alternate flip_bit_challenge solution

This solution is clearer

Algorithm description updated for new solution.
Comments added to code
This commit is contained in:
andy.boot 2017-04-06 23:02:33 +01:00
parent 075b51cde3
commit 7dfee269f1

View File

@ -71,35 +71,25 @@
"source": [ "source": [
"## Algorithm\n", "## Algorithm\n",
"\n", "\n",
"* seen = []\n", "* Overview:\n",
"* Build a list of sequence counts\n", " * Use 2 counters. \n",
" * Look for 0's\n", " * Each counter will track the number of 1's it has seen. \n",
" * This will be 0 length if the input has trailing ones\n", " * When we see a 0 this indicates the end of the run for the long_counter and indicates the bit to flip for the short_counter. We then throw the long counter away, promote the short_counter to the long_counter and reset the short_counter\n",
" * Add sequence length to seen\n", "\n",
" * Look for 1's\n", "\n",
" * Add sequence length to seen\n", "* For each bit:\n",
"* Find the largest sequence of ones looking at seen\n", " * If we see a '1' \n",
" * Loop through seen\n", " * Increment both counters.\n",
" * On each iteration of the loop, flip what we are looking for from 0 to 1 and vice versa\n", " * If we see a '0' \n",
" * If seen[i] represents 1's, continue, we only want to process 0's\n", " * For the long_counter this indicates the end of the row of '1's. Compare our count to best (and update it if better).\n",
" * If this is our first iteration:\n", " * For the short_counter this indicates the bit we should flip from 0 to 1. Promote it to be the long_counter, it will continue to count bits as the long_counter\n",
" * max_result = seen[i+1] + 1 if seen[i] > 0\n", " * Reset the short_counter to 0\n",
" * continue\n",
" * If we are looking at leading zeroes (i == len(seen)-1):\n",
" * result = seen[i-1] + 1\n",
" * If we are looking at one zero:\n",
" * result = seen[i+1] + seen[i-1] + 1\n",
" * If we are looking at multiple zeroes:\n",
" * result = max(seen[i+1], seen[i-1]) + 1\n",
" * Update max_result based on result\n",
" \n", " \n",
"We should make a note that Python does not have a logical right shift operator built in. We can either use a positive number or implement one for a 32 bit number:\n",
" \n", " \n",
" num % 0x100000000 >> n\n",
"\n", "\n",
"Complexity:\n", "Complexity:\n",
"* Time: O(b)\n", "* Time: O(lg(n))\n",
"* Space: O(b)" "* Space: O(1)\n"
] ]
}, },
{ {
@ -118,62 +108,37 @@
"outputs": [], "outputs": [],
"source": [ "source": [
"class Bits(object):\n", "class Bits(object):\n",
"\n",
" MAX_BITS = 32\n", " MAX_BITS = 32\n",
" \n", " \n",
" def _build_seen_list(self, num):\n",
" seen = []\n",
" looking_for = 0\n",
" count = 0\n",
" for _ in range(self.MAX_BITS):\n",
" if num & 1 != looking_for:\n",
" seen.append(count)\n",
" looking_for = not looking_for\n",
" count = 0\n",
" count += 1\n",
" num >>= 1\n",
" seen.append(count)\n",
" return seen\n",
" \n",
" def flip_bit(self, num):\n", " def flip_bit(self, num):\n",
" if num is None:\n", " if num is None:\n",
" raise TypeError('num cannot be None')\n", " raise TypeError\n",
" if num == -1:\n", " count_no_bit_flip = 0\n",
" return self.MAX_BITS\n", " count_after_bit_flip = 0\n",
" if num == 0:\n", " best = 0\n",
" return 1\n", " \n",
" seen = self._build_seen_list(num)\n", " # This handles negative numbers.\n",
" max_result = 0\n", " if num < 0:\n",
" looking_for = 0\n", " num += 2**self.MAX_BITS\n",
" for index, count in enumerate(seen):\n", " \n",
" result = 0\n", " for i in range(0, 32):\n",
" # Only look for zeroes\n", " c = (num >> i) & 1\n",
" if looking_for == 1:\n", " if c == 1:\n",
" looking_for = not looking_for\n", " count_no_bit_flip += 1\n",
" continue\n", " count_after_bit_flip += 1\n",
" # First iteration, take trailing zeroes\n",
" # or trailing ones into account\n",
" if index == 0:\n",
" if count != 0:\n",
" # Trailing zeroes\n",
" try:\n",
" result = seen[index + 1] + 1\n",
" except IndexError:\n",
" result = 1\n",
" # Last iteration\n",
" elif index == len(seen) - 1:\n",
" result = 1 + seen[index - 1]\n",
" else:\n", " else:\n",
" # One zero\n", " # If the count after the bit flip > best -> save it.\n",
" if count == 1:\n", " if count_after_bit_flip > best:\n",
" result = seen[index + 1] + seen[index - 1] + 1\n", " best = count_after_bit_flip\n",
" # Multiple zeroes\n", " \n",
" else:\n", " # This counter now assumes we flipped this bit\n",
" result = max(seen[index + 1], seen[index - 1]) + 1\n", " count_after_bit_flip = count_no_bit_flip + 1\n",
" if result > max_result:\n", " \n",
" max_result = result\n", " # We start a new counter from this index.\n",
" looking_for = not looking_for\n", " count_no_bit_flip = 0\n",
" return max_result" " \n",
" return max(count_after_bit_flip, best)\n",
" "
] ]
}, },
{ {
@ -210,6 +175,9 @@
" assert_raises(TypeError, bits.flip_bit, None)\n", " assert_raises(TypeError, bits.flip_bit, None)\n",
" assert_equal(bits.flip_bit(0), 1)\n", " assert_equal(bits.flip_bit(0), 1)\n",
" assert_equal(bits.flip_bit(-1), bits.MAX_BITS)\n", " assert_equal(bits.flip_bit(-1), bits.MAX_BITS)\n",
" assert_equal(bits.flip_bit(-2), bits.MAX_BITS)\n",
" assert_equal(bits.flip_bit(-3), bits.MAX_BITS)\n",
" assert_equal(bits.flip_bit(-4), bits.MAX_BITS - 1)\n",
" num = int('00001111110111011110001111110000', base=2)\n", " num = int('00001111110111011110001111110000', base=2)\n",
" expected = 10\n", " expected = 10\n",
" assert_equal(bits.flip_bit(num), expected)\n", " assert_equal(bits.flip_bit(num), expected)\n",