1

我是python新手，请温柔:)

我仍在复习我的 Python 技能，并希望养成用我的程序创建函数的习惯。我希望与人们联系并询问有关将程序​​程序分解为功能的任何提示/技术。我现在正在做一些事情，我想我有一个很好的方向，但希望看看是否有人有经验法则或他们可以提供的一些建议。

def main():
    print("Welcome to Invitational Chess Tournament.1st Round Pairing below:")

def openlist:     
    myfile = open('CHESS.txt','r')
    players = []

def creatlist(players):
    for line in myfile:         
        line = line.rstrip('\n')
        players.append(line)
        players.sort()

def findpairs(players):                             
    m= len(players)//2          
    for i in range(m):
        print (players[i],"&", players[i+m])

 myfile.close()  

main ()
4

2 回答 2

2

只是一些（未分类的）建议：

  • 不返回任何内容的函数称为“过程”
  • 让函数做一件事，只做一件事，没有副作用。考虑给函数一些东西，修改它并返回它。通过这种方式，您可以使它们保持简短，并且很容易重复使用它们。
  • 学习面向对象编程（oop），它允许您从事物中创建“对象”，并使编程对于新手程序员来说更加直观。作为一个非常基本的示例，您可以执行以下操作： player.move(right);
  • 与上一点相关，请考虑您的功能范围。如果一个函数仅由它的封闭类使用，请将其设为私有。
  • 记录您的功能。在每个函数的开头，创建一个小注释来解释它的作用（而不是如何！）并可能解释参数。不是每个人都喜欢这个，但是如果你曾经在一个库中分享你的代码，这是必须的。

具体的：

与您的设计保持一致，openlist可以改进。它打开myfile了，但它也创建了一个数组players[]，但也从不返回。您可以删除数组声明并添加一个 return 语句，但是您只剩下一个仅初始化变量的单行函数。这不是很有帮助，考虑删除该功能并直接使用myfile = open...。

def creatlist(players)是一个更有用的过程，但您可以通过将其定义为def createlistfromfile(file)并使其返回初始化和填充的玩家数组来使其更清晰。

def findpairs(players)很好，但尝试让它返回一个带有对的数组，而不是直接打印它。您可能需要稍后使用它。使用不同的过程打印数组。

于 2013-10-30T14:19:35.153 回答
0

您的函数打开列表什么也不做。它只创建 2 个变量，但从不返回它们。我建议您阅读一些基本的 Python 教程。它们都涵盖了函数式编程。例如 Wikibooks 有一些很好的文章：http ://en.wikibooks.org/wiki/Python_Programming

于 2013-10-30T14:20:26.660 回答