diff --git a/metrix-simulator/src/calculecrirecontraintesdodu.cpp b/metrix-simulator/src/calculecrirecontraintesdodu.cpp index 4e7c43d83..1f7be62be 100644 --- a/metrix-simulator/src/calculecrirecontraintesdodu.cpp +++ b/metrix-simulator/src/calculecrirecontraintesdodu.cpp @@ -1978,10 +1978,8 @@ double Calculer::transitSurQuad(const std::shared_ptr& quad, double puissance = 0.; int numQuadSurveille = quad->num_; - int i_q; - int i_p; - for (i_q = 0; i_q < icdt->nbLignes_; ++i_q) { + for (int i_q = 0; i_q < icdt->nbLignes_; ++i_q) { auto& quad2 = icdt->listeQuads_[i_q]; // verification sur la presence du quad dans l'incident if (quad2->num_ == quad->num_) { @@ -1991,7 +1989,7 @@ double Calculer::transitSurQuad(const std::shared_ptr& quad, variationTranSurIncident += puissance * icdt->rho_[i_q][numQuadSurveille]; } - for (i_q = 0; i_q < icdt->nbCouplagesFermes_; ++i_q) { + for (int i_q = 0; i_q < icdt->nbCouplagesFermes_; ++i_q) { auto& quad2 = icdt->listeCouplagesFermes_[i_q]; // verification sur la presence du quad dans l'incident puissance = quad2->u2Yij_ * (theta[quad2->norqua_->num_] - theta[quad2->nexqua_->num_]); @@ -2000,7 +1998,7 @@ double Calculer::transitSurQuad(const std::shared_ptr& quad, if (icdt->nbGroupes_ >= 1) { int nbQuadsInc = icdt->nbLignes_ + icdt->nbCouplagesFermes_; - for (i_p = 0; i_p < icdt->nbGroupes_; ++i_p) { + for (int i_p = 0; i_p < icdt->nbGroupes_; ++i_p) { auto& grpe = icdt->listeGroupes_[i_p]; if (!grpe->etat_) { continue; @@ -2029,7 +2027,7 @@ double Calculer::transitSurQuad(const std::shared_ptr& quad, } if (icdt->nbLccs_ >= 1) { - for (i_q = 0; i_q < icdt->nbLccs_; ++i_q) { // ATTENTION AU SIGNE - + for (int i_q = 0; i_q < icdt->nbLccs_; ++i_q) { // ATTENTION AU SIGNE - auto& hvdc = icdt->listeLccs_[i_q]; if (!hvdc->connecte()) { continue; @@ -2045,7 +2043,7 @@ double Calculer::transitSurQuad(const std::shared_ptr& quad, } // Annulation de l'action des TD fictifs en N - for (i_p = 0; i_p < res_.nbCCEmulAC_; ++i_p) { + for (int i_p = 0; i_p < res_.nbCCEmulAC_; ++i_p) { double varPuiss = 0.; auto& tdfictif = res_.TDFictifs_[i_p]; diff --git a/metrix-simulator/src/cte.h b/metrix-simulator/src/cte.h index d5cf9cca2..31eaaff06 100644 --- a/metrix-simulator/src/cte.h +++ b/metrix-simulator/src/cte.h @@ -31,7 +31,7 @@ std::string c_fmt(const char* format, Args... args) std::array buf; buf.fill('\0'); - sprintf(buf.data(), format, args...); + snprintf(buf.data(), max, format, args...); return std::string(buf.data()); } diff --git a/metrix-simulator/src/metrix2assess.cpp b/metrix-simulator/src/metrix2assess.cpp index 73e043e95..eab97984c 100644 --- a/metrix-simulator/src/metrix2assess.cpp +++ b/metrix-simulator/src/metrix2assess.cpp @@ -1508,7 +1508,7 @@ bool Calculer::calculVariationsMarginales(FILE* fr, continue; } string nom; - int numIncident; + int numIncident = -1; std::tie(nom, numIncident, std::ignore) = (*opt_inc); double costVal = (config::configuration().redispatchCostOffset() > 0) ? std::accumulate(cost[i].begin(), diff --git a/metrix-simulator/src/reseau.cpp b/metrix-simulator/src/reseau.cpp index e0c71c21f..9910dd68d 100644 --- a/metrix-simulator/src/reseau.cpp +++ b/metrix-simulator/src/reseau.cpp @@ -148,8 +148,6 @@ void Reseau::update_with_configuration() //-------------------- void Reseau::lireDonnees() { - unsigned int i; - auto& config = config::configuration(); update_with_configuration(); @@ -169,7 +167,7 @@ void Reseau::lireDonnees() // Traitement des noeuds //---------------------- - for (i = 0; i < static_cast(nbNoeuds_); ++i) { + for (int i = 0; i < nbNoeuds_; ++i) { auto noeud = std::make_shared(i, config.cpposregDIE()[i] - 1); noeuds_[i] = noeud; noeud->typeNoeud_ = Noeud::NOEUD_REEL; @@ -181,7 +179,7 @@ void Reseau::lireDonnees() vector> quadsParIndice; quadsParIndice.reserve(nbQuads_); - for (i = 0; i < static_cast(nbQuads_); ++i) { + for (unsigned int i = 0; i < static_cast(nbQuads_); ++i) { nomQuadripole = config.cqnomquaDIE()[i]; rtrim(nomQuadripole); if (i >= config.tnnorquaDIE().size() || i >= config.tnnexquaDIE().size() || config.tnnorquaDIE()[i] == 0 @@ -264,7 +262,7 @@ void Reseau::lireDonnees() nbNoeuds_ = nbNoeuds_ + nbTd_; noeuds_.resize(nbNoeuds_); - for (i = 0; i < static_cast(nbTd_); ++i) { + for (unsigned int i = 0; i < static_cast(nbTd_); ++i) { auto quadVrai = quadsParIndice[config.dttrdequDIE()[i] - 1]; auto typeP = static_cast(config.dtmodregDIE()[i]); @@ -343,7 +341,7 @@ void Reseau::lireDonnees() noeuds_.resize(nbNoeuds_ + nbCCEmulAC_); } - for (i = 0; i < static_cast(nbCC_); ++i) { + for (unsigned int i = 0; i < static_cast(nbCC_); ++i) { auto& nor = noeuds_[config.dcnorquaDIE()[i] - 1]; auto& nex = noeuds_[config.dcnexquaDIE()[i] - 1]; string nom = config.dcnomquaDIE()[i]; @@ -498,7 +496,7 @@ void Reseau::lireDonnees() //---------------------- // Types des groupes - for (i = 0; i < static_cast(nbTypesGroupes_); ++i) { + for (unsigned int i = 0; i < static_cast(nbTypesGroupes_); ++i) { string typeGroupe = config.trnomtypDIE()[i]; rtrim(typeGroupe); typesGroupes_.push_back(typeGroupe); @@ -508,12 +506,11 @@ void Reseau::lireDonnees() nbVarGroupes_ = 0; double sumMaxGroupes = 0.0; DBandeRegGlobale_ = 0; - std::shared_ptr prod; cptPtIncident = 0; vector> groupesParIndice; groupesParIndice.reserve(nbGroupes_); - for (i = 0; i < static_cast(nbGroupes_); ++i) { + for (unsigned int i = 0; i < static_cast(nbGroupes_); ++i) { nomGroupe = config.trnomgthDIE()[i]; rtrim(nomGroupe); if (i >= config.tnneurgtDIE().size() || config.tnneurgtDIE()[i] == 0 || config.tnneurgtDIE()[i] > nbNoeuds_) { @@ -597,7 +594,7 @@ void Reseau::lireDonnees() string nomConso; vector> consommationsParIndice; consommationsParIndice.reserve(nbConsos_); - for (i = 0; i < static_cast(nbConsos_); ++i) { + for (unsigned int i = 0; i < static_cast(nbConsos_); ++i) { nomConso = config.tnnomnoeDIE()[i]; rtrim(nomConso); @@ -722,7 +719,7 @@ void Reseau::lireDonnees() int nbIncValides = 0; incidentsEtParades_.reserve(nbIncidents_); - for (i = 0; i < static_cast(nbIncidents_); ++i) { + for (unsigned int i = 0; i < static_cast(nbIncidents_); ++i) { auto icdt = std::make_shared(Incident::INCONNU); icdt->num_ = i; icdt->nbLignes_ = 0; @@ -2699,11 +2696,11 @@ void Reseau::updateBase(const config::VariantConfiguration::VariantConfig& confi } } - for (auto& td : config.tdPhasing) { - const auto& str = std::get(td); + for (auto& tdPhasing : config.tdPhasing) { + const auto& str = std::get(tdPhasing); auto tdIt = TransfoDephaseurs_.find(str); if (tdIt != TransfoDephaseurs_.end()) { - auto var_int = std::get(td); + auto var_int = std::get(tdPhasing); auto& td = tdIt->second; if (!td->tapdepha_.empty() && (var_int >= td->lowtap_) && (var_int < td->lowtap_ + td->nbtap_)) { td->puiConsBase_ = td->angle2Power(td->tapdepha_[var_int - td->lowtap_]); @@ -2946,11 +2943,11 @@ void Reseau::updateVariant(MapQuadinVar& mapping, const config::VariantConfigura } } - for (auto& td : config.tdPhasing) { - const auto& str = std::get(td); + for (auto& tdPhasing : config.tdPhasing) { + const auto& str = std::get(tdPhasing); auto tdIt = TransfoDephaseurs_.find(str); if (tdIt != TransfoDephaseurs_.end()) { - auto var_int = std::get(td); + auto var_int = std::get(tdPhasing); auto& td = tdIt->second; if (!td->tapdepha_.empty() && (var_int >= td->lowtap_) && (var_int < td->lowtap_ + td->nbtap_)) { auto power = td->angle2Power(td->tapdepha_[var_int - td->lowtap_]); @@ -2963,13 +2960,13 @@ void Reseau::updateVariant(MapQuadinVar& mapping, const config::VariantConfigura } for (auto& threshold : config.tresholds) { - for (auto quad_cfg : threshold.second) { + for (const auto& quad_cfg : threshold.second) { const auto& str = std::get(quad_cfg); - auto& mapping = (threshold.first == VariantConfiguration::VariantConfig::Threshold::MAX_N) - ? elementsASurveillerN_ - : elementsASurveillerNk_; - auto quadSurvIt = mapping.find(str); - if (quadSurvIt != mapping.end()) { + auto& map = (threshold.first == VariantConfiguration::VariantConfig::Threshold::MAX_N) + ? elementsASurveillerN_ + : elementsASurveillerNk_; + auto quadSurvIt = map.find(str); + if (quadSurvIt != map.end()) { auto& quad = quadSurvIt->second; auto var_dbl = std::get(quad_cfg); diff --git a/metrix-simulator/tests_reference/connexite/composante_non_connexe/metrixOut.txt b/metrix-simulator/tests_reference/connexite/composante_non_connexe/metrixOut.txt index c1c79bb6e..f93aeb95e 100644 --- a/metrix-simulator/tests_reference/connexite/composante_non_connexe/metrixOut.txt +++ b/metrix-simulator/tests_reference/connexite/composante_non_connexe/metrixOut.txt @@ -38368,4 +38368,4 @@ Noeud '42248 [CONSO_BJMH]' est hors composante connexe principale (composante 23 Noeud '42249 [CONSO_BJMI]' est hors composante connexe principale (composante 2310) Noeud '42250 [CONSO_BJMJ]' est hors composante connexe principale (composante 2310) Noeud '42251 [CONSO_BJMK]' est hors composante connexe principale (composante 2310) -Not recoverable error: Réseau non connexe (file=reseau.cpp ,line=665) +Not recoverable error: Réseau non connexe (file=reseau.cpp ,line=662)