写代码是一种艺术。使用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
连接:?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;