寫出優雅簡明代碼的論題集 -- Csharp(C#)篇[1]
最近和一些朋友討論如何寫出優雅的代碼，我們都很喜歡C#，所以以C#為例。主要一共有三位程序員在一起討論，為簡單起見我用ABC代表我們三個人。
有時候我們會針對一些代碼進行討論，有時候我們會提出一些觀點，有時候我們會一起學習網上一些現有的博客，為了便于大家引用，我給每一個論題都編上號。
在很多情況下，我們的意見統一，那么我會給大家呈現我們的結論；但是有些情況我們有分歧。
你可以加入我們的討論，我非常也希望能夠獲知你的意見，讓我們一起茁壯成長！
好吧，讓我們今天就開始。
論題一：函數越小越好!
相信絕大部分程序員會認同這一點，維護一個超過100行的函數會讓人抓狂。
我記得我以前修改過一個用cobol寫的程序，一個文件超過10萬行，我為了進行一個極其小的修改花了3天的時間，而且最后自己也不知道會不會造成什么嚴重的后果。-- 這已經過去8年了，希望那段代碼運行良好。
到底理想狀態下，我們的函數應該不大于多少行？我們三個人的答案是：
A： 10 行
B： 15 行
C： 20 行
論題二：用 Linq 簡化代碼
Linq有時可以幫助我們寫出一些非常“人性”的語句。
下面的這個函數是用于在數據庫中插入新的評論：
{
// validate params
if (null == cn) throw new ArgumentNullException("cn");
if (cn.State != ConnectionState.Open) throw new ArgumentException("Invalid parameter: connection is not open.", "cn");
if (null == Comments) throw new ArgumentNullException("Comments");
foreach (CommentData data in Comments)
{
if (data.CommentId.HasValue)
throw new ArgumentNullException("Create is only for saving new data. Call save for existing data.", "data");
}
...
其中foreach這一部分可以簡化為：
{
throw new ArgumentNullException("Create is only for saving new data. Call save for existing data.", "data");
}
在這一點上，我們存在分歧，A認為沒有必要進行簡化，因為原來的已經很明確了；但B認為簡化后的代碼可讀性更強，看上去更加直接。
論題三：集合初始值
希望每個人都已經知道C#的這個用法了，直接上一些代碼：
3.1
原始代碼：
idsToFind.Add(1);
idsToFind.Add(2);
修改后：
3.2
原始代碼：
startingPoint.X = 5;
startingPoint.Y = 13;
修改后：
論題四：運用 ？：和？？
據說，有些公司會拿這個來測試入門的程序員：
4.1
原始代碼：
System.Console.WriteLine(c.Name);
else
System.Console.WriteLine("List element has null value.");
修改后：
4.2
原始代碼：
if (value == null)
{
name = string.Empty;
}
修改后：
還可以更簡單，變成：
論題五： 運用AS
原始代碼：
{
var salEmp = (SalariedEmployee)employee;
pay = salEmp.WeeklySalary;
// ...
}
修改后：
if (salEmployee != null)
{
pay = salEmployee.WeeklySalary;
// ...
}
論題六： 運用 using
原始代碼：
{
var orders = new List<Order>();
var con = new SqlConnection("some connection string");
var cmd = new SqlCommand("select * from orders", con);
var rs = cmd.ExecuteReader();
while (rs.Read())
{
// ...
}
rs.Dispose();
cmd.Dispose();
con.Dispose();
return orders;
}
這是一段非常丑陋的代碼，我們完全迷失在dispose群中，什么時候要調用哪個dispose啊？ 天哪？ 如果我們用 finally， 可以將代碼寫為：
{
SqlConnection con = null;
SqlCommand cmd = null;
SqlDataReader rs = null;
var orders = new List<Order>();
try
{
con = new SqlConnection("some connection string");
cmd = new SqlCommand("select * from orders", con);
rs = cmd.ExecuteReader();
while (rs.Read())
{
// ...
}
}
finally
{
rs.Dispose();
cmd.Dispose();
con.Dispose();
}
return orders;
}
看看using到底給我們帶來了什么：
{
var orders = new List<Order>();
using (var con = new SqlConnection("some connection string"))
{
using (var cmd = new SqlCommand("select * from orders", con))
{
using (var rs = cmd.ExecuteReader())
{
while (rs.Read())
{
// ...
}
}
}
}
return orders;
}
{
var orders = new List<Order>();
using (var con = new SqlConnection("some connection string"))
using (var cmd = new SqlCommand("select * from orders", con))
using (var rs = cmd.ExecuteReader())
{
while (rs.Read())
{
// ...
}
}
return orders;
}
未完待繼…
留言列表
