单一职责原则 - 从文件加载列表？

发布于 2024-11-25 11:27:56 字数 589 浏览 1 评论 0原文

假设我有一个 Car 类：

class Car 
{
  string GetMake()
  string GetModel()
  int GetYear()
}

我有一个自定义 CarService 类，其中包含汽车列表：

class CarService
{
  void AddCar(Car car)
  void RemoveCar(Car car)
  List<Car> GetCars()
   ... other methods removed for clarity...
}

现在我想将汽车列表从文件加载到 CarService 类。我以前的 OOP 直觉是将其作为 CarService 类上的 LoadFromFile() 方法。然而，当我现在正在学习 SRP 和可测试性时，我不太确定。

遵循单一职责原则，正确的设计方法是什么？我应该有一个 CarLoader 类吗？

更新

我认为各种语言的解决方案应该是相同的，但我将使用 C++。如果我使用 C#、Java 或 python，我的问题将是相同的。

原文

Say I have a Car class:

class Car 
{
  string GetMake()
  string GetModel()
  int GetYear()
}

And I have a custom CarService class that holds a list of cars:

class CarService
{
  void AddCar(Car car)
  void RemoveCar(Car car)
  List<Car> GetCars()
   ... other methods removed for clarity...
}

Now I want to load a list of cars from a file to the CarService class. My old OOP instinct would be to put this as a method like LoadFromFile() on the CarService class. However as I'm learning about SRP and testability now I'm not so sure.

Following the single responsibility principle, what is the correct way to design this? Should I have a CarLoader class?

UPDATE

I think the solution should be the same in a wide variety of languages but I'll be using C++. My question would be identical if I was using C#, Java, or python.

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

離人涙 2024-12-02 11:27:56

根据您要实现此功能的语言，我认为 load_carservice 函数或等效的静态方法就足够了。

静态方法解决方案可能会违反 SRP，因为序列化格式可能会发生变化，而类的其余部分保持不变。这就是为什么我从不使用强迫我将所有内容都放在一个类中的语言进行编程的原因之一。如果您的语言强迫您这样做，并且您想严格遵守 SRP，那么将需要额外的课程。

回复收藏 0 原文

农村范ル 2024-12-02 11:27:56

您可能不应该拥有 CarLoader 类。至少根据您所显示的内容，您的 CarService 类看起来也不是很有用。至少马上，它看起来就像（如果它做了任何有用的事情的话）您的CarService基本上是试图在之上构建一个set一个列表。我想我会编写这样的代码：

class Car { 
// ...
    friend std::istream &operator>>(std::istream &is, std::Car &c) { 
        return is >> c.model >> c.year >> c.color;
    }
    friend std::ostream &operator<<(std::ostream &os, std::Car const &c) {
       return os << c.model << "\t" << c.year << "\t" << c.color;
};

std::set<Car> cars;

std::ifstream car_file("cars.txt");

// read data from the file:
std::copy(std::istream_iterator<Car>(car_file),
          std::istream_iterator<Car>(),
          std::inserter(cars));

std::set 已经知道如何添加和删除项目...

You probably should not have a CarLoader class. At least based on what you've shown, your CarService class doesn't look very useful either. At least right off, it looks like (if it does anything useful at all) your CarService is basically trying to build a set on top of a List. I think I'd write the code something like this:

class Car { 
// ...
    friend std::istream &operator>>(std::istream &is, std::Car &c) { 
        return is >> c.model >> c.year >> c.color;
    }
    friend std::ostream &operator<<(std::ostream &os, std::Car const &c) {
       return os << c.model << "\t" << c.year << "\t" << c.color;
};

std::set<Car> cars;

std::ifstream car_file("cars.txt");

// read data from the file:
std::copy(std::istream_iterator<Car>(car_file),
          std::istream_iterator<Car>(),
          std::inserter(cars));

std::set already knows how to add and remove items...

回复收藏 0 原文

~没有更多了~