这样写 if/else 的人，内心一定很扭曲吧

honeycomb

2017 年 7 月 11 日

@liyuhang
不考虑格式(缩进等)，原先的代码在逻辑上没什么不合适，难道这里希望把它改用逆否命题式的 guard 条件来写？

pysama

2017 年 7 月 11 日

@caniuse 这字体还行把。我见过坚决不用等宽字体写代码的。而且只要是带 tab 的工具，肯定是齐刷刷二十几个 tab 全开着，找文件的时候一个个 tab 来回切着找。
我是佩服的不行。。

bk201

2017 年 7 月 11 日

no bb，show me your code.除了啰嗦，也没觉得有多奇怪，引申到别人心里扭曲是什么鬼？

wotemelon

2017 年 7 月 11 日

@redtea 你这代码写的。。。无力吐槽

wangxiaoer

2017 年 7 月 11 日

我觉得楼主贴的代码逻辑没啥大问题，就是有些重复了，总比 x 马培训那个手动递归的强吧。

这样会不会好点

mrcn

2017 年 7 月 11 日

不是很懂到底哪里不对
除了里面的 if 可以跟着 else 直接写以外

hzw94

2017 年 7 月 11 日

肯定是新手，基本不注意格式规范，应该从未考虑当别人阅读的情况

jianghu52

2017 年 7 月 11 日

如果工期只有半天。我指不定也会写出这样的代码。。。。

wizardforcel

2017 年 7 月 11 日

就不知道拿个变量缓存一下。

doubleflower

2017 年 7 月 11 日

写代码对格式没有强迫症的都可以开除了

syyy

2017 年 7 月 11 日

我曾经因为 else 两边打空格被吐槽，后来当着那人的面格式化代码，完胜。

zjsxwc

2017 年 7 月 11 日

感觉是 else 里的是原来正常的代码，后来产品经理要求马上加个组合支付（ gPayMethod==99 ）功能，就写成楼主这种代码了

suckli

2017 年 7 月 11 日

这种魔鬼数字看着就很恐怖。。。

MiguelValentine

2017 年 7 月 11 日

看了一些回答。。我才发现 v2 的水平也就这样。就看到一个提三目的，好几个说没什么大问题。。？？可以。难怪你们的代码量 5x。是 kpi 指标吗。

Arainc

2017 年 7 月 11 日

@am241 同想脱嵌入式圈。。。无数个全局变量

walleL

2017 年 7 月 11 日

有没有大佬指出都有哪些问题？我学习一下

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

https://v2ex.ih06.com/t/374340

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

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