“你代碼寫得這么爛林说,有問過代碼是什么感受嗎?”
一屯伞,完全不能被讀懂的代碼
哎,憋了這么多天豪直,今天我實(shí)在憋不住了劣摇!
本小姐我,快-被-一-段-代-碼-氣-炸-了弓乙!
是的末融,你沒看錯(cuò)钧惧,惹我生氣的是一段根本就沒有生命的代碼。
但是不得不說勾习,這段代碼有夠魔性浓瞪!
一個(gè)明明沒有生命的東西,硬生生把一個(gè)活的人給惹炸毛了巧婶。
它還不夠魔性嗎乾颁?
事情的經(jīng)過是這樣的。
姐姐我最近負(fù)責(zé)跟進(jìn)的是一個(gè)教育類軟件的ios客戶端艺栈。
已經(jīng)上線運(yùn)營(yíng)了英岭,但是最近甲方領(lǐng)導(dǎo)覺得以前的ui不夠美觀,于是提出了我們這個(gè)月要做個(gè)小改版湿右。
我看了需求诅妹,無非調(diào)調(diào)顏色,改改圖標(biāo)排列毅人,再添加幾個(gè)詳情頁(yè)吭狡。
好像也沒啥難搞的。
于是我就跟身邊一個(gè)小伙子簡(jiǎn)單交待幾句丈莺,讓他去做這個(gè)事情赵刑。
臨了,我還不忘記叮囑了句场刑,我們現(xiàn)在代碼里般此,這一塊之前改來改去,代碼結(jié)構(gòu)有點(diǎn)亂牵现,你看時(shí)間允許的話铐懊,就一起重構(gòu)下吧。
就是這句話瞎疼,讓我想狠抽自己嘴巴科乎。
是的,代碼他重構(gòu)過了贼急,但是重構(gòu)后的代碼茅茂,我更加看不懂了!
昨天轉(zhuǎn)測(cè)的版本太抓,今天看到有個(gè)問題空闲,通訊錄的列表里,有個(gè)人的電話號(hào)碼顯示不對(duì)走敌。
粗粗一看碴倾,這不就是tableview的cell復(fù)用數(shù)據(jù)沒更新好的問題嘛?
這種小case,姐姐順手就可以給處理了跌榔。
誰(shuí)知道啊异雁,姐姐還是太年輕,就這么栽這坑里了僧须!
當(dāng)我進(jìn)到這個(gè)界面的- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath函數(shù)中纲刀,一下子就傻眼了。
切切實(shí)實(shí)的陰溝里翻船的感覺担平。
首先示绊,這個(gè)函數(shù)有200多行!
直接懵圈了驱闷。
但是我也不能這么久放棄耻台,有點(diǎn)沒面子。
硬著頭皮往下看空另。
令人想不到的是盆耽,這個(gè)函數(shù)中使用的圖片資源,名字竟然還是中文扼菠!
怎么樣摄杂?有沒有要吐血的感覺?
哦循榆,那你的承受能力就太差了些析恢。
因?yàn)椋钭屛页惺懿涣说倪€不是這些秧饮。
而是這200多行的代碼竟然做的都是cell上控件的創(chuàng)建初始化賦值...
我實(shí)在沒法淡定啊映挂,對(duì)著這樣的代碼,我竟然不知道還要cell對(duì)應(yīng)的類干嘛盗尸。
跟幾個(gè)朋友吐槽了下柑船,做測(cè)試的美眉說我承受不了的這條她不能理解。
我?guī)捉叵母v——
“你想想直接讓奶奶甩開兒子媳婦泼各,生個(gè)孫子是什么感觸吧”
還有比這更形象的比喻了么鞍时?
肯定有人不相信我講的話,好吧扣蜻,我就貼一小段給大家感受下逆巍。
ContacterDetailCell * cell = [tableView dequeueReusableCellWithIdentifier:cellId];
if (!cell) {
cell = [[ContacterDetailCell alloc]initWithStyle:UITableViewCellStyleDefault reuseIdentifier:cellId];
cell.selectionStyle = UITableViewCellSeparatorStyleNone;
}
BOOL isHasShortDn = [[[NSUserDefaults standardUserDefaults]objectForKey:KEY_ISOPEN_VNET]integerValue];
ADTIMContacter * contacter = self.contacterArray[indexPath.row];
_phoneNum = contacter.m_strDn;
cell.contentView.userInteractionEnabled = YES;
cell.shortDnLabel = [UILabel new];
cell.telLabel = [UILabel new];
cell.classNameLabel = [[UILabel alloc]initWithFrame:CGRectMake(5, 5, MAIN_WIDTH, 40)];
cell.classNameLabel.textColor = [UIColor darkGrayColor];
cell.classNameLabel.font = [UIFont systemFontOfSize:16.0];
[cell.contentView addSubview:cell.classNameLabel];
cell.publicLable = [[UILabel alloc]initWithFrame:CGRectMake(MAIN_WIDTH - 80, 135, 50, 40)];
cell.publicLable.text = contacter.m_isOpen_phone ? @"已公開":@"未公開";
cell.publicLable.textColor = [UIColor redColor];
cell.publicLable.font = [UIFont systemFontOfSize:16.0];
cell.publicLable.textAlignment = NSTextAlignmentCenter;
[cell.contentView addSubview:cell.publicLable];
UIImageView * line = [UIImageView new];
line.backgroundColor = kBWDarkGray;
[cell.contentView addSubview:line];
if ([[LoginUserUtil userId] isEqualToString:_userID[1]]) {
cell.classNameLabel.hidden = NO;
cell.classNameLabel.text = self.classNameArray[indexPath.row];
cell.shortDnLabel.frame = CGRectMake(5, 45, MAIN_WIDTH, 55);
cell.telLabel.frame = CGRectMake(5, 105, MAIN_WIDTH, 55);
line.frame = CGRectMake(0, CGRectGetMaxY(cell.shortDnLabel.frame), MAIN_WIDTH, 1);
UISwitch * my_switch = [[UISwitch alloc]initWithFrame:CGRectMake(cell.publicLable.frame.origin.x, CGRectGetMinY(cell.publicLable.frame) - 25, 0, 0)];
[my_switch setOn:contacter.m_isOpen_phone animated:YES];
[my_switch addTarget:self action:@selector(switchValueChanged:) forControlEvents:UIControlEventValueChanged];
my_switch.tag = 100 + indexPath.row;
if (!IsStrEmpty(contacter.m_strDn)) {
[cell.contentView addSubview:my_switch];
}
我就想問誰(shuí)特么能把這段代碼讀完,并理解了莽使?
我肯定是不行锐极!
還不死心的我,又去看了看其他地方吮旅。
真心想講臟話溪烤,好嘛味咳?
充斥滿屏的魔鬼數(shù)字庇勃,
完全不知所謂的變量命名檬嘀,
不考慮屬性的使用范圍,統(tǒng)一采用public
不考慮內(nèi)存的優(yōu)化使用
......
這位大哥责嚷,你這樣寫代碼鸳兽,有沒有問問代碼什么心情?
最終罕拂,我也沒找到上面所說問題的解決辦法揍异。只好找來制造這段代碼的小哥。
沒錯(cuò)爆班,這小哥就在剛才處理完問題后衷掷,已經(jīng)領(lǐng)了我甩給他的編碼規(guī)范去面壁了。
我曾一直天真(臭不要臉)的以為柿菩,我的團(tuán)隊(duì)能做出完美的應(yīng)用戚嗅。經(jīng)此事件,我突然感覺枢舶,對(duì)于我目前的團(tuán)隊(duì)來說懦胞,做出完美的app是多么遙不可及的夢(mèng)想。
我們甚至都寫不出來凉泄,能讓別人(注意躏尉,這里的別人不是所有人,而是你的同行們)看懂的代碼后众!
二胀糜、我們?yōu)槭裁磳懗隽瞬灰锥拇a?
不要問我什么是容易被別人讀懂的代碼蒂誉。
你只要想想你自己去讀別人的代碼教藻,你希望看到什么樣的代碼心中就有答案?
告訴我拗盒,你在讀代碼時(shí)
是希望看到排版整齊的怖竭,還是很隨性的?
是喜歡看變量命名使用同一規(guī)則的陡蝇,還是根本就看不懂使用了什么規(guī)則的痊臭?
是更樂于看到函數(shù)的功能單一的,還是樂于看到囊括了好幾個(gè)不相關(guān)的能力的函數(shù)登夫?
是更接受邏輯清晰的還是更接受邏輯繁復(fù)的广匙?
是一看就知道作者意圖,還是讀了半天也不明白作者寫這段代碼要干嘛恼策?
......
所以鸦致,你還要問我什么樣的代碼是容易被別人讀懂的嗎潮剪?
你自己心中明明早就有很明確的答案了好嗎?
你之所以寫出這么爛的代碼分唾,無非是你對(duì)別人(一般是跟你合作項(xiàng)目的伙伴)有明確的要求抗碰,而對(duì)自己沒有!
完全分不清楚好代碼和爛代碼的人除外绽乔。
相信我弧蝇,在你需要讀別人代碼的時(shí)候,你心中對(duì)好代碼的定義能細(xì)分到三級(jí)目錄折砸。
然而看疗,到你自己寫的時(shí)候,都是干脆交給編譯器睦授,自己都懶得讀两芳,更別說給自己的代碼套自己的標(biāo)準(zhǔn),檢驗(yàn)下是否易讀去枷。
三怖辆、怎么寫出容易被讀懂的代碼?
怎么寫出容易讀的代碼沉填?
方法就一個(gè)疗隶,寫完后自己去讀!
然后想象一下翼闹,別人讀這段代碼的心里感受斑鼻。
當(dāng)然不排除,有很多自認(rèn)為大神的人猎荠,經(jīng)常會(huì)寫出一些晦澀難懂的代碼坚弱,以此來彰顯自己水平牛X。
但是关摇,請(qǐng)看明白荒叶,這些人都僅僅是自以為是的牛X。
真正的牛X是什么樣的输虱?把復(fù)雜的問題用很通俗淺顯的方法解決了些楣,那才是真正的牛X!
舉個(gè)很簡(jiǎn)單的例子宪睹,《天龍八部》中的掃地僧愁茁,他出手時(shí),擺什么花里胡哨的招數(shù)了嗎亭病?沒有鹅很!
所以你是希望自己做個(gè)隨便一舉手投足都自帶功力的高手,還是做個(gè)金玉其外的花架子罪帖,選擇權(quán)在你自己促煮。
畢竟邮屁,代碼是承載我們技術(shù)水平的東西,它跟我們沒有深仇大恨菠齿。
在敲下每行代碼的時(shí)候佑吝,我們還是稍微用點(diǎn)心,不要總滿足于功能似乎實(shí)現(xiàn)了就萬(wàn)事大吉的入門級(jí)別泞当!