说说你们在项目中遇到过的最糟糕的代码

[letitbesqzr]

最新接手的一个模块....一看代码,哭了

[miemiekurisu]

400 行满是深度嵌套的 if else （ 20 层），考验你的视力

[yxzblue]

[xiaowangge]

性别字段为 : sex ， int 类型，并且 0 表示男， 1 表示女 :-)

[update]

@gongpeione
这个 function zhuru (zhi ) 真是萌萌的。。

[update]

@gongpeione
这个 function zhuru (zhi ) 真是萌萌的。。
😊

[pedestrianc]

- (void )setAudiecenModel:(QIAudienceViewCellModel *)audiecenModel {
_audiecenModel = audiecenModel;

CGFloat restrictionWidth = kDeviceWidth - self.groupNameLabel.frame.origin.x - MARGIINOFCONTROLS - 20;
NSDictionary* groupLevel = audiecenModel.groupLevelDict[audiecenModel.userID];
NSString* fansID = groupLevel[@"fansID"];
NSString* rankName = groupLevel[@"level"];
if (nil != rankName ) {
audiecenModel.rankName = rankName;
} else {
rankName = audiecenModel.rankName;
}

NSString* rankImageName = [NSString stringWithFormat:@"group_level%ld",audiecenModel.rankName.integerValue];
if (nil == rankName ) {
rankImageName = nil;
}
if ([_audiecenModel.anchorRank integerValue] > 0 ) {
rankImageName = [NSString stringWithFormat:@"live_anchor_nobility%@",_audiecenModel.anchorRank];
}
if (nil == rankName && [_audiecenModel.anchorRank integerValue] <= 0 ) {
rankImageName = nil;
}

self.rankImageView.image = [UIImage imageNamed:rankImageName];
if (nil == audiecenModel.rankName || [audiecenModel.rankName isEqualToString:@""]) {
self.rankImageViewWidthCons.constant = 11;
self.groupNameLabelLeadingCons.constant = 12;
} else {
self.rankImageViewWidthCons.constant = 11;
self.groupNameLabelLeadingCons.constant = 12;
}

NSString* fansGroupName = audiecenModel.groupNameDict[fansID][@"fansName"];
if (nil == rankName ) {
fansGroupName = nil;
}
if (nil != fansGroupName ) {
audiecenModel.fansGroupName = fansGroupName;
} else {
fansGroupName = audiecenModel.fansGroupName;
}

CGSize groupNameSize = [fansGroupName sizeWithFont:[UIFont systemFontOfSize:COMMONFONTSIZE]];
if (nil == audiecenModel.fansGroupName || [audiecenModel.fansGroupName isEqualToString:@""]) {
self.userNameLeadingCons.constant = 0;
} else {
self.userNameLeadingCons.constant = MARGIINOFCONTROLS;
}
self.groupNameLabel.text = fansGroupName;
NSString* userName = audiecenModel.userName;
if ([audiecenModel.userID isEqualToString:[UserManager shareUserManager].userItem.userID]) {
NSString *name = [UserManager shareUserManager].userItemDic[@"nickname"];
self.userNameLabel.text = name?name:self.userNameLabel.text;

}

CGSize size = [userName sizeWithFont:[UIFont systemFontOfSize:COMMONFONTSIZE]]; // wwc 过时的方法
self.userNameLabel.text = userName;


NSDictionary* medalInfor = _audiecenModel.medalDict[_audiecenModel.userID];
NSString* medalImageName = nil;
if ([medalInfor isKindOfClass:[NSString class]]) {
medalImageName = (NSString*)medalInfor;
} else {
medalImageName = medalInfor[@"image"];
}

[self.medalImageView sd_setImageWithURL:[NSURL URLWithString:medalImageName] placeholderImage:nil];
if (nil == medalImageName || [medalImageName isEqualToString:@""]) {
self.medalLeadingCons.constant = 0;
self.medalWidthCons.constant = 0;
} else {
self.medalLeadingCons.constant = MARGIINOFCONTROLS;
self.medalWidthCons.constant = 18;
}

self.vestImageView.image = [UIImage imageNamed:audiecenModel.vestName];
self.nobilityImageView.image = [UIImage imageNamed:audiecenModel.nobilityName];
if (![@"noble100" isEqualToString:audiecenModel.nobilityName]) {
self.nobilityImageView.hidden = NO;
self.eventBgView.hidden = NO;
_tap.enabled = YES;
} else {
_tap.enabled = NO;
}

if (restrictionWidth > 150 && [audiecenModel.userID isEqualToString:@"20066"]) {
GGLogDebug (@"=======restrictionWidth======%f",restrictionWidth );
}

if (medalImageName ) {
restrictionWidth -= (self.medalWidthCons.constant + MARGIINOFCONTROLS );
}
if (self.vestImageView.image ) {
restrictionWidth -= (self.vestImageView.frame.size.width + MARGIINOFCONTROLS );
}
if (self.nobilityImageView.image ) {
restrictionWidth -= (self.nobilityImageView.frame.size.width + MARGIINOFCONTROLS );
}
if (restrictionWidth < groupNameSize.width + size.width ) {
self.groupNameWidthCons.constant = groupNameSize.width > restrictionWidth * LABELWIDTHRADIO ? restrictionWidth* LABELWIDTHRADIO : groupNameSize.width;
self.userNameLabelWidthCons.constant = restrictionWidth - self.groupNameWidthCons.constant;
} else {
self.groupNameWidthCons.constant = groupNameSize.width;
self.userNameLabelWidthCons.constant = size.width;
}

if ([audiecenModel.userID isEqualToString:@"20066"]) {
GGLogDebug (@"=======restrictionWidth======%f",restrictionWidth );
}
}








