Using Python textwrap.shorten for string but with bytes width
This solution is inefficient, but it does appear to always work correctly and without ever shortening excessively. It serves as a canonical baseline for testing any efficient solutions.
It first shortens pretending that the text is an ASCII string; this can shorten insufficiently but never excessively. It then inefficiently shortens one character at a time, and no more than necessary.
import textwrap
_MIN_WIDTH = 5 # == len(textwrap.shorten(string.ascii_letters, len(string.ascii_letters) - 1)) == len('[...]')
def shorten_to_bytes_width(text: str, width: int) -> str:
# Ref: https://stackoverflow.com/a/56401167/
width = max(_MIN_WIDTH, width) # This prevents ValueError if width < _MIN_WIDTH
text = textwrap.shorten(text, width) # After this line, len(text.encode()) >= width
while len(text.encode()) > width:
text = textwrap.shorten(text, len(text) - 1)
assert len(text.encode()) <= width
return text
Credit: Thanks to Sanyash for an improvement.
Test
>>> s = '☺ Ilsa, le méchant ☺ ☺ gardien ☺'
>>> shorten_to_bytes_width(s, 27)
'☺ Ilsa, le méchant [...]'
>>> len(_.encode())
27
Testing a candidate answer
Any candidate answer can be tested by comparing its outputs with the outputs of my function for width
of range(50, -1, -1)
or at minimum range(50, 5, -1)
. Given a candidate
function, the code below implements the unit test:
import unittest
class TestShortener(unittest.TestCase):
def test_candidate(self):
text = '☺ Ilsa, le méchant ☺ ☺ gardien ☺'
for width in range(50, -1, -1):
with self.subTest(width=width):
self.assertEqual(shorten_to_bytes_width(text, width), candidate(text, width))
In theory it's enough to encode
your string, then check if it fits in the "width" constraint. If it does, then the string can be simply returned. Otherwise you can take the first "width" bytes from the encoded string (minus the bytes needed for the placeholder). To make sure it works like textwrap.shorten
one also needs to find the last whitespace in the remaining bytes and return everything before the whitespace + the placeholder. If there is no whitespace only the placeholder needs to be returned.
Given that you mentioned that you really want it byte-amount constrained the function throws an exception if the placeholder is too large. Because having a placeholder that wouldn't fit into the byte-constrained container/datastructure simply doesn't make sense and avoids a lot of edge cases that could result in inconsistent "maximum byte size" and "placeholder byte size".
The code could look like this:
def shorten_rsplit(string: str, maximum_bytes: int, normalize_spaces: bool = False, placeholder: str = "[...]") -> str:
# Make sure the placeholder satisfies the byte length requirement
encoded_placeholder = placeholder.encode().strip()
if maximum_bytes < len(encoded_placeholder):
raise ValueError('placeholder too large for max width')
# Get the UTF-8 bytes that represent the string and (optionally) normalize the spaces.
if normalize_spaces:
string = " ".join(string.split())
encoded_string = string.encode()
# If the input string is empty simply return an empty string.
if not encoded_string:
return ''
# In case we don't need to shorten anything simply return
if len(encoded_string) <= maximum_bytes:
return string
# We need to shorten the string, so we need to add the placeholder
substring = encoded_string[:maximum_bytes - len(encoded_placeholder)]
splitted = substring.rsplit(b' ', 1) # Split at last space-character
if len(splitted) == 2:
return b" ".join([splitted[0], encoded_placeholder]).decode()
else:
return '[...]'
And a simple test case:
t = '☺ Ilsa, le méchant ☺ ☺ gardien ☺'
for i in range(5, 50):
shortened = shorten_rsplit(t, i)
byte_length = len(shortened.encode())
print(byte_length <= i, i, byte_length, shortened)
Which returns
True 5 5 [...]
True 6 5 [...]
True 7 5 [...]
True 8 5 [...]
True 9 9 ☺ [...]
True 10 9 ☺ [...]
True 11 9 ☺ [...]
True 12 9 ☺ [...]
True 13 9 ☺ [...]
True 14 9 ☺ [...]
True 15 15 ☺ Ilsa, [...]
True 16 15 ☺ Ilsa, [...]
True 17 15 ☺ Ilsa, [...]
True 18 18 ☺ Ilsa, le [...]
True 19 18 ☺ Ilsa, le [...]
True 20 18 ☺ Ilsa, le [...]
True 21 18 ☺ Ilsa, le [...]
True 22 18 ☺ Ilsa, le [...]
True 23 18 ☺ Ilsa, le [...]
True 24 18 ☺ Ilsa, le [...]
True 25 18 ☺ Ilsa, le [...]
True 26 18 ☺ Ilsa, le [...]
True 27 27 ☺ Ilsa, le méchant [...]
True 28 27 ☺ Ilsa, le méchant [...]
True 29 27 ☺ Ilsa, le méchant [...]
True 30 27 ☺ Ilsa, le méchant [...]
True 31 31 ☺ Ilsa, le méchant ☺ [...]
True 32 31 ☺ Ilsa, le méchant ☺ [...]
True 33 31 ☺ Ilsa, le méchant ☺ [...]
True 34 31 ☺ Ilsa, le méchant ☺ [...]
True 35 35 ☺ Ilsa, le méchant ☺ ☺ [...]
True 36 35 ☺ Ilsa, le méchant ☺ ☺ [...]
True 37 35 ☺ Ilsa, le méchant ☺ ☺ [...]
True 38 35 ☺ Ilsa, le méchant ☺ ☺ [...]
True 39 35 ☺ Ilsa, le méchant ☺ ☺ [...]
True 40 35 ☺ Ilsa, le méchant ☺ ☺ [...]
True 41 41 ☺ Ilsa, le méchant ☺ ☺ gardien ☺
True 42 41 ☺ Ilsa, le méchant ☺ ☺ gardien ☺
True 43 41 ☺ Ilsa, le méchant ☺ ☺ gardien ☺
True 44 41 ☺ Ilsa, le méchant ☺ ☺ gardien ☺
True 45 41 ☺ Ilsa, le méchant ☺ ☺ gardien ☺
True 46 41 ☺ Ilsa, le méchant ☺ ☺ gardien ☺
True 47 41 ☺ Ilsa, le méchant ☺ ☺ gardien ☺
True 48 41 ☺ Ilsa, le méchant ☺ ☺ gardien ☺
True 49 41 ☺ Ilsa, le méchant ☺ ☺ gardien ☺
The function also has an argument for normalizing the spaces. That could be helpful in case you have different kind of whitespaces (newlines, etc.) or multiple sequential spaces. Although it will be a bit slower.
Performance
I did a quick test using simple_benchmark
(a library I wrote) to ensure it's actually faster.
For the benchmark I create a string containing random unicode characters where the (on average) one out of 8 characters is a whitespace. I also use half the length of the string as byte-width to split. Both have no special reason, it could bias the benchmarks though, that's why I wanted to mention it.
The functions used in the benchmark:
def shorten_rsplit(string: str, maximum_bytes: int, normalize_spaces: bool = False, placeholder: str = "[...]") -> str:
encoded_placeholder = placeholder.encode().strip()
if maximum_bytes < len(encoded_placeholder):
raise ValueError('placeholder too large for max width')
if normalize_spaces:
string = " ".join(string.split())
encoded_string = string.encode()
if not encoded_string:
return ''
if len(encoded_string) <= maximum_bytes:
return string
substring = encoded_string[:maximum_bytes - len(encoded_placeholder)]
splitted = substring.rsplit(b' ', 1) # Split at last space-character
if len(splitted) == 2:
return b" ".join([splitted[0], encoded_placeholder]).decode()
else:
return '[...]'
import textwrap
_MIN_WIDTH = 5
def shorten_to_bytes_width(text: str, width: int) -> str:
width = max(_MIN_WIDTH, width)
text = textwrap.shorten(text, width)
while len(text.encode()) > width:
text = textwrap.shorten(text, len(text) - 1)
assert len(text.encode()) <= width
return text
def naive(text: str, width: int) -> str:
width = max(_MIN_WIDTH, width)
text = textwrap.shorten(text, width)
if len(text.encode()) <= width:
return text
current_width = _MIN_WIDTH
index = 0
slice_index = 0
endings = ' '
while True:
new_width = current_width + len(text[index].encode())
if new_width > width:
break
if text[index] in endings:
slice_index = index
index += 1
current_width = new_width
if slice_index:
slice_index += 1 # to include found space
text = text[:slice_index] + '[...]'
assert len(text.encode()) <= width
return text
MAX_BYTES_PER_CHAR = 4
def bytes_to_char_length(input, bytes, start=0, max_length=None):
if bytes <= 0 or (max_length is not None and max_length <= 0):
return 0
if max_length is None:
max_length = min(bytes, len(input) - start)
bytes_too_much = len(input[start:start + max_length].encode()) - bytes
if bytes_too_much <= 0:
return max_length
min_length = max(max_length - bytes_too_much, bytes // MAX_BYTES_PER_CHAR)
max_length -= (bytes_too_much + MAX_BYTES_PER_CHAR - 1) // MAX_BYTES_PER_CHAR
new_start = start + min_length
bytes_left = bytes - len(input[start:new_start].encode())
return min_length + bytes_to_char_length(input, bytes_left, new_start, max_length - min_length)
def shorten_to_bytes(input, bytes, placeholder=' [...]', start=0):
if len(input[start:start + bytes + 1].encode()) <= bytes:
return input
bytes -= len(placeholder.encode())
max_chars = bytes_to_char_length(input, bytes, start)
if max_chars <= 0:
return placeholder.strip() if bytes >= 0 else ''
w = input.rfind(' ', start, start + max_chars + 1)
if w > 0:
return input[start:w] + placeholder
else:
return input[start:start + max_chars] + placeholder
# Benchmark
from simple_benchmark import benchmark, MultiArgument
import random
def get_random_unicode(length): # https://stackoverflow.com/a/21666621/5393381
get_char = chr
include_ranges = [
(0x0021, 0x0021), (0x0023, 0x0026), (0x0028, 0x007E), (0x00A1, 0x00AC), (0x00AE, 0x00FF),
(0x0100, 0x017F), (0x0180, 0x024F), (0x2C60, 0x2C7F), (0x16A0, 0x16F0), (0x0370, 0x0377),
(0x037A, 0x037E), (0x0384, 0x038A), (0x038C, 0x038C)
]
alphabet = [
get_char(code_point) for current_range in include_ranges
for code_point in range(current_range[0], current_range[1] + 1)
]
# Add more whitespaces
for _ in range(len(alphabet) // 8):
alphabet.append(' ')
return ''.join(random.choice(alphabet) for i in range(length))
r = benchmark(
[shorten_rsplit, shorten_to_bytes, shorten_to_bytes_width, naive, bytes_to_char_length],
{2**exponent: MultiArgument([get_random_unicode(2**exponent), 2**exponent // 2]) for exponent in range(4, 15)},
"string length"
)
I also did a second benchmark excluding the shorten_to_bytes_width
function so I could benchmark even longer strings:
r = benchmark(
[shorten_rsplit, shorten_to_bytes, naive],
{2**exponent: MultiArgument([get_random_unicode(2**exponent), 2**exponent // 2]) for exponent in range(4, 20)},
"string length"
)
I will propose a naive solution with a loop and checking len of encoded characters like len(text[index].encode())
. Also added timings for improvement proposed in this comment
import textwrap, timeit
_MIN_WIDTH = 5
def A_B_B(text: str, width: int) -> str:
width = max(_MIN_WIDTH, width) # This prevents ValueError if width < _MIN_WIDTH
text = textwrap.shorten(text, width) # After this line, len(text.encode()) >= width
while len(text.encode()) > width:
text = textwrap.shorten(text, len(text) - 1)
assert len(text.encode()) <= width
return text
def naive(text: str, width: int) -> str:
width = max(_MIN_WIDTH, width) # This prevents ValueError if width < TEXTWRAP_MIN_WIDTH
# textwrap.shorten does a lot of work like merging several spaces into one,
# so we will use it first
text = textwrap.shorten(text, width)
if len(text.encode()) <= width:
return text
current_width = _MIN_WIDTH # len of placeholder
index = 0
slice_index = 0 # we will do a slice on a last found space if necessary
# (to avoid slicing in a middle of a word, for example)
endings = ' ' # there also can be some more endings like \t \n
while True:
# we will use the fact that if str = str1 + str2 then
# len(str.encode()) = len(str1.encode()) + len(str2.encode())
new_width = current_width + len(text[index].encode()) # taking one more character
if new_width > width:
break
if text[index] in endings:
slice_index = index
index += 1
current_width = new_width
if slice_index: # slice_index = 0 is a special case
# when we dont go further than end of first word
slice_index += 1 # to include found space
text = text[:slice_index] + '[...]'
assert len(text.encode()) <= width
return text
s = '☺ Ilsa, le méchant ☺ ☺ gardien ☺'
n = 27
print(timeit.timeit(lambda: A_B_B(s, n), number=1000))
print(timeit.timeit(lambda: naive(s, n), number=1000))
Timings:
0.032570790994213894
0.0206866109801922