Como dijo 🌻 pt2, son muchos archivos, pero calma, solamente

borro comentarios

Signed-off-by: Tomas Fabrizio Orsi <[email protected]>

src/server/aceptador.cpp

@@ -5,12 +5,10 @@
 Aceptador::Aceptador(const char *puerto) : socket(puerto) {};
 
 void Aceptador::aceptarClientes() {
-    //TODO: Hacer que en vez de true sea socket cerrado
     while (!this->socket.is_closed()) {
         try {
             Socket conexionEntrante = this->socket.accept();
 
-            //TODO Anadir un reap dead ACA que busque en una lista de jugadores
             Cliente* cliente = new Cliente(std::move(conexionEntrante), this->escenariosDisponibles, std::ref(this->partidas));
 
             // Primero el reap_dead para que la lista tenga un elemento menos

src/server/arma.h

@@ -4,14 +4,6 @@
 
 #include "defs.h"
 
-// struct MisArmas
-
-// Gusano {
-//     Arma armaEquipada; 
-
-//     EstadoArma 
-// }
-
 struct CaracteristicasArma {
     bool tieneMira;
     bool esCuerpoACuerpo;
@@ -72,25 +64,6 @@ class Arma {
 
     std::pair<coordX, coordY> getCoordenadasTeletransporte();
     void setCoordenadasTeletransporte(std::pair<coordX, coordY> coordenadas);
-    // TODO: hacer todo el cpp y revisar la estructura
-    // (esta horrible, lo se)
-
 };
 
-
-/* CHARLA DE GUIA: 
-c: equipar X
-s: ok, estado
-c: arranco a atacar
-s: estadoGusano = configurarAngulo | estadoArma: apuntando X angulo
-c: cambio angulo
-s: ok, seteo y mando estado
-c: listo el angulo
-s: ok, estadoGusano = configurarPotencia
-c: cambio potencia
-s: ok, seteo y mando estado
-c: listo el potencia
-s: ok, ataco :p + muchos estados de como pasan las cosas
-*/
-
 #endif

src/server/box2dDefs.h

@@ -52,7 +52,6 @@ typedef b2Vec2 finCaja;
 enum class TipoEntidad { GUSANO = 1, VIGA = 2, ARMA = 4, PROYECTIL = 8, OCEANO = 16, PROYECTILREAL = 32, PROVISION = 64};
 
 // se usa para poder saber que tipo de proyectil es
-// TODO: agregar si es fragmento
 struct ProyectilAsociado {
     b2Body *proyectil;
     ArmaProtocolo arma;

src/server/cliente.cpp

@@ -1,6 +1,5 @@
 #include "cliente.h"
 
-
 Cliente::Cliente(Socket&& socket, Mapas mapasDisponibles, MonitorPartida& monitorPartidas) 
     : protocolo(std::move(socket)), sender(std::ref(protocolo)), 
       reciever(std::ref(protocolo), mapasDisponibles, monitorPartidas, this) {
@@ -31,7 +30,6 @@ Cliente::~Cliente() {
     std::cout << "despues sender\n";
 }
 
-
 bool Cliente::estaMuerto() {
     return !this->reciever.is_alive();
 }

src/server/configServer.h

@@ -5,7 +5,6 @@
 #include "defs.h"
 #include <cstdint>
 
-
 // Clase que se encarga de leer el archivo de configuracion del server.
 class ConfiguracionServer {
 public:

src/server/gusano.cpp

@@ -41,7 +41,7 @@ void Gusano::setEstado(EstadoGusano nuevoEstado) {
     this->estado = nuevoEstado;
 
     //Si se esta cayendo queremos cancelar todo tipo de velocidad y
-    //que caiga directo. TODO: Esto lo hace "re duro". Podriamos hacer
+    //que caiga directo. GOTCHA: Esto lo hace "re duro". Podriamos hacer
     //que se conserve ALGO de la velocidad
     if (this->estado == CAYENDO || this->estado == QUIETO) {
         b2Vec2 direccion;
@@ -111,7 +111,6 @@ std::pair<b2Vec2, std::pair<inicioCaja, finCaja>> Gusano::ejecutarGolpe() {
 
     //Angulo
     //Usando trigonometria  vamos a obtener los vectores dado el angulo
-    //WARNING: HARDCODEO La potencia
     /*
            /| 
           / |
@@ -175,7 +174,6 @@ void Gusano::realizarMovimiento(Direccion direccionDeseada) {
         this->estado == HACE_PIRUETA)
         return;
 
-    // b2Vec2 direccion;
     std::pair<coordX, coordY> direccion;
     switch (direccionDeseada) {
     case INICIO_DER:
@@ -244,16 +242,13 @@ void Gusano::recibirDano(b2Vec2 golpe, Entidad *entidad) {
     ArmaProtocolo tipoArma;
     tipoArma = entidad->proyectil.arma;
     if (this->estado == MUERTO || false) {
-        // ((tipoArma != GRANADA_VERDE_P) // && (velocidadActual.x != 0 && velocidadActual.y != 0)
-        //  )) {
         return;
     }
 
     std::cout << "RECIBO DANIO\n";
 
     float distanciaGusanoBomba = b2Distance(entidad->proyectil.posInicial, this->cuerpo->GetPosition());
 
-    //TODO cambiar a que no tenga que crear el arma para obtener el danio
     Arma armaUsada(tipoArma);
     u_int danio;
     int radio;
@@ -274,7 +269,7 @@ void Gusano::recibirDano(b2Vec2 golpe, Entidad *entidad) {
 
     float porcentaje = 1.0f / (1.0f + std::pow(distanciaGusanoBomba / radio,2));
 
-    // // Calculate the graduated damage
+    // Calculate the graduated damage
     danioReal = (distanciaGusanoBomba > radio) ? 0 : danio * porcentaje;
     if (tipoArma == BATE_P && distanciaGusanoBomba <= radio) {
         radio = 2;
@@ -386,7 +381,6 @@ Ataque Gusano::ejecutar(Accion accion) {
         ataqueARealizar.posicion = posicion;
         ataqueARealizar.tiempoEspera = tiempoEspera;
         ataqueARealizar.arma = armaQueQuiero;
-        // this->estado = QUIETO;
         break;
     case MOVERSE:
         {
@@ -536,6 +530,7 @@ Ataque Gusano::ejecutar(Accion accion) {
 
     //Hago que el gusano se acuerde que fue lo ulitmo que realizo
     // si es ataco hay que "mentirle"
+    //:0
     if (this->ultimaAccion.accion == ATACO) {
         this->ultimaAccion.accion = ESTAQUIETO;
     } else {
@@ -593,11 +588,7 @@ RepresentacionGusano Gusano::getRepresentacion() {
     repre.dir = this->direccion;
     repre.estado = this->estado;
     repre.posicion = this->getCoords();
-    //TODO Ahora est hardcodeado. Hacer algo generico.
-    //Esto solo aplica al bate
     RepresentacionArma arma = this->armaSeleccionada->getRepresentacion();
-    // arma.municiones = 100000;
-    // arma.arma = this->armaEquipada;
 
     repre.armaEquipada = arma;
 

src/server/gusano.h

@@ -44,7 +44,6 @@ class Gusano {
 
     condicionDeTurno turno;
 
-    //Arma armaequipada;
     std::vector<Arma> armas;
     ArmaProtocolo armaEquipada;
     Arma* armaSeleccionada;
@@ -53,11 +52,9 @@ class Gusano {
 
     //Conceptos de b2box
     b2Body* cuerpo;
-    // int tiempoEspera;
 
     void setDireccion(DireccionGusano nuevaDireccion);
 
-    //TODO Cambiar nombre
     void realizarMovimiento(Direccion direccionDeseada);
 
     Mapa& mapa;
@@ -92,10 +89,8 @@ class Gusano {
 
     void preparar(Accion& accion);
 
-    //TODO: Esta funcion recibe el arma o tipo de dano
     void recibirDano(b2Vec2 golpe, Entidad *entidad);
     void recibirDanioCaida(b2Vec2 velocidad);
-    // [[nodiscard]] std::pair<cambioX, cambioY> cambio(Accion accion);
 
     //WARNING: Esta funcion tiene que se ser llamada CERCA, sino AL
     //AL LADO del constructor. Gracias por nada c++, null pointers para
@@ -117,10 +112,7 @@ class Gusano {
 
     void recibirDanoPorCheat();
 
-    [[nodiscard]] std::pair<
-        b2Vec2,
-        std::pair<inicioCaja, finCaja>
-        > ejecutarGolpe();
+    [[nodiscard]] std::pair<b2Vec2, std::pair<inicioCaja, finCaja>> ejecutarGolpe();
 };
 
 #endif

src/server/jugador.cpp

@@ -16,14 +16,9 @@ Gusano *Jugador::getGusanoDeTurno() {
 Gusano *Jugador::getGusanoActual() {
     Gusano *gusanoActual = nullptr;
 
-    // int posicionInicial = this->gusanoActualPos;
     //Busco el primer gusano no muerto
     for (int i = 0; i <= (int)this->gusanos.size(); i++) {
         this->gusanoActualPos += 1;
-        // if (this->gusanoActualPos == posicionInicial) {
-        //     return nullptr;
-        // }
-
         //WARNING Casteo falopa. En teoria nada deberia explotar
         if (this->gusanoActualPos >= (int) this->gusanos.size()) {
 	        this->gusanoActualPos = 0;
@@ -40,7 +35,6 @@ Gusano *Jugador::getGusanoActual() {
 
     //Una vez encontrado dicho gusano, la proxima iteracion ya arranca
     //con el proximo
-    std::cout << "gusano actual: " << gusanoActualPos << "\n";
     return gusanoActual;
 }
 
@@ -99,7 +93,6 @@ void Jugador::anadirGusano(Gusano *nuevoGusano) {
 }
 void Jugador::darMasVidaAGusanos() {
     for (auto &&gusano : this->gusanos) {
-        // TODO: hacer que lea el 25 del yaml
         gusano->anadirVida(25);
     }
 

src/server/mapa.h

@@ -1,7 +1,6 @@
 #ifndef MAPA_HEADER
 #define MAPA_HEADER
 
-
 #include "defs.h"
 #include "yaml-cpp/yaml.h"
 

src/server/monitorPartida.cpp

@@ -23,7 +23,6 @@ id MonitorPartida::anadirPartida(Mapa mapa) {
     id idPartidaNueva;
     idPartidaNueva = this->contador;
 
-    //TODO: Check new for no memory
     Partida *partidaNueva = new Partida(mapa);
     partidaNueva->start();
 
@@ -49,8 +48,6 @@ void MonitorPartida::anadirJugadorAPartida(Cliente *nuevoCliente, id partidaEspe
     Partida *partidaRecibidora;
     partidaRecibidora = this->mapa.at(partidaEspecifica);
 
-    // InformacionInicial infoInicial;
-    // infoInicial = partidaRecibidora->anadirCliente(nuevoCliente);
     partidaRecibidora->anadirCliente(nuevoCliente);
 }
 
@@ -103,22 +100,9 @@ void MonitorPartida::kill() {
             continue;
         }
 
-        // TODO: verificar que el destructor de partida esta bien y no leakea memoria
         partida->stop();
         partida->join();
         delete partida;
         this->mapa[idPartida] = nullptr;
     }
 }
-
-// MonitorPartida::~MonitorPartida() {
-//     for (auto const& [idPartida, partida] : this->mapa) {
-//         if (partida == nullptr || !partida->is_alive()) {
-//             continue;
-//         }
-
-//         // TODO: verificar que el destructor de partida esta bien y no leakea memoria
-//         partida->join();
-//         delete partida;
-//     }
-// }

src/server/monitorPartida.h

@@ -30,8 +30,6 @@ class MonitorPartida {
 
     void kill();
 
-    // ~MonitorPartida();
-
 };