一个赋值方法一百多行，不建 model 数据混乱。赋值之后各种计算 layout 。。。。。同事的代码。

[letitbesqzr]

@xiaowangge 哈哈 以前遇到过... 身份信息..本来只有两个 1 代表医生 0 代表居民 。。。在程序里用 true 和 false ... 后来说要加一个 管理员身份，就傻逼了。

[zacard]

糟糕代码不报错，你就谢天谢地吧。遇到不测试（甚至有编译时错误）就直接提交代码的，你哭都来不及。。。

[andybest]

@blacklee "服务端程序语言对方选择的是 Java
MySQL 里的 bool 字段， 1 表示 false ， 0 表示 true "

那 bool 应如何在 MySQL 中储存？

[aivier]

JS 里用 JSON ， C#配合 PHP 服务器端一直用 XML ，这写法也是够惊人的....

[mozartgho]

楼主，体现你 performance 的时候到了！

[iugo]

@andybest 简单来说, 你不觉得有点写反了吗?

[laucie]

百度单词 来全拼

[akira]

for a:=1 to a1 do
for b:=1 to b1 do
for c:=1 to c1 do
for d:=1 to d1 do
for e:=1 to e1 do
for f:=1 to f1 do
for g:=1 to g1 do
for h:=1 to h1 do
for i:=1 to i1 do
for j:=1 to j1 do
for k:=1 to k1 do
for l:=1 to l1 do
for m:=1 to m1 do
for n:=1 to n1 do
for o:=1 to o1 do
for p:=1 to p1 do
for q:=1 to q1 do
for r:=1 to r1 do
for s:=1 to s1 do
for t:=1 to t1 do

我们问他，万一字母不够用怎么办

[scarlex]

@xiaowangge ...我觉得应该反过来才对啊， 0 表示女， 1 表示男...

[xuewl]

@blacklee MYSQL 的布尔值使用 tinyint (1 ) 的 1 和 0 来表示难道不是科学的方法吗？

[reinhardzoe]

坐等 "我就是那个 1 表示 girl,0 表示 boy 的程序员"

[xiaowangge]

@scarlex 是啊。「常识」，大家都认为 0 表示女， 1 表示男。

但代码中，就是相反的啊。害我还出了个 bug 呢 :-)

[jjed0119cs04]

我擦。这么多人。。。

[siteshen]

#!/usr/bin/env python

# example 1
def get_something (self, a, b, get_c=True ):
if get_c:
c = a + b
return a, b, c
else
return a, b

# example 2
def list_users_handler (self, request ):
user_ids = exec (request.params['user_ids'])