4
1

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

More than 1 year has passed since last update.

PONOSAdvent Calendar 2022

Day 1

Unityエディタ上で循環的複雑度(CCN)を計測する

Last updated at Posted at 2022-11-30

PONOS Advent Calendar 2022の1日目の記事です。

はじめに

ポノスではソースコードの保守性が担保されているか判断するために、循環的複雑度(以下、CCN) の数値を参考にしています。(循環的複雑度 - Wikipedia)
この数値はソースコードがどれだけ複雑に構成されているかを表現しており、処理の分岐(if、for、switch等)が増えるに従って上昇していきます。
一般的に、CCNが高いソースコードほど保守しづらくバグが混入しやすい傾向があります。
CCNの数値が高いメソッドを発見したら、リファクタリングを行って分岐を減らしたり処理を分散させたりして、数値を下げるようにしましょう。
(なお、ポノスではCCNが30を超えるコードがないことがリリース要件とされています)

今回は、Unity開発においてCCNを確認しやすくする方法を紹介したいと思います。

lizardでCCNを計測する

今回の記事ではCCNを計測するためのプログラムとしてlizardを利用します。
lizardはPython製のプログラムとなりますので、pipを使用してインストールしてください。

$ pip install lizard

計測対象のフォルダへ移動し、lizardコマンドを実行することで計測が実施されます。

$ cd Assets
$ lizard -l csharp

-lは計測対象のプログラミング言語を指定するオプションです。
今回はUnityプロジェクトのC#スクリプトファイルが対象となるので、csharpを指定します。

試しに、UnityでCreate/C# Scriptメニューから作成されるC#スクリプトファイルを対象に計測してみます。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class Sample : MonoBehaviour
{
    void Start()
    {
    }

    void Update()
    {   
    }
}

このC#スクリプトファイルに対して、lizardを実行してみます。

$ lizard -l csharp
================================================
  NLOC    CCN   token  PARAM  length  location
------------------------------------------------
       3      1      5      0       4 Sample::Start@8-11@./Sample.cs
       3      1      5      0       4 Sample::Update@14-17@./Sample.cs
1 file analyzed.
==============================================================
NLOC    Avg.NLOC  AvgCCN  Avg.token  function_cnt    file
--------------------------------------------------------------
     12       3.0     1.0        5.0         2     ./Sample.cs

===============================================================================================================
No thresholds exceeded (cyclomatic_complexity > 15 or length > 1000 or nloc > 1000000 or parameter_count > 100)
==========================================================================================
Total nloc   Avg.NLOC  AvgCCN  Avg.token   Fun Cnt  Warning cnt   Fun Rt   nloc Rt
------------------------------------------------------------------------------------------
        12       3.0     1.0        5.0        2            0      0.00    0.00

結果出力の中の「CCN」の数値が循環的複雑度を指しています。
Start()Update()の2つのメソッドについて計測していることがわかりますが、どちらもCCNの値は1となっています。
これはメソッド内にifやforなどの分岐が存在していないためです。

このままだとちゃんとCCNが計測されているのかが分かりづらいため、Start()メソッドに分岐を増やしてみます。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class Sample : MonoBehaviour
{
    void Start()
    {
        int number = 1;
        if(0 < number) // まったく意味のない分岐
        {
            Debug.Log("numberは0よりも大きい!");
        }
    }

    void Update()
    {
    }
}

再度、lizardの計測を実施してみましょう。

$ lizard -l csharp
================================================
  NLOC    CCN   token  PARAM  length  location
------------------------------------------------
       8      2     24      0       8 Sample::Start@7-14@./Sample.cs
       3      1      5      0       3 Sample::Update@16-18@./Sample.cs
1 file analyzed.
==============================================================
NLOC    Avg.NLOC  AvgCCN  Avg.token  function_cnt    file
--------------------------------------------------------------
     17       5.5     1.5       14.5         2     ./Sample.cs

===============================================================================================================
No thresholds exceeded (cyclomatic_complexity > 15 or length > 1000 or nloc > 1000000 or parameter_count > 100)
==========================================================================================
Total nloc   Avg.NLOC  AvgCCN  Avg.token   Fun Cnt  Warning cnt   Fun Rt   nloc Rt
------------------------------------------------------------------------------------------
        17       5.5     1.5       14.5        2            0      0.00    0.00

