Q&A
コーの表示が崩れています。直してください。
Unity のタグをつけてください。
> 値渡しのコピーなどをすれば良いと思うのですが
意味不明です。具体的にコードのどこをどうしたいのか書いてください。
C#
1 2using System.Collections; 3using System.Collections.Generic; 4using UnityEngine; 5using System.Linq; 6 7public class Test : MonoBehaviour 8{ 9 [System.Serializable] 10 public class Party 11 { 12 public List<Char> Chars; 13 14 public object Clone() 15 { 16 Party tmpParty; 17 tmpParty = (Party)this.MemberwiseClone(); 18 19 tmpParty.Chars = tmpParty.Chars.ToList(); 20 21 return tmpParty; 22 } 23 } 24 25 26 [System.Serializable] 27 public class Char 28 { 29 public int Char_ID; 30 public string Char_Name; 31 public List<Item> Char_Items = new List<Item>(); 32 } 33 34 [System.Serializable] 35 public class Item 36 { 37 public string Name; 38 public int Count; 39 } 40 41 public List<Party> SaveData = new List<Party>(); 42 43 [SerializeField] 44 public Party SaveData_Playing; 45 46 void Start() 47 { 48 Item Item1 = new Item(); 49 Item1.Name = "回復剤"; 50 Item1.Count = 1; 51 52 Item Item2 = new Item(); 53 Item2.Name = "武器"; 54 Item2.Count = 2; 55 56 Char Man=new Char(); 57 Man.Char_ID = 1; 58 Man.Char_Name = "オトコ"; 59 Man.Char_Items.Add(Item1); 60 Man.Char_Items.Add(Item2); 61 62 Item Item3 = new Item(); 63 Item1.Name = "魔法剤"; 64 Item1.Count = 3; 65 66 Item Item4 = new Item(); 67 Item2.Name = "なんか"; 68 Item2.Count = 4; 69 Char WoMan = new Char(); 70 WoMan .Char_ID = 2; 71 WoMan .Char_Name = "オンナ"; 72 WoMan .Char_Items.Add(Item3); 73 WoMan .Char_Items.Add(Item4); 74 75 SaveData_Playing.Chars.Add(Man); 76 SaveData_Playing.Chars.Add(WoMan); 77 78 SaveData.Add(new Party()); 79 SaveData[0]=(Party)SaveData_Playing.Clone(); 80 81 } 82}
上記のようなスクリプトを作りました。
ざっくりですが、ゲームを遊んでいてプレイ状況をセーブしたときの挙動想定しています。
79行目の
SaveData[0]=(Party)SaveData_Playing.Clone();
でセーブスロットにセーブデータを保存したつもりです。
ただ、実際に動かしてみると
SaveData_PlayingとSaveDataが一部同期してしまっており困っております。
例えば、Manの回復剤を2にすると、セーブをしたデータの
SaveData[0]
こちらまで一緒に２に切り替わってしまいます。
ネットで調べた限りだと値渡しのコピー(?)や、DeepCopyをする、と書いてあるのですが、どこをどうしたら良いかわからず、お知恵を貸していただけますと幸いです
このままではコードが読みづらいので、質問を編集し、</>(コードの挿入)ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください
> 出てくる’’’の枠の中に・・・
毎回間違えてますが ’’’ ではなくて ``` です。（バッククオート ３ つ）
おっしゃりたい意味はわかるので些事なのですが
C#では「参照渡し」は全く別の事象を指しています。
WoManとManを間違えているところがありますが、元ソースでもそうなっているのですか？
失礼しました。
コード挿入とUnityタグについて追加させていただきました。
また、質問文を補足させていただきました。
また、「WoManとManを間違えているところ」コチラも修正させていただきました。
Microsoft のドキュメントによると、
"MemberwiseClone メソッドは、新しいオブジェクトを作成し、現在のオブジェクトの非静的フィールドをその新しいオブジェクトにコピーすることによって、簡易コピーを作成します。フィールドが値型の場合、そのフィールドはビット単位でコピーされます。フィールドが参照型の場合、参照はコピーされますが、参照先オブジェクトはコピーされないため、元のオブジェクトとその複製は同じオブジェクトを参照します。"
・・・ということです。 上の一番最後のところが原因では？
回答1件
0
ベストアンサー
tmpParty = (Party)this.MemberwiseClone();
Microsoft のドキュメントによると、
"MemberwiseClone メソッドは、新しいオブジェクトを作成し、現在のオブジェクトの非静的フィールドをその新しいオブジェクトにコピーすることによって、簡易コピーを作成します。フィールドが値型の場合、そのフィールドはビット単位でコピーされます。フィールドが参照型の場合、参照はコピーされますが、参照先オブジェクトはコピーされないため、元のオブジェクトとその複製は同じオブジェクトを参照します。"
・・・ということです。 上の文の一番最後のところ「参照先オブジェクトはコピーされないため、元のオブジェクトとその複製は同じオブジェクトを参照します」が原因では？
Party クラスの Clone メソッドを手直しして、参照先のクラス定義から新たに別インスタンスを作って上で言う「参照先オブジェクト」の中身をコピーするようにしてはいかが？
以下のような感じ。
C#
1using System; 2using System.Collections.Generic; 3using System.Globalization; 4 5namespace ConsoleApp5 6{ 7 internal class Program 8 { 9 static void Main(string[] args) 10 { 11 Party SaveData_Playing = new Party(); 12 SaveData_Playing.Chars = new List<Char>(); 13 14 Item Item1 = new Item(); 15 Item1.Name = "回復剤"; 16 Item1.Count = 1; 17 18 Item Item2 = new Item(); 19 Item2.Name = "武器"; 20 Item2.Count = 2; 21 22 Char Man = new Char(); 23 Man.Char_ID = 1; 24 Man.Char_Name = "オトコ"; 25 Man.Char_Items.Add(Item1); 26 Man.Char_Items.Add(Item2); 27 28 Item Item3 = new Item(); 29 Item3.Name = "魔法剤"; 30 Item3.Count = 3; 31 32 Item Item4 = new Item(); 33 Item4.Name = "なんか"; 34 Item4.Count = 4; 35 36 Char WoMan = new Char(); 37 WoMan.Char_ID = 2; 38 WoMan.Char_Name = "オンナ"; 39 WoMan.Char_Items.Add(Item3); 40 WoMan.Char_Items.Add(Item4); 41 42 SaveData_Playing.Chars.Add(Man); 43 SaveData_Playing.Chars.Add(WoMan); 44 45 Party saveData = SaveData_Playing.Clone(); 46 47 Console.WriteLine($"Name; {SaveData_Playing.Chars[0].Char_Items[0].Name}, " + 48 $"Count: {SaveData_Playing.Chars[0].Char_Items[0].Count}"); 49 50 SaveData_Playing.Chars[0].Char_Items[0].Count = 2; 51 Console.WriteLine($"Name; {SaveData_Playing.Chars[0].Char_Items[0].Name}, " + 52 $"Count: {SaveData_Playing.Chars[0].Char_Items[0].Count}"); 53 Console.WriteLine($"Name: {saveData.Chars[0].Char_Items[0].Name}, " + 54 $"Count: {saveData.Chars[0].Char_Items[0].Count}"); 55 } 56 } 57 58 public class Party 59 { 60 public List<Char> Chars; 61 62 public Party Clone() 63 { 64 Party tempParty = new Party(); 65 tempParty.Chars = new List<Char>(); 66 foreach (Char c in this.Chars) 67 { 68 Char tempChar = new Char 69 { 70 Char_ID = c.Char_ID, 71 Char_Name = c.Char_Name, 72 Char_Items = new List<Item>() 73 }; 74 foreach (Item i in c.Char_Items) 75 { 76 Item tempItem = new Item 77 { 78 Name = i.Name, 79 Count = i.Count 80 }; 81 tempChar.Char_Items.Add(tempItem); 82 } 83 tempParty.Chars.Add(tempChar); 84 85 } 86 87 return tempParty; 88 } 89 } 90 91 public class Char 92 { 93 public int Char_ID; 94 public string Char_Name; 95 public List<Item> Char_Items = new List<Item>(); 96 } 97 98 public class Item 99 { 100 public string Name; 101 public int Count; 102 } 103}
結果は:
Name; 回復剤, Count: 1
Name; 回復剤, Count: 2
Name: 回復剤, Count: 1
投稿2023/01/23 14:16
退会済みユーザー
総合スコア0
ありがとうございます！
頂いた改修内容で試してみたいと思います。
たすかります!!
途中経過のセーブであれば、JSON 文字列にシリアル化してファイルに保存するようにしてはいかがですか？
あなたの回答
tips
プレビュー
