2048合并功能

Question

2048合并功能

来自：Coursera计算原理第1部分课程

我在我的OWLTEST上获得-2分，它使用Pylint作为样式指南。错误说明：

分支太多（17/12）功能“合并”，第7行

这是什么意思？

我非常努力地使这个程序工作。我是从头开始写的。我还想知道是否有一些技术可以使这种更清洁和/或一些最佳实践改进？我知道有可能以更好的方式写这个，因为现在我的代码看起来很乱。

  # -*- coding: utf-8 -*-
    """
    Created on Thu Sep  3 17:55:56 2015
    2048_merge_attempt1.py
    @author: Rafeh
    """
    def merge(nums):
        '''
        Takes a list as input 
        returns merged pairs with
        non zero values shifted to the left.
        [2, 0, 2, 4] should return [4, 4, 0, 0]
        [0, 0, 2, 2] should return [4, 0, 0, 0]
        [2, 2, 0, 0] should return [4, 0, 0, 0]
        [2, 2, 2, 2, 2] should return [4, 4, 2, 0, 0]
        [8, 16, 16, 8] should return [8, 32, 8, 0]
        '''
        slide = []  # Append non-zeroes first
        for num in nums:
            if num != 0:
                slide.append(num)
        for num in nums:
            if num == 0:
                slide.append(num)
        pairs = []
        for idx, num in enumerate(slide):
            if idx == len(slide)-1:
                pairs.append(num)
                if len(pairs) != len(nums):
                    pairs.append(0)
                break
            if num == slide[idx+1]:
                if num != 0:
                    pairs.append(num*2)
                    slide[idx+1] -= slide[idx+1]
                    # slide[idx+1], slide[idx+2] = slide[idx+2], slide[idx+1]
                else:
                    pairs.append(num)
            else:
                    pairs.append(num)  # Even if they don't match you must append
        slide = []  # Append non-zeroes first
        for num in pairs:
            if num != 0:
                slide.append(num)
        for num in nums:
            if num == 0:
                slide.append(num)
        for _ in range(len(nums) - len(slide)):
            if len(nums) != len(slide):
                slide.append(0)
        return slide

Answer 1

“分支”是程序控制流程中的一个位置，控件可以以多种方式“分支”，即执行不同的语句集。一个很好的例子是'if'语句，它总是有两个分支。 E.g。

a()
if e:
  b()
c()

如果忽略异常或sys.exit()调用等的可能性，则有两个分支：a(); b(); c()，如果e为真。如果a(); c();为假，则为e。

在您的情况下，有许多if语句可以替换。例如，而不是

for num in nums:
    if num != 0:
        slide.append(num)

你可以写

slide += [num in nums if num != 0]

现在，这个循环遍布整个地方。所以我建议只做一次，在顶部，然后

nonZeroNums = [num in nums if num != 0]

然后使用它。

2048合并功能

2048合并功能

来自：Coursera计算原理第1部分课程

1 个答案: