use full classes for stringtable

Author: LeftofZen

OpenLocoTool/DatFileParsing/SawyerStreamReader.cs

@@ -123,13 +123,10 @@ public static ILocoObject LoadFull(string filename, ILogger? logger = null, bool
 					var lang = (LanguageId)data[ptr++];
 					var ini = ptr;
 
-					while (data[ptr++] != '\0')
-					{
-						;
-					}
+					while (data[ptr++] != '\0') ;
 
 					var str = Encoding.ASCII.GetString(data[ini..(ptr - 1)]); // do -1 to exclude the \0
-					if (!languageDict.TryAdd(lang, str))
+					if (!languageDict.TryAdd(lang, new StringTableEntry { String = str }))
 					{
 						//Logger.Error($"Key {(i, lang)} already exists (this shouldn't happen)");
 						break;

OpenLocoTool/DatFileParsing/Typedefs.cs

@@ -9,7 +9,6 @@
 global using Speed32 = System.Int32;
 global using MicroZ = System.Byte;
 global using SoundObjectId = System.Byte;
-global using StringTable = System.Collections.Generic.Dictionary<string, System.Collections.Generic.Dictionary<OpenLocoTool.LanguageId, string>>;
 using System.ComponentModel;
 
 namespace OpenLocoTool.DatFileParsing
@@ -23,4 +22,30 @@ public record Pos2(
 	{
 		public static int StructSize => 0x04;
 	}
-}
+
+	// this is only required because WinForms' DataGridView cannot handle plain old dictionary<foo, string>, because string type doesn't have the getter+setter it needs
+	//public class StringTable
+	//{
+	//	Dictionary<string, StringTableInner> table = new();
+	//}
+
+	public class StringTable
+	{
+		public Dictionary<string, Dictionary<LanguageId, StringTableEntry>> table { get; set; } = new();
+
+		public void Add(string key, Dictionary<LanguageId, StringTableEntry> value) => table.Add(key, value);
+
+		public Dictionary<LanguageId, StringTableEntry> this[string key]
+		{
+			get => table[key];
+			set => table[key] = value;
+		}
+
+		public Dictionary<string, Dictionary<LanguageId, StringTableEntry>>.KeyCollection Keys => table.Keys;
+	}
+
+	public class StringTableEntry
+	{
+		public string String { get; set; }
+	}
+}

OpenLocoToolGui/StringTableUserControl.cs

@@ -1,4 +1,5 @@
 using OpenLocoTool;
+using OpenLocoTool.DatFileParsing;
 using System.ComponentModel;
 
 namespace OpenLocoToolGui
@@ -9,7 +10,7 @@ public partial class StringTableUserControl : UserControl
 
 		public StringTableUserControl() => InitializeComponent();
 
-		public void SetDataBinding(Dictionary<string, Dictionary<LanguageId, string>> data)
+		public void SetDataBinding(StringTable data)
 		{
 			blKeys.Clear();
 			foreach (var key in data.Keys)