more on major code smells

AntaresSimulatorTeam · Sep 6, 2023 · 1e65059 · 1e65059
1 parent c81b801
commit 1e65059
Show file tree

Hide file tree

Showing 9 changed files with 3 additions and 94 deletions.
diff --git a/src/solver/optimisation/MinDownTime.cpp b/src/solver/optimisation/MinDownTime.cpp
@@ -12,21 +12,13 @@ void MinDownTime::add(int pays, int cluster, int clusterIndex, int pdt, bool Sim
     CorrespondanceCntNativesCntOptim.NumeroDeContrainteDesContraintesDeDureeMinDArret[cluster] = -1;
     if (!Simulation)
     {
-        double pminDUnGroupeDuPalierThermique
-          = PaliersThermiquesDuPays.pminDUnGroupeDuPalierThermique[clusterIndex];
-
         int NombreDePasDeTempsPourUneOptimisation
           = problemeHebdo->NombreDePasDeTempsPourUneOptimisation;
 
         const std::vector<int>& NombreMaxDeGroupesEnMarcheDuPalierThermique
           = PaliersThermiquesDuPays.PuissanceDisponibleEtCout[clusterIndex]
               .NombreMaxDeGroupesEnMarcheDuPalierThermique;
 
-        int t1 = pdt - DureeMinimaleDArretDUnGroupeDuPalierThermique;
-        if (t1 < 0)
-            t1 = NombreDePasDeTempsPourUneOptimisation + t1;
-        double rhs = NombreMaxDeGroupesEnMarcheDuPalierThermique[t1]; // /!\ TODO check
-
         builder.updateHourWithinWeek(pdt).include(Variable::NODU(cluster), 1.0);
 
         for (int k = pdt - DureeMinimaleDArretDUnGroupeDuPalierThermique + 1; k <= pdt; k++)

diff --git a/src/solver/optimisation/MinHydroPower.cpp b/src/solver/optimisation/MinHydroPower.cpp
@@ -25,8 +25,6 @@ void MinHydroPower::add(int pays)
             builder.updateHourWithinWeek(pdt);
             builder.include(Variable::HydProd(pays), 1.0);
         }
-        // const double rhs = problemeHebdo->CaracteristiquesHydrauliques[pays]
-        //                      .MinEnergieHydrauParIntervalleOptimise[NumeroDeLIntervalle];
 
         problemeHebdo->NumeroDeContrainteMinEnergieHydraulique[pays]
           = problemeHebdo->ProblemeAResoudre->NombreDeContraintes;

diff --git a/src/solver/optimisation/MinHydroPower.h b/src/solver/optimisation/MinHydroPower.h
@@ -1,8 +1,9 @@
 #pragma once
 #include "constraint_builder.h"
 
-struct MinHydroPower : private Constraint
+class MinHydroPower : private Constraint
 {
+public:
     using Constraint::Constraint;
     void add(int pays);
 };
diff --git a/src/solver/optimisation/NbUnitsOutageLessThanNbUnitsStop.cpp b/src/solver/optimisation/NbUnitsOutageLessThanNbUnitsStop.cpp
@@ -10,8 +10,6 @@ void NbUnitsOutageLessThanNbUnitsStop::add(int pays,
     {
         const PALIERS_THERMIQUES& PaliersThermiquesDuPays
           = problemeHebdo->PaliersThermiquesDuPays[pays];
-        double pminDUnGroupeDuPalierThermique
-          = PaliersThermiquesDuPays.pminDUnGroupeDuPalierThermique[clusterIndex];
         const int DureeMinimaleDArretDUnGroupeDuPalierThermique
           = PaliersThermiquesDuPays.DureeMinimaleDArretDUnGroupeDuPalierThermique[clusterIndex];
 
@@ -20,18 +18,6 @@ void NbUnitsOutageLessThanNbUnitsStop::add(int pays,
         CorrespondanceCntNativesCntOptim.NumeroDeContrainteDesContraintesDeDureeMinDeMarche[cluster]
           = -1;
 
-        int NombreDePasDeTempsPourUneOptimisation
-          = problemeHebdo->NombreDePasDeTempsPourUneOptimisation;
-
-        int t1 = pdt - DureeMinimaleDArretDUnGroupeDuPalierThermique;
-
-        if (t1 < 0)
-            t1 = NombreDePasDeTempsPourUneOptimisation + t1;
-
-        const std::vector<int>& NombreMaxDeGroupesEnMarcheDuPalierThermique
-          = PaliersThermiquesDuPays.PuissanceDisponibleEtCout[clusterIndex]
-              .NombreMaxDeGroupesEnMarcheDuPalierThermique;
-
         builder.updateHourWithinWeek(pdt)
           .include(Variable::NumberBreakingDownDispatchableUnits(cluster), 1.0)
           .include(Variable::NumberStoppingDispatchableUnits(cluster), -1.0)

diff --git a/src/solver/optimisation/PMaxDispatchableGeneration.cpp b/src/solver/optimisation/PMaxDispatchableGeneration.cpp
@@ -12,8 +12,6 @@ void PMaxDispatchableGeneration::add(int pays,
           = problemeHebdo->PaliersThermiquesDuPays[pays];
         double pmaxDUnGroupeDuPalierThermique
           = PaliersThermiquesDuPays.PmaxDUnGroupeDuPalierThermique[clusterIndex];
-        const int DureeMinimaleDArretDUnGroupeDuPalierThermique
-          = PaliersThermiquesDuPays.DureeMinimaleDArretDUnGroupeDuPalierThermique[clusterIndex];
 
         builder.updateHourWithinWeek(pdt)
           .include(Variable::DispatchableProduction(cluster), 1.0)

diff --git a/src/solver/optimisation/PMinDispatchableGeneration.cpp b/src/solver/optimisation/PMinDispatchableGeneration.cpp
@@ -13,13 +13,11 @@ void PMinDispatchableGeneration::add(int pays,
         double pminDUnGroupeDuPalierThermique
           = PaliersThermiquesDuPays.pminDUnGroupeDuPalierThermique[clusterIndex];
 
-        int NombreDePasDeTempsPourUneOptimisation
-          = problemeHebdo->NombreDePasDeTempsPourUneOptimisation;
         builder.updateHourWithinWeek(pdt)
           .include(Variable::DispatchableProduction(cluster), 1.0)
           .include(Variable::NODU(cluster), -pminDUnGroupeDuPalierThermique)
           .greaterThan();
-        /*consider Adding rename constraint inside the builder*/
+        /*consider Adding naming constraint inside the builder*/
         if (builder.NumberOfVariables() > 0)
         {
             ConstraintNamer namer(problemeHebdo->ProblemeAResoudre->NomDesContraintes,

diff --git a/src/solver/optimisation/constraint_builder.h b/src/solver/optimisation/constraint_builder.h
@@ -242,32 +242,6 @@ class Visitor
 };
 } // namespace Variable
 
-// enum class Constraint {
-//   FlowDissociation,
-//   AreaBalance,
-//   FictiveLoads,
-//   HydroPower,
-//   HydroPowerSmoothingUsingVariationSum,
-//   HydroPowerSmoothingUsingVariationMaxDown,
-//   HydroPowerSmoothingUsingVariationMaxUp,
-//   MinHydroPower,
-//   MaxHydroPower,
-//   MaxPumping,
-//   AreaHydroLevel,
-//   FinalStockEquivalent,
-//   FinalStockExpression,
-//   NbUnitsOutageLessThanNbUnitsStop,
-//   NbDispUnitsMinBoundSinceMinUpTime,
-//   MinDownTime,
-//   PMaxDispatchableGeneration,
-//   PMinDispatchableGeneration,
-//   ConsistenceNODU,
-//   ShortTermStorageLevel,
-//   BindingConstraintHour,
-//   BindingConstraintDay,
-//   BindingConstraintWeek
-// };
-
 class ConstraintBuilder
 {
 public:
@@ -307,18 +281,6 @@ class ConstraintBuilder
 
         return *this;
     }
-    // ConstraintBuilder& operatorRHS(char op, double rhs)
-    // {
-    //     if (op == '<' || op == '=' || op == '>')
-    //     {
-    //         operator_ = op;
-    //         rhs_ = rhs;
-    //     }
-    //     else
-    //         throw std::runtime_error("Invalid operator");
-
-    //     return *this;
-    // }
 
     ConstraintBuilder& equalTo()
     {
@@ -336,25 +298,6 @@ class ConstraintBuilder
         return *this;
     }
 
-    // ConstraintBuilder& equalTo(double rhs)
-    // {
-    //     operator_ = '=';
-    //     rhs_ = rhs;
-    //     return *this;
-    // }
-    // ConstraintBuilder& lessThan(double rhs)
-    // {
-    //     operator_ = '<';
-    //     rhs_ = rhs;
-    //     return *this;
-    // }
-    // ConstraintBuilder& greaterThan(double rhs)
-    // {
-    //     operator_ = '>';
-    //     rhs_ = rhs;
-    //     return *this;
-    // }
-
     void build();
     int NumberOfVariables() const
     {

diff --git a/src/solver/optimisation/opt_construction_contraintes_couts_demarrage.cpp b/src/solver/optimisation/opt_construction_contraintes_couts_demarrage.cpp
@@ -64,10 +64,6 @@ void OPT_ConstruireLaMatriceDesContraintesDuProblemeLineaireCoutsDeDemarrage(
         {
             PMaxDispatchableGeneration pMaxDispatchableGeneration(problemeHebdo);
             PMinDispatchableGeneration pMinDispatchableGeneration(problemeHebdo);
-            double pminDUnGroupeDuPalierThermique
-              = PaliersThermiquesDuPays.pminDUnGroupeDuPalierThermique[index];
-            double pmaxDUnGroupeDuPalierThermique
-              = PaliersThermiquesDuPays.PmaxDUnGroupeDuPalierThermique[index];
             const int palier
               = PaliersThermiquesDuPays.NumeroDuPalierDansLEnsembleDesPaliersThermiques[index];
 
@@ -156,8 +152,6 @@ void OPT_ConstruireLaMatriceDesContraintesDuProblemeLineaireCoutsDeDemarrage(
         MinDownTime minDownTime(problemeHebdo);
         for (int index = 0; index < PaliersThermiquesDuPays.NombreDePaliersThermiques; index++)
         {
-            int DureeMinimaleDArretDUnGroupeDuPalierThermique
-              = PaliersThermiquesDuPays.DureeMinimaleDArretDUnGroupeDuPalierThermique[index];
             const int palier
               = PaliersThermiquesDuPays.NumeroDuPalierDansLEnsembleDesPaliersThermiques[index];
 

diff --git a/src/solver/optimisation/opt_construction_matrice_des_contraintes_cas_lineaire.cpp b/src/solver/optimisation/opt_construction_matrice_des_contraintes_cas_lineaire.cpp
@@ -133,7 +133,6 @@ void OPT_ConstruireLaMatriceDesContraintesDuProblemeLineaire(PROBLEME_HEBDO* pro
 
     if (nombreDePasDeTempsPourUneOptimisation > nombreDePasDeTempsDUneJournee)
     {
-        int semaine = problemeHebdo->weekInTheYear;
         CORRESPONDANCES_DES_CONTRAINTES_HEBDOMADAIRES& CorrespondanceCntNativesCntOptimHebdomadaires
           = problemeHebdo->CorrespondanceCntNativesCntOptimHebdomadaires;
         for (uint32_t cntCouplante = 0; cntCouplante < problemeHebdo->NombreDeContraintesCouplantes;