Общие вопросы программирования в C++Builder

[Marvin]

Изучаю процессы...
есть класс

Код:

class TMyThread : public TThread
{           
private:
protected:
  void __fastcall Execute();
public:
  AnsiString str;
  void __fastcall AddText();
  __fastcall TMyThread(bool CreateSuspended);
};

Код:

void __fastcall TMyThread::AddText()
{
  Form1->Memo1->Lines->Add(str);
}

void __fastcall TMyThread::Execute()
{
  Synchronize(AddText);
}

Код:

void __fastcall TForm1::Button1Click(TObject *Sender)
{
  TMyThread *MyThread;
  MyThread = new TMyThread(true);
  MyThread->str = "qqqwweewqqwe";
  MyThread->Resume();
  delete MyThread;
}

насколько я понимаю, методу Execute() управление должно передаться после выполнения команды MyThread->Resume(); ?
но этого непроисходит, и соответствнно в мемо ничего не выводится. Может я неправильно понимаю процесс выполнения TThread...

в инете нету таких примеров на си чтобы в подробностях разобраться, если ктонибудь знает где можно раздобыть боль-мень подробную инфу, пошлите меня туда по ссылке плиз (исключая гугл и пр. поисковики ) или посоветуйте книгу какую-нибудь где это описывается достаточно хорошо. Даже у Страуструпа нету....
спасибо

[Marvin]

по поводу ошибки я поспешил, извините, просто я недождавшись завершения процесса удалил его... втыкнув WaitFor() в нужное место проблемма решилась сама собой

но про инфу вопрос остается открыт

[Marvin]

а что может означать [linker Error]Unresolved External referenced from ?
функция такая существует, модуль подключил...

[IT-IR]

скорее всего необходимо дописать #pragma link "файл"

[Marvin]

скорее всего необходимо дописать #pragma link "файл"

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

[Zor]

// кнопка "дополнит опции"

Resume не запускает поток, а помечает его как готовый для запуска. В след раз когда система будет нарезать время задачам, она даст квант времени и этому. Но не ранее чем закончится время у текущего потока (откуда вы и делали Resume)

[Marvin]

Zor, теперь понятно)))


P/S/ вот выкладываю проект, кому не лень, посмотрите пожалста, может еще чего посоветуете - с прочессами в си имею дело второй раз, так что код местами может быть карявым проект только на начальной стадии разработки, я его некомпилировал даже неразу.

спасибо.

[Marvin]

все, с линкер ерророй покончено, я просто тупо подключил не тот файл, вместо MD5.cpp подключил MD5.h)))
так же разобрался с ошибкой в строке ledtFileHash->Text = GHash->AHash; я просто ненаписал метод который возвращает значение этого свойства)))

З.Ы. как я уже писал, я только начал изучать процессы, и важи замечания/комментарии мне очень важны. прошу прощения за то, что выкладываю неготовый проект, просто хочу продолжить его написание с учетом ваших замечаний. тут я выложу подправленную версию (исправил ошибки о которых писал тут), жду ваших комментарий.
спасибо

[oxotnik]

подправить надо бы
void GetSHash(); заменить на
AnsiString GetSHash() {return FHash;};
только для разных файлов один и тот же хэш выводит

[Marvin]

подправить надо бы
void GetSHash(); заменить на
AnsiString GetSHash() {return FHash;};

только для разных файлов один и тот же хэш выводит

да, это я уже исправил в моем последнем сообщении выложил
про одинаковые хэши - действительно незаметил, спасибо oxotnik.
изменил чтобы в btnGetHashClick создавался экземпляр моего класса и там же по завершению работы удалялся, теперь хэши разные, но почему так? вроде поток завершается к тому времени когда выполняется WaitFor()... а так создавать/удалять его каждый раз при нажатии на кнопку как то "не экономично" получается. есть другой способ решения?

[Zor]

"не экономично", зато правильно. Завершенный по WaitFor() поток уже мертвый. Совсем.
К тому же создание потока для системы - милли/микро секунды, а расчет хеша - весьма долгий процесс.
Зачем экономить там где этого делать не надо?

подразумевается так:

