lautarofritz / tp3

0 stars 0 forks source link

Nota: Reentrega #7

Open eze210 opened 4 years ago

eze210 commented 4 years ago

El TP no está tan mal, pero tiene dos detalles claves que te pido que corrijas:

1 - Evitar el uso de malloc. Casi siempre lo podés reemplazar por un std::vector<> en el stack (dónde ponga la memoria interna std::vector no importa), y en caso de que no puedas, y sí o sí tengas que usar heap, la herramienta en C++ es new, no malloc 2 - El protocolo está repartido entre el socket y las clases que usan al socket. Abstraerlo en una clase Protocolo. Esta puede contener al socket o recibirlo en sus operaciones, pero tiene que ser la encargada de pasar acciones de nivel aplicación a cadenas de bytes (y viceversa)

El resto de las cosas marcadas son detalles que suman

lautarofritz commented 4 years ago

Sobre el punto 2, la idea sería tener una clase Protocolo que sea una intermediaria entre el cliente y el socket no? Osea que el cliente lo único que haría sería recibir el comando por entrada estándar y a lo sumo verificar si es válido o no, o ni siquiera eso?