Return static constant data by constant reference

This commit is contained in:
Dan Paulat 2022-01-23 19:44:42 -06:00
parent a202e0e2a0
commit c47a569645
6 changed files with 16 additions and 16 deletions

View file

@ -68,9 +68,9 @@ enum class Phenomenon
Unknown Unknown
}; };
Phenomenon GetPhenomenon(const std::string& code); Phenomenon GetPhenomenon(const std::string& code);
std::string GetPhenomenonCode(Phenomenon phenomenon); const std::string& GetPhenomenonCode(Phenomenon phenomenon);
std::string GetPhenomenonText(Phenomenon phenomenon); const std::string& GetPhenomenonText(Phenomenon phenomenon);
} // namespace awips } // namespace awips
} // namespace scwx } // namespace scwx

View file

@ -61,11 +61,11 @@ public:
bool Parse(const std::string& s); bool Parse(const std::string& s);
static ProductType GetProductType(const std::string& code); static ProductType GetProductType(const std::string& code);
static std::string GetProductTypeCode(ProductType productType); static const std::string& GetProductTypeCode(ProductType productType);
static Action GetAction(const std::string& code); static Action GetAction(const std::string& code);
static std::string GetActionCode(Action action); static const std::string& GetActionCode(Action action);
private: private:
std::unique_ptr<PVtecImpl> p; std::unique_ptr<PVtecImpl> p;

View file

@ -19,9 +19,9 @@ enum class Significance
Unknown Unknown
}; };
Significance GetSignificance(const std::string& code); Significance GetSignificance(const std::string& code);
std::string GetSignificanceCode(Significance significance); const std::string& GetSignificanceCode(Significance significance);
std::string GetSignificanceText(Significance significance); const std::string& GetSignificanceText(Significance significance);
} // namespace awips } // namespace awips
} // namespace scwx } // namespace scwx

View file

@ -154,12 +154,12 @@ Phenomenon GetPhenomenon(const std::string& code)
return phenomenon; return phenomenon;
} }
std::string GetPhenomenonCode(Phenomenon phenomenon) const std::string& GetPhenomenonCode(Phenomenon phenomenon)
{ {
return phenomenonCodes_.left.at(phenomenon); return phenomenonCodes_.left.at(phenomenon);
} }
std::string GetPhenomenonText(Phenomenon phenomenon) const std::string& GetPhenomenonText(Phenomenon phenomenon)
{ {
return phenomenonText_.at(phenomenon); return phenomenonText_.at(phenomenon);
} }

View file

@ -242,7 +242,7 @@ PVtec::ProductType PVtec::GetProductType(const std::string& code)
return productType; return productType;
} }
std::string PVtec::GetProductTypeCode(PVtec::ProductType productType) const std::string& PVtec::GetProductTypeCode(PVtec::ProductType productType)
{ {
return productTypeCodes_.left.at(productType); return productTypeCodes_.left.at(productType);
} }
@ -266,7 +266,7 @@ PVtec::Action PVtec::GetAction(const std::string& code)
return action; return action;
} }
std::string PVtec::GetActionCode(PVtec::Action action) const std::string& PVtec::GetActionCode(PVtec::Action action)
{ {
return actionCodes_.left.at(action); return actionCodes_.left.at(action);
} }

View file

@ -56,12 +56,12 @@ Significance GetSignificance(const std::string& code)
return significance; return significance;
} }
std::string GetSignificanceCode(Significance significance) const std::string& GetSignificanceCode(Significance significance)
{ {
return significanceCodes_.left.at(significance); return significanceCodes_.left.at(significance);
} }
std::string GetSignificanceText(Significance significance) const std::string& GetSignificanceText(Significance significance)
{ {
return significanceText_.at(significance); return significanceText_.at(significance);
} }