Понимание принципа единой ответственности дизайна SOLID

Какова цель интерфейса ICheckValid? Вы звоните check_valid в другое место? На мой взгляд, поскольку frame_number на самом деле является свойством Frame только для чтения, почему было бы неправильно проверять его согласованность прямо в конструкторе без каких-либо дополнительных интерфейсов для этого? (Предполагается, что конструкторы создают непротиворечивые объекты и поэтому могут свободно проверять входящие параметры по своему усмотрению.)

Однако вместо того, чтобы спрашивать, как правильно проверить это поле, было бы лучше спросить, зачем вам действительно нужно свойство frame_number в Frame? Похоже, это индекс этого элемента в каком-то массиве - можно просто использовать индекс, зачем хранить его в Frame? Позже вы можете написать некоторую логику if/else, например:

if (frame_number == 10) {
     // some rules
} else {
     // other rules
}

Однако вряд ли это НАДЕЖНЫЙ подход, поскольку вы, вероятно, в конечном итоге будете писать операторы if/else во многих частях файла Frame. Вместо этого вы можете создать базовый класс FrameBase, определить в нем большую часть логики, а также некоторые абстрактные методы, которые будут реализованы в OrdinaryFrame и TenthFrame, где вы определите разные правила. Это позволит вам полностью избежать frame_number — вы просто создадите девять OrdinaryFrames и один TenthFrame.

Что касается критики: ваш код, похоже, абстрагирует шары и рамки, но по какой-то причине игнорирует «броски» или «броски». Учтите необходимость добавления информации о траектории каждого броска, вам нужно будет изменить интерфейс IFrame, добавив что-то вроде void SetThrowTrajectory(int throwNumber, IThrowTrajectory trajectory). Однако, если вы абстрагируете отбросы, например. IBallRoll туда легко поместилась бы функциональность, связанная с траекторией (а также некоторые логические вычисляемые свойства, например, IsStrike, IsSpare).

Когда я думаю о SRP, я обычно делаю акцент на аспекте ответственности. Название класса, в свою очередь, должно в идеале описывать его Ответственность. Для некоторых классов это то, чем должен «быть» класс (хорошим примером может быть фрейм, если ему не хватает поведения и он просто представляет состояние), но когда у вас есть поведенческая ответственность, имя соответствует тому, чем должен быть класс. делать'.

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

//My job is to keep score; I don't interpret the scores
public interface IScoreKeeper : IScoreReporter
{
    void SetScore(int bowlerIndex, int frameIndex, int throwIndex, int score);
}

//My job is to report scores to those who want to know the score, but shouldn't be allowed to change it
public interface IScoreReporter
{
    int? GetScore(int bowlerIndex, int frameIndex, int throwIndex);        
}

//My job is to play the game when told that it's my turn
public interface IBowler
{
    //I'm given access to the ScoreReporter at some point, so I can use that to strategize
    //(more realisically, to either gloat or despair as applicable)

    //Throw one ball in the lane, however I choose to do so
    void Bowl(IBowlingLane lane);
}

//My job is to keep track of the pins and provide score feedback when they are knocked down
//I can be reset to allow bowling to continue
public interface IBowlingLane
{
    int? GetLastScore();

    void ResetLane();
}

//My job is to coordinate a game of bowling with multiple players
//I tell the Bowlers to Bowl, retrieve scores from the BowlingLane and keep
//the scores with the ScoreKeeper.
public interface IBowlingGameCoordinator
{
    //In reality, I would probably have other service dependencies, like a way to send feedback to a monitor
    //Basically anything that gets too complicated and can be encapsulated, I offload to some other service to deal with it
    //I'm lazy, so all I want to do is tell everybody else what to do.

    void PlayGame(IScoreKeeper gameScore, IEnumerable<IBowler> bowlers, IBowlingLane lane);
}

Обратите внимание: если вы хотите использовать эту модель для простого подсчета очков (без участия в реальной игре), у вас может быть заглушка Bowler (которая ничего не делает) и MockBowlingLane, который выдает ряд значений очков. BowlingGameCoordinator заботится о текущем боулер, фрейме и броске, поэтому очки накапливаются.