unityでアイテムを集めるとスコアが増えるようにしたいのに増えない

Question

###何がしたいか
プレイヤーがアイテムを集めると変数「sc_count」を1ずつ増やしcanvas内のtextに「sc_count」を表示させたい
###問題
最初にプレイヤーがアイテムに触れたときは変数が増えるがそれ以降は増えなくなる。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;
public class item : MonoBehaviour
{
    public GameObject target ;
     public Text scoreLabel;
     int sc_count;
    void Start (){
        sc_count =0;
        scoreLabel.text = sc_count.ToString();
   }
   
 
    void OnTriggerEnter (Collider hit)
    {
        if (hit.CompareTag ("Player"))
        {
            
            Destroy(gameObject);
//新しくランダムでアイテムを配置する
            Instantiate (target,new Vector3(UnityEngine.Random.Range(-9.0f,9.0f),0.5f,UnityEngine.Random.Range(-9.0f,9.0f)),Quaternion.identity);
         sc_count++;
         scoreLabel.text = sc_count.ToString();
    
         
            
        }
    }
   
    
}

追記

画像


Creator.cs

//変更前
var script = Instantiate<item>(ItemPrefab, pos, Quaternion.identity);
//変更後
var script = Instantiate<item>(ItemPrefab, new Vector3(UnityEngine.Random.Range(-9.0f,9.0f),0.5f,UnityEngine.Random.Range(-9.0f,9.0f)), Quaternion.identity);

Answer 1

[追記] 追記していただいた画像について

どちらも合ってます。Creator.csのコードも問題ありません。

[追記] スコアが９ずつ増えていきます

これは、今分かりません。下記コードで試してみて9つ増えるのであれば言ってください。

[追記] コードを一部修正します。

コード見ました。itemCreatorをstaticにしてましたがstaticは理解するまで使わない方が良いです。といっても慣れた頃には使わない方が良いコードが書けると理解すると思います。今はDataManagerぐらいにしておくのがおすすめです。DataManagerは変えず、下記のような形でどうでしょうか？

itemCreator.cs

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

public class itemCreator : MonoBehaviour
{
    //フォルダに入れたitemをアタッチ
    public GameObject target;

    void Start()
    {
        DataManager.Instance.OnAddScore += OnAddScore;
    }

    public void Create()
    {
        var pos = GetRandomPos();
        Instantiate(target, pos, Quaternion.identity);
    }

    //スコアが加算された時に呼ばれる
    public void OnAddScore(int score)
    {
        //スコアが加算されたので生成
        Create();
    }

    Vector3 GetRandomPos()
    {
        return new Vector3(
            UnityEngine.Random.Range(-9.0f, 9.0f),
            0.5f,
            UnityEngine.Random.Range(-9.0f, 9.0f)
        );
    }
}

item.cs

using UnityEngine;

public class item : MonoBehaviour
{
    void OnTriggerEnter(Collider hit)
    {
        if (hit.CompareTag("Player"))
        {
            DataManager.Instance.AddScore(100);
            Destroy(gameObject);
        }
    }
}

過去回答

回答させていいただきます。int sc_countをstatic int sc_countに変更し、Startのsc_count =0;を削除するとカウントされるはずです。

ただ、あまり良い方法とは言えません。
スコアを管理するスクリプトを用意して、そちらに通知するのが良いと思います。
僕なら下記のようにします。

・DataManager スコアを管理するスクリプト
・ScoreTxt DataManagerのスコアが加算されたら表示するスクリプト
・ItemCreator itemの生成、itemに対して一括処理したい内容(消した時の獲得スコアなど)を記述するスクリプト
・Item 現在のitemスクリプトのスコア関連を無くしたスクリプト

[追記]書いてみました

書いてみたので共有します。参考になりますと幸いです。

DataManager データの管理を担当
DontDestroyOnLoadを付与したシングルトンです。

using System;
using UnityEngine;

public class DataManager : MonoBehaviour
{
    //シングルトン化 DataManager.Instanceで何処からでもアクセスできます。
    public static DataManager Instance
    {
        get{
            if (_instance == null)
            {
                _instance = LoadInstance();
            }
            return _instance;
        }
    }
    static DataManager _instance;

