2

我一直在学习 Python 并使用 Project Euler 来修复我的一些数学技能。我遇到了问题 35的问题。我已经生成了 100 万以下的所有素数，消除了包含任何偶数的素数，现在我只是尝试用剩余的约 3k 素数进行最后一次检查。

该功能应该：

  1. 列出约 3k 个素数。
  2. 返回一个新列表，该列表由原始列表中每个项目的所有旋转列表组成。

这就是我所拥有的，以及我对每一行的理解：

def rotations(lst):
    newlist = []
    for i in lst:                                              # Take each int item in list. 
        s = [int(j) for j in str(i)]                           # Turn each item into a list of digit strings
        num = ([s[k:]+s[:-len(s)+k] for k in range(len(s))])   # Generate list of rotations of strings 
        tmplst = [] 
        for l in num:                                          # For each string rotation
            tmplst.append(int(''.join(map(str,l))))            # Turn it into an integer, add that int to tmplst
        newlist.append(tmplst)                                 # Add each tmplist to 'newlist'
    return newlist

仅输入rotations([123,456])产生：

[[123, 231, 312]]

当我期待

[[123, 231, 312],[456,564,645]]

任何可能出现问题的线索？

4

2 回答 2

8

每当有人拥有其他人（包括我自己）无法重现其奇怪行为的代码时，我立即想到：空格错误。并查看您的代码：

'    def rotations(lst):'
'            newlist = []'
'            for i in lst:                                              # Take each int item in list. '
'            \t    s = [int(j) for j in str(i)]                           # Turn each item into a list of digit strings'
'            \t    num = ([s[k:]+s[:-len(s)+k] for k in range(len(s))])   # Generate list of rotations of strings '
'            \t    tmplst = [] '
'            \t    for l in num:                                          # For each string rotation'
"                \t        tmplst.append(int(''.join(map(str,l))))            # Turn it into an integer, add that int to tmplst"
"                \t    newlist.append(tmplst)                                 # Add each tmplist to 'newlist'"
'                    return newlist'
'        '

在我看来，您可能正在混合制表符和空格。这可能会导致非常神秘的行为，因为某些行的缩进并不像您想象的那样多。运行你的程序

python -tt your_program_name.py

确认，如果是这样，切换到使用四个空格缩进（我的意思是“制表符”，它由四个空格组成，而不是\t。）

于 2013-03-05T17:43:04.887 回答
1

上一个答案中提到的空白错误解决了直接的问题——在 的第一遍结束时返回结果for i in lst:，而不是在循环完成后返回结果——但简化代码也很有意义，因此它和错误更少更容易被发现。这是一个示例（将函数重命名为 fromrotations以rotor进行区分）。

def rotor(lst):
    out = []
    for i in lst:
        s = str(i)
        out.append([int(s[k:]+s[:k]) for k in range(len(s))])
    return out

鉴于上述情况，rotor([456, 1789, 25887])返回

[[456, 564, 645],
 [1789, 7891, 8917, 9178],
 [25887, 58872, 88725, 87258, 72588]]
于 2013-03-05T18:28:51.850 回答