search

EscobarMariaSol / TP2-Taller-de-Programacion

0 stars 0 forks source link

Estos constructor y asignación por copia son frágiles #5

Open fedemgp opened 4 years ago

fedemgp commented 4 years ago

https://github.com/EscobarMariaSol/TP2-Taller-de-Programacion/blob/0a0d7173507bd69c7ffbd22175c3c005f42cd581/file_repository.cpp#L26-L33

Primero y principal, este objeto no debería de poder copiarse (querés una intancia única para no tener comportamiento indefinido). Segundo, que pasa si hago:

FileRepository r1;
// agrego cosas
r1 = r1;

Esto debería estar protegido como se dijo en clase.

fedemgp commented 4 years ago

Lo mismo para OutputRepository. No permitas las copias, y si necesitás copias/movimiento, protegete de no copiarte a vos mismo.