    /// <summary>
    /// スコアが追加されたのを通知
    /// </summary>
    public event Action<int> OnAddScore = delegate {};

    /// <summary>
    /// 現在のスコア
    /// </summary>
    /// <value>The score.</value>
    public int Score{ get; private set; }

    void Awake()
    {
        if(_instance != null
           && _instance != this)
        {
            Destroy(gameObject);
        }
    }

    public void AddScore(int addScore)
    {
        //スコアを加算
        Score += addScore;
        //通知
        OnAddScore(Score);
    }
    
    /// <summary>
    /// インスタンスの設定
    /// </summary>
    public static DataManager LoadInstance()
    {
        var instance = FindObjectOfType<DataManager>();

        if(instance == null)
        {
            var go = new GameObject("DataManager", new []{typeof(DataManager) });
            instance =  go.GetComponent<DataManager>();
        }

        DontDestroyOnLoad(instance.gameObject);
        return instance;
    }
}

ItemCreator アイテムの生成とアイテムに一括処理したい内容を記述

using UnityEngine;

public class ItemCreator : MonoBehaviour
{
    /// <summary>
    /// アイテムのPrefabをHierarchyから設置
    /// </summary>
    [SerializeField] item ItemPrefab;

    void Awake()
    {
        //すでにHierarchyに設置されているitemがあれば初期化します。
        //FindObjectsOfTypeは重い処理なのであまり使わない方が良いです。
        foreach (var item in FindObjectsOfType<item>())
        {
            SetUpItem(item);
        }
    }

    /// <summary>
    /// アイテムの生成
    /// </summary>
    /// <param name="pos">Position.</param>
    public void Create(Vector3 pos)
    {
        var script = Instantiate<item>(ItemPrefab, pos, Quaternion.identity);
        //アイテムの初期化
        SetUpItem(script);
    }

    /// <summary>
    /// アイテムの初期化
    /// </summary>
    /// <param name="item">Item.</param>
    void SetUpItem(item item)
    {
        //アイテムの初期化
        item.Initialized(1);
        item.OnDestroyItem += OnDestroyItem;
    }

    /// <summary>
    /// itemが破棄されたタイミングで呼ばれるイベント
    /// </summary>
    /// <param name="item">Item.</param>
    void OnDestroyItem(item item)
    {
        //itemが破棄されたのでDataManagerにスコア通知
        DataManager.Instance.AddScore(item.AddScore);
    }
}

ScoreTxt スコアの表示を担当

using UnityEngine;
using UnityEngine.UI;

public class ScoreTxt : MonoBehaviour
{
    /// <summary>
    /// テキストを設定
    /// </summary>
    [SerializeField] Text TxtRender;

    void Start()
    {
        //初期スコアを設定
        OnAddScore(DataManager.Instance.Score);

        // スコア追加イベントを受け取る
        DataManager.Instance.OnAddScore += OnAddScore;
    }

    /// <summary>
    /// スコアがカウントされると呼ばれる
    /// </summary>
    /// <param name="score">Score.</param>
    void OnAddScore(int score)
    {
        TxtRender.text = score.ToString();
    }
}

item 少々改変しました

using System;
using UnityEngine;

public class item : MonoBehaviour
{
    /// <summary>
    /// itemが削除される際に呼ばれるイベント
    /// </summary>
    public event Action<item> OnDestroyItem = delegate {};

    /// <summary>
    /// 追加するスコア
    /// </summary>
    /// <value>The add score.</value>
    public int AddScore{ get; private set; }

    public void Initialized(int addScore)
    {
        AddScore = addScore;
    }

    void OnTriggerEnter(Collider hit)
    {
        if (hit.CompareTag("Player"))
        {
            OnDestroyItem(this);
            Destroy(gameObject);
        }
    }

    void OnDestroy()
    {
        //イベントがバインドされている場合、GCに回収されない場合があるので破棄する
        OnDestroyItem = null;
    }
}

Comments

[naotiki]

回答ありがとうございます。
明日試してみます。

[naotiki]

