search

RodrigoToroIcarte / IIC2113-2023-1

6 stars 0 forks source link

CleanCode #185

Open iledantec opened 1 year ago

iledantec commented 1 year ago

Que sería más correcto, tener un código más corto y tener warnings amarillos o viceversa? Además, debemos solucionar todos estos warnings?

RodrigoToroIcarte commented 1 year ago

¿Qué warnings?

iledantec commented 1 year ago

"Dereference of a possibly null reference" por ejemplo, ya que hay algunos que no puedo solucionar (a pesar de que checkeo las condiciones de nulidad antes de ejecutar ciertos métodos, me parece que rider no detecta correctamente cuando un método solo es llamado por métodos que ya checkearon la nulidad, por lo que se forma una cadena horrible en que checkeo nulidad (de los players por ejemplo) que alarga mucho el codigo.

RodrigoToroIcarte commented 1 year ago

Ya veo. No, yo no agregaría código para chequear nulidad sobre una variable que nunca es nula. Tampoco consideramos el número de warnings al momento de evaluar tu entrega. Los warning son eso. Te avisan de que algo podría estar mal. Uno lo ve y decide si efectivamente está malo. Si está malo lo arregla. Si está bien se deja como está :)