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

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

ただいまの
回答率

90.61%

  • C#

    6880questions

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

  • Unity

    3827questions

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

Unity List型から要素取り出し

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 441

JectABC

score 21

前提・実現したいこと

とても初歩的なことで、いつもできていたことなのですが突然できなくなっていたので質問します
以下のコードのDebug.Log(v)でコンソールにクラス名が出てこずnullと表示されています。
しかしv.GetType()やvの要素にアクセスしたところ、その内容が出力できます.

質問は2つです
・vがnullなのにその要素にアクセスできるということはvはいったい何を出力しているのか(本来ならCharTemplateのはず)
・3つ目のクラスGamemanagerでのエラー解決方法

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

エラーメッセージ

該当のソースコード

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

public class CharTemplete :MonoBehaviour{

    public int id;
    public int rank;
    public int rear;
    public string name;


    public CharTemplete(int i,int r,int l,string n){
        id = i;
        rank = r;
        rear = l;
        name = n;
    }

    public void InitStatus(CharTemplete t){
        this.id = t.id;
        this.rank = t.rank;
        this.rear = t.rear;
        this.name = t.name;
    }


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

public class Deckmanager : MonoBehaviour {


    public List<CharTemplete> chars = new List<CharTemplete>();
    public List<CharTemplete> mydekki = new List<CharTemplete>();
     public List<CharTemplate> mylist  = new List<CharTemplate>();

    public void firstDekkiInit(){
        Addin(0);
        foreach(var v in mydekki){
            mylist.Add(v);
            Debug.Log(v);           //null
            Debug.Log(v.GetType());     //CharTemplate
            Debug.Log(v.name);             //"歩"
        }
    }

        //charsに全追加
    public void AllAdd(){
        chars.Add (new CharTemplete (0,1,0,"歩"));
        chars.Add (new CharTemplete (1,2,0,"香"));
        chars.Add (new CharTemplete (2,2,0,"馬"));
        chars.Add (new CharTemplete (3,3,0,"銀"));
        chars.Add (new CharTemplete (4,3,0,"金"));
        chars.Add (new CharTemplete (5,4,0,"飛"));
        chars.Add (new CharTemplete (6,4,0,"角"));
        chars.Add (new CharTemplete (7,5,0,"王"));
        chars.Add (new CharTemplete (8,5,0,"玉"));
    }

     //dekkiにcharsの一部を追加
    public void Addin(int id){
        foreach(var v in chars){
        if(v.id == id)
            mydekki.Add(v);
        }
    }


}
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class GameManager : MonoBehaviour {

    private Deckmanager deck;

    public Button koma; //駒オブジェクト、CharTemplateコンポーネントアタッチ済み
    public Button[] prefab = new Button[20]; //prefab
    public int preCnt = 0;//prefabの添字
    public Transform ImageTransform;

    void Start () {
        koma = Resources.Load<Button>("path");
        deck = new Deckmanager();
        deck.AllAdd();
        deck.firstDekkiInit();
    create_pre();
    }

        //prefabをセット,komaのCharTemplateの値をInitStatus()で変更
    public void create_pre(){
        prefab[preCnt] = Instantiate(koma,ImageTransform.localPosition,ImageTransform.localRotation);
        prefab[preCnt].GetComponent<CharTemplete>().InitStatus(deck.mydekki[preCnt]);//ここでnull refelenceエラー
        prefab[preCnt].transform.SetParent(ImageTransform);
        prefab[preCnt].transform.localScale = Vector3.one;
        prefab[preCnt].GetComponentInChildren<Text>().text = deck.mylist[preCnt].name;
        preCnt++;

    }




}

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

Unity 5.5.3
C#
これらのコードは質問で見やすくするため無駄なコードを省いています。必要なコードも消している恐れがありますので指摘していただければ補足を記述します。
お願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • ozwk

    2017/10/02 09:56 編集

