目次

SOLID原則を勉強するその1～単一責任の原則 (SRP)～
SOLID原則を勉強するその2～オープン・クローズド原則(OCP)～
SOLID原則を勉強するその3～リスコフの置換原則(LSP)～
SOLID原則を勉強するその4～インターフェース分離の原則(ISP)～　←いまここ
SOLID原則を勉強するその5～依存性逆転の原則(DIP)～

前置き

書籍を読んだり、ググったりして、自分に分かりやすいようにまとめた記事です。
より詳しく知りたい方は、下記の参考文献を読んでみてください。

参考文献

Clean Architecture　達人に学ぶソフトウェアの構造と設計 | Amazon
Adaptive Code　～　C#実践開発手法 | Amazon
C#の設計の基本【SOLID原則】まとめ
 Unity開発で使える設計の話＋Zenjectの紹介
 C# で SOLID の原則に違反する危険性

インターフェース分離の原則(ISP)

クライアントが利用しないメソッドの実装を強制してはいけない
- 「不必要なメソッドなのに実装しなければならない」を防ぐ
- インターフェースには最小限のものだけ定義するべき

コード例

それぞれの敵の行動パターンを管理する Shadow クラス、 Gigas クラス、 DarkInferno クラスを作成します。
（敵の元ネタはキングダムハーツです。分からない人はすみません…）
それぞれのクラスは、以下の行動を行います。

	`Shadow`	`Gigas`	`DarkInferno`
攻撃	○	○	○
歩行	○	○	×
飛行	×	○	○

before

IEnemy インターフェースを用意
- 敵クラスは全てこのインターフェースを実装する

IEnemy.cs

public interface IEnemy
{
    void Attack();
    void Walk();
    void Fly();
}

Shadow.cs

using UnityEngine;

public class Shadow : IEnemy
{
    public void Attack()
    {
        Debug.Log("シャドウの攻撃");
    }

    public void Fly()
    {
        // 空を飛ばないので、このメソッドでは何もしない
    }

    public void Walk()
    {
        Debug.Log("シャドウが歩いて移動");
    }
}

Gigas.cs

using UnityEngine;

public class Gigas : IEnemy
{
    public void Attack()
    {
        Debug.Log("ギガースの攻撃");
    }

    public void Fly()
    {
        Debug.Log("ギガースが空を飛ぶ");
    }

    public void Walk()
    {
        Debug.Log("ギガースが歩く");
    }
}

DarkInferno.cs

using UnityEngine;

public class DarkInferno : IEnemy
{
    public void Attack()
    {
        Debug.Log("ダークインフェルノの攻撃");
    }

    public void Fly()
    {
        Debug.Log("ダークインフェルノが空を飛んで移動");
    }

    public void Walk()
    {
        // 歩かずに常に空を飛んで移動するので、このメソッドでは何もしない
    }
}

ダメなところ

Gigas クラスは全てのメソッドを必要としているが…
- Shadow クラスは Fly メソッドは不要（空を飛ばないから）
- DarkInferno クラスは Walk メソッドは不要（歩かないから）
  - 不要なメソッドの実装を強要している
  - 新たな敵クラスを実装するときや、メソッドを呼び出すクライアント側も混乱してしまう

After

IEnemy インターフェースを IAttack, IWalk, IFly の各インターフェースに分離
- 敵クラスは必要なインターフェースを選んで実装する

IAttack.cs

public interface IAttack
{
    void Attack();
}

IWalk.cs

public interface IWalk
{
    void Walk();
}

IFly.cs

public interface IFly
{
    void Fly();
}

Shadow.cs

using UnityEngine;

public class Shadow : IAttack, IWalk
{
    public void Attack()
    {
        Debug.Log("シャドウの攻撃");
    }

    public void Walk()
    {
        Debug.Log("シャドウが歩いて移動");
    }
}

Gigas.cs

using UnityEngine;

public class Gigas : IAttack, IFly, IWalk
{
    public void Attack()
    {
        Debug.Log("ギガースの攻撃");
    }

    public void Fly()
    {
        Debug.Log("ギガースが空を飛ぶ");
    }

    public void Walk()
    {
        Debug.Log("ギガースが歩く");
    }
}

DarkInferno.cs

using UnityEngine;

public class DarkInferno : IAttack, IFly
{
    public void Attack()
    {
        Debug.Log("ダークインフェルノの攻撃");
    }

    public void Fly()
    {
        Debug.Log("ダークインフェルノが空を飛んで移動");
    }
}

良いところ

不要なインターフェースの実装がなくなった

終わりに

もし変なところがあったら教えて下さい。
（特にクラス図とか…）