推荐学习书目
› Learn Python the Hard Way
Python Sites
› PyPI - Python Package Index
› http://diveintopython.org/toc/index.html
› Pocoo
值得关注的项目
› PyPy
› Celery
› Jinja2
› Read the Docs
› gevent
› pyenv
› virtualenv
› Stackless Python
› Beautiful Soup
› 结巴中文分词
› Green Unicorn
› Sentry
› Shovel
› Pyflakes
› pytest
Python 编程
› pep8 Checker
Styles
› PEP 8
› Google Python Style Guide
› Code Style from The Hitchhiker's Guide
这是一个创建于 1961 天前的主题，其中的信息可能已经有所发展或是发生改变。
新手，按照逻辑写了个循环。如下
list_a= []
old = []
new = []
test = []
...
for i in list_a:
if 'keyword' in list_a[0]:
if list_a[1] == A:
new.append[list[2]]
else:
old.append[list[2]]
else:
if list_a[1]== B and ConditionB:
test.append[list[2]]
if len(old) == 0 and len(new)>10:
dosomething()
else:
if len(new) < len(old) and len(test) ==0:
dosomething2(0)
else:
if len(test)==0:
dosomething3()
else:
if len(test) > 5:
dosomething4()
老码农同事说我 if else 写的太多，让别人不容易看懂，建议优化下代码比如用 If .. continue。 但是我想了下似乎这种有众多子条件的似乎不太适合用 if ...contiune 来做啊。 请教大家这段代码有啥优化建议么
第 1 条附言 · 2019-07-13 22:55:18 +08:00
补充下，因为程序有字数限制，所以像下面这写法，字数就会比我主楼里的多很多。。。容易超标
if 'keyword' in list_a[0] and list_a[1] == A:
elif 'keyword' in list_a[0] and list_a[1] != A:
if 'keyword' in list_a[0] and list_a[1] == A:
elif 'keyword' in list_a[0] and list_a[1] != A:
 |
1
Yvette 2019-07-13 20:44:56 +08:00  1
|
 |
2
misaka19000 2019-07-13 20:56:18 +08:00 via Android
下面那个为啥不用 elif
|
 |
3
uyhyygyug1234 2019-07-13 21:03:39 +08:00
@Yvette 这个，，，把所有的 css 都拿掉了，有 gnu 的感觉。。。。
|
 |
4
ladypxy OP @misaka19000 可以用 elseif。。。主要用 else 我个人感觉更好理解啊。。
|
 |
5
secondwtq 2019-07-13 21:13:52 +08:00 via iPad
确实是有一定优化空间的，我能想到的：
第一段的 append[list[2]] 这个逻辑是重复的，你可以把 if 判断抽出来成一个变量或者函数，然后再 append 下面那个可以单独搞一个函数，然后 if len(old) == 0 and len(new)>10: return dosomething() 这样 |
 |
6
cdwyd 2019-07-13 21:17:06 +08:00 via Android
看着不舒服，我习惯把一种情况这写在一起
比如 if 'keyword' in list_a[0] and if list_a[1] == A: pass |
 |
7
lhx2008 2019-07-13 21:17:57 +08:00
if len(old) == 0 and len(new)>10:
dosomething() return; if len(new) < len(old) and len(test) ==0: dosomething2(0) return; if len(test)==0: dosomething3() return if len(test) > 5: dosomething4() return |
 |
8
ysoserious 2019-07-13 21:48:07 +08:00 via Android
这样写，时间久了可能自己都忘了，实在不行就写点注释把逻辑注释出来就好了。同 5 楼，一般遇到这样的情况就把重复部分提取出来做 function 调用。
|
 |
9
starsriver 2019-07-13 21:51:50 +08:00 via Android
同七楼，面向未来写法
|
|
10
ladypxy OP @lhx2008 我也考虑过这种写法，但是感觉这样写效率不如我那样写啊。我的写法满足了 if 额度条件，就不会再去 elif 里做判断。而你这个写法，会把所有的 if 做一个判断啊，哪怕是在第一个 if 就满足了。
|
 |
11
wi 2019-07-13 22:10:23 +08:00
多用 return，少用 else，也许有的根本就不好优化，看你这个方法在干什么咯，看着头就大，怎么有这么多 else
|
 |
12
yumenoks 2019-07-13 22:16:35 +08:00
if 'keyword' in list_a[0]:
if list_a[1] == A: ============= 这里可不可以这个 if 'keyword' in list_a[0] and list_a[1] == A: |
 |
15
BruceLi 2019-07-13 22:30:47 +08:00
可以用 filter 和 lambda 表达式
|
|
16
ladypxy OP @yumenoks 可以。但是这么写有个问题。
``` if 'keyword' in list_a[0]: if list_a[1] == A: new.append[list[2]] else: old.append[list[2]] ``` 这段我就要写成 ``` if 'keyword' in list_a[0] and list_a[1] == A: elif 'keyword' in list_a[0] and list_a[1] != A: ``` 这样字数就变多了。。。这程序有大小限制。。。这么写容易超出字数限制。。 |
 |
17
skywatcher 2019-07-14 00:14:08 +08:00
说实话，真的很难看懂😂你可以把这串代码要做什么详细描述一下，可能大家能提供更有效、简洁的写法
|
|
18
skywatcher 2019-07-14 00:37:37 +08:00
```python
def process(): for i in range(len(list_a)): if 'keyword' in list_a[i]: if list_a[i] == A: new_.append(list_[i]) else: old.append(list_[i]) elif list_a[i]== B and ConditionB: test.append(list_[i+1]) # situation 1: some description if len(old) == 0 and len(new_)>10: do_something() return # situation 2: some description if len(new_) < len(old) and len(test) ==0: do_something2() return # situation 3: some description if len(test) != 0 and len(test) > 5: do_something4() else: do_something3() ``` |
 |
19
lynskylate 2019-07-14 00:47:10 +08:00
因为不知道你具体逻辑 只能简单的帮你优化一下了
```python list_a = [] old_list = [] new_list = [] for i in list_a: k, w, res, *_ = list_a # 使用解包，变量名再优化下可以增加可读性 if 'keyword' in k and w == A: new_list.append(res) else: old_list.append(res) old_list_length = len(old_list) new_list_length = len(new_list) if old_list_length == 0 and new_list_length > 10: do_something() return # 如果后面没有的话，直接在此处 return 减少 if else if new_list_length < old_list_length and len(test) == 0 dosomething2(2) return # 同上 if len(test) == 0: dosomething3() return # 同上 if len(test) > 5: dosomething4() ``` 如果知道你具体逻辑，这个判断应该是能优化不少的。 |
 |
20
vkhsyj 2019-07-14 17:02:54 +08:00
把这段代码拆分成两个函数看起来就会好很多了
|
 |
21
imycc 2019-07-14 23:30:09 +08:00
`list_a`跟`i`跟`keyword`应该是你简写给我们看的时候写错了，几个变量有点乱，不好说怎么优化。
单说下面那一部分用`if...elif...elif...elif`会好点，减少缩进的层级。 |
Select Language