mirror of
https://github.com/donnemartin/interactive-coding-challenges.git
synced 2024-03-22 13:11:13 +08:00
Improved compression by only appending count if the count is more than 2
This commit is contained in:
parent
6276624914
commit
ef77603f9b
|
@ -18,7 +18,7 @@
|
||||||
"cell_type": "markdown",
|
"cell_type": "markdown",
|
||||||
"metadata": {},
|
"metadata": {},
|
||||||
"source": [
|
"source": [
|
||||||
"## Problem: Compress a string such that 'AAABCCDDDD' becomes 'A3B1C2D4'. Only compress the string if it saves space.\n",
|
"## Problem: Compress a string such that 'AAABCCDDDD' becomes 'A3BCCD4'. Only compress the string if it saves space.\n",
|
||||||
"\n",
|
"\n",
|
||||||
"* [Constraints](#Constraints)\n",
|
"* [Constraints](#Constraints)\n",
|
||||||
"* [Test Cases](#Test-Cases)\n",
|
"* [Test Cases](#Test-Cases)\n",
|
||||||
|
@ -51,7 +51,7 @@
|
||||||
"* None -> None\n",
|
"* None -> None\n",
|
||||||
"* '' -> ''\n",
|
"* '' -> ''\n",
|
||||||
"* 'AABBCC' -> 'AABBCC'\n",
|
"* 'AABBCC' -> 'AABBCC'\n",
|
||||||
"* 'AAABCCDDDD' -> 'A3B1C2D4'"
|
"* 'AAABCCDDDD' -> 'A3BCCD4'"
|
||||||
]
|
]
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
|
@ -69,10 +69,20 @@
|
||||||
" * For each char in string\n",
|
" * For each char in string\n",
|
||||||
" * If char is the same as last_char, increment count\n",
|
" * If char is the same as last_char, increment count\n",
|
||||||
" * Else\n",
|
" * Else\n",
|
||||||
" * Append last_char to compressed_string\n",
|
" * If the count is more than 2\n",
|
||||||
" * append count to compressed_string\n",
|
" * Append last_char to compressed_string\n",
|
||||||
" * count = 1\n",
|
" * append count to compressed_string\n",
|
||||||
" * last_char = char\n",
|
" * count = 1\n",
|
||||||
|
" * last_char = char\n",
|
||||||
|
" * If count is 1\n",
|
||||||
|
" * Append last_char to compressed_string\n",
|
||||||
|
" * count = 1\n",
|
||||||
|
" * last_char = char\n",
|
||||||
|
" * If count is 2\n",
|
||||||
|
" * Append last_char to compressed_string\n",
|
||||||
|
" * Append last_char to compressed_string once more\n",
|
||||||
|
" * count = 1\n",
|
||||||
|
" * last_char = char\n",
|
||||||
" * Append last_char to compressed_string\n",
|
" * Append last_char to compressed_string\n",
|
||||||
" * Append count to compressed_string\n",
|
" * Append count to compressed_string\n",
|
||||||
" * Return compressed_string\n",
|
" * Return compressed_string\n",
|
||||||
|
@ -91,47 +101,66 @@
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
"cell_type": "code",
|
"cell_type": "code",
|
||||||
"execution_count": 1,
|
"execution_count": 14,
|
||||||
"metadata": {
|
"metadata": {
|
||||||
"collapsed": false
|
"collapsed": false
|
||||||
},
|
},
|
||||||
"outputs": [],
|
"outputs": [],
|
||||||
"source": [
|
"source": [
|
||||||
"def compress_string(string):\n",
|
"def compress_string(string):\n",
|
||||||
" if string is None or len(string) == 0:\n",
|
"\tif string is None or len(string) == 0:\n",
|
||||||
" return string\n",
|
"\t\treturn string\n",
|
||||||
"\n",
|
"\n",
|
||||||
" # Calculate the size of the compressed string\n",
|
"\t# Calculate the size of the compressed string\n",
|
||||||
" size = 0\n",
|
"\tsize = 0\n",
|
||||||
" last_char = string[0]\n",
|
"\tlast_char = string[0]\n",
|
||||||
" for char in string:\n",
|
"\tfor char in string:\n",
|
||||||
" if char != last_char:\n",
|
"\t\tif char != last_char:\n",
|
||||||
" size += 2\n",
|
"\t\t\tsize += 2\n",
|
||||||
" last_char = char\n",
|
"\t\t\tlast_char = char\n",
|
||||||
" size += 2\n",
|
"\tsize += 2\n",
|
||||||
"\n",
|
"\n",
|
||||||
" # If the compressed string size is greater than\n",
|
"\t# If the compressed string size is greater than\n",
|
||||||
" # or equal to string size, return original string\n",
|
"\t# or equal to string size, return original string\n",
|
||||||
" if size >= len(string):\n",
|
"\tif size >= len(string):\n",
|
||||||
" return string\n",
|
"\t\treturn string\n",
|
||||||
"\n",
|
"\n",
|
||||||
" # Create compressed_string\n",
|
"\t# Create compressed_string\n",
|
||||||
" compressed_string = list()\n",
|
"\t# New objective:\n",
|
||||||
" count = 0\n",
|
"\t\t# Single characters are to be left as is\n",
|
||||||
" last_char = string[0]\n",
|
"\t\t# Double characters are to be left as are\n",
|
||||||
" for char in string:\n",
|
"\tcompressed_string = list()\n",
|
||||||
" if char == last_char:\n",
|
"\tcount = 0\n",
|
||||||
" count += 1\n",
|
"\tlast_char = string[0]\n",
|
||||||
" else:\n",
|
"\tfor char in string:\n",
|
||||||
" compressed_string.append(last_char)\n",
|
"\t\tif char == last_char:\n",
|
||||||
" compressed_string.append(str(count))\n",
|
"\t\t\tcount += 1\n",
|
||||||
" count = 1\n",
|
"\t\telse:\n",
|
||||||
" last_char = char\n",
|
"\t\t\t# Do the old compression tricks only if count exceeds two\n",
|
||||||
" compressed_string.append(last_char)\n",
|
"\t\t\tif count > 2:\n",
|
||||||
" compressed_string.append(str(count))\n",
|
"\t\t\t\tcompressed_string.append(last_char)\n",
|
||||||
|
"\t\t\t\tcompressed_string.append(str(count))\n",
|
||||||
|
"\t\t\t\tcount = 1\n",
|
||||||
|
"\t\t\t\tlast_char = char\n",
|
||||||
|
"\t\t\t# If count is either 1 or 2 (or 0?)\n",
|
||||||
|
"\t\t\t# FIXME: CAN COUNT BE 0?\n",
|
||||||
|
"\t\t\telse:\n",
|
||||||
|
"\t\t\t\t# If count is 1, leave the char as is\n",
|
||||||
|
"\t\t\t\tif count == 1:\n",
|
||||||
|
"\t\t\t\t\tcompressed_string.append(last_char)\n",
|
||||||
|
"\t\t\t\t\tcount = 1\n",
|
||||||
|
"\t\t\t\t\tlast_char = char\n",
|
||||||
|
"\t\t\t\t# If count is 2, append the character twice\n",
|
||||||
|
"\t\t\t\telse:\n",
|
||||||
|
"\t\t\t\t\tcompressed_string.append(last_char)\n",
|
||||||
|
"\t\t\t\t\tcompressed_string.append(last_char)\n",
|
||||||
|
"\t\t\t\t\tcount = 1\n",
|
||||||
|
"\t\t\t\t\tlast_char = char\n",
|
||||||
|
"\tcompressed_string.append(last_char)\n",
|
||||||
|
"\tcompressed_string.append(str(count))\n",
|
||||||
"\n",
|
"\n",
|
||||||
" # Convert the characters in the list to a string\n",
|
"\t# Convert the characters in the list to a string\n",
|
||||||
" return \"\".join(compressed_string)"
|
"\treturn \"\".join(compressed_string)"
|
||||||
]
|
]
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
|
@ -143,7 +172,7 @@
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
"cell_type": "code",
|
"cell_type": "code",
|
||||||
"execution_count": 2,
|
"execution_count": 15,
|
||||||
"metadata": {
|
"metadata": {
|
||||||
"collapsed": false
|
"collapsed": false
|
||||||
},
|
},
|
||||||
|
@ -167,7 +196,7 @@
|
||||||
" assert_equal(func(None), None)\n",
|
" assert_equal(func(None), None)\n",
|
||||||
" assert_equal(func(''), '')\n",
|
" assert_equal(func(''), '')\n",
|
||||||
" assert_equal(func('AABBCC'), 'AABBCC')\n",
|
" assert_equal(func('AABBCC'), 'AABBCC')\n",
|
||||||
" assert_equal(func('AAABCCDDDD'), 'A3B1C2D4')\n",
|
" assert_equal(func('AAABCCDDDD'), 'A3BCCD4')\n",
|
||||||
" print('Success: test_compress')\n",
|
" print('Success: test_compress')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"\n",
|
"\n",
|
||||||
|
@ -182,7 +211,7 @@
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
"cell_type": "code",
|
"cell_type": "code",
|
||||||
"execution_count": 3,
|
"execution_count": 16,
|
||||||
"metadata": {
|
"metadata": {
|
||||||
"collapsed": false
|
"collapsed": false
|
||||||
},
|
},
|
||||||
|
@ -198,25 +227,34 @@
|
||||||
"source": [
|
"source": [
|
||||||
"%run -i test_compress.py"
|
"%run -i test_compress.py"
|
||||||
]
|
]
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"cell_type": "code",
|
||||||
|
"execution_count": null,
|
||||||
|
"metadata": {
|
||||||
|
"collapsed": true
|
||||||
|
},
|
||||||
|
"outputs": [],
|
||||||
|
"source": []
|
||||||
}
|
}
|
||||||
],
|
],
|
||||||
"metadata": {
|
"metadata": {
|
||||||
"kernelspec": {
|
"kernelspec": {
|
||||||
"display_name": "Python 2",
|
"display_name": "Python 3",
|
||||||
"language": "python",
|
"language": "python",
|
||||||
"name": "python2"
|
"name": "python3"
|
||||||
},
|
},
|
||||||
"language_info": {
|
"language_info": {
|
||||||
"codemirror_mode": {
|
"codemirror_mode": {
|
||||||
"name": "ipython",
|
"name": "ipython",
|
||||||
"version": 2
|
"version": 3
|
||||||
},
|
},
|
||||||
"file_extension": ".py",
|
"file_extension": ".py",
|
||||||
"mimetype": "text/x-python",
|
"mimetype": "text/x-python",
|
||||||
"name": "python",
|
"name": "python",
|
||||||
"nbconvert_exporter": "python",
|
"nbconvert_exporter": "python",
|
||||||
"pygments_lexer": "ipython2",
|
"pygments_lexer": "ipython3",
|
||||||
"version": "2.7.10"
|
"version": "3.4.3"
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
"nbformat": 4,
|
"nbformat": 4,
|
||||||
|
|
|
@ -7,7 +7,7 @@ class TestCompress(object):
|
||||||
assert_equal(func(None), None)
|
assert_equal(func(None), None)
|
||||||
assert_equal(func(''), '')
|
assert_equal(func(''), '')
|
||||||
assert_equal(func('AABBCC'), 'AABBCC')
|
assert_equal(func('AABBCC'), 'AABBCC')
|
||||||
assert_equal(func('AAABCCDDDD'), 'A3B1C2D4')
|
assert_equal(func('AAABCCDDDD'), 'A3BCCD4')
|
||||||
print('Success: test_compress')
|
print('Success: test_compress')
|
||||||
|
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue
Block a user