    そもそもforeachでmydekkiリストの要素を列挙しながらその要素をmydekkiにAddしている時点でおかしいんですが(少なくとも素のC#では例外投げられる)、これは省いたためですか?

    キャンセル

  • JectABC

    2017/10/02 18:23

    申し訳ございません。不備がありました。訂正します mydekkiに入れるのではなく他の同じ型のリストへ格納します

    キャンセル

  • ozwk

    2017/10/02 21:09

    このコードで問題が発生することを確認しましたか?

    キャンセル

回答 1

checkベストアンサー

0

You are trying to create a MonoBehaviour using the 'new' keyword.  This is not allowed.  MonoBehaviours can only be added using AddComponent().  Alternatively, your script can inherit from ScriptableObject or no base class at all

といった内容の警告は表示されませんでしたでしょうか?
どうもMonoBehaviourGameObjectと組み合わせてやらないとToStringやらnull判定やらが期待通りにならないようですので、これを何とかしたほうがよさそうです。

参考1: UnityEngine.ObjectとSystem.Objectの注意点 - Qiita
参考2: Unityのnullはnullじゃないかもしれない - Qiita

DeckmanagerはMonoBehaviourの継承をやめることはできそうですが、CharTemplateは駒にアタッチしているようなのでMonoBehaviourを継承しないとまずいでしょうから、無理やり感がありますが下記のようにしてみました。
※いくつかCharTemplateCharTempleteになっていた箇所がありましたが、「CharTemplate」に統一しました。

CharTemplate

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

public class CharTemplate : MonoBehaviour{

    public int id;
    public int rank;
    public int rear;
    public string name;

    // 生成用のメソッドを追加
    public static CharTemplate Create(int i, int r, int l, string n){
        var gameObject = new GameObject(n); // GameObjectを作って...
        var charTemplate = gameObject.AddComponent<CharTemplate>(); // CharTemplateを新規アタッチし...
        charTemplate.InitStatus(i, r, l, n); // 内容をi、r、l、nで初期化して...
        return charTemplate; // できあがった新しいCharTemplateを返す
    }

    // コンストラクタは廃止
    /*
    public CharTemplate(int i,int r,int l,string n){
        id = i;
        rank = r;
        rear = l;
        name = n;
    }
    */

    // 代わりにi、r、l、nをid、rank、rear、nameにセットするメソッドを追加
    public void InitStatus(int i, int r, int l, string n){
        id = i;
        rank = r;
        rear = l;
        name = n;
    } 

    public void InitStatus(CharTemplate t){
        this.id = t.id;
        this.rank = t.rank;
        this.rear = t.rear;
        this.name = t.name;
    }
}

Deckmanager

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

public class Deckmanager { // MonoBehaviourの継承をやめる


    public List<CharTemplate> chars = new List<CharTemplate>();
    public List<CharTemplate> mydekki = new List<CharTemplate>();
    public List<CharTemplate> mylist  = new List<CharTemplate>();

    public void firstDekkiInit(){
        Addin(0);
        foreach(var v in mydekki){
            mylist.Add(v);
            Debug.Log(v);                  //null
            Debug.Log(v.GetType());        //CharTemplate
            Debug.Log(v.name);             //"歩"
        }
    }

    //charsに全追加
    public void AllAdd(){
        // newで生成する代わりに、独自の生成メソッドを使用
        chars.Add (CharTemplate.Create (0,1,0,"歩"));
        chars.Add (CharTemplate.Create (1,2,0,"香"));
        chars.Add (CharTemplate.Create (2,2,0,"馬"));
        chars.Add (CharTemplate.Create (3,3,0,"銀"));
        chars.Add (CharTemplate.Create (4,3,0,"金"));
        chars.Add (CharTemplate.Create (5,4,0,"飛"));
        chars.Add (CharTemplate.Create (6,4,0,"角"));
        chars.Add (CharTemplate.Create (7,5,0,"王"));
        chars.Add (CharTemplate.Create (8,5,0,"玉"));
    }

    //dekkiにcharsの一部を追加
    public void Addin(int id){
        foreach(var v in chars){
            if(v.id == id)
                mydekki.Add(v);
        }
    }


}

GameManagerでのnull参照エラーについてはうまく再現しなかったのですが、komaのプレハブがResourcesフォルダに入っているか、そのアセットの名前は「path」になっているか、そのプレハブにCharTemplateがアタッチされているか...といったところをチェックしてみてはいかがでしょうか?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/10/03 23:21

    わかりづらい質問なのに、ご丁寧にありがとうございます。null参照エラーは出ずにきちんと参照できて思った通りに動きましたが、やはりDebug.logがnullのままでした。komaにもCharTemplateがアタッチされています。もしかしたらlogの出し方など悪かったのかもしれません、ですが参照エラーの解決はしたのでまた別の機会に質問しようと思います。ありがとうございました。

    キャンセル

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

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

関連した質問

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

  • C#

    6880questions

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

  • Unity

    3827questions

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