我的第一个python程序：你能告诉我我做错了什么吗？

Question

我希望这个问题适合stackoverflow。 如果没有，我会马上删除这个问题。

我刚刚编写了我的第一个python程序。 我们的想法是你可以发出一个命令，并将它并行发送到几个服务器。

这仅用于个人教育目的。 该计划有效！ 我真的想在python上变得更好，因此我想问下列问题：

1. 与PHP（我以前习惯）相比，我的风格看起来很混乱。 您对样式改进有什么建议吗？
2. 我使用正确的库吗？ 我正确使用它们吗？
3. 我使用正确的数据类型吗？ 我正确使用它们吗？

我有一个很好的编程背景，但是我花了很长时间才开发出适合PHP的样式（PEAR编码标准，知道使用什么工具以及何时使用）。

来源（一个文件，92行代码）

http://code.google.com/p/floep/source/browse/trunk/floep

Answer 1

通常最好是在句子结束后的内容:在一个单独的行中（也不要在它之前添加空格）

if options.verbose:
  print ""

代替

if options.verbose : print ""

如果要迭代它，则无需检查列表的len

if len(threadlist) > 0 : 
  for server in threadlist :
    ...

是多余的，更“可读”（python足够聪明，不能遍历空列表）：

for server in threadlist:
  ...

另外一个更“'pythonistic'是使用列表的理解（但肯定是一个有争议的意见）

server = []
for i in grouplist : servers+=getServers(i)

可以缩短为

server = [getServers(i) for i in grouplist]

Answer 2

在卸载任何批评之前，首先让我祝贺你的第一个Python程序正常运行。 从一种语言转移到另一种语言可能是一件苦差事，不断摸索语法问题并通过不熟悉的库进行搜索。

最引用的风格指南是PEP-8 ，但这只是一个指南，至少有一部分被忽略......不，我的意思是认为不适用于某些特定情况，并对指南作者和贡献者给予应有的尊重： - ）。

我无法将它与PHP进行比较，但与其他Python应用程序相比，很明显您遵循其他语言的样式约定。 我并不总是同意其他开发人员说你必须做的很多事情，但随着时间的推移，我认识到为什么使用约定有助于传达应用程序正在做的事情，并将帮助其他开发人员帮助你。

---

提高异常，而不是字符串。

raise RuntimeError('Server or group ' + sectionname + ' not found in ' + configfile)

变

  引发RuntimeError（'+ configfile'中找不到'服务器或组'+ sectionname +'） 

---

在'if'或'for'结尾处的'：'之前没有空格，并且不要在同一行上放置多个语句，并且要在运算符周围放置空格。 使用对象的变量名称并坚持使用i和j来表示循环索引变量（比如我们熟练的FORTRAN前辈）：

  for group in grouplist：servers + = getServers（i） 

变为：

while len(threadlist) > 0 :

---

容器可以在不获取长度的情况下测试内容：

if command.strip():

变

  而threadlist： 

和

(options,args) = parser.parse_args()

if options.verbose : print "floep 0.1" 

command = " ".join(args)

if command.strip() == "" : parser.error('no command given')

变

options, args = parser.parse_args()
if options.verbose:
    print "floep 0.1" 

if not args:
    parser.error('no command given')
command = " ".join(args)

---

拆分元组通常不会放在语句左侧的括号中，命令逻辑有点复杂。 如果没有args那么“.join（...）将是一个空字符串：

\n （options，args）= parser.parse_args（）\n\n 如果options.verbose：print“floep 0.1” \n\n command =“”。join（args）\n\n if command.strip（）==“”：parser.error（'no command given'）\n

变

result = proc.readlines()
        strresult = ''
        for line in result : strresult+=line 
        self.result = strresult

---

一个python for循环有一个不寻常的'else'子句，如果循环遍历所有元素而没有'break'，则执行该子句：

\n     对于threadlist中的服务器：\n         foundOne =假 \n         如果不是server.isAlive（）：\n             ...略...\n             foundOne = True\n\n         如果没有找到一个：\n             time.sleep（0.010）\n

变

\n     对于threadlist中的服务器：\n         如果不是server.isAlive（）：\n             ...略...\n             打破\n     其他：\n         time.sleep（0.010）\n

---

获取行列表然后将它们重新组合在一起有点长篇大论：

\n         result = proc.readlines（）\n         strresult =''\n         对于结果中的行：strresult + = line \n         self.result = strresult\n

变

\n         self.result = proc.read（）\n

---

您的库使用很好，查看子进程模块，它更新一点。

你的数据类型没问题。

你会得到很多其他的东西:-)

Answer 3

在Python中不推荐使用字符串异常，因此这一行：

if not config.has_section(sectionname): 
    raise 'Server or group ' + sectionname + ' not found in ' + configfile

应该重做这样的事情：

if not config.has_section(sectionname):
    raise ConfigNotFoundError(
        "Server or group" + sectionname + "not found in" + configfile)

class ConfigNotFoundError(Exception):
    pass

[编辑在评论中反映了dangph的建议]

这是更多的代码行，但它对未来的升级更好。

为了便于阅读，可以这样：

parser.add_option('-q','--quiet',action="store_false", help="Display only server output", dest="verbose", default=True)

可以像这样重写：

parser.add_option('-q',
                  '--quiet',
                  action="store_false",
                  help="Display only server output", 
                  dest="verbose", 
                  default=True)

您可能更喜欢另一种分割方法调用的方法，但其想法是长行很难读取。

您还应该阅读PEP 8以了解Python风格。

Answer 4

通常，为了重复使用，我们执行以下操作，从程序中的第48行开始

def main():
    config = ConfigParser.RawConfigParser()
    etc.

if __name__ == "__main__":
    main()

这只是一个起点。

完成此操作后，您会发现main（）实际上是两部分：解析命令行界面并完成工作。 然后你想重构一下这样的东西。

def serverWork(group,...):
    servers = getServers(group)
    etc.

def main():
    config = ConfigParser.RawConfigParser()

    if command.strip() == "":
        parser.error('no command given')
    else:
        serverWork( options.group, options.etc., ... )

现在，您已将实际工作提升到此模块中的函数。 您的serverWork功能现在可以被其他程序或脚本轻松重用。