Simplifica ControlPizzas::obtener_total_objetivos

autosquash · Oct 14, 2023 · a523e35 · a523e35
1 parent b24d9cf
commit a523e35
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/src/modelo/control_pizzas.cpp b/src/modelo/control_pizzas.cpp
@@ -23,6 +23,9 @@ ControlPizzas::ControlPizzas(Pedidos pedidos_, bool es_estatico_)
             }
         }
     }
+    if (es_estatico) {
+        assert(pedidos.size() == 1);
+    }
 }
 
 const std::vector<TipoPizza> & //
@@ -72,7 +75,11 @@ int ControlPizzas::obtener_total_preparadas() const {
     return total_preparadas;
 }
 
-int ControlPizzas::obtener_total_objetivos(Pedido &pedido) const {
+/* Calcular total objetivos. Solo tiene sentido con pedidos estaticos. */
+int ControlPizzas::obtener_total_objetivos() const {
+    assert(es_estatico);
+    assert(pedidos.size() == 1);
+    auto &pedido = pedidos[0];
     int total_objetivos = 0;
     for (auto &par : pedido.contenido) {
         auto tp = par.first;

diff --git a/src/modelo/control_pizzas.h b/src/modelo/control_pizzas.h
@@ -22,6 +22,6 @@ struct ControlPizzas {
     const std::vector<TipoPizza> &get_tipos_disponibles() const;
     void procesar_despacho(const TipoPizza tp);
     int obtener_total_preparadas() const;
-    int obtener_total_objetivos(Pedido &pedido) const;
+    int obtener_total_objetivos() const;
     bool faltan_pedidos_por_cubrir() const;
 };
diff --git a/src/nivel.cpp b/src/nivel.cpp
@@ -130,11 +130,7 @@ AccionGeneral Nivel::ejecutar() {
     // std::cout << "Número de tipos de pizza disponibles: "
     //           << control_pizzas.get_tipos_disponibles().size() << std::endl;
     if (datos_nivel.es_estatico) {
-        // Calcular total objetivos
-        auto &pedidos = control_pizzas.pedidos;
-        assert(pedidos.size() == 1);
-        auto &pedido = pedidos[0];
-        total_objetivos = control_pizzas.obtener_total_objetivos(pedido);
+        total_objetivos = control_pizzas.obtener_total_objetivos();
     }
     Vista vista(
         datos_nivel.es_estatico,               //