关键时刻,第一时间送达!
你在公司项目里面看到过哪些操蛋的代码?
String result;
......
return result.toString();
算吗?
尽情吐槽吧
Cyandev,编程语言 本地应用开发 Android开发
if (m_doc->isModified() == true)
{
for (int i = 0; i
{
save();//Save the document for 100 times to ensure it has been saved successfully.
}
}
不知道谁写的,总之我都看哭了
侯杰,Code and Game
enum FiveLine
{
Gold,
Wood,
Water,
Fire,
Earth,
};
看枚举名字不知道五行(hang)是什么鬼,看了枚举内容恍然大雾,原来是五行(xing)……
李旭东,https://rrke.cc/
…………
…………
…………
白砂糖,大三,傲娇的处女座程序猿,伪美工
//以下所有left代表右
// 以下所有right代表左
Kun Andy,卖油翁
public enum ColorType
{
Green, // 绿色
Hong, // 红色
}
//......
this.curColortype= ColorType.Hong;
......
毫无违和感! 英语和汉语的激烈碰撞,一时愣是没读懂代码,“Hong”为何物,你看注释才恍然大悟!犀利
可能是绿的拼音(lv) 有点像等级,所以为了防止歧义,用了Green!赞一个!
!!!!!!!!!!!!!
程序员很可爱,我爱程序猿大家庭!
这些蕴含智慧的代是礼物也是惊喜!
可以帮你拉这么多赞!感谢!
!!!!!!!!!!!!!
真爱身边的程序员!
爱他们
匿名用户
我软有不少代码看上去挺好的但是会被标识成操蛋,这都归功于各种静态检查工具。
在我软代码里面起个变量名叫WhiteList都会被自动发bug,因为用White来表示允许就属于种族歧视,一定要改成AllowList。我会告诉你我们因此还真的改了服务接口而且还要做向下兼容吗?
还有变量名叫country也会被查,因为正确的说法是country and region。
我不是说public的类成员哦,private的都不行。
另外kill,dead之类的字眼都是有问题的。
当然你真的一定要用这些词你就得说明理由,然后加supression。
要像 @vczh 那样变量起名bitch什么的那是要死一百遍啊。
所以你现在知道写点代码有多难了吧。
孙传武,http://chuanwu.me
有次看一项目源码,注释里有个warning。
// warning: Do not make any changes here.
// I got confused why it can run successfully. PLEASE Do not make any changes before you figure it out.
绅士提督不笑船,我听见月光洒在湖面上。
变量命名
class UplaodDuiLie // 上传队列
// Uplaod 非手误,完全原文。
// 为什么你们只吐槽 typo 而不吐槽中英文混合呢。。
myd7349,前花样作死世界男单总冠军。
从长度上来说,完爆楼上各位的。感受下:
if(m_Str2.Find("'")>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find(':')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('/')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('!')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('@')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('#')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('$')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('%')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('&')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('^')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('*')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('(')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find(')')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('/')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('|')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find(':')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('[')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find(']')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('{')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
if(m_Str2.Find('}')>=0)
{
AfxMessageBox("您输入的包含非法字符");
m_Str2="";
UpdateData(FALSE);
return;
}
哪怕不知道 RE,也该知道 FindOneOf 啊?!!!!
就是再不济,也写个循环吧。
谨遵 DRY 原则的程序员看到这段代码受到了 1024 Kg 的伤害。
当然,还有什么:
一个线程回调函数函数写了 700 多行的事情咱就不说了。
一个小小的项目专门定义了名叫 Global.h/cpp 的文件来放置全局变量咱也不说了。一个小小的项目中一二百个全局变量咱也不说了,这些全局变量中大部分还是指针变量咱也甭说了!!!
我去哭一会去。
青苔,Web前端失足青年 https://github.com/Dannyxie
看见这段代码的时候,我拍了很久的掌
来自:知乎
https://www.zhihu.com/question/30776912
程序员共读整理发布,转载请联系作者获得授权
领取专属 10元无门槛券
私享最新 技术干货