treat major code smells 2

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

Hide file tree

Showing 7 changed files with 6 additions and 69 deletions.
diff --git a/src/solver/optimisation/ConsistenceNODU.cpp b/src/solver/optimisation/ConsistenceNODU.cpp
@@ -6,30 +6,17 @@ void ConsistenceNODU::add(int pays, int cluster, int clusterIndex, int pdt, bool
     {
         const PALIERS_THERMIQUES& PaliersThermiquesDuPays
           = problemeHebdo->PaliersThermiquesDuPays[pays];
-        double pminDUnGroupeDuPalierThermique
-          = PaliersThermiquesDuPays.pminDUnGroupeDuPalierThermique[clusterIndex];
-        const int DureeMinimaleDArretDUnGroupeDuPalierThermique
-          = PaliersThermiquesDuPays.DureeMinimaleDArretDUnGroupeDuPalierThermique[clusterIndex];
 
         int NombreDePasDeTempsPourUneOptimisation
           = problemeHebdo->NombreDePasDeTempsPourUneOptimisation;
 
-        int t1 = pdt - DureeMinimaleDArretDUnGroupeDuPalierThermique;
         int Pdtmoins1 = pdt - 1;
         if (Pdtmoins1 < 0)
             Pdtmoins1 = NombreDePasDeTempsPourUneOptimisation + Pdtmoins1;
 
         CORRESPONDANCES_DES_VARIABLES& CorrespondanceVarNativesVarOptimTmoins1
           = problemeHebdo->CorrespondanceVarNativesVarOptim[Pdtmoins1];
 
-        if (t1 < 0)
-            t1 = NombreDePasDeTempsPourUneOptimisation + t1;
-
-        const std::vector<int>& NombreMaxDeGroupesEnMarcheDuPalierThermique
-          = PaliersThermiquesDuPays.PuissanceDisponibleEtCout[clusterIndex]
-              .NombreMaxDeGroupesEnMarcheDuPalierThermique;
-        double rhs = 0; // /!\ TODO check
-
         builder.updateHourWithinWeek(pdt)
           .include(Variable::NODU(cluster), 1.0)
           .updateHourWithinWeek(Pdtmoins1)
@@ -38,6 +25,7 @@ void ConsistenceNODU::add(int pays, int cluster, int clusterIndex, int pdt, bool
           .include(Variable::NumberStartingDispatchableUnits(cluster), -1)
           .include(Variable::NumberStoppingDispatchableUnits(cluster), 1)
           .equalTo();
+
         if (builder.NumberOfVariables() > 0)
         {
             ConstraintNamer namer(problemeHebdo->ProblemeAResoudre->NomDesContraintes,

diff --git a/src/solver/optimisation/FictitiousLoad.cpp b/src/solver/optimisation/FictitiousLoad.cpp
@@ -25,42 +25,10 @@ void FictitiousLoad::add(int pdt, int pays)
 
     builder.updateHourWithinWeek(pdt);
     exportPaliers(*problemeHebdo, builder, pays);
-    builder.include(Variable::HydProd(pays), -problemeHebdo->DefaillanceNegativeUtiliserHydro[pays])
+    auto coeff = problemeHebdo->DefaillanceNegativeUtiliserHydro[pays] ? -1 : 0;
+    builder.include(Variable::HydProd(pays), coeff)
       .include(Variable::NegativeUnsuppliedEnergy(pays), 1.0);
 
-    {
-        // double rhs = 0;
-        // // Private members ?
-        // const ALL_MUST_RUN_GENERATION& AllMustRunGeneration
-        //   = problemeHebdo->AllMustRunGeneration[pdtHebdo];
-        // const CONSOMMATIONS_ABATTUES& ConsommationsAbattues
-        //   = problemeHebdo->ConsommationsAbattues[pdtHebdo];
-        // const std::vector<bool>& DefaillanceNegativeUtiliserConsoAbattue
-        //   = problemeHebdo->DefaillanceNegativeUtiliserConsoAbattue;
-        // const std::vector<bool>& DefaillanceNegativeUtiliserPMinThermique
-        //   = problemeHebdo->DefaillanceNegativeUtiliserPMinThermique;
-
-        // double MaxAllMustRunGeneration = 0.0;
-
-        // if (AllMustRunGeneration.AllMustRunGenerationOfArea[pays] > 0.0)
-        //     MaxAllMustRunGeneration = AllMustRunGeneration.AllMustRunGenerationOfArea[pays];
-
-        // double MaxMoinsConsommationBrute = 0.0;
-        // if (-(ConsommationsAbattues.ConsommationAbattueDuPays[pays]
-        //       + AllMustRunGeneration.AllMustRunGenerationOfArea[pays])
-        //     > 0.0)
-        //     MaxMoinsConsommationBrute = -(ConsommationsAbattues.ConsommationAbattueDuPays[pays]
-        //                                   +
-        //                                   AllMustRunGeneration.AllMustRunGenerationOfArea[pays]);
-
-        // rhs = DefaillanceNegativeUtiliserConsoAbattue[pays]
-        //       * (MaxAllMustRunGeneration + MaxMoinsConsommationBrute);
-
-        // if (DefaillanceNegativeUtiliserPMinThermique[pays] == 0)
-        // {
-        //     rhs -= OPT_SommeDesPminThermiques(problemeHebdo, pays, pdtHebdo);
-        // }
-        builder.lessThan();
-    }
+    builder.lessThan();
     builder.build();
 }
diff --git a/src/solver/optimisation/FinalStockExpression.h b/src/solver/optimisation/FinalStockExpression.h
@@ -2,7 +2,7 @@
 
 #include "constraint_builder.h"
 
-struct FinalStockExpression : private Constraint
+class FinalStockExpression : private Constraint
 {
 public:
     using Constraint::Constraint;

diff --git a/src/solver/optimisation/FlowDissociation.cpp b/src/solver/optimisation/FlowDissociation.cpp
@@ -33,11 +33,6 @@ void FlowDissociation::add(int pdt, int interco)
           .include(Variable::IntercoDirectCost(interco), -1.0)
           .include(Variable::IntercoIndirectCost(interco), 1.0);
 
-        // if (CoutDeTransport.IntercoGereeAvecLoopFlow)
-        //     builder.equalTo(
-        //       problemeHebdo->ValeursDeNTC[pdtHebdo].ValeurDeLoopFlowOrigineVersExtremite[interco]);
-        // else
-        //     builder.equalTo(0.);
         builder.equalTo();
 
         builder.build();

diff --git a/src/solver/optimisation/HydroPower.h b/src/solver/optimisation/HydroPower.h
@@ -1,7 +1,7 @@
 #pragma once
 #include "constraint_builder.h"
 
-struct HydroPower : private Constraint
+class HydroPower : private Constraint
 {
 public:
     using Constraint::Constraint;

diff --git a/src/solver/optimisation/HydroPowerSmoothingUsingVariationMaxDown.cpp b/src/solver/optimisation/HydroPowerSmoothingUsingVariationMaxDown.cpp
@@ -7,13 +7,6 @@ void HydroPowerSmoothingUsingVariationMaxDown::add(int pays, int pdt)
         return;
     }
 
-    const int nombreDePasDeTempsPourUneOptimisation
-      = problemeHebdo->NombreDePasDeTempsPourUneOptimisation;
-
-    const auto& CorrespondanceVarNativesVarOptim
-      = problemeHebdo->CorrespondanceVarNativesVarOptim[pdt];
-    int nombreDeTermes = 0;
-
     ConstraintNamer namer(problemeHebdo->ProblemeAResoudre->NomDesContraintes,
                           problemeHebdo->NamedProblems);
     namer.UpdateArea(problemeHebdo->NomsDesPays[pays]);

diff --git a/src/solver/optimisation/HydroPowerSmoothingUsingVariationMaxUp.cpp b/src/solver/optimisation/HydroPowerSmoothingUsingVariationMaxUp.cpp
@@ -7,13 +7,6 @@ void HydroPowerSmoothingUsingVariationMaxUp::add(int pays, int pdt)
         return;
     }
 
-    const int nombreDePasDeTempsPourUneOptimisation
-      = problemeHebdo->NombreDePasDeTempsPourUneOptimisation;
-
-    const auto& CorrespondanceVarNativesVarOptim
-      = problemeHebdo->CorrespondanceVarNativesVarOptim[pdt];
-    int nombreDeTermes = 0;
-
     ConstraintNamer namer(problemeHebdo->ProblemeAResoudre->NomDesContraintes,
                           problemeHebdo->NamedProblems);
     namer.UpdateArea(problemeHebdo->NomsDesPays[pays]);