netaneldj 7542-Taller-TP3 issues

netaneldj / 7542-Taller-TP3

0 stars 0 forks source link

issues

Newest

Newest Most commented Recently updated Oldest Least commented Least recently updated

Esto debe ser privado, estás rompiendo encapsulamiento

#30 fedemgp opened 4 years ago
1
Esto te dije que debería ser privado para no romper encapsulamiento

#29 fedemgp closed 4 years ago
0
Otra race condition (relacionada con la de la RC del Aceptador)

#28 fedemgp opened 4 years ago
0
Rompés el encapsulamiento

#27 fedemgp opened 4 years ago
1
Esto es otra race condition

#26 fedemgp opened 4 years ago
1
El close del socket debería ser anterior a borrar los clientes. Esto es una Race

#25 fedemgp opened 4 years ago
1
El cliente no tenia que tener hilos. Hiciste el binario mas pesado al linkear la librería de pthread

#24 fedemgp opened 4 years ago
0
Usá std::vector que es una clase RAII y mas robusta

#23 fedemgp opened 4 years ago
0
El comando no debería saber nada del protocolo.

#22 fedemgp opened 4 years ago
3
Esto denota falta de modularización

#21 fedemgp opened 4 years ago
1
Varias cosas acopladas y logica del juego en el cliente

#20 fedemgp opened 4 years ago
0
Estás silenciando todas las excepciones que heredan de std:exception

#19 fedemgp opened 4 years ago
0
Acordate de implementar al menos un operador ()

#18 fedemgp closed 4 years ago
1
Te falla un caso de prueba privado

#17 fedemgp closed 4 years ago
2
Esto es feo, silenciás todas las excepciones

#16 fedemgp closed 4 years ago
0
Race condition

#15 fedemgp closed 4 years ago
1
Rompés encapsulamiento, esto debería ser construido internamente en el constructor

#14 fedemgp closed 4 years ago
0
Esto no es RAII, el destructor debe llamar a close y a shutdown.

#13 fedemgp closed 4 years ago
0
Esta clase no debe ser common, el cliente no tiene que conocerla.

#12 fedemgp closed 4 years ago
0
Rompés encapsulamiento, Este constructor debe ser privado

#11 fedemgp closed 4 years ago
0
Por qué el cv? no se requería en este TP.

#10 fedemgp closed 4 years ago
0
Te mataste con algo que pudo haber sido muy simple (y mezclaste protocolo con comando)

#9 fedemgp closed 4 years ago
1
?? (el servidor recibe esto?)

#8 fedemgp closed 4 years ago
1
No respetás la endiannes del enunciado

#7 fedemgp closed 4 years ago
0
El protocolo no debería saber de esto

#6 fedemgp closed 4 years ago
1
Logica del cliente y servidor (y del juego) dentro del protocolo

#5 fedemgp closed 4 years ago
3
Alteraste el protocolo, y te quedó un poco complejo

#4 fedemgp closed 4 years ago
7
Instancía los objetos a medida que los vayas usando, no le des un lifetime mayor del que necesitan

#3 fedemgp closed 4 years ago
0
Ojo con las copias de los objetos (estas copiando siempre los strings)

#2 fedemgp closed 4 years ago
1
Pasaje por copia te copia el string

#1 fedemgp closed 4 years ago
0