VPF::Asm вставки - Форум программистов

valex13

Дата 21.3.2005, 06:33 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 243
Регистрация: 29.1.2003
Где: Иркук. область, г . Иркутск

Репутация: нет
Всего: 1

Господа!
Имеется следующий код на С++ Builder 5. Осуществляет поиск в TList элемента.

Код


...
    TList* ContractList;
...

ontract* UnContracts::FindContract (int ClientID)
{

   bool bFind= false;
        for (int i=0;i<=ContractList->Count-1;i++)
        {
          Contract *m_Dog = (Contract*)ContractList->Items[i];
          if (m_Dog->ClientID ==ClientID)
           {
                //Запись найдена!
                bFind = true;
                //Удаляю ссылку, чтобы не наткнуться повторно
                ContractList->Delete (i);
                return  (m_Dog);
           }
        }
        if (!bFind)
          return (NULL);
}

Я выяснил, что имеенно эта часть кода работает дольше всега, т.к. количество записей велико.
Я бы хотел заменить поиск реквизита asm вставкой
Как мне на asm написать следующую строку ?

Код


Contract *m_Dog = (Contract*)ContractList->Items[i];

UhaMurad

Дата 21.3.2005, 08:24 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 103
Регистрация: 22.9.2004
Где: Махачкала

Репутация: нет
Всего: нет

А что за тип - Contract? Видимо преобразование типов идёт долго.

valex13

Дата 21.3.2005, 10:00 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Код


struct Contract
{
  int ClientID;                          //Код клиента 
  int iContrTypeID;                 //Код типа контракта
  AnsiString strDogNumber;    //Номер договора
  AnsiString strDateDogOpen;    //Дата заключения договора
};

UhaMurad

Дата 21.3.2005, 21:32 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 103
Регистрация: 22.9.2004
Где: Махачкала

Репутация: нет
Всего: нет

Ну не знаю, думаю, что asm-вставки особо не помогут. Могу кинуть пару советов, хотя и они вряд ли помогут:

-> вместо

Код


Contract *m_Dog = (Contract*)ContractList->Items[i];
if (m_Dog->ClientID ==ClientID)

лучше будет

Код


if (((Contract*)ContractList->Items[i])->ClientID == ClientID)

-> на всякий случай лучше будет передать ContractList в качестве параметра функции (можно параметром по умолчанию)

-> лучше не использовать в качестве аргумента ClientID (сделай лучше argCilentID)

valex13

Дата 22.3.2005, 04:05 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Код


if (((Contract*)ContractList->Items[i])->ClientID == ClientID)

А в чем разница? Переменная так же создается, только код компактней. Давно известно, что цикл на asm в 6 раз быстрее работают.

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C++ Builder \| Следующая тема »