爲何今天忽然想說這個話題,那是由於在工做當中,被隊友坑過。一樣的一個功能,你讓不一樣的人去實現,可能會有不一樣的實現方式,一樣,就會獲得不同的結果。有些人作出來性能好,代碼精簡,可讀性好,易於維護等,有些人寫的代碼不獨代碼凌亂堪,更能讓服務器崩潰。程序員的基礎和思惟很重要,解決問題的思惟,要多思考,而不能只考慮完成功能,其它就啥都無論了,說得嚴重點,給人條活路好麼?人都是爹媽生的!html
一、我先列一個最最簡單的基礎編程題,原本很簡單,至於爲何寫成了這樣,不要問我,問寫的那位大神吧,我徹底相信我如今的公司中有人看到這樣的代碼(雖然工做幾年了...)會給我一個錯誤的答案mysql
int a = 5; int b = a++ + (++a) * 2 + ++a; Console.WriteLine(a); Console.WriteLine(b);
解:5+ a=a+1 a=a+1 a=7 a=a+1=5 +7*2+8;
注意優先順序,從左至右,一元運算符高於二元運算符
答案是:
8
27
一眼看上去,精簡是精簡了,但是可讀性呢?這是給人看的麼?若是你口算不出來,請複習下這一塊的基礎知識,補一補。
算運運算符 ++ --
++;分爲前++和後++,不論是前++仍是後++,最終的結果都是這個變量的值自身加1.
區別:若是在一個表達式中遇到了++運算符,
若是是前++,則首先這個變量的值自身加一,而後拿着這個加一後的值去參與運算。
若是是後++,則首先拿原值參與運算,運算完成後,自身再加一。
這裏只是舉個例子,現實中看到了相似的代碼,我呢個去,我費了好大勁........
我用reflector工具查看了下,解析以下:
計算機會理解成以下: int num = 5; int num2 = (num++ + (++num * 2)) + ++num; 也就是說: ++在後,也就是說num依舊爲5,等+運算後再執行++ num=num+1;//6 num=num+1; //7 兩次++,num=7 5+7*2 num=num+1;//8 num2=5+7*2+8
IL語言以下:程序員
.method private hidebysig static void Main(string[] args) cil managed { .entrypoint .maxstack 3 .locals init ( [0] int32 num, [1] int32 num2) L_0000: nop L_0001: ldc.i4.5 L_0002: stloc.0 L_0003: ldloc.0 L_0004: dup L_0005: ldc.i4.1 L_0006: add L_0007: stloc.0 L_0008: ldloc.0 L_0009: ldc.i4.1 L_000a: add L_000b: dup L_000c: stloc.0 L_000d: ldc.i4.2 L_000e: mul L_000f: add L_0010: ldloc.0 L_0011: ldc.i4.1 L_0012: add L_0013: dup L_0014: stloc.0 L_0015: add L_0016: stloc.1 L_0017: ldloc.0 L_0018: call void [mscorlib]System.Console::WriteLine(int32) L_001d: nop L_001e: ldloc.1 L_001f: call void [mscorlib]System.Console::WriteLine(int32) L_0024: nop L_0025: ret }
IL語言看得夠明瞭了吧!這是要考驗個人理解能力和基礎能力仍是要顯示得你牛逼呢?因此啊,仍是要自我修煉,才能不管別人怎麼坑我,我都能有辦法應付哇
二、舉例,控制檯輸入一個成績,若是 成績>=90 :輸出A,若是90>成績>=80 輸出B,若是80>成績>=70 輸出C ,若是70>成績>=60 輸出D,若是 成績<60輸出E
(考慮用if好仍是用if-else好仍是if else if好,思考,爲何好?)
如下是我在項目中看見的三種寫法:
代碼以下:
Console.WriteLine("請輸入你的考試成績"); int score = Convert.ToInt32(Console.ReadLine());
if else 的作法
#region if -else的作法 if (score >= 90) { Console.WriteLine("A"); } else//<90 { if (score >= 80) { Console.WriteLine("B"); } else//<80 { if (score >= 70) { Console.WriteLine("C"); } else//<70 { if (score >= 60) { Console.WriteLine("D"); } else { Console.WriteLine("E"); } } } } #endregion
if的作法
#region if的作法 if (score >= 90) { Console.WriteLine("A"); } if (score >= 80) { Console.WriteLine("B"); } if (score >= 70) { Console.WriteLine("C"); } if (score >= 60) { Console.WriteLine("D"); } if (score < 60) { Console.WriteLine("E"); } #endregion
if else if的作法
#region if else if的作法 if (score >= 90) { Console.WriteLine("A"); } else if (score >= 80) { Console.WriteLine("B"); } else if (score >= 70) { Console.WriteLine("C"); } else if (score >= 60) { Console.WriteLine("D"); } #endregion
請比對下這三種實現方式有什麼不一樣,哪種好,爲何?發現什麼問題了麼?我在工做中,常常會看到一些程序員亂用if else,明明能夠有更好的實現方式。redis
分支結構:if結構 if-else結構
選擇結構:if else-if sql
swich case的作法(這裏只考慮100分制)數據庫
public static string GetLevel(int score) { string level = ""; switch (score / 10) { case 10: case 9: level = "A"; break; case 8: level = "B"; break; case 7: level = "C"; break; case 6: level = "D"; break; default: level = "E"; break; } return level; }
三、使用一個同步程序或者定時任務計劃來往Ftp服務器上面拋送訂單報文,而後獲取Ftp服務器上面的回執報文編程
公司程序是這樣寫的:服務器
一、從數據庫中查詢須要拋送的訂單cookie
二、在服務器上面生成訂單報文並存在一個文件夾下面網絡
三、而後使用Ftp上傳這些訂單報文,已經上傳成功的就修改這個報文名稱(後面+_1)
四、上傳完報文後,而後根據這些報文名稱就去Ftp服務器上面下載回執報文,又存到一個文件夾裏面
五、再讀取這個文件夾裏面的回執文件,已經讀取過的,就把回執文件更名
.......
我真心說不下去了,第一次看到的時候,我就以爲寫這代碼的人應該抓去牢房裏關着,不要放出來
結果程序跑了一個月,服務器奔潰了,我上服務器上面刪報文文件刪除了2個小時,幾十個G,而後看下程序代碼,一千萬匹草泥馬奔騰而過,而後安排人重寫代碼...
我到如今都想不明白,程序爲何要這樣寫?即使你要生成訂單報文,你按年、月、日生成報文文件,處理完成以後,把報文文件移動到特定目錄下面,總比你更名快吧,而後按期把一個月前的交易記錄刪除(不要就直接刪除)或者壓縮包(要的話就壓縮備份),怎麼就不想一下一個月,這個文件夾下面會有幾十萬個文件呢,這是典型要乾死服務器的節奏啊。至於根據文件名去下載Ftp上面的回執文件,也不須要再從這個文件夾裏面遍歷出全部_1的文件名,再去下載,這多慢啊,何不記錄在redis裏面或者memory存儲類型的mysql數據表中呢?還有生成交易報文,上傳交易報文,取交易報文回執都是在同一個線程裏面的,爲何不使用多我的定時任務計劃,多個進程呢?我一直想不明白.........
不過海關的技術人人員也真是奇葩,訂單報文竟然用XML,還一條記錄一個XML文件,也不讓壓縮上傳再解壓,回執目錄裏面還存放幾家企業的回執報文.........只能說國企真心有錢,根本不用擔憂網絡流量和咱們這邊拋單的性能,卡死咱們這些傻不拉唧的電商企業,找一點存在感。
我之前在的一家公司是作城市一卡通的,如今深圳通啊XX通啊基本上都是那公司作的,很老了,可是作的程序很穩定,天天上傳交易報文,一個客戶端的報文是一天一個文件,裏面的記錄是16進制,壓縮上傳,再解壓,分目錄存儲、分目錄備份,爲何?不闊氣啊,省錢啊,公交車的交易報文是用gprs無線傳輸的,若是按照海關這樣搞,不被搞死?也用過Ftp傳輸,交易量那麼大,沒卡死過,很穩定,由於我發現他們解決交易報文的方式不錯,而不是用了啥新技術啊什麼的,如今的公司剛成立2年,一天幾千單訂單報文,竟然把本身的Web服務器搞死了,哪裏的移植?從網上copy了正確的代碼可是用在了錯誤的解決問題的思惟上面。海關報文弄XML,還一個文件一個,我呢個去,用FTP上傳,那麼多電商企業往你海關FTP服務器上面傳,你還不讓人家打包,做死了.....海關把本身服務器做死了,咱們不能把本身的服務器也做死啊!
四、foreach或者for循環中,N次更新、插入、刪除數據庫操做
我很好奇,爲何在循環裏面要進行一次數據庫鏈接、而後執行數據庫操做,而後關閉鏈接,爲何不把須要進行數據庫操做的內容保存在一個集合裏面,最後一次性去執行Sql操做,讓循環裏面只進行一次打開、執行、關閉操做呢?人家ORM框架裏面都有了批處理操做....
我在項目中看見不少地方,批量導入,批量更新回執狀態的時候,常常看見這樣的C#代碼,抓狂,想把寫代碼的人拉去吊着打...........
五、系統後臺,是一個完整的界面,也就是說,不管你點擊啥,整個界面刷新,對就是整個界面刷新,刷新還要每次從session裏面讀出html把菜單從新加載一遍,我第一次看到,眼睛就被刷瞎了.......而後左側菜單欄,爲了保持記錄選中狀態,而後各類js+cookies就出來了,幾百行js穿插其中,我艹,當時想重構的心都沒有,就想重寫(但是誰給我時間?爛攤子一個...),還有那奇葩的資源角色權限管理,竟然是半自動的,市面上面早就是全自動了,什麼是半自動,就是後臺界面配置了,代碼裏面還要去修改,並且仍是硬編碼,我真的是長見識了,隨便從網上亂下載一個源碼也不至於這樣啊!至於爲何能作出這樣的項目,我不想說話。最後客戶終於忍受不了,要求改,我就安排一個同事改了,用最簡單的方式弄個frameset,而後把那幾百行戶js刪除,終於只刷新中間部分了,唉!記得當時美工給個人(我當時作財務系統http://www.cnblogs.com/jiekzou/p/4508392.html)和給這個項目開發者的是同一個界面,可作出來後就是天壤之別了....
省略一萬字,這些都是最最基本的東西,爲何一些工做幾年了的程序員會犯這樣的錯誤,估計是懶得思考吧,程序員是要學會偷懶,可是要勤于思考.....
提示:細心的朋友,會發現本文if的實現方式有bug,少了return,項目中就有人是這樣乾的......