Question

好吧，伙计们，我一直试图找出我正在制造的每一个可能的错误，但我放弃......我需要帮助！我正在写的是一个应用程序来管理我的工作租金，当日期过去时，我的应用程序从2个文本文件中删除名称。我编写了3个小函数（程序）来完成这项工作。这里：

这个从dates.dat文件加载并删除包含该员工姓名的行。

procedure remDate(emp: String);/// Removes employee from date file
var
  pos1, i: integer;
  dateList: TStringList;
begin
  dateList:=TStringList.Create;
  dateList.LoadFromFile('Data\dates.dat');
  for i:=0 to dateList.Count-1 do begin
    pos1:=AnsiPos(emp, dateList[i]);
    if pos1<>0 then begin
      dateList.Delete(i);
      dateList.SaveToFile('Data\dates.dat');
    end;
  end;
  dateList.Free;
end; //eo remDate

这个从perm.dat文件中删除包含员工姓名的行。

procedure remPerm(emp: String);/// Removes employee from perm file
var
  pos1, i: integer;
  permList: TStringList;
begin
  permList:=TStringList.Create;
  permList.LoadFromFile('Data\perm.dat');
  for i:=0 to permList.Count-1 do begin
    pos1:=AnsiPos(emp, permList[i]);
    if pos1<>0 then begin
      permList.Delete(i);
      permList.SaveToFile('Data\perm.dat');
    end;
  end;
  permList.Free;
end; //eo remPerm

这个把它们粘在一起。 isDue是一个简单的函数，它比较2个日期，如果日期是今天或过去，则返回TRUE。

procedure updatePerms;
var
  empList: TStringList;
  i: integer;
begin
  empList:=TStringList.Create;
  empList.LoadFromFile('Data\employes.dat');
  for i:=0 to empList.Count-1 do begin
    if isDue(empList[i]) then begin
      remDate(empList[i]);
      remPerm(empList[i]);  (*) Here is where the error points.
    end;
  end;
  empList.Free;
end;

我得到的错误是它在updatePerms过程中到达remPerm。（*）我得到一个EStringList错误，超出范围（＃）。通过多次尝试得出结论，只有当员工的截止日期是今天时才会发生。如果您需要更多信息，请评论！在此先感谢，非常感谢任何帮助！

Answer 1

问题是你正在使用for循环。只有在输入循环时才会评估for循环的结束点。此时您可能有100个项目，但一旦开始删除，将会有更少的项目。这将导致列表索引超出范围错误。

简单的解决方法是反转for循环：

procedure remDate(emp: String);
/// Removes employee from date file
var
  pos1, i: integer;
  dateList: TStringList;
begin
  dateList := TStringList.Create;
  dateList.LoadFromFile('Data\dates.dat');
  for i := dateList.Count - 1 downto 0 do
  begin
    pos1 := AnsiPos(emp, dateList[i]);
    if pos1 <> 0 then
    begin
      dateList.Delete(i);
      dateList.SaveToFile('Data\dates.dat');
    end;
  end;
  dateList.Free;
end; // eo remDate

如果员工多次出现，这将有效。

但是，如果员工只出现一次，您可以使用break提前退出循环：

procedure remDate(emp: String);
/// Removes employee from date file
var
  pos1, i: integer;
  dateList: TStringList;
begin
  dateList := TStringList.Create;
  dateList.LoadFromFile('Data\dates.dat');
  for i := 0 to dateList.Count - 1 do
  begin
    pos1 := AnsiPos(emp, dateList[i]);
    if pos1 <> 0 then
    begin
      dateList.Delete(i);
      dateList.SaveToFile('Data\dates.dat');
      Break; // <-- early exit
    end;
  end;
  dateList.Free;
end; // eo remDate

另一种解决方案是使用while循环。

Delphi for循环和StringList错误

1 个答案: