Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > Delphi: Общие вопросы > Как красиво реализовать


Автор: RinOSpro 11.12.2008, 15:36
Есть множество однотипных процедур в основе которых лежит цикл. Вместо того чтобы в каждой организовывать цикл. Лучше сделать основной цикл. и как то передавать туда в цикл нужную процедуру:
Код

  ADataSours.DisableControls;
  ADataSours.First;

  while not ADataSours.Eof do
  begin
    // а вот тут должны вызываться множество однотипных процедур
  end;

    ADataSours.Next;

    end;


Как это красиво реализовать?

Автор: Poseidon 11.12.2008, 15:46
Цитата(RinOSpro @  11.12.2008,  15:36 Найти цитируемый пост)
Есть множество однотипных процедур в основе которых лежит цикл.
Не мешало бы и их увидеть

Автор: Bose 11.12.2008, 15:47
Например так:

Код

// объявляя типы процедур
type
  TMyDatasetProcObj = procedure (aDataset:TDataset) of object; // для методов классов
  TMyDatasetProc = procedure (aDataset:TDataset); // для процедур

procedure ForEachRecordPerform(aDataset:TDataset; aProc:TMyDatasetProcObj);
begin
  aDataset.DisableControls;
  try
    aDataset.First;
    while not aDataset.Eof do
    begin
      aProc(aDataset);
      aDataset.Next;
    end;
  finally
    aDataset.EnableControls;
  end;
end;


Минус в том, что для процедур и методов класса придётся писать отдельные реализации ForEachRecordPerform, лучше всего выбрать один тип и работать только с ним.

п.с. не уверен что понял вопрос  smile 

Автор: RinOSpro 11.12.2008, 15:55
Bose ты правильно понял вопрос... я тоже думал об этом но почему то этот вариант меня не устроил... Это нормально так делать?

Автор: bems 11.12.2008, 16:00
Цитата(RinOSpro @  11.12.2008,  15:55 Найти цитируемый пост)
Это нормально так делать? 
хочешь об этом поговорить?  ))
Тут все нормально.

Автор: RinOSpro 11.12.2008, 16:10
У меня не процедуры а функции, и проблема будет вот в чем: обычно я возвращаю Boolean но в одном случае придется string, вот тут то придется думать...

А так действительно красиво smile

Код

  LoopAlongTable(DataUpdate, FindOldVersion);

Автор: Bose 11.12.2008, 17:25
Цитата(RinOSpro @  11.12.2008,  15:55 Найти цитируемый пост)
Это нормально так делать? 

Нормально писать для каждого случая свой код. В том смысле нормально, что чаще всего делают именно так.  smile 


Цитата(RinOSpro @  11.12.2008,  16:10 Найти цитируемый пост)
У меня не процедуры а функции, и проблема будет вот в чем: обычно я возвращаю Boolean но в одном случае придется string, вот тут то придется думать...

В этом случае наверное так:
Код

type
  TMyDatasetObjFuncBool = function (aDataset:TDataset):Boolean of object; // для методов классов
  TMyDatasetObjFuncString = function (aDataset:TDataset):string of object; // для методов классов
procedure ForEachRecordPerformBool(aDataset:TDataset; aProc:TMyDatasetObjFuncBool); 
begin
  aDataset.DisableControls;
  try
    aDataset.First;
    while not aDataset.Eof do
    begin
      aProc(aDataset);
      aDataset.Next;
    end;
  finally
    aDataset.EnableControls;
  end;
end;

procedure ForEachRecordPerformString(aDataset:TDataset; aProc:TMyDatasetObjFuncString ); 
begin
  aDataset.DisableControls;
  try
    aDataset.First;
    while not aDataset.Eof do
    begin
      aProc(aDataset);
      aDataset.Next;
    end;
  finally
    aDataset.EnableControls;
  end;
end;


Как это лучше сделать в принципе зависит от задачи. Наверное можно вместо двух типов объявить один, с функцией возвращающей variant.
Код

type
  TMyDatasetObjFuncVar = function (aDataset:TDataset):variant of object; // для методов классов





п.с. Можно конечно объявить функции с одинаковыми названиями и overload-ом. Но в последнее время overloading мне не нравится.  smile 

Автор: RinOSpro 11.12.2008, 18:58
Bose в той функции что возвращала string, сделал ее так что теперь она возвращает boolean а string пишет в глобальную переменную.

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)