Delphi中使代碼簡潔的 5 條忠告(轉)

標籤：

 

寫代碼是一種藝術。使用Delphi，任何人都可以輕而易舉地開發出某種軟體、完成某些任務。而完美的代碼則只有真正的高手才能寫出。除了正確的縮排、大小寫、命名規則之外，請時刻牢記愛因斯坦的名言--簡單就是美。下面將談及的五個代碼問題，可能是初學者、甚至一些老鳥都會犯的錯誤。

忠告一

布爾型變數的賦值操作應該是直接的。例如，在一個if/then/else語句中，if子句將布爾型變數賦值為True，而else子句將其賦為False。下面這段代碼的寫法是不好的：

if If_Love_Delphi then
  Result:=True
else
  Result:=False;

而這樣寫就比較好：

Result:= If_Love_Delphi;

忠告二

避免使用嵌套的if/then/if語句，而用and來代替。下面這段代碼太羅嗦：

if If_Love_Delphi then
  if If_Love_Linux then
TryKylix(Now);

應該這樣寫：

if If_Love_Delphi and If_Love_Linux then
  TryKylix(Now);

不用擔心後面的判斷語句會超前執行。Project|Options|Compiler|Syntax Options|Complete Boolean eval選項通常是關閉的（除非你選定這個項），這保證了執行順序不會顛倒。

綜合前兩個忠告，假如你有一段這樣的代碼：

if If_Love_Delphi then
  if If_Love_Linux then
Result:=True;

就可以把它改成：

Result:= If_Love_Delphi and If_Love_Linux；

簡單而言，假如結果取決於一個條件判斷，那麼，Result:=True或者Result:=False這樣的語句就是多此一舉。在初始化布爾型變數的時候，可以給它們賦值。不過根本用不著把一個布爾型變數初始化為False--Delphi在建立這個變數的時候就已經把它賦職位False了。相似的情況還有：

對象的布爾型屬性(Boolean)，自動被初始化為False (0)；
整型變數(Integer)，自動被初始化為 0；
字串(String)，自動被初始化為空白字串。

忠告三

判斷布爾型變數的值時，無需用"=True"或者"=False"這樣的語句。下面的寫法不好：

if (If_Love_Delphi=True) and
  (If_Love_Linux=False) then
    DoNotTryLinux;

對於函數的傳回值或者一個屬性是布爾型的情況，應該這樣寫：

if If_Love_Delphi and
  not If_Love_Linux then
DoNotTryLinux;

忠告四

盡量不要用"+"操作符進行字串合并。這樣做效率太低了。下面的例子不好：

ShowMessage(‘在下身高‘+IntToStr(iHeight)+‘米，體重‘+IntToStr(iWeight)+‘公斤。‘);

這樣寫會較好：

ShowMessage(Format(‘在下身高%d，體重%d。‘, [iHeight,iWeight]));

忠告五

盡量多用with語句。它不僅效率高，而且使代碼更加易讀。比如，這段代碼：

if Sender if TEdit then
  if (TEdit(Sender).Text=‘) or
(TEdit(Sender).Text[TEdit(Sender).SelStart]=‘) or
  (TEdit(Sender).SelLength=
Length(TEdit(Sender).Text)) 
and (Key in [‘a‘..‘z‘]) then
Key:=UpperCase(Key);

就不如這樣的代碼來得簡潔易讀：

if Sender is TEdit then
  with Sender as TEdit do