Click()
{
создается поток по new()
устанавливаются параметры (файл, тип хеша и тд)
размораживается поток
// механизм ожидания расчета - на совести автора
дождались завершения - считываем результат
удаляется поток.
}

// опять Synchronize - я счаз заплачу 

[Marvin]

"не экономично", зато правильно...

теперь понял

// опять Synchronize - я счаз заплачу 

Zor, а чем вам Synchronize ненравится?

PS наталкнулся на такую ошибку
есть класс:

Код:

class TGetHash : public TThread
{
private:
  ...
  TStringList *FFileList;
  ...
  void SetFileList(TStringList *list);

protected:
  ...
  __property TStringList *AFileList={read=FFileList , write=SetFileList};
  ...

  void __fastcall Execute();
public:
  __fastcall TGetHash(bool CreateSuspended);
  __fastcall ~TGetHash();
};

при опсании метода SetFileList возникает ошибка Too few parametres in call to '__fastcall TStrings::Get(int)'

Код:

  ...
void TGetHash::SetFileList(TStringList *list)
{
  FFileList->Strings = list->Strings;
};
  ...

[Zor]

"а чем вам Synchronize ненравится" - нет реальной многозадачности. Практически прямой вызов CalcMDX() из вашей программы, никаких потоков 

FFileList->Strings = list->Strings; = Сбой программы во время работы. FFileList не создан

[Marvin]

нет реальной многозадачности. Практически прямой вызов CalcMDX() из вашей программы, никаких потоков

что такое CalcMDX()? в хелпе такого нету, по запросу в гугле ничего ненайдено...

FFileList->Strings = list->Strings; = Сбой программы во время работы. FFileList не создан

нет, не вовремя работы, а на этапе компиляции.
я же создаю FFileList в конструкторе:

Код:

__fastcall TGetHash::TGetHash(bool CreateSuspended)
  : TThread(CreateSuspended)
{
  FolderName = ExtractFileDrive(Application->ExeName);
  FFileList = new TStringList;
  FTypeHash = 0;
};

и удаляю его так:

Код:

__fastcall TGetHash::~TGetHash()
{
  delete FFileList;
};

неужели неправильно?

[Zor]

в архиве ее еще нет правильно перекинуть список:
FFileList->Text = list->Text;
// CalcMDx - подразумевались ваши функции подсчета хеша.

[Marvin]

в архиве ее еще нет правильно перекинуть список:
FFileList->Text = list->Text;
// CalcMDx - подразумевались ваши функции подсчета хеша.

спасибо Zor, все начало работать 

PS вот искал по форуму вопос, почему у меня даже с использованием процесса программа виснет если считать хэш большого файла? по идее процесс для того и создан, чтобы иметь доступ к главному окну приложения, и чтобы сохранить его функциональность во время выполнения процесса, на то он и процесс... вобщем ответа ненашол. кто сталкивался (наверня много кто) дайте знать решение этой проблеммы пожалста

[Zor]

старенький глухой дедушка повтор:
Synchronize = нет реальной многозадачности. Практически прямой вызов CalcMDX() из вашей программы, никаких потоков

[Marvin]

старенький глухой дедушка повтор:
Synchronize = нет реальной многозадачности. Практически прямой вызов CalcMDX() из вашей программы, никаких потоков

это я понял. но некоторые приложения так делают же под виндой. или они создают новую форму, из нее запускают задачу, прогрессят ее Update()'ами...? тогда какой вообще смысл в процессах?

[Zor]

Подсказка - убрать Synchronize? (Да!/Есть!/Так точно!) 
В основной программе мониторить завершение потока, затем выводить результат.
// чем проще - тем надежнее.

[Marvin]

гы, синхронайз я убрал после первого вашего поста об его ненужности)))

из проги мониторить завершение потока... а WaitFor() неподойдет?
мне кажется что на зависание проги влияет то что после поток->WaitFor() и прога сама ожидает завершения и ниче неможет больше делать... но убирать WaitFor() бессмысленно, тогда надо поместить его куда-нибудь, где он непомешает.. но куда?

или типо создать еще одну bool переменную в классе потока и менять ее при завершении определеного действия, мне кажется так проще, но первый вариант мне больше нравится если его можно осуществить конечно?

