1
thedevil7 2013-01-17 13:41:07 +08:00
昨晚手机看了代码, 今天用电脑来回复.
1. 不知道你的环境是什么, 不过从代码本身来看, 没有明显的地方指明到底用的是 Python2, 还是 Python3. 从另一个角度讲, 为了方便 unix 用户, 首行加上 #!/usr/bin/env python2/3 后可以直接执行.(由 print 发现, 原来是 Python2 啊) 2. 代码风格. 参考: PEP8 和 PyQt 库的命名方式. -- 2.1 不规则的空行. -- 2.2 命名, 特别是函数的命名. ---- 个人认为, 如果用 PyQt 的库的话, 和它保持一致使用 mixedCase 比较统一, 而不用 CamelCase. -- 2.3 "=" 左右的空格. -- 2.4 引入库的顺序, 先 os 再第三方库, 再自己的库. ---- import os,sys,ftplib,time 这样的引入一般分为多行. -- 2.5 对于 None 的比较, 用 is/is not 而不是 =/!=. line 78. 3. 既然是 Gui, 弄几个 QLineEdit 和几个 QPushButton 设置一下图片路径和 ftp 地址以及用户名密码应该不麻烦的. 4. -- textEdit=QTextEdit() -- self.textEdit=textEdit 我觉得如果只是这样, 不需要对 textEdit 进行更多的操作(setText等), 直接 self.textEdit = QTextEdit() 就好了. ps. 个人比较喜欢 Python3 的 connect 方式. self.pushButton.connect(self.buttonPushed) |