治りました
試してみたのですがこのようなエラーが出ました。
~~~
You are trying to replace or create a Prefab from the instance 'item' that references a missing script. This is not allowed.
Please change the script or remove it from the GameObject.
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)
~~~

[IShix]

削除したオブジェクトをInstantiateしようとしてエラーになってます。
アタッチしたItem Prefabですが、Hierarchyに設置したItem Prefabを設定していませんか？
フォルダに置いたItem Prefabを設定するようにしてください。

[naotiki]

なぜかitemが増えません原因わかりますでしょうか？
お手数おかけして申し訳ありません。

[IShix]

なぜ増えないのかの判断できません。
情報もらえませんか？

[naotiki]

エラー
NullReferenceException: Object reference not set to an instance of an object
ScoreTxt.OnAddScore (System.Int32 score) (at Assets/script/ScoreTxt.cs:26)
ScoreTxt.Start () (at Assets/script/ScoreTxt.cs:14)

[naotiki]

警告
DontDestroyOnLoad only work for root GameObjects or components on root GameObjects.
UnityEngine.Object:DontDestroyOnLoad(Object)
DataManager:LoadInstance() (at Assets/script/DataManager.cs:60)
DataManager:get_Instance() (at Assets/script/DataManager.cs:12)
ScoreTxt:Start() (at Assets/script/ScoreTxt.cs:14)

[naotiki]

scriptの場所の写真追加します

[naotiki]

情報の追記をしました

[naotiki]

エラーなどは消えました
しかしもう一つ問題があったのをわすれていましたなぜかスコアが９ずつ増えていきます

[IShix]

今、回答修正するので少々お待ちください。

[IShix]

追記しました。

[naotiki]

ありがとうございました
無事解決しました

[IShix]

解決できてよかったです。

[naotiki]

https://teratail.com/questions/88395

[naotiki]

https://teratail.com/questions/183640

Answer 2

もう分らなかったので初心者なりに頑張りました
itemCreator.cs

using UnityEngine;
using System.Collections;
using System.Collections.Generic;
public class itemCreator : MonoBehaviour
{
     public static itemCreator Instance
    {
        get{
            if (_instance == null)
            {
                _instance = LoadInstance();
            }
            return _instance;
        }
    }
    static itemCreator _instance;
    
    public GameObject target ;

    
       public void Create()
    {
        
        var pos = new Vector3(UnityEngine.Random.Range(-9.0f,9.0f),0.5f,UnityEngine.Random.Range(-9.0f,9.0f));
   
        Instantiate(target, pos, Quaternion.identity);
    
    
    }
        public static itemCreator LoadInstance()
    {
        var instance = FindObjectOfType<itemCreator>();

        if(instance == null)
        {
            var go = new GameObject("itemCreator", new []{typeof(itemCreator) });
            instance =  go.GetComponent<itemCreator>();
        }

        DontDestroyOnLoad(instance.gameObject);
        return instance;
    }

}

item.cs

using System;
using UnityEngine;

public class item : MonoBehaviour
{
    /// <summary>
    /// itemが削除される際に呼ばれるイベント
    /// </summary>
    public event Action<item> OnDestroyItem = delegate {};

    /// <summary>
    /// 追加するスコア
    /// </summary>
    /// <value>The add score.</value>
    public int AddScore{ get; private set; }

    public void Initialized(int addScore)
    {
        AddScore = addScore;
    }

    void OnTriggerEnter(Collider hit)
    {
        if (hit.CompareTag("Player"))
        {
            OnDestroyItem(this);
            Destroy(gameObject);
            
            itemCreator.Instance.Create();
           DataManager.Instance.AddScore(100);
        }
    }

  
}

この書き方やっぱあまり良くないですか？

Answer 3

この場合、itemごとにsc_countがカウントされます。
例えば、アイテムAとアイテムBが存在する場合に、アイテムAを取った場合、アイテムAのsc_countは増えますが、その後即座にアイテムAが消えるため、意味ないです（当然、アイテムBのsc_countは増えません）。

アイテムではなく、1つだけ存在する独立したゲームオブジェクトで、スコアを管理してください。

Comments

[naotiki]

https://teratail.com/questions/88395

[naotiki]

https://teratail.com/questions/183640