質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.50%

  • C#

    7116questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • Unity

    4002questions

    Unityは、ユニティテクノロジーが開発したゲームエンジンです。 主にモバイルやブラウザ向けのゲーム製作に利用されていましたが、3Dの重力付きゲームが簡単に作成できることから需要が増え、現在はマルチプラットフォームに対応しています。 言語はC言語/C++で書かれていますが、C#、JavaScript、Booで書かれたコードにも対応しています。

  • 配列

    522questions

    配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

Unity C# Listをエディタ拡張で2次元配列を扱いたい

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,756

suvera

score 87

前提・実現したいこと

エディタ拡張で2次元配列を扱って
inspector(EditorWindow)で色々値を変更し
実行時にその値が反映されるようにしたいです。

発生している問題・エラーメッセージ

inspector上では成功しているように見えます。
具体的にはサイズを増やせば欄縦にも横にも増えて
値の変更なども維持されます。

しかし、実行ボタンを押すと初期化されるのか何故かNullに変わってしまいます。
当然inspectorもNullに変わります。

該当のソースコード

# スクリプトの一部

MonoBehaviour継承クラス

MapManagerClass
public List<List<int>> map;

void Start() {
Debug.Log(map);
}
public void SetMap(List<List<BlockStatus>> newMap) {
map = newMap;
}

Editorスクリプト


