c++ - 如何在没有重复代码的情况下处理 getter 的 const/non const 组合？

Question

这个问题已经有答案了:

How do I remove code duplication between similar const and non-const member functions? (21 个回答)

已关闭 3 年前。

假设我有一个结构，它有一个名称和一些与之关联的值:

struct Season {
  std::string name;
  // Mean temperature over the season
  double meanTemperature;
  // Days since the start of year when it's strongest
  float strongestAt;
  // Fraction of a year it applies to, compared to other seasons
  float yearFraction;
}

本类(class)描述每年的一个季节。假设我有一个它们的集合，其中包含全年:

// made up method that is supposed to find a season (I don't use it in real code)
int findIndex(const std::vector<Season>& in, std::function<bool(const Season&)>);
class SeasonCollection
{
public:
  const Season* GetSeason(const std::string& name) const
  {
    const int index = findIndex(_seasons, [name](const Season& s) { return s.seasonName == name; });
    return index != -1 ? &_seasons[index] : nullptr;
  }
  Season* GetSeason(const std::string& name)
  {
    const int index = findIndex(_seasons, [name](const Season& s) { return s.seasonName == name; });
    return index != -1 ? &_seasons[index] : nullptr;
  }
private:
  //! Settings for seasons
  std::vector<Season> _seasons;
};

在该集合中，您可以看到我需要同时获取 const Season*和Season* 。这是因为在某些情况下，该集合是只读的，而在其他情况下，它是可写的。

还有其他获取季节的方法，例如一年中的某一天(例如圣诞节 24.12 )。我想要一个const getter 用于获取它们的每种方法，但我也不想复制粘贴它们中的每一个，只需添加 const .

最好的方法是什么？

Answer 1

我不想这么说，但是const_cast。您要做的就是创建并调用 const getter，然后删除它返回的 const，因为您知道自己位于非 const getter 内的非常量对象中。这看起来像

  const Season* GetSeason(const std::string& name) const
  {
    const int index = findIndex(_seasons, [name](const Season& s) { return s.seasonName == name; });
    return index != -1 ? &_seasons[index] : nullptr;
  }
  Season* GetSeason(const std::string& name)
  {
    return const_cast<Season*>(const_cast<SeasonCollection const *>(this)->GetSeason(name));
    //     ^ remove the const^ ^ add const to call the const getter     ^^call const getter^
  }