if (Text=‘) or
   (Text[SelStart]=‘) or
   (SelLength=Length(Text)) and
   (Key in [‘a‘..‘z‘] then
  Key:=UpCase(Key);

 

 

 

(*//
標題:Delphi中建議使用的語句
整理:Zswang
串連:http://www.csdn.net/Expert/TopicView1.asp?id=724036
日期:2002-06-22
支援:wjhu111#21cn.com
//*)

{ No.1 判斷邏輯類型 }
var B: Boolean;
begin
  B := Boolean(2); //這樣只是為了調試//B := True;
  if B = True then ShowMessage(‘B = True‘); //不建議//不安全
  ///////
  if B then ShowMessage(‘B‘); //建議//簡短
end;

var B: Boolean;
begin
  if Edit1.Text = ‘是‘ then //不建議//煩瑣
    B := True
  else B := False;
  ///////
  B := Edit1.Text = ‘是‘; //建議//簡短
end;

{ No.2 臨時SQL查詢 }
begin
  QueryTemp.Close;
  QueryTemp.SQL.Text := ‘SELECT SUM(金額) AS 合計 FROM 銷售表‘;
  QueryTemp.Open; //不建議//資料沒有關閉造成資源浪費
  ShowMessage(Query1.FieldByName(‘合計‘).AsString);
  /////
  QueryTemp.SQL.Text := ‘SELECT SUM(金額) AS 合計 FROM 銷售表‘;
  QueryTemp.Open;
  ShowMessage(Query1.FieldByName(‘合計‘).AsString);
  QueryTemp.Close; //建議用//使用完就關閉
end;

{ No.3 擷取記錄數 }
var
  vRecordCount: Integer;
begin
  Query1.SQL.Text := ‘SELECT * FROM Table1‘; //不建議//嚴重浪費資源，會取得很多不必要得資訊
  Query1.Open;
  vRecordCount := Query1.RecordCount;
  Query1.Close;
  /////
  Query1.SQL.Text := ‘SELECT COUNT(*) AS 記錄數 FROM Table1‘; //建議//快速有效、只處理一條記錄
  Query1.Open;
  vRecordCount := Query1.FieldByName(‘記錄數‘).AsInteger;
  Query1.Close;

  ShowMessage(IntToStr(vRecordCount));
end;

{ No.4 欄位賦值 }
begin
  Table1.Edit;
  Table1.FieldByName(‘姓名‘).AsString := Edit1.Text; //不建議
  Table1.FieldByName(‘日期‘).AsDateTime := Date;
  /////
  Table1[‘姓名‘] := Edit1.Text; //建議//簡短、擴充性好
  //Table1.Fieldvalues[‘姓名‘] := Edit1.Text; //Borland建議的方法。以及ParamValues[]
  Table1[‘日期‘] := Date;
end;

{ No.5 使用Self指標 }
begin
  Edit1.Parent := Form1; //不建議//Form1隻是一個變數//如果沒有分配資源怎麼辦？
  ///////
  Edit1.Parent := Self; //建議
end;

{ No.6 遍曆資料集 }
var
  I: Integer;
begin
  Query1.First;
  for I := 0 to Query1.RecordCount - 1 do begin //不建議//容易被影響
    Query1.Next;
    { };
  end;
  /////
  Query1.First;
  while not Query1.Eof do begin //建議
    { };
    Query1.Next;
  end;
end;

{ No.7 利用Sender參數，使代碼通用 }
procedure TForm1.Edit1Change(Sender: TObject);
begin
  if Edit1.Text = ‘‘ then //不建議
  Edit1.Color := clRed;
  ///////
  if TEdit(Sender).Text = ‘‘ then //建議//複製到EditXChange中很方便
  TEdit(Sender).Color := clRed;
end;

{ No.8 使用預設轉換函式 }
var
  I: Integer;
begin
  I := StrToInt(Edit1.Text); //不建議
  ///////
  I := StrToIntDef(Edit1.Text, 0);//建議//參考StrToFloatDef,StrToDateDef....不過這些只有Delphi6才有
end;

{ No.9 遍曆數組 }
var
  I: Integer;
  A: array[0..9] of Integer;
begin
  for I := 0 to 9 do //不建議
  A[I] := I;
  ///////
  for I := Low(A) to High(A) do //建議//擴充性好
  A[I] := I;
end;

{ No.10 利用MaxInt常量 }
begin
  Caption := Copy(Edit1.Text, 3, Length(Edit1.Text) - 3 + 1); //不建議
  ///////
  Caption := Copy(Edit1.Text, 3, MaxInt); //建議//嘻嘻，少計算一次
end;

{ No.11 Result函數指標 }
function FuncName: Boolean;
begin
  FuncName := True; //不建議//並且放在賦值號右邊不能當普通變數
  ///////
  Result := True; //建議//擴充性好
end;

function FuncSum(A: array of Integer): Integer;
var I: Integer;
begin
  Result := 0;
  for I := Low(A) to High(A) do
    Result := Result + A[I]; //可不能用 FuncSum := FuncSum + A[I];
end;

{ No.12 必須執行的代碼、使用try ... finally ... end語句 }
var
  vStringList: TStringList;
begin
  vStringList := TStringList.Create;
  vStringList.LoadFromFile(‘c:\temp.txt‘);
  ShowMessage(vStringList.Text);
  vStringList.Free; //不建議//如果出現異常資源將無法釋放
  ///////
  vStringList := TStringList.Create;
  try
    vStringList.LoadFromFile(‘c:\temp.txt‘);
    ShowMessage(vStringList.Text);
  finally //建議//即使出現Exit都會執行
    vStringList.Free;
  end;
end;

//其他情況1
begin
  Screen.Cursor := crHourGlass;
  try
  { 耗時操作 }
  finally
  Screen.Cursor := crDefault;
  end;
end;
//其他情況2
begin
  Query1.DisableControls;
  try
  { 操作資料集 }
  finally
    Query1.EnableControls;
  end;
end;




>>對象的布爾型屬性(Boolean)，自動被初始化為False (0)；
>>整型變數(Integer)，自動被初始化為 0；
>>字串(String)，自動被初始化為空白字串。

這個不一定

局部變數最好初始化一下,不然會產生莫名錯誤.
特別是: Integer, string, Pointer
偶的經驗
Table1[‘姓名‘] := Edit1.Text;   這個可以  但要是：
Edit1.Text := Table1[‘姓名‘]   //這個絕對不行，因為這時候要是NULL就會報錯 這時候要FieldByName

己找到答案了，AnsiString也可以在Format和sprintf裡用"%s"裡指代.

俺想少數兩三個AnsiString還是可以用"+"來串連，而更多的AnsiString應該用sprintf或Format。

比較一下

strMsg ="姓名: " + name + "年齡: " + IntToStr(age) + "號碼: " + IntToStr(num) + "工資: " + IntToStr(salary);

和

strMsg.sprintf("姓名: %s年齡: %d號碼: %d工資: %d",name,age,num,salary);

後者可讀性比前者要略好，產生的機器碼只有1/3, 更重要的是可以把sprintf裡的格式字串提出來集中維護，而前者用"+"導致的大量零碎字串就不好處理。

後者唯一缺點是當有大量參數時，需要花一點工夫將格式字串和相應的參數對齊。









忠告1: 如果要用到with的地方,請想想能不能重構,將其作為一個函數使用
忠告2: 對於資料集的CLOSE請使用
       try
         DataSet.Open;
       finally
         DataSet.Close;
       end;
忠告3: 使用Sender時,請先判斷一下是不是nil, 是不是非你要的那個類型
忠告4: FieldValue對null的處理實在是差,請還是使用FieldByName().AsXXXXX這個
忠告5: 如果你要寫一個長長的判斷,還是寫成一個函數吧,不要寫進一個if then中,那樣,幾天后,你就將看不懂了









1、關於WITH，我個人基本不喜歡用，因為他帶來的是可讀性非常差。一個參數類，幾個屬性的時候，用的還好，而且項目組人同要比較熟練這個類！如果二個以上的參數類，天知道怎麼讀這一小段的WITH代碼。
2、關於字串，用+和Format這兩個我一般是看情況的。有些字串邊境用Format就很不好使，比如對一些SQL語句的拼字，多個SQL用+串連代碼易讀性高。而且這個效率還算好！然而一些樓主都提及的字串，當然用Format要好，同樣易讀性好，我覺得在使用到字元（串）操作的時候可讀性是主要的，如果跟效率搭上邊了，長字串處理，我覺得已經是設計問題了。
3、關於FOR和WHILE的比較，資料集的遍曆當然用WHILE。RecordCount除非你不用資料集過渡，如果用了，這個系統大了，維護死你，不可預見性太高了。這不是效率問題，是你寫出來的軟體能不能用的問題。
4、關於FieldByName。這個嘛提下，我一直都是用它的。除非特殊性情會用Fields[]。要不然，別人的軟體交給你維護的時候，你就等著哭吧，查欄位你就得花上所有的加班時間。。

可能我說的沒道理，大家就指出來，如果能證明就更好了，這樣我又學到東西了。。

Delphi中使代碼簡潔的 5 條忠告(轉)