Coder-Sakura / PixiC

Pixiv插画批量下载,提供关注画师插画、收藏作品下载(单/多/动图)及API - pixiv爬虫
http://mybot.top/blog/2020/06/24/pixic-bu-shu/
MIT License
153 stars 21 forks source link

关于代码可读性 #1

Closed winfcloud closed 3 years ago

winfcloud commented 3 years ago

PixiC功能挺好的,感觉也一直在维护 不过我在看项目的时候,发现项目里,经常用到python的条件表达式

python的条件表达式可读性并不如C语言里的三元运算符好,这个看个人喜好

再就是会引入模块的时候会使用

from [module] import *

这样引入,后面代码确实可以省略一点代码,但是命名可能冲突,而且可读性和维护性都下降了, 我并不知道后面突然出现的变量到底是从哪个模块引入的,为了弄明白有必要看一眼(现代IDE优化了这个过程), 感觉可以使用别名减少代码量的同事,这样可读性也会高一点

当然,只是我个人看法,并不知道开发者你的开发思路和构建想法

Coder-Sakura commented 3 years ago

1、第一点在后续更新中会进行一些优化的。不过列表推导式(类似上面说的条件表达式)用的地方大多数都是一些简单的if-else判断。 2、第二点的话,确实是我偷懒引起的,确实没考虑到。

感谢提供宝贵的建议!

Coder-Sakura commented 3 years ago

v2.1.5已作相关处理