1

编辑：这似乎不仅仅是一个错误的错误。

我在以下简单算法中遇到了一个错误，该算法应该显示字符串中的字母数，沿run-length encoding.

我可以看到为什么最后一个字符没有添加到结果字符串中，但是如果我增加rangeofi我会得到index out of range明显的原因。

我想从算法设计的角度了解这里的概念问题，以及让我的代码正常工作。

我是否需要一些特殊情况代码来处理原始字符串中的最后一项？或者将当前字符与previous字符进行比较可能更有意义，尽管这在算法开始时会造成问题？

这种算法是否有一种通用方法，将当前元素与前一个/下一个元素进行比较，从而避免索引超出范围的问题？

def encode(text):
    # stores output string
    encoding = ""
    i = 0

    while i < len(text) - 1:
        # count occurrences of character at index i
        count = 1
        while text[i] == text[i + 1]:
            count += 1
            i += 1

        # append current character and its count to the result
        encoding += text[i] + str(count) 
        i += 1

    return encoding

text = "Hello World"
print(encode(text))
# Gives H1e1l2o1 1W1o1r1l1
4

2 回答 2

1

如果你保持你的策略，你将不得不检查i+1 < len(text)。这给出了类似的东西：

def encode(text): 
    L = len(text) 
    start = 0 
    encoding = '' 
    while start < L: 
        c = text[start] 
        stop = start + 1 
        while stop < L and text[stop] == c: 
            stop += 1 
        encoding += c + str(stop - start) 
        start = stop 
    return encoding

另一种做事的方法是记住每次运行的开始：

def encode2(text): 
     start = 0 
     encoding = '' 
     for i,c in enumerate(text): 
         if c != text[start]: 
             encoding += text[start] + str(i-start) 
             start = i
     if text:
         encoding += text[start] + str(len(text)-start) 
     return encoding

这使您可以仅枚举感觉更 Pythonic 的输入。

于 2019-11-04T17:28:42.520 回答
1

你是对的，while i < len(text)如果它与前一个字符不同（d在你的情况下），你应该让外部循环处理最后一个字符。

然后，您的算法在全局范围内都很好，但是在查找最后一个字符的出现时它会崩溃。此时，text[i+1]就成为非法。

要解决这个问题，只需在内部循环中添加一个安全检查：while i+1 < len(text)

def encode(text):
    # stores output string
    encoding = ""
    i = 0

    while i < len(text):
        # count occurrences of character at index i
        count = 1
        # FIX: check that we did not reach the end of the string 
        # while looking for occurences
        while i+1 < len(text) and text[i] == text[i + 1]:
            count += 1
            i += 1

        # append current character and its count to the result
        encoding += text[i] + str(count) 
        i += 1

    return encoding

text = "Hello World"
print(encode(text))
# Gives H1e1l2o1 1W1o1r1l1d1
于 2019-11-04T16:48:19.783 回答