Многоугольник, Fpc, Ооп, Оптимизация Опции

[Bokul]

Реализовал объект многоугольник, который имеет неопределенное количество вершин. Но скорость работы с ним критическая, поэтому надо как можно сильнее его оптимизировать.
Вот, что он из себя представляет:

  
unit Polygone;
interface
uses UnitTVector,ListOfVertex,objects;
    TPPolygone=^TPolygone;
    TPolygone=object
       List:TVertexList;//Кольцевой список
       constructor init;//инициализируем поля
       procedure   AddVertex(x,y:integer);//добавляем вершину, x,y - ее координаты
       procedure   Increase(Const vector:TVector);//Увеличиваем координаты всех вершины на вектор vector, описание - ниже
       procedure   Decrease(Const vector:TVector);//Уменьшаем координаты всех вершины на вектор vector, описание - ниже
       procedure   ForEach(proc:ToDO);//Проделываем процедуру типа ToDO с каждой вершиной
       function    ContainsPoint(point:TPoint):boolean;//входит ли точка point в объект
       function    IsInside(OtherPolyg:TPolygone):boolean;//входит ли объект в OtherPolyg
       function    ContainsOtherPolyg(OtherPolyg:TPolygone):boolean;// входит ли OtherPolyg в объект
       destructor  done;//Удаляем все поля
    end;

Кольцевой список

unit ListOfVertex;
interface
uses objects;
type
   ToDo=procedure(point:TPoint);//тип процедуры, вызываемой с ForEach

   TPVertexItem=^TVertexItem;//элемент списка
   TVertexItem=record
      vertex:TPoint;//информационная часть - координаты вершины
      next:TPVertexItem;
   end;

   TPVertexList=^TVertexList;
   TVertexList=object
      Last:TPVertexItem;
      constructor init;//инициализируем поля
      procedure   AddElement(data:TPoint);//добавляем новый элемент
      function    DeleteElement(p:TPVertexItem):boolean;//удаляем элемент
      procedure   ForEach(proc:ToDo);//Проделываем процедуру типа ToDO с каждым элементом
      destructor  done;//удаляемся
   end;

Модуль для работы с векторами

unit UnitTVector;
interface
Type
    TVector=record
       X,Y:integer;
    end;
Operator + (Const v1,v2:TVector) R:TVector; //складываем два вектора
Operator - (Const v1,v2:TVector) R:TVector;//отнимаем от первого вектора второй
Operator * (Const k:double; v:TVector) R:TVector;//перемножаем вектор на коэффициент k
Operator * (Const v:TVector; k:double) R:TVector;//тоже самое, только другой порядок
Operator / (Const v:TVector; k:double) R:TVector;//делим вектор на коэффициент 

Программа для проверки минимальной работоспособности

uses crt,ListOfVertex,objects, UnitTVector, Polygone;
procedure Show(p:TPoint);
begin
  writeln('X = ',p.x,' Y = ',p.y);
end;

var obj1,obj2:TPPolygone;
    p1:TPoint;
    mem:longint;
    vector:TVector;
begin
  clrscr;
  mem:=GetHeapStatus.TotalFree;
  vector.x:=4;
  vector.y:=8;
  p1.x:=40;
  p1.y:=50;
  new(obj2,init);
  with obj2^ do
       begin
            AddVertex(30,10);
            AddVertex(30,20);
            AddVertex(50,20);
            AddVertex(50,10);
       end;
  new(obj1,init);
  with obj1^ do
       begin
            AddVertex(10,10);
            AddVertex(40,60);
            AddVertex(1000,10);
            writeln('First');
            ForEach(show);
            writeln;
            writeln('Point x=',p1.x,' y=',p1.y);
            if ContainsPoint(p1) then writeln('First contains Point')
            else writeln('First does not contain Point');
            writeln; 
            Writeln('After decrease: ');
            writeln('Vector: x=',vector.x,' y=',vector.y);
            decrease(vector);
            ForEach(show);
            writeln;
            writeln('Second');
            obj2^.ForEach(show);
            if ContainsOtherPolyg(obj2^) then writeln('First contains second object')
            else writeln('First does not contain second object');
            if IsInside(obj2^) then writeln('Second is inside first object')
            else writeln('Second is not inside first object');
       end;
  dispose(obj1,done);
  dispose(obj2,done);
  writeln;
  writeln('Difference: ',mem-GetHeapStatus.TotalFree);
  readln;
end.

Три модуля вмести:  Polygone.rar ( 13.71 килобайт ) Кол-во скачиваний: 447


PS в принципе код прозрачен, но если что-то не понятно - спрашивайте.

[volvo]

где у тебя произошла такая утечка?

У меня? Я запустил только твою программу, буква в букву, как ты ее привел, так что утечка - у тебя...

Как в Fpc засекать время?

uses windows;
var T: dword;
...
T := gettickcount();
// ...
writeln('time = ', gettickcount() - T);

Я думаю, этого будет достаточно, если сделать огромное количество вычислений... Можно, конечно, и поточнее время замерять, но с этим, я думаю, не стОит заморачиваться...

А почему без ForEach - для этого процедуре надо передавать еще один параметр - вектор, но ForEach работает только с таким типом

Ну, во-первых, перегрузку функций еще никто не отменял, ты не в TP, а в ObjectPascal-е, кстати, можно было бы и перегрузить, если, конечно, такие функции будут вызываться достаточно часто... А во-вторых,

Вообще то сначала они у меня были в виде перегруженных операторов (+-), но никак не получалось заставить их правильно работать

- показывай, как перегружал, у меня после добавления перегрузки

operator + (const p: TPoint; const v: TVector) R: TPoint;
operator - (const p: TPoint; const v: TVector) R: TPoint;

в модуль UnitTVector вот такой код:

procedure TPolygone.Decrease(Const Vector:TVector);
var p, the_last:TPVertexItem;
begin
     the_last:=List.last^.next;
     p := the_last;
     repeat
          p^.vertex -= vector; // Как тебе такая запись ?
          p:=p^.next;
     until p = the_last;
end;

отрабатывает без проблем... Кстати, это тебе хинт, как немного оптимизировать программу, вместо того, чтоб каждый раз брать List^.Last^.next, берешь его один раз, и сравниваешь с текущим значением P