ガチャで当たった物が増えない

unityでゲームを作っていますが、このコードでGManager.instance～～が増えません。
どうしたら増えるのか教えて頂きたいです

c#
1using System;
2using System.Collections;
3using System.Collections.Generic;
4using UnityEngine;
5using UnityEngine.UI;
6using System.Linq;
7
8class Data
9{
10    public string name;
11    public int rarity;
12    public int id;
13    public int numbers;
14}
15
16public class gacha : MonoBehaviour
17{
18    public void itiren()
19    {
20        List<Data> myData = new List<Data>
21        {
22            new Data
23            {
24                name = "kaze",
25                rarity = 1,
26                id = 1001,
27                numbers = GManager.instance.kazeint
28            },
29
30            new Data
31            {
32                name = "bomu",
33                rarity = 1,
34                id = 1002,
35                numbers = GManager.instance.bomuint
36            },
37
38            new Data
39            {
40                name = "speeddown",
41                rarity = 1,
42                id = 1003,
43                numbers = GManager.instance.speeddownint
44            },
45
46            new Data
47            {
48                name = "mpkotei",
49                rarity = 1,
50                id = 1004,
51                numbers = GManager.instance.mpkoteiint
52            },
53
54            new Data
55            {
56                name = "mpzouka",
57                rarity = 1,
58                id = 1005,
59                numbers = GManager.instance.mpzoukaint
60            },
61
62            new Data
63            {
64                name = "mpkyuushuu",
65                rarity = 1,
66                id = 1006,
67                numbers = GManager.instance.mpkyuushuuint
68            },
69
70            new Data
71            {
72                name = "hpkaifuku",
73                rarity = 1,
74                id = 1007,
75                numbers = GManager.instance.hpkaifukuint
76            },
77
78            new Data
79            {
80                name = "hpzouka",
81                rarity = 1,
82                id = 1008,
83                numbers = GManager.instance.hpzoukaint
84            }
85        };
86
87        int gachane = 50;
88        if (GManager.instance.coin >= gachane)
89        {
90            GManager.instance.coin = GManager.instance.coin - gachane;
91
92
93            float gint = UnityEngine.Random.Range(0f, 100f);
94            if (gint < 50f)
95            {
96                var iti =
97                    from x in myData
98                    where x.rarity == 1
99                    orderby x.id ascending
100                    select x;
101
102                System.Random r1 = new System.Random();
103                int r2 = r1.Next(0, 8);
104
105                Debug.Log(myData[r2].numbers);
106
107                foreach (Data a in iti)
108                {
109                    ++ myData[r2].numbers;
110                    Debug.Log(r2 + "+1");
111                }
112            }
113
114            else if (gint < 90f)
115            {
116                Debug.Log("2");
117            }
118
119            else if (gint < 97f)
120            {
121                Debug.Log("3");
122            }
123
124            else
125            {
126                Debug.Log("4");
127            }
128        }
129    }
130}
131
132using System.Collections;
133using System.Collections.Generic;
134using UnityEngine;
135
136public class GManager : MonoBehaviour
137{
138    public static GManager instance = null;
139
140    //hosiiti
141    public int kazeint;
142    public int bomuint;
143    public int speeddownint;
144    public int mpkoteiint;
145    public int mpzoukaint;
146    public int mpkyuushuuint;
147    public int hpkaifukuint;
148    public int hpzoukaint;
149    //hosiitiowari
150
151
152    private void Awake()
153    {
154        if (instance == null)
155        {
156            instance = this;
157            DontDestroyOnLoad(this.gameObject);
158        }
159        else
160        {
161            Destroy(this.gameObject);
162        }
163    }
164}

行動規範の内容に同意します

回答2件

ベストアンサー

質問の内容としては++ myData[r2].numbersの部分でGManagerクラスの

    public int kazeint;
～
    public int hpzoukaint;

