Design: condivisione di una classe tra più forme

While coding, I came across a realization that I am not creating 'loose coupling'. The way I am trying to solve the problem right now is creating an instance of the Log Collector Class in MAIN and passing it around to each form classes constructor.

Questo è il modo migliore per andare.

Un singleton funzionerà, ma porta con sé tutti i tipi di rischi man mano che la tua applicazione cresce, specialmente se hai intenzione di implementare i test delle unità.

Il problema più grande con un singleton è che sarà disponibile per ogni singola classe nella tua applicazione, non solo per quelli a cui hai consentito l'accesso. E, come vedrai più avanti, non puoi più separare l'accesso a determinate funzionalità tramite l'interfaccia.

but than thought that if two forms need to update this class at the same time, there would be an issue.

Questo vale sia che si tratti di un Singleton o di una singola istanza detenuta da ciascun modulo. Ma, se è il modulo che invia il messaggio, anche in un'applicazione multi-thread, tutto ciò si verificherà nel thread dell'interfaccia utente, quindi non è un problema.

Se c'è qualche tipo di pericolo allora un semplice lock () risolverà il problema per te.

Would an Interface for the Log Collector Class be best used here for all forms wanting to communicate with it?

Sì. Decisamente. Anche se estendi il tuo mediatore in un secondo momento per ricevere più messaggi da ogni modulo, disponi di più interfacce e tieni lo stesso oggetto in ogni campo.

ad es. (nota come un singleton non è così attraente ora?)

public class FooForm : Form
{
    private ILogCollector _logCollector;
    private IDataRequestor _dataRequestor;

    public FooForm (FormMediator mediator)
    {
        _logCollector = mediator;
        _dataRequestor = mediator;
    }

    // Stuff here
}

In questo modo, se un altro modulo può solo registrare eccezioni e non richiedere dati, non è necessario esporre tale funzionalità al modulo.

Questo è il "I" in SOLID : Principio di segregazione dell'interfaccia .

if two forms need to update this class at the same time, there would be an issue

Due moduli non possono aggiornare lo stesso oggetto collettore di log esattamente nello stesso momento, a meno che non si trovino in thread diversi. Se i moduli sono in esecuzione in thread diversi ma tutti condividono lo stesso oggetto collector di log (o qualsiasi altro oggetto), è necessario riflettere attentamente su come mantenere il log collector in uno stato coerente. Potrebbe diventare piuttosto complicato.

Una soluzione comune a questo tipo di problema riguarda l'uso di un mutex per impedire che due thread eseguano entrambe le modifiche contemporaneamente. Un altro è che i thread postino messaggi a una coda che il log collector può quindi elaborare in sequenza. Prima di saltare a provare a risolvere il problema, leggi la programmazione multithread. È facile sbagliare qui e notoriamente è difficile eseguire il debug dei problemi di interazione dei thread.