[Marvin]

тут вот какая ошибка еще возникла, в одной функции я использую SelectDirectory():

Код:

#include
  ...
  ...
  AnsiString Dir;
  bool status = SelectDirectory("Укажите папку, файлы из которой будут добавлены на рассчет хэшей", "", Dir);
  ... 
  ...

все безупречно работало пока я не снял галочки с Project->Options: вкладка Packagest строка Build with runtime packages и вкладка Linker строка Use Dynamic RTL.
теперь при компиляции возникает ошибка [linker Error]Unresolved External '__fastcall Filectrl::SelectDirectory(...)' referenced from .../main.obj

скорее всего необходимо дописать #pragma link "файл"

это тоже непомогает

из за чего ошибка возникла, и как ее исправить?

[IT-IR]

в  Directory укажи папку с obj

[Zor]

[linker Error] Unresolved external '__fastcall Filectrl::SelectDirectory(System::AnsiString&, System::Set<:tselectdiropt>, int)' referenced from ...\MAIN.OBJ

A posting from John Moore solves this longtime problem:
The problem is that for some reason, everything in filectrl.hpp is in
vclx50.lib.  There are 2 basic ways to fix this:
1)  Close your project, edit the .bpr file and add vclx50.lib to the
LIBRARIES and SPARELIBS lines:
   
   
2)  Add a DriveComboBox (or any other component in filectrl), in the Win 3.1
palette tab, to your form and then delete it.  A side effect of doing this
is that vclx50.lib is left in your .bpr file.


вам лучше по методу #2.
WaitFor собственно и тормозит основной поток пока работает другой. отслеживайте готовность таймером, что ли?

[Marvin]

[linker Error] Unresolved external '__fastcall Filectrl::SelectDirectory(System::AnsiString&, System::Set<:tselectdiropt>, int)' referenced from ...\MAIN.OBJ

A posting from John Moore solves this longtime problem:
The problem is that for some reason, everything in filectrl.hpp is in
vclx50.lib.  There are 2 basic ways to fix this:
1)  Close your project, edit the .bpr file and add vclx50.lib to the
LIBRARIES and SPARELIBS lines:
   
   
2)  Add a DriveComboBox (or any other component in filectrl), in the Win 3.1
palette tab, to your form and then delete it.  A side effect of doing this
is that vclx50.lib is left in your .bpr file.


вам лучше по методу #2.
WaitFor собственно и тормозит основной поток пока работает другой. отслеживайте готовность таймером, что ли?

да, сделал по 2 методу, и все начало рабортать.. почему так?
а при использовании первого метода у меня была Linker Fatal Error, там что то вроде ненайден vclx50.lib...

в  Directory укажи папку с obj

это где?)))

P/S/ всем спасибо, все работает с таймером ничего несделал пока, но в ближайшем будущем доделаю.
выкладываю проект, может кому-то будет интересно тут же можно посмотреть скриншот. жду ваших замечаний.

[Zor]

v2.0 - оптимист. я бы сказал v0.8 

[Romzes]

А что такое хаш, и в чем суть программы??? Я просто не знаю!

[Marvin]

v2.0 - оптимист. я бы сказал v0.8 

да, поторопился с версией, но пусть будет

А что такое хаш, и в чем суть программыЯ просто не знаю!

Хеш-суммой (хешем, хеш-образом, хеш-кодом) называется значение хеш-функции. В криптографии хеш-сумму иногда также называют дайджестом сообщения. Значение хеш-суммы может использоваться для проверки целостности данных, а также заменять собой данные, которые небезопасно хранить в явном виде (например, пароли, ответы на вопросы тестов и т.д.)

Явное значение хеш-суммы, как правило, записывается в шестнадцатеричном виде. Так, утилита md5sum, вычисляющая значение хеш-функции MD5 от данного файла, выдает результат в виде строки из 32-х шестнадцатеричных цифр - например, 026f8e459c8f89ef75fa7a78265a0025.

в моей програме я считаю хэш по алгоритму MD5, подробнее здесь http://ru.wikipedia.org/wiki/MD5
википедия рулит