Chi dovrebbe essere responsabile della selezione della classe derivata appropriata?

Question

Ho recentemente scritto una libreria di classi che include alcuni oggetti che modellano determinati tipi di file. Ad esempio, v'è un estratto Document di classe, con le classi derivate PdfDocument (cemento) e OfficeDocument (astratto, con classi concrete derivate come WordDocument e ExcelDocument), ecc

attualmente il modo i clienti a creare un nuovo oggetto è selezionando la classe derivata appropriata e passandogli la matrice di byte. Così, per esempio, se ho un array di byte di un PdfDocument e un WordDocument, vorrei fare qualcosa di simile:

var wordDocument = new WordDocument(wordDocumentByteArray); 
var pdfDocument = new PdfDocument(pdfDocumentByteArray); 

È questo disegno accettabili, che il cliente deve sapere cosa classe derivata da usare? O sarebbe meglio nascondere tutti tranne la classe astratta Document e utilizzare qualcosa come un modello di fabbrica astratto per restituire il tipo corretto derivato? es .:

var wordDocument = DocumentFactory.GetDocument(wordDocumentByteArray, "docx"); 
// pass file extension so we know what the file is 

Nota che i tipi di derivati ​​non aggiungono ulteriori proprietà/metodi per la classe astratta, hanno appena implementare i metodi astratti in modi diversi.

Answer 1

Il secondo approccio è molto meglio del primo, perché nasconde il fatto stesso dell'esistenza di documenti Word e Pdf dagli utenti della libreria. Ciò diventa particolarmente importante quando decidi di aggiungere altri tipi di documenti, ad es. Rtf, Html e così via: gli utenti otterrebbero i benefici dei nuovi tipi aggiunti senza dover ricompilare il loro codice. Infatti, non si accorgerebbero nemmeno di aver cambiato nulla: se fatto bene, il loro codice "funzionerà" semplicemente con i documenti di tipo che non hanno mai saputo esistere.

P.S. Se riesci a scansionare l'array di byte e a capire il tipo corretto da esso, la tua API può "guadagnare alcuni punti per stile" eliminando il secondo parametro.

Answer 2

Se i tipi derivati ​​non aggiungono proprietà/metodi e si ha la capacità tecnica di determinare il tipo da utilizzare per un determinato byte [], non renderei pubbliche le classi derivate ... semplicemente aumentano la superficie di materiale che il consumatore dovrà analizzare durante l'apprendimento della libreria. Basta avere un metodo factory statico come public static Document OpenDocument(byte[] data) nella classe Document.