回答編集履歴

回答修正します。

2019/04/06 11:14

投稿

IShix

スコア1729

answer CHANGED Viewed

@@ -1,3 +1,73 @@
+### [追記] 追記していただいた画像について
+どちらも合ってます。Creator.csのコードも問題ありません。
+### [追記] スコアが９ずつ増えていきます
+これは、今分かりません。下記コードで試してみて9つ増えるのであれば言ってください。
+### [追記] コードを一部修正します。
+コード見ました。```itemCreator```をstaticにしてましたがstaticは理解するまで使わない方が良いです。といっても慣れた頃には使わない方が良いコードが書けると理解すると思います。今は```DataManager```ぐらいにしておくのがおすすめです。```DataManager```は変えず、下記のような形でどうでしょうか？
+itemCreator.cs
+```C#
+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
+```C#
+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;```を削除するとカウントされるはずです。
 ただ、あまり良い方法とは言えません。

追記

2019/04/06 11:14

投稿

IShix

スコア1729

answer CHANGED Viewed

@@ -9,9 +9,10 @@
 ・ItemCreator itemの生成、itemに対して一括処理したい内容(消した時の獲得スコアなど)を記述するスクリプト
 ・Item 現在のitemスクリプトのスコア関連を無くしたスクリプト
-### [追記]書いてみたので共有します
+### [追記]書いてみました
+書いてみたので共有します。参考になりますと幸いです。
-DataManager データの管理を担当
+**DataManager データの管理を担当**
 DontDestroyOnLoadを付与したシングルトンです。
 ```C#
@@ -80,7 +81,7 @@
 }
 ```
-ItemCreator アイテムの生成とアイテムに一括処理したい内容を記述
+**ItemCreator アイテムの生成とアイテムに一括処理したい内容を記述**
 ```C#
 using UnityEngine;
@@ -135,7 +136,7 @@
 }
 ```
-ScoreTxt スコアの表示を担当
+**ScoreTxt スコアの表示を担当**
 ```C#
 using UnityEngine;
 using UnityEngine.UI;
@@ -167,7 +168,7 @@
 }
 ```
-item アイテム 少々改変しました
+**item 少々改変しました**
 ```C#
 using System;
 using UnityEngine;

追記

2019/04/02 14:10

投稿

IShix

スコア1729

answer CHANGED Viewed

@@ -1,5 +1,4 @@
-回答させていいただきます。
-```int sc_count```を```static int sc_count```に変更し、```Start```の```sc_count =0;```を削除するとカウントされるはずです。
+回答させていいただきます。```int sc_count```を```static int sc_count```に変更し、```Start```の```sc_count =0;```を削除するとカウントされるはずです。
 ただ、あまり良い方法とは言えません。
 スコアを管理するスクリプトを用意して、そちらに通知するのが良いと思います。
@@ -8,4 +7,202 @@
 ・DataManager スコアを管理するスクリプト
 ・ScoreTxt DataManagerのスコアが加算されたら表示するスクリプト
 ・ItemCreator itemの生成、itemに対して一括処理したい内容(消した時の獲得スコアなど)を記述するスクリプト
-・Item 現在のitemスクリプトのスコア関連を無くしたスクリプト
+・Item 現在のitemスクリプトのスコア関連を無くしたスクリプト
+### [追記]書いてみたので共有します
+DataManager データの管理を担当
+DontDestroyOnLoadを付与したシングルトンです。
+```C#
+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 アイテムの生成とアイテムに一括処理したい内容を記述
+```C#
+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 スコアの表示を担当
+```C#
+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 アイテム 少々改変しました
+```C#
+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;
+    }
+}
+```

追記

2019/04/02 14:08

投稿

IShix

スコア1729

answer CHANGED Viewed

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

追記

2019/04/02 12:18

投稿

IShix

スコア1729

answer CHANGED Viewed

@@ -1,2 +1,10 @@
 回答させていいただきます。
-```int sc_count```を```static int sc_count```に変更し、```Start```の```sc_count =0;```を削除するとカウントされるはずです。
+```int sc_count```を```static int sc_count```に変更し、```Start```の```sc_count =0;```を削除するとカウントされるはずです。
+ただ、あまり良い方法とは言えません。
+スコアを管理するスクリプトを用意して、そちらに通知するのが良いと思います。
+僕なら下記のようにします。
+・DataManager スコアを管理するスクリプト
+・ScoreTxt DataManagerのスコアが加算されたら表示するスクリプト
+・Item 現在のitemスクリプトのスコア関連を無くしたスクリプト