Design del modello di oggetto: raccolte su classi

Il modo più semplice è usare IEnumerable<Passenger> . Se hai esposto un List<Passenger> come IEnumerable, potresti preoccuparti che qualcuno lo trasmetta a un Elenco, ma personalmente non lo farei. Stanno rompendo il contratto se lo fanno, e tu non sei qui per impedirgli di spararsi ai piedi se vogliono. Il tuo compito è comunicare chiaramente ciò che dovrebbero fare con esso.

Se vuoi fornire la possibilità di ottenere un conteggio dei passeggeri o di recuperare un passeggero per indice, hai due opzioni:

1. Esporre metodi / proprietà pubbliche: public int PassengerCount { get; } , public Passenger GetPassengerByIndex(int index) (e potresti sbarazzarti dell'IEnumerable se lo desideri)
2. Esporre un ReadOnlyCollection<Passenger> anziché IEnumerable<Passenger> che accetta una lista nel suo costruttore.

Personalmente ritengo che la prima opzione sia migliore se stai definendo un'API pubblica perché ti offre più spazio per modificare l'implementazione senza influire sull'interfaccia. Inoltre, se il consumatore deve chiamare train.Passengers.Count , questo tipo di interruzione di uno dei principi OO (sui metodi di chiamata sul risultato dei metodi di un oggetto).

C'è un'altra differenza tra le due opzioni se la tua classe Train non è immutabile. Se ottieni un riferimento a train.Passengers e tienilo premuto, e poi qualcuno scende dal treno, quel cambiamento non si riflette nel riferimento che hai. Se usi la prima opzione, intendi che il numero di passeggeri che ricevi è solo "a partire da questo momento". Quindi la prima opzione implica la mutabilità, ma la seconda opzione implica l'immutabilità (almeno nella lista dei passeggeri).

Di solito quando passi le raccolte il modo più flessibile è passare IEnumerable < class >

Se hai davvero bisogno di impedire ad altri di modificare la tua raccolta che non funzionerà, dato che possono essere lanciati nell'Elenco originale o qualunque cosa fosse, allora ReadOnlyCollection sembra una buona scelta.

Non ho mai visto alcuna documentazione Microsoft che specifichi che un elenco non debba essere esposto pubblicamente. Se si osserva la definizione Elenco classi su MSDN, non si troverà alcuna menzione di questo . Dovrai fornire la documentazione per eseguire il backup delle asserzioni.

Nelle nostre applicazioni usiamo di solito Elenco (Of T) per le nostre collezioni. Sono semplici, digitano sicuri e quando li esponiamo tramite un servizio Web (e così via) vengono convertiti automaticamente in una matrice semplice (Of T). Abbiamo utilizzato questo approccio su innumerevoli sistemi senza molti problemi.

Modifica

Ho letto i post nei commenti, grazie a tutti per aver fornito la documentazione. Per noi, estendere le nostre lezioni non è un problema e il fatto che la serializzazione sia un problema con le raccolte ci mette nel campo List / IList.

Anche se sono d'accordo sul fatto che le raccomandazioni in generale abbiano senso, sono solo raccomandazioni e che se la tua situazione non si adatta a quel modello, allora dovresti fare ciò che funziona per te. Nel nostro caso esponiamo List (Of T) o talvolta implementiamo il nostro IList.

In Java, si restituisce un Set e si utilizza Collections.unmodifiableSet per rendere il set da restituire. Immagino che questo business ReadOnlyCollection / asReadOnly sia il tuo equivalente.

IEnumerable non mi sembra una buona scelta. Voglio poter dire:

Passenger roger = rememberRoger();
if (train.getPassengers().contains(roger)) {
    tellEveryoneRogerIsOnHisWayHome();
}

Ciò significa che getPassengers deve restituire l'equivalente di Raccolta.

Considererei i comportamenti dei passeggeri. Per esempio. se non vuoi comportamenti forniti da tipi di base, è meglio creare tipi personalizzati ...

In Object Oriented un oggetto dovrebbe esistere solo se ha "comportamenti" (vedi ad esempio cosa dice di questo Allen Holub). Quindi, la mia risposta significa che nella mia domanda progetterò Passeggeri come raccolta di Passeggero se non avrò alcun comportamento aggiuntivo. Ma una volta trovato ho bisogno di un comportamento come "rimuovere tutti i Passeggeri sotto i 10 anni", creerò Passeggeri come oggetto personalizzato con una raccolta interna che mostra un messaggio "removePassengersYoungerThan (int years)".

Penso che ogni collezione abbia bisogno di un design specifico. Anche nella stessa applicazione.

Spero che questo sia più chiaro della risposta precedente.