吐槽下最近接手一个项目的代码，还有这种操作？

Markdown

Save

吐槽

接手

47 条回复 • 2017-07-17 11:34:32 +08:00

1

knva

2017-07-13 15:57:03 +08:00

没准是测试代码呢.

2

Famio

2017-07-13 15:59:19 +08:00

function clean_cache()
{
alert("缓存清除成功！")
}

前两天 V 站看到的

3

kinghui

2017-07-13 16:02:47 +08:00

没准是兼容代码呢.

4

chazz

2017-07-13 16:04:06 +08:00

哈哈哈，跟前段时间的清楚缓存一摸一样的操作

5

claysec

2017-07-13 16:05:23 +08:00

def random():
a = random(10)
return a

6

surfire91

2017-07-13 16:07:49 +08:00

我咋觉得没啥问题，可能是为了方便调试，或者是预留接口之类的

7

Lonely

2017-07-13 16:12:52 +08:00

这一般是预留的吧

8

lixiangzaizheli

2017-07-13 16:20:17 +08:00

如果项目写完了就不会要你接手了先写完逻辑再补代码而已

9

cod

2017-07-13 16:20:50 +08:00

没毛病。第一段预留，第二段一般都是测试代码，改成 false 就快速屏蔽。
经常这么写，尤其是脚本类语言。
当然 c/c++的话，就不可能这样，改一下再编译太费劲。

10

momocraft

2017-07-13 16:24:41 +08:00

我做到一半时也这样..不过会写个 FIXME

11

reus

2017-07-13 16:27:01 +08:00

7

少见多怪，你经验还是太少

用来调试的，用来预留的，各种技巧，多学习，少吐槽

12

blacklee

2017-07-13 16:31:37 +08:00

1. 很多人说了，这没毛病，是你少见多怪。多写几年，你就会用这种方法来调试代码了。
2. 话虽如此，但这种代码应该是留给自己的，把这种代码交出去就不太好了。

13

iyangyuan

2017-07-13 16:33:24 +08:00

没什么大问题，方便扩展，很有先见之明

14

autoxbc

2017-07-13 18:30:04 +08:00

附加操作，快速屏蔽
if(true)
{...}

快速略过判断条件
if( false || ... )
{...}
else
{...}

多方案备选
switch(1)
{
case 1:
...
break;
case 2:
...
break;
}

15

printempw

2017-07-13 19:01:28 +08:00

挺正常的，我也常写，不过一般会标明，也不会就这么放着交给别人

16

nutting

2017-07-13 19:32:59 +08:00 via Android

第一个应该留个 TODO 吧，第二个测试用的，但是 true 建议定义到一个常量里起个名字吧

17

kamen

2017-07-13 19:36:33 +08:00

写法没毛病，但是好歹加个注释啊

18

Mikewu

2017-07-13 19:43:18 +08:00

测试的时候我也常这样写

19

mingyun

2017-07-13 22:08:43 +08:00

// to do

20

Ouyangan

2017-07-13 22:11:23 +08:00

因为没写完逻辑的话 , 不这样写编译器就会报错啊...

21

oneonesv

2017-07-13 22:30:38 +08:00

测试用的没毛病

22

hellojeana

2017-07-13 23:42:38 +08:00 via iPhone

@Famio 好好笑！！😂😂

23

wemore

2017-07-13 23:55:59 +08:00 via Android

uc 看不到图，顺便绘麻好评

24

weyou

2017-07-13 23:59:27 +08:00 via Android

@autoxbc 快速略过判断条件有点问题吧？应该 false &&或者 true || 吧，还是我没看懂你的目的

25

ditel

2017-07-14 00:33:56 +08:00 via Android

第一个基本是写个接口，后面再填

26

autoxbc

2017-07-14 00:38:25 +08:00

@weyou 我写的是常规状态，所谓常规状态就是不影响条件判断，略过状态是调试时快速切换用

略过判为真
常规状态 if( false || ... )
略过状态 if( true || ... )

略过判为假
常规状态 if( true && ... )
略过状态 if( false && ... )

27

autoxbc

2017-07-14 00:47:33 +08:00

@weyou 再写个明显点的

if(1)
{...}
1 => 0 此段被注释掉

if( 0 || ... )
0 => 1 此段被判真

if( 1 && ... )
1 => 0 此段被判假

28

autoxbc

2017-07-14 01:23:23 +08:00

@weyou 抱歉刷屏，刚发现我们俩的出发点不一样

你的写法在调试的"冷区"适用，写 true || 就快速判真，不过出调试时略麻烦，需要删除 true ||

我的写法在"热区"适用，布尔值和逻辑符号就放在那，平时也有，但是不影响逻辑。进调试时只改动布尔值，出调试也只改布尔值，尤其是弱类型语言，0 和 1 切换就行。

乍一看比较怪异，这样扣题"还有这种操作"。