void OnGUI() {
mapManager = EditorGUILayout.ObjectField("MapManagerコンポーネント を持ったオブジェクト", mapManager, typeof(MapManager), true) as MapManager;

if (mapManager == null)
return;

List<List<int>> map = mapManager.map;
int mapSize = 0;
Debug.Log(map);
if (map != null) {
mapSize = map.Count;
}
else {
mapManager.map = new List<List<int>>();
map = mapManager.map;
}

foldoutMapList = EditorGUILayout.Foldout(foldoutMapList, "map");
if (this.foldoutMapList) {
mapSize = EditorGUILayout.IntField("Size", mapSize);
mapSize = Mathf.Clamp(mapSize, 1, 5);
EditorGUILayout.BeginHorizontal(GUI.skin.box);
{
for (int index = 0; index < mapSize; index++) {
EditorGUILayout.BeginVertical(GUI.skin.box);
{
if (index < map.Count) {
EditorGUILayout.LabelField(index.ToString());
}
else {
map.Add(new List<int>());
}
EditorGUILayout.EndVertical();
}
}
EditorGUILayout.EndHorizontal();
if (mapSize < map.Count) {
map.RemoveRange(mapSize, map.Count - mapSize);
}
}
mapManager.SetMap(map);
}

試したこと

1次元で同じような処理をやってみました。
こちらは思ったように動作して、実行時にNullになるようなことはありませんでした。

補足情報(言語/FW/ツール等のバージョンなど)

Unity2017
C#

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

Serialize Nested Lists - Unity Answersによると、どうやらUnityは入れ子のListをシリアライズしてくれないようですね...
回答者の方は、下記のようにラッパークラスを作る方法を提案されていますが、この形に書き換えてみるとどうでしょうか?


absameen による 質問 · 2012年07月24日 17:29

Serialize Nested Lists

Will Unity not serialize nested lists such as:

 List<List<int>> nList = new List<List<int>>();


How can we serialize this list?


CapnCromulent による 回答 · 2012年09月14日 04:16

Unfortunately, you'll have to write a wrapper class, like:

 [System.Serializable]
 public class ListWrapper
 {
      public List<int> myList;
 }


then declare within your ScriptableObject

 List<ListWrapper> nList =new List<ListWrapper>();

[追記]
多次元のListをInspectorに表示する【Unity】 - (:3[kanのメモ帳]で、同様にラッパー方式で多次元Listをインスペクタ上でも取り扱えるようにする方法が紹介されていました。こちらもご参考になりそうです。

[コメントを受けて追記]
いまいちしっくりいく回答を提示できず申し訳ないです...
下記のような感じでインデクサーを追加してやれば多少はスクリプトの見た目がすっきりしますかね?あんまり大差はないですが...

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

public class List2DTest : MonoBehaviour
{
    public List<ListInt> Test;

    private void Start()
    {
        Debug.Log(this.Test[0][0]);
    }
}

[Serializable]
public class ListInt
{
    [SerializeField]
    private List<int> storage = new List<int>();

    public int this[int i]
    {
        get
        {
            return this.storage[i];
        }

        set
        {
            this.storage[i] = value;
        }
    }
}

[改善案を思いついたので追記]
ISerializationCallbackReceiverを実装することでシリアライズ時の挙動をコントロールできるらしいので、今さらながら試してみました。

まず、MapManagerの方の基本戦略としては、シリアライズして保存しておくためのラッパークラス方式のデータ(コード中のserializableMap)と、他のスクリプトから使うときのためのList<List<int>>(コード中のmap)に二重化しておいて、シリアライズ・デシリアライズ時に一方を他方に変換して同期することにしました。データ量が増えてしまいますが、一応それっぽく動くのではないでしょうか(同期部分は手抜きして毎回新しいオブジェクトを再生成していますが、既存のオブジェクトを再利用する形にすればいくらか効率的だと思います)。

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

public class MapManager : MonoBehaviour, ISerializationCallbackReceiver
{
    [NonSerialized]
    public List<List<int>> map;

    [SerializeField, HideInInspector]
    private List<ListInt> serializableMap;

    void Start() {
        Debug.Log(map);
    }

    public void SetMap(List<List<int>> newMap) {
        map = newMap;
    }

    public void OnBeforeSerialize()
    {
        this.serializableMap = this.map == null ? null : this.map.Select(subList => new ListInt(subList)).ToList();
    }

    public void OnAfterDeserialize()
    {
        this.map = this.serializableMap == null ? null : this.serializableMap.Select(listInt => listInt.data).ToList();
    }
}

[Serializable]
public class ListInt
{
    public List<int> data;

    public ListInt(List<int> list)
    {
        this.data = list ?? new List<int>();
    }
}

エディターウィンドウの方ですが、プレイモードにすると操作対象のMapManagerの参照が失われるので、代わりにMapManagerを持っているゲームオブジェクトへの参照を保持させることにしました。
それ以外は、2次元表示を試すためにIntFieldを入れてみたりしましたが、mapへのアクセス自体はご質問者さんご提示のコードに倣っています。

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

public class MapManagerEditorWindow : EditorWindow
{
    private const float MinWidth = 16.0f;
    private const float MaxWidth = 1024.0f;

    private GameObject mapManagerObject;
    private bool foldoutMapList;

    [MenuItem("Window/Map Editor")]
    static void Open() {
        EditorWindow.GetWindow<MapManagerEditorWindow>("Map Editor");
    }

    void OnGUI() {
        MapManager mapManager = EditorGUILayout.ObjectField("MapManagerコンポーネント を持ったオブジェクト", mapManagerObject == null ? null : mapManagerObject.GetComponent<MapManager>(), typeof(MapManager), true) as MapManager;
        if (mapManager == null)
            return;
        mapManagerObject = mapManager.gameObject;
        List<List<int>> map = mapManager.map;
        int mapSize = 0;
        Debug.Log(map);
        if (map != null) {
            mapSize = map.Count;
        } else {
            mapManager.map = new List<List<int>>();
            map = mapManager.map;
        }
        foldoutMapList = EditorGUILayout.Foldout(foldoutMapList, "map");
        if (this.foldoutMapList) {
            mapSize = EditorGUILayout.IntField("Size", mapSize);
            mapSize = Mathf.Clamp(mapSize, 1, 5);
            EditorGUILayout.BeginHorizontal(GUI.skin.box);
            {
                for (int index = 0; index < mapSize; index++) {
                    EditorGUILayout.BeginVertical(GUI.skin.box);
                    {
                        if (index < map.Count) {
                            EditorGUILayout.LabelField(index.ToString(), GUILayout.MinWidth(MinWidth), GUILayout.MaxWidth(MaxWidth));
                        } else {
                            map.Add(new List<int>());
                        }
                        List<int> column = map[index];
                        for (int rowIndex = 0; rowIndex < mapSize; rowIndex++) {
                            if (rowIndex >= column.Count) {
                                column.Add(0);
                            }
                            column[rowIndex] = EditorGUILayout.IntField(column[rowIndex], GUILayout.MinWidth(MinWidth), GUILayout.MaxWidth(MaxWidth));
                        }
                        EditorGUILayout.EndVertical();
                        if (mapSize < column.Count) {
                            column.RemoveRange(mapSize, column.Count - mapSize);
                        }
                    }
                }
                EditorGUILayout.EndHorizontal();
                if (mapSize < map.Count) {
                    map.RemoveRange(mapSize, map.Count - mapSize);
                }
            }
            mapManager.SetMap(map);
        }
    }
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/24 19:37

    ありがとうございます。
    自分も追記のところのサイトに行きつきましてその方法で解決しました。
    まぁ本当はList<List<int>>の形がきれいだとは思うんですが無理なようですしList<Class>で行こうと思います。

    なお、Nullになるのはinspectorに表示して扱っていなかったため実行時に初期化されていたからでした。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.50%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • C#

    7116questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • Unity

    4002questions

    Unityは、ユニティテクノロジーが開発したゲームエンジンです。 主にモバイルやブラウザ向けのゲーム製作に利用されていましたが、3Dの重力付きゲームが簡単に作成できることから需要が増え、現在はマルチプラットフォームに対応しています。 言語はC言語/C++で書かれていますが、C#、JavaScript、Booで書かれたコードにも対応しています。

  • 配列

    522questions

    配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。