È buona pratica restituire un array di oggetti?

I principali motivi per non seguire l'approccio ItemContainer->getItems() e restituire un array sono:

1. È probabile che l'implementazione iniziale restituisca semplicemente un riferimento all'array originale, che consentirebbe al chiamante di modificare la raccolta sottostante. Questo è considerato una cattiva pratica.
2. Il prossimo lavoro è quindi creare una copia dell'array e restituirlo. Ciò è negativo perché un chiamante può pensare che stia ancora modificando la raccolta sottostante ed è anche costoso per le raccolte di grandi dimensioni.

Se il chiamante vuole davvero avere una copia dell'array, quindi itemContainer->cloneItems() è molto più chiaro e meno probabilità di essere usato in modo inappropriato.

Se il chiamante desidera semplicemente avere un singolo elemento, quindi fornire un

itemContainer->getItem(index && key)

è chiaro ed efficiente

Se il chiamante vuole iterare sugli articoli, fornendo un

itemContainer->getItemIterator()

è più chiaro. A seconda della lingua che puoi implementare

itemContainer->VisitItems(visitor)

dove visitor è un visitor class, delegate o function pointer.

Spero di averti dato alcune idee su come questo possa essere affrontato in modo diverso

Perché non entrambi? Generalmente, poiché prima di 5.4 il PHP non supportava il dereferenziamento del ritorno di una funzione, era normale restituire una matrice di oggetti, e quindi se la chiave dell'array restituito era passata in quella funzione, la funzione restituiva solo quel valore.

È PHP, quindi stai lavorando in un linguaggio tipizzato dinamicamente. Se vuoi maggiore sicurezza, puoi sempre creare un oggetto separato per la memorizzazione degli elementi, come ItemCollection . Questo sarebbe un approccio che vedresti in linguaggi come Java o C# .

dipende:

• la restituzione di un array ti consente di passare direttamente al foreach idiomatico della lingua

• ma utilizzando get basato su indice significa che non è necessario allocare l'array e spingere tutti gli elementi in esso.