专栏名称: 程序员大咖
为程序员提供最优质的博文、最精彩的讨论、最实用的开发资源;提供最新最全的编程学习资料:PHP、Objective-C、Java、Swift、C/C++函数库、.NET Framework类库、J2SE API等等。并不定期奉送各种福利。
目录
相关文章推荐
程序员的那些事  ·  趣图:前端开发者站着吃饭的原因找到了 ·  1 周前  
程序猿  ·  支付宝推出独立App,网友不淡定了...... ·  1 周前  
OSC开源社区  ·  鸿蒙生态繁荣背后:WPS全面适配Harmon ... ·  1 周前  
51好读  ›  专栏  ›  程序员大咖

你在公司项目里面看过哪些操蛋的代码?

程序员大咖  · 公众号  · 程序员  · 2017-02-13 19:15

正文

转自:知乎 

https://www.zhihu.com/question/30776912


整理自知乎的同名问答贴。题主的补充:


String result;

......

return result.toString();


算吗?


尽情吐槽吧


Cyandev 的分享


if (m_doc->isModified() == true)

{

    for (int i = 0; i

    {

        save(); 

//Save the document for 100 times to ensure it has been saved successfully.

    }

}


不知谁写的,总之我都看哭了。


侯杰 的分享


enum FiveLine

{

  Gold,

  Wood,

  Water,

  Fire,

  Earth,

};


看枚举名字不知道五行(hang)是什么鬼,看了枚举内容恍然大雾,原来是五行(xing)……


李旭东 的吐槽


   

       

 

       

       

   

 

…………

 

   

 

   

 

…………

 

   

 

…………

 


白砂糖 的分享


// 以下所有left代表右

// 以下所有right代表左


Kun Andy 的分享


public enum ColorType

{

  Green, 

// 绿色

  Hong, 

// 红色

}

//......

this.curColortype= ColorType.Hong;

......


毫无违和感! 英语和汉语的激烈碰撞,一时愣是没读懂代码,“Hong”为何物,你看注释才恍然大悟!犀利

可能是绿的拼音(lv) 有点像等级,所以为了防止歧义,用了Green!赞一个!



!!!!!!!!!!!!!

程序员很可爱,我爱程序猿大家庭!

这些蕴含智慧的代是礼物也是惊喜!

可以帮你拉这么多赞!感谢!

!!!!!!!!!!!!!

真爱身边的程序员!

爱他们


孙传武 的分享


有次看一项目源码,注释里有个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 的文件来放置全局变量咱也不说了。一个小小的项目中一二百个全局变量咱也不说了,这些全局变量中大部分还是指针变量咱也甭说了!!!

我去哭一会去。


青苔 的分享



看见这段代码的时候,我拍了很久的掌


虎耳 的吐槽


曾经在一个家公司,有一次有人发现对每个请求,有个地方都会空等了一秒。

后来问老大,说,如果客户有新要求,随时可以把性能提高一倍以上。。。


欢迎大家在评论中补充 :)