if分の追加により、処理の分岐が増えたためStart()メソッドのCCNの値が1から2へ上昇していることがわかります。

lizardのプログラムを使用すれば、非常に手軽にメソッドのCCNを取得することができました。

Unityエディタ上で循環的複雑度を計測する

では、このlizardのプログラムをUnityから呼び出し、Unity上で計測結果を確認できるようにしてみましょう。
Unityから外部のプログラムを実行するためにはProcessクラスを使用します。

Processを介してlizardのプロジェクトを実行し結果をConsoleウィンドウへ出力するメソッドを作成します。

using System.Diagnostics;
using UnityEditor;

public static class CyclomaticComplexityInstrument
{
    [MenuItem("循環的複雑度/計測")]
    public static void Perform()
    {
        var process = new Process();
        // lizardのプログラム指定にはフルパスが必要でした。
        process.StartInfo.FileName = "/Library/Frameworks/Python.framework/Versions/Current/bin/lizard";
        process.StartInfo.Arguments = "-l csharp";
        process.StartInfo.RedirectStandardOutput = true;
        process.StartInfo.UseShellExecute = false;
        process.StartInfo.CreateNoWindow = true;
        process.Start();

        var output = process.StandardOutput.ReadToEnd();
        process.WaitForExit();
        process.Close();

        UnityEngine.Debug.Log(output);
    }
}

メニューからこのプログラムを実行すると、Consoleウィンドウへ以下のような結果が出力されます。
(新規プロジェクトを作成したときに自動でインストールされたパッケージ内のC#スクリプトファイルが計測されている模様です)
image.png
Unity上でCCNの計測結果を出力することができました。

Projectウィンドウで選択されたC#スクリプトファイルに対してのみCCNを計測する

ここまでで、UntiyプロジェクトのC#スクリプトファイルのCCNを計測して出力することができました。
しかし、現在のソースコードはプロジェクト内のすべてのC#スクリプトファイルを計測対象としているため、プロジェクト内のC#スクリプトファイルの数が多くなると、目的のC#スクリプトファイルの結果を確認しづらくなってしまいます。

そこで、Projectウィンドウで選択中のC#スクリプトファイルに対してのみ計測が実行されるに改修し、目的のC#スクリプトファイルの計測結果がすぐに確認できるようにしてみましょう。
lizardは実行時の引数で計測対象のファイルを指定することができるため、Projectウィンドウで選択中のアセットのパスを取得し、lizardの引数に設定します。

using System.Diagnostics;
using System.Linq;
using UnityEditor;

public static class CyclomaticComplexityInstrument
{
    [MenuItem("Assets/循環的複雑度/計測")]
    public static void Perform()
    {
        var assetPaths = Selection.assetGUIDs.Select(e => AssetDatabase.GUIDToAssetPath(e));
        var targets = assetPaths.Aggregate((a, b) => a + " " + b);
       
        var process = new Process();
        process.StartInfo.FileName = "/Library/Frameworks/Python.framework/Versions/Current/bin/lizard";
        process.StartInfo.Arguments = "-l csharp " + targets;
        process.StartInfo.RedirectStandardOutput = true;
        process.StartInfo.UseShellExecute = false;
        process.StartInfo.CreateNoWindow = true;
        process.Start();

        var output = process.StandardOutput.ReadToEnd();
        process.WaitForExit();
        process.Close();

        UnityEngine.Debug.Log(output);
    }
}

機能が完成したら先ほど計測したSampleのC#スクリプトファイルに対して計測を実施してみましょう。
Projectウィンドウ上で対象となるC#スクリプトファイルを選択し、右クリックから循環的複雑度/計測を選択します。
image.png
以下のように、選択中のC#スクリプトファイル内に定義されているメソッドの情報だけが出力されています。
image.png

これでUnityエディタ上から気軽に目的のC#スクリプトファイルのCCNが計測できるようになりました。
プルリクエストを作成する前などにこの機能を使い、CCNの高いコードを書いていないかを確認すると良いかと思います。

まとめ

CCNが大きい数値にならないように心がけ、ソースコードの保守性を保っていきたいですね!

明日は@blockさんです!

4
1
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
4
1

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?