1
Jayce245 2017-07-10 19:08:04 +08:00
哈哈哈
|
2
bidongliang 2017-07-10 19:12:44 +08:00 2
不光 if/else,整个代码都光芒四射吧!
|
3
leejanfin 2017-07-10 19:13:01 +08:00 via iPhone
喜感...
另外等号两边没有空格 差评... |
4
wangdu2012 2017-07-10 19:13:39 +08:00 via iPhone
这个怎么修改好?
|
5
RiceNoodle 2017-07-10 19:14:44 +08:00
感觉在看面试题里面的风格纠错题似的
|
6
wmc 2017-07-10 19:15:24 +08:00 via Android 2
我怎么觉得把 else 后面的{收到同一行更美观。。。
|
7
domty 2017-07-10 19:15:47 +08:00
这不就是一行三目运算符的事吗?😮
|
8
shihira 2017-07-10 19:18:23 +08:00 via Android
除了风格上有点松垮之外,还好啊觉得,没有太多重的 if/else,逻辑也算清晰
|
9
minami 2017-07-10 19:19:11 +08:00 3
×代码格式化工具
√上 V2 发帖 |
10
herozzm 2017-07-10 19:25:57 +08:00
|
11
0915240 2017-07-10 19:26:32 +08:00 via iPhone
估计才写代码没多久吧
|
12
herozzm 2017-07-10 19:27:45 +08:00 6
|
13
caniuse 2017-07-10 19:28:39 +08:00
你这字体也不像是写代码的人
|
14
redtea 2017-07-10 19:33:09 +08:00
var payMethod = $("#selPayMethod").val();
if(gPayMethod == "99"){ payMethod = "99"; }else if(payMethod == undefined || payMethod == ""){ payMethod = "00"; } myPacket.data.add("payMethod", payMethod); |
15
qdwang 2017-07-10 19:33:54 +08:00
这可能是某种需求变动导致的,应对后续的需求变动增减代码也会比较方便
|
16
explon 2017-07-10 19:34:07 +08:00 via iPhone
傻逼赶紧开除
|
18
hjc4869 2017-07-10 19:42:21 +08:00
楼主给的这个 if / else 的代码格式倒是挺扭曲的……
|
19
zhoufenfens 2017-07-10 19:57:29 +08:00
我还见过这样的
if (xxx) doSth() else { doSthElse() } 想骂人.. |
20
rubyvector 2017-07-10 20:27:12 +08:00
格式怪了点。逻辑没问题吧
|
21
SourceMan 2017-07-10 20:32:14 +08:00 via iPhone
上 eslint 不就没有吐槽的机会了吗?只能说你们本身开发不规范
|
22
cxbig 2017-07-10 20:50:39 +08:00
新手的话难免,尝试沟通,介绍 IDE 或 Lint 等自动排版工具。拒不接受的再来吐槽也不迟。
代码是啰嗦了一点,但是思路 比较清晰。 - 中文注释这个不喜欢 - 表单取值要加这么多判断说明设计就有问题,尝试在源头就处理好,比方说: -- 变量 gPayMethod 直接输入到#selPayMethod -- #selPayMethod 不给空值,默认为“ 00 ” |
23
ecloud 2017-07-10 20:59:01 +08:00
这哥们儿是双子座的吧,人格比较分裂……
|
24
timothyye 2017-07-10 21:04:22 +08:00 via Android 1
这种写法是异端,得烧屎,哈哈
|
25
solobat 2017-07-10 21:23:21 +08:00
if else 的放置确实有点儿另类,倒是可以再优化下
|
26
acrisliu 2017-07-10 21:29:56 +08:00
|
27
am241 2017-07-10 22:07:42 +08:00 4
见过的最奇葩的代码,这也是我为什么总想着逃离嵌入式圈的原因
![]( ) |
28
liyuhang 2017-07-10 22:08:18 +08:00 2
![image]( https://ooo.0o0.ooo/2017/07/10/59638a3ba2e4a.png)
|
29
liyuhang 2017-07-10 22:09:54 +08:00
回复怎么使用 Markdown 语法?
|
30
breestealth 2017-07-10 22:10:26 +08:00 11
看别人代码并且像 LZ 这么评价的人,内心一定很扭曲吧
看代码就看代码,凭什么说人家内心扭曲,LZ 你自己才是内心扭曲的那个吧。 |
31
greatghoul 2017-07-10 22:13:32 +08:00
花式秀代码。
|
33
mingyun 2017-07-10 22:47:27 +08:00
代码格式化下先
|
34
dream7758522 2017-07-10 22:53:27 +08:00 via Android
我觉得格式别扭了一些,
代码挺好的。 |
35
mahone3297 2017-07-10 22:54:20 +08:00
lz,那你说应该怎么写?
大家都吐槽,那应该怎么写? 大家在吐槽 else 的位置?还好吧。。。 |
37
NicholasWangC 2017-07-10 23:02:28 +08:00
@am241 ta 是不是用 excel 写的代码。。。呸,拖的代码?
|
40
BoBoy 2017-07-10 23:50:55 +08:00 via iPhone
又见回帖的好多装逼侠。。。。
|
41
honeycomb 2017-07-11 00:08:37 +08:00
@liyuhang
不考虑格式(缩进等),原先的代码在逻辑上没什么不合适,难道这里希望把它改用逆否命题式的 guard 条件来写? |
42
pysama 2017-07-11 00:57:05 +08:00
@caniuse 这字体还行把。 我见过 坚决不用等宽字体写代码的。 而且只要是带 tab 的工具,肯定是齐刷刷二十几个 tab 全开着,找文件的时候一个个 tab 来回切着找。
我是佩服的不行。。 |
43
bk201 2017-07-11 07:02:52 +08:00 via iPhone
no bb,show me your code.除了啰嗦,也没觉得有多奇怪,引申到别人心里扭曲是什么鬼?
|
45
wangxiaoer 2017-07-11 08:37:31 +08:00
|
46
mrcn 2017-07-11 08:38:06 +08:00 via Android
不是很懂到底哪里不对
除了里面的 if 可以跟着 else 直接写以外 |
47
hzw94 2017-07-11 08:41:23 +08:00
肯定是新手,基本不注意格式规范,应该从未考虑当别人阅读的情况
|
48
lrh3321 2017-07-11 08:41:39 +08:00
需要格式化下代码
|
49
auhah 2017-07-11 08:58:04 +08:00
写完肯定没自己看过。。。。
|
50
xujialiang 2017-07-11 08:59:29 +08:00
一起优化代码,面试题
|
51
jianghu52 2017-07-11 09:03:47 +08:00
如果工期只有半天。我指不定也会写出这样的代码。。。。
|
52
wizardforcel 2017-07-11 09:04:38 +08:00 via Android
就不知道拿个变量缓存一下。
|
53
doubleflower 2017-07-11 09:07:16 +08:00
写代码对格式没有强迫症的都可以开除了
|
54
syyy 2017-07-11 09:09:15 +08:00
我曾经因为 else 两边打空格被吐槽,后来当着那人的面格式化代码,完胜。
|
55
zjsxwc 2017-07-11 09:14:39 +08:00
感觉是 else 里的是 原来正常的代码, 后来产品经理要求马上加个组合支付( gPayMethod==99 )功能,就写成楼主这种代码了
|
56
suckli 2017-07-11 09:15:47 +08:00
这种魔鬼数字看着就很恐怖。。。
|
57
MiguelValentine 2017-07-11 09:15:54 +08:00 1
看了一些回答。。我才发现 v2 的水平也就这样。就看到一个提三目的,好几个说没什么大问题。。??可以。难怪你们的代码量 5x。是 kpi 指标吗。
|
59
jydeng 2017-07-11 09:19:09 +08:00
shift + alt + f
|
60
walleL 2017-07-11 09:28:11 +08:00
有没有大佬指出都有哪些问题?我学习一下
|
61
chenyu0532 2017-07-11 09:32:17 +08:00
说代码逻辑写的不好就说逻辑的事,跟个 if else 格式叫什么劲
|
62
deadEgg 2017-07-11 09:33:44 +08:00
if else 的写法精简度 是判断大学生代码清洁的基本门槛之一 :doge
|
63
heaton_nobu 2017-07-11 09:39:59 +08:00
感觉楼主内心也挺扭曲的
|
64
whisperer 2017-07-11 09:48:24 +08:00
说 if else 的各位,我想请教一下我这个怎么解释,原代码如下:
执行代码格式化,PhpStorm 把格式改成楼主同事的风格了: |
66
lianxiaoyi 2017-07-11 09:53:29 +08:00 1
为什么要给一个有强迫症的程序员看这个。。。。。。妈的。。。。。。心扭一起了。。。。。我先去捏包方便面缓解一下。。。。。。。。。。。。
|
67
Curtion 2017-07-11 10:06:49 +08:00
实话告诉你,我们老师都是这么写的,不过是搞嵌入式的
|
68
aitaii 2017-07-11 10:07:06 +08:00
@lianxiaoyi 你的强迫症不能强迫你把。。。写整齐点吗,。。。。,,。。,。》。,。,。,。,。,。,。,。,
|
71
liu35118665 2017-07-11 10:09:01 +08:00
var payMethod=gPayMethod;
if(payMethod !="99"){ payMethod=$("#selPayMethod").val()||"00"; } myPacket.data.add("payMethod",payMethod); |
72
fds 2017-07-11 10:11:57 +08:00
强制开启自动格式化
|
73
lianxiaoyi 2017-07-11 10:12:00 +08:00
@aitaii 这个到不是强迫症。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。
|
74
lianxiaoyi 2017-07-11 10:13:21 +08:00
@aitaii 我草。。。。。。为毛后面少了一个。 我靠 。。。。。。
|
75
whisperer 2017-07-11 10:19:07 +08:00
|
76
slion 2017-07-11 10:25:30 +08:00
先把代码格式化一次
|
79
maplerecall 2017-07-11 10:39:15 +08:00
简直难受,究竟是经历了什么才会写出这样的括号
|
80
sonyxperia 2017-07-11 10:51:32 +08:00
我觉得楼主的内心扭曲多了
|
81
zhengxiaowai 2017-07-11 10:53:57 +08:00
换行党和不换行党看着都很无奈
|
82
bk201 2017-07-11 10:56:02 +08:00
@MiguelValentine 算了吧,这里几个不懂三目,别乱喷,如果你厉害你写个代码给大家瞧瞧就是了。
|
84
XiaoFaye 2017-07-11 10:59:43 +08:00
大家别 BB 了,估计写这代码的人 5 万一个月呢,尔等屌丝哪有资格评论。
|
86
QAPTEAWH 2017-07-11 11:00:22 +08:00
格式有点问题,内容不评价。
有些人不要少见多怪 - “ logic follow structure ” 根据这个原则写出多重嵌套 if-else 很正常,反倒是各种合并会让逻辑不清晰 - 状态机写个大 switch 太正常了,搞不好是代码生成的 |
87
blacklee 2017-07-11 11:12:40 +08:00
好像还没人说这一点,题中的代码上是
```if (cond1) { xx1(); } else { if (cond2) xx2(); else xx3(); }``` 这就是 `if - else if` 的经典套路了,应该改成 ```if (cond1) { xx1(); } else if (cond2) { xx2(); } else { xx3(); }``` 不知道 md 语法行不行 |
88
Sapp 2017-07-11 11:38:28 +08:00
@MiguelValentine 有些人就是不喜欢用三元运算符,这不是喷点,喷点是他这么写确实很难看,就算用 if 也不该用这么难看,不知道你的关注点为什么在可以用三元运算符上。
|
89
Sapp 2017-07-11 11:44:17 +08:00
|
90
yanghao633785 2017-07-11 11:46:56 +08:00
var payMethod = $("#selPayMethod").val();
if(payMethod == undefined || payMethod == ""){ payMethod = "00"; } myPacket.data.add("payMethod", payMethod); //"99"为啥要单独拿出来额 |
91
yuchenyang1994 2017-07-11 11:52:41 +08:00
不想吐槽。。。。 |
92
laoyur 2017-07-11 11:54:49 +08:00
能用即合理,能赚钱就行,吹毛求疵干吗
说非要优化成 if - else if 结构的,不知道业务是随时会变的吗,楼主这种风格的代码应对随时会加上的各种龟毛逻辑,更从容,不容易出 bug |
93
Sapp 2017-07-11 11:55:12 +08:00
@yanghao633785
你这么写似乎是有 bug,第一个 if 里是 gPayMethod,gPayMethod 应该是单独获取的。 而且也没必要这么写。 const payMethod = gPayMethod === '99' ? '99' : $("#selPayMethod").val() || '0' xxxx('payMethod', payMethod) |
94
mineqiqi 2017-07-11 12:02:07 +08:00
var payMethod = gPayMethod=="99"?gPayMethod:$(#selPayMethod).val||"00";
myPacket.data.add("payMethod",payMethod); |
95
yanghao633785 2017-07-11 12:05:36 +08:00
@Sapp 确实,受教了。
|
96
Jimrussell 2017-07-11 12:14:35 +08:00
顶楼那个确实有待提高,但也就是别人可能菜了点新手了点。说他内心扭曲我实在看不出来。
|
97
ZzFoo 2017-07-11 12:22:06 +08:00
楼主的槽点居然在 if else, 不应该是 "99" 和 "00" 那两个 magic number 以及写了三次的 myPacket.data.add() 吗
|
98
stzz 2017-07-11 12:25:14 +08:00 via Android
其他就算了,主要是括号看得人难受吧,看得头疼
|
100
liangWL 2017-07-11 12:36:39 +08:00
Ctrl+Shift+Alt+T 搞定
|