search

andy0013 / TP2

TP2 Hilos
0 stars 0 forks source link

Usá mas la member initializer list #25

Open fedemgp opened 2 years ago

fedemgp commented 2 years ago

https://github.com/andy0013/TP2/blob/1bffc6e02134bd4ff4eae8332b10db3ccf65d17b/TP2/src/LectorDeArchivo.cpp#L11-L17

Estás construyendo los atributos 2 veces, haciendo que tu programa sea mas lento (en un programa profesional que requiera tener alta performance, esto te va a matar). Los objetos de la STL respetan el concepto de RAII y tienen constructores parametrizados que ya te inicializan el objeto. Usá mas la member initializer list.

fedemgp commented 2 years ago

Corregido, pero quedan constructores de otras clases a los que se podía haber usado la MIL (por ejemplo, en Partición).