-1

我的查询，运行时大约需要 7 秒来完成应该做的事情。但是，由于它插入了大约 30 条记录，我认为它太慢了。现在，要么我运行写得不好的查询，要么它确实需要这么多时间。但这会很奇怪。底层数据库是 SQLite，查询如下所示：

procedure TForm1.cxButton1Click(Sender: TObject);
begin
with UNIquery2 do begin
  Close;
  SQL.Clear;
UNIQuery1.First;
while Uniquery1.EOF = false do begin
SQL.Text:= 'INSERT INTO MYTABLE (FIELD1,FIELD2,FIELD3,FIELD4) VALUES (:a1,:a2,:a3,:a4)';
         ParamByName('a1').asString := AdvOfficeStatusBar1.Panels[0].Text;
         ParamByName('a2').asString := UniTable1.FieldByName('FIELD2').asString;
         ParamByName('a3').asString := Uniquery1.FieldByName(',FIELD3').asString;
         ParamByName('a4').Value := Uniquery1.FieldByName('FIELD4').Value;//boolean field true/false
         Uniquery1.Next;
         ExecSQL;
end;
end;
end;

那么有人可以告诉我这是否可以或我错过了什么？除了布尔值（真/假）的“a4”之外，所有字段都是文本。

修改后的答案（基于 LS_dev 的建议）：

procedure TForm1.cxButton1Click(Sender: TObject);
begin
    with UNIquery2 do begin
        Close;
        SQL.Clear;
        SQL.Add('INSERT INTO MYTABLE (FIELD1,FIELD2,FIELD3,FIELD4) VALUES (:a1,:a2,:a3,:a4)');
        SQL.Prepare;
        UniTransaction.AddConnection(UniConnection2);
        UniTransaction.StartTransaction;
try
        UNIQuery1.First;
        while Uniquery1.EOF = false do begin
            Params[0].asString := AdvOfficeStatusBar1.Panels[0].Text;
            Params[1].asString := UniTable1.FieldByName('FIELD2').asString;
            Params[2].asString := Uniquery1.FieldByName(',FIELD3').asString;
            Params[3].Value := Uniquery1.FieldByName('FIELD4').Value;//boolean field true/false
            Uniquery1.Next;
            ExecSQL;
        end;
        UniTransaction.Commit;
finally
  if UNIquery2.Connection.InTransaction then
    UNIquery2.Connection.Rollback;
    end;
    end;
    end;
4

2 回答 2

8

不了解 Delphi，但会提出一些改进建议：

  1. 您没有使用交易。COMMIT在所有插入之后，您应该有类似自动提交禁用和命令之类的东西；

  2. 你SQL.Text:=...应该没时间了。如果此属性集编译 SQL 语句，将其置于 while 之外将防止不必要的 VDBE 编译；

  3. 如果您的意图是将行从一个表复制到另一个表（使用静态字段），您可以使用单个 SQL 命令，例如INSERT INTO MYTABLE SELECT :a1, FIELD2, FIEDL3, FIELD4 FROM source_table，设置ParamByName('a1').asString := AdvOfficeStatusBar1.Panels[0].Text

这是通用的数据库使用改进，希望能给你一些方向。

使用唯一 SQL 的建议：

procedure TForm1.cxButton1Click(Sender: TObject);
begin
    with UNIquery2 do
    begin
        SQL.Clear;
        SQL.Add('INSERT INTO MYTABLE (FIELD1,FIELD2,FIELD3,FIELD4) SELECT ?,FIELD2,FIELD3,FIELD4 FROM UNIquery1_source_table');
        Params[0].asString := AdvOfficeStatusBar1.Panels[0].Text;
        ExecSQL;
    end;
end;

使用改进的数据库处理的建议：

procedure TForm1.cxButton1Click(Sender: TObject);
begin
    with UNIquery2 do 
    begin
        Close;
        SQL.Clear;
        SQL.Add('INSERT INTO MYTABLE (FIELD1,FIELD2,FIELD3,FIELD4) VALUES (:a1,:a2,:a3,:a4)');
        SQL.Prepare;
        UniTransaction.AddConnection(UniConnection2);
        UniTransaction.StartTransaction;
        UNIQuery1.First;
        while Uniquery1.EOF = false do 
        begin
            Params[0].asString := AdvOfficeStatusBar1.Panels[0].Text;
            Params[1].asString := UniTable1.FieldByName('FIELD2').asString;
            Params[2].asString := Uniquery1.FieldByName(',FIELD3').asString;
            Params[3].Value := Uniquery1.FieldByName('FIELD4').Value;//boolean field true/false
            Uniquery1.Next;
            ExecSQL;
        end;
        UniTransaction.Commit;
    end;
end;
于 2013-12-10T10:06:46.837 回答
0

如果 SQL INSERT 本身很慢，我建议先在交互式客户端中测试其执行速度。或者编写一个简单的测试应用程序来执行一个硬编码的 INSERT 并测量其执行时间。

您还可以使用调试器、日志记录或分析器来找出代码中消耗时间的操作 - 例如Uniquery1.Next可能ExecSQL是

于 2013-12-10T08:35:32.753 回答