监控路由在线设备

2018-10-09 20:24:19 +08:00
 liuwenxu

那天自己一整天都不在线，可以 sos

3073 次点击
所在节点    分享创造
13 条回复
gstqc
2018-10-09 21:00:51 +08:00
这个代码的格式看着不能忍啊

0. 头部不声明编码格式
1. 函数前不空两行
2. 用 + 拼接字符串
3. 写文件不用 with
4. 有些等号没有前后空格
5. 函数内的注释没有缩进，并且这么长不换行
6. 函数结束没有空白行
7. 不兼容 Python 3
8. 有 bug
CEBBCAT
2018-10-09 21:47:57 +08:00
嗯，开源？
楼上太搞笑啦 哈哈哈
kslr
2018-10-10 01:56:50 +08:00
入门级代码质量
vHypnos
2018-10-10 02:09:45 +08:00
@gstqc #1 我也是用+ 拼接字符串， 压力甚大。
dlsflh
2018-10-10 02:12:32 +08:00
@vHypnos 我看的 Python 书上讲字符串也是用+拼接的啊？
ck65
2018-10-10 02:24:20 +08:00
❌ 嵌入代码
❌ 截图
✅ 给屏幕拍照
adrianXu
2018-10-10 09:27:52 +08:00
饶老师教你截图系列
E0
2018-10-10 09:55:42 +08:00
@gstqc 2，3 两条不认同
python 用+拼接字符串在变量比较少的情况下效率并不比 f-string 和 format 低

文件操作用 with 不一定比手动操作(open,close)差。
copfee
2018-10-10 11:39:44 +08:00
小试牛刀 vs 工业规范
gstqc
2018-10-10 12:04:36 +08:00
@E0 format 不会出现类型错误。当然写个简单脚本不会有什么实质上的区别
with 不是强制，不过还是推荐用 with
tyhtao1990
2018-10-10 14:10:23 +08:00
这是在开代码修正大会？
liuwenxu
2018-10-10 15:59:22 +08:00
这楼歪的，本意是分享下通过在线设备查看家里有没有人或者设备长期不在线自动 sos
Doragd
2018-10-12 18:37:42 +08:00
@liuwenxu 道理我都懂，可是屏幕拍照干嘛 23333

这是一个专为移动设备优化的页面（即为了让你能够在 Google 搜索结果里秒开这个页面），如果你希望参与 V2EX 社区的讨论，你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/496093

V2EX 是创意工作者们的社区，是一个分享自己正在做的有趣事物、交流想法，可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX