你在公司項目裡面看到過哪些操蛋的代碼？

你在公司項目裡面看到過哪些操蛋的代碼？

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)……

匿名用戶

以前所在公司做激光雕刻軟體，客戶對雕刻速度有要求，於是乎大神把顯示在客戶端的所需時間乘上4/5。關注公眾號：程序員大咖。後來軟體交給我維護，重寫了界面，新版本到了客戶手裡雕刻速度變慢了，無論怎麼優化雕刻演算法，都不能達到原來的雕刻速度。

後來忘記是怎麼看到那個* 4/5了，真是買了表的

李旭東，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

看見這段代碼的時候，我拍了很久的掌

喜歡這篇文章嗎？立刻分享出去讓更多人知道吧！