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