Когда нарушать принцип единой ответственности?

Question

Как вы решаете, когда и как нарушать single responsability principle?

Например, скажем, у меня есть сетевая камера со следующим интерфейсом (интерфейс продолжал глупо и «неправильно» для простоты):

class Camera 
{ 
    string user(); 
    void set_user(string user); 

    string password(); 
    void set_password(string password); 

    string url(); 
    void set_url(string url); 

    image take_snapshot(); 

    bool reboot(); 
} 

Это выглядит естественно, но это выглядит как класс камеры имеет 3 responsabilties: хранить метаданные, делать снимок, перезагружать. После SRP, вы могли бы написать это следующим образом:

class Camera 
{ 
    string user(); 
    void set_user(string user); 

    string password(); 
    void set_password(string password); 

    string url(); 
    void set_url(string url); 
} 

image take_snapshot(camera c); 
bool reboot_camera(camera c); 

Здесь питания аккуратно разделены по ОБЯЗАННОСТИ, но сейчас он очень похож на C с немыми и функцией структурами ... Возникает вопрос о том, зачем нам ООП в первую очередь.

Как вы балансируете между удобством и SRP?

[EDIT]

@stjin идея была показана в ответ на @John Zwinck

Answer 1

Я бы написать пример так:

class Session 
{ 
public: 
    Session(string url, string user, string password); 
}; 

class Camera 
{ 
public: 
    Camera(Session); 

    image take_snapshot(); 
    bool reboot(); 
}; 

Основная идея здесь заключается в разделении аутентификацию и определение сеанса/конечной точки (и, возможно, соединение) с помощью элементов управления камерой. Класс Camera теперь более точно моделирует настоящую камеру: у нее есть что-то вроде кнопки питания и кнопки спуска затвора. Виртуализация камеры находится в другом месте, сама по себе. Это также делает более очевидным, что делать, если кто-то хочет сделать сеанс USB для другой камеры и т. Д.

Вторичная идея состоит в том, что объекты создаются с действующим состоянием с самого начала. Здесь нет способа настроить камеру без пароля, например, вызов функции take_snapshot() без учетных данных просто невозможно. Конечно, учетные данные могут быть недействительными, но это может быть проверено каким-либо методом, который может быть вызван в одном из конструкторов.

В стороне, нет ничего плохого в бесплатных функциях. ООП переоценивается, мы все это знаем, и вы не должны испытывать никакой необходимости извиняться, если бесплатные функции работают в вашем случае использования. Dumb-структуры могут быть лучше, чем бессмысленные геттеры и сеттеры, особенно если вы не создаете повторно используемую библиотеку с требованиями совместимости ABI.