Indy TCP-клиент в потоке

Должен ли TIdTCPClient быть в основной форме (как компонент времени разработки) или он должен быть переменной-членом класса потока?

Либо будет работать нормально. Важно то, что вы вызываете Connect() и другие методы ввода-вывода в контексте рабочего потока (внутри его Execute() метода).

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

Да, если они обращаются к элементам управления пользовательского интерфейса или другим общим данным, которые необходимо защитить.

Безопасно ли как для основного потока VCL, так и для клиентского потока выполнять вызовы функций для объекта TIdTCPClient?

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

На данный момент мой код заполнен временными переменными и функциями-заглушками, потому что Synchronize требует закрытия void (void), и это довольно спагетти.

Вы можете использовать классы Indy _3 _ / _ 4_, чтобы помочь вам лучше организовать этот спагетти-код. Например, унаследуйте класс от TIdSync, переместите в него свои переменные и переопределите его виртуальный метод DoSynchronize() для вызова ваших методов (ов) формы по мере необходимости. Затем вы можете создать экземпляр своего класса, при необходимости заполнить его переменные, вызвать его Synchronize() метод, прочитать его переменные, если необходимо, а затем освободить его.

#include <IdSync.hpp>

class TMySync : public TIdSync
{
protected:
    virtual void __fastcall DoSynchronize();
public:
    // variables...
};

void __fastcall TMySync::DoSynchronize()
{
    // call Form methods, use variables as needed...
}

void __fastcall TMyThread::Execute()
{
    //...
    TMySync *sync = new TMySync;
    // set variables as needed...
    sync->Synchronize();
    // read variables as needed...
    delete sync;
    //...
}