🚀 go-pugleaf

> That is dependant on the speed of the DB,

Inserting new record in MS ACCESS (*.mdb) with
TADOTable is prety fast from my experience.

> and the speed of the protection that you use for
> guarding the DB from threading issues.

The only protection I know for are CriticalSections
(which I learned from you).

// PSEUDO !!!
bool AddNewRecord()
{
   RAII_LOCK();

   DB->Open();
   DB->TABLE->InsertNewRow(DATA);
   DB->Close();
}


> Why are you writing to the DB in the OnGetThread event
> handler, though? That is a very odd place to do it.  Or do
> you mean that your DB code is in the thread that is started
> by OnGetThread and is not inside of OnGetThread itself?

This is what I ment / I shortened the code as far as I could /

void __fastcall TAccountsService::ServerSocketGetThread(TObject *Sender,
      TServerClientWinSocket *ClientSocket,
      TServerClientThread *&SocketThread)
{
   SocketThread = new TAccountsServerThread( false, ClientSocket );
}
//---------------------------------------------------------------------------

void __fastcall TAccountsServerThread::ClientExecute()
{
   TMemoryStream *InboxMemoryStream = new TMemoryStream;
   TWinSocketStream *Stream = new TWinSocketStream ( ClientSocket, 5000 );

   try
   {
      while ( ( !Terminated ) && ( ClientSocket->Connected ) )
      {
         if ( Stream->WaitForData ( 5000 ) )
         {
            if( !ReadData ( Stream, InboxMemoryStream, 1000 ) )
            {
               ClientSocket->Close();
               break;
            }

            // SAVING TO DB CODE (PSEUDO, SHOWN ABOVE)
           bool Result = AddNewRecord();

            SendingNotificationToClientIfTheTaskWasSuccessfulOrNot( Stream, 
InboxMemoryStream, Result );
         }
      }
      // From here connection is closed ...
   }
   __finally
   {
      delete Stream;
      delete InboxMemoryStream;
   }
}


-- 
Best regards,
Vladimir Stefanovic