上記の値が変化しないという話かと思いますがList<Data> myDataを宣言時のnumbers = GManager.instance.kazeint以下は値渡しなのでmyData内のnumbersを増やしても親元の値は変化はしません。

・GManager.instance.kazeint++の様に直接加算させる
・参照渡しになる作りに変える

どちらかおこなって下さい。

ちなみに現状は8回ループ加算されるのでその部分を修正しても一回で8増えます

・別質問で理解出来ない方がいたようなので追記

GManager.instance.kazeint++の様に直接加算させる

意味的には https://teratail.com/questions/317983 で退会済みユーザーさんが指摘していた「GManager.Instance.変数名 = 数値;」と同じです。

参照渡しになる作りに変える

退会済みユーザーさんが指摘していた様にrefを使ったint型での参照渡しはここから作り変えるのは難しいかと思います。

参照渡しでは無く参照型を使って作り変えて下さい。
int[]にするだけで良いです。

スパゲティなのでintのインクリメント部分だけ抜き出して書くと下記になります。

C#
1using System.Collections.Generic;
2using UnityEngine;
3
4class Data
5{
6    public int numbers;
7}
8
9public class gacha : MonoBehaviour
10{
11    public GManager g;
12
13    public void itiren()
14    {
15        List<Data> myData = new List<Data>
16        {
17            new Data
18            {
19                numbers = GManager.instance.kazeint
20            },
21        };
22
23        myData[ 0 ].numbers++;
24
25        Debug.Log( "gachaクラスのmyData[ 0 ].numbers: " + myData[ 0 ].numbers );
26        Debug.Log( "GManagerクラスのg.kazeint: " + g.kazeint );
27    }
28}
29
30public class GManager : MonoBehaviour
31{
32    public static GManager instance = null;
33
34    public int kazeint;
35
36    private void Awake()
37    {
38        kazeint = 0;
39
40        if( instance == null )
41        {
42            instance = this;
43            DontDestroyOnLoad( this.gameObject );
44        }
45        else
46        {
47            Destroy( this.gameObject );
48        }
49    }
50}

コレが質問で問題となっている値渡しの弊害でmyData[ 0 ].numbersをインクリメントしてもログは下記のようになりGManagerクラスのg.kazeintは0のままです。

gachaクラスのmyData[ 0 ].numbers: 1
GManagerクラスのg.kazeint: 0

次に両方のクラスのintをint[]配列に変えてやると

C#
1using System.Collections.Generic;
2using UnityEngine;
3
4class Data
5{
6    public int[] numbers;
7}
8
9public class gacha : MonoBehaviour
10{
11    public GManager g;
12
13    public void itiren()
14    {
15        List<Data> myData = new List<Data>
16        {
17            new Data
18            {
19                numbers = GManager.instance.kazeint
20            },
21        };
22
23        myData[ 0 ].numbers[ 0 ]++;
24
25        Debug.Log( "gachaクラスのmyData[ 0 ].numbers[ 0 ]: " + myData[ 0 ].numbers[ 0 ] );
26        Debug.Log( "GManagerクラスのg.kazeint[ 0 ]: " + g.kazeint[ 0 ] );
27    }
28}
29
30public class GManager : MonoBehaviour
31{
32    public static GManager instance = null;
33
34    public int[] kazeint;
35
36    private void Awake()
37    {
38        kazeint = new int[] { 0 };
39
40        if( instance == null )
41        {
42            instance = this;
43            DontDestroyOnLoad( this.gameObject );
44        }
45        else
46        {
47            Destroy( this.gameObject );
48        }
49    }
50}

ログは下記のようになり渡し元のGManagerクラスの数値も増加させる事が可能になります。

gachaクラスのmyData[ 0 ].numbers[ 0 ]: 1
GManagerクラスのg.kazeint[ 0 ]: 1

別質問ではDictinaryに変更して解決されたようで良かったです。

投稿2021/01/18 17:30

『🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中！

＼teratail特別グッズやAmazonギフトカード最大2,000円分が当たる！／

関連した質問