Changeset - 6919080271d5
[Not reviewed]
Tess Snider (Malkyne) - 3 years ago 2021-07-21 17:47:38
this@malkyne.org
Implemented a system that will allow ingredients to be safely renamed, in the
future, without invalidating profiles. Standardized to PP's "FalconBait."
7 files changed with 129 insertions and 40 deletions:
0 comments (0 inline, 0 general)
Data/ingredients.txt
Show inline comments
...
 
@@ -8,7 +8,7 @@ Carrot          | Carrot      | 224, 112
 
Clay            | Clay        | 128, 96, 32
 
Dead Tongue     | DeadTongue  | 112, 64, 64
 
Toad Skin       | ToadSkin    | 48, 96, 48
 
Falcons Bait    | FalconsBait | 128, 240, 224
 
Falcons Bait    | FalconBait  | 128, 240, 224
 
Red Sand        | RedSand     | 144, 16, 24
 
Lead            | Lead        | 80, 80, 96
 
Silver Powder   | Silver	  | 16, 16, 32
Data/renames.txt
Show inline comments
 
new file 100644
 
# PracticalPaint
 

	
 
EarthLight | FalconBait
 
FalconsBait | FalconBait
 

	
 
# Internal
 

	
 
Earth Light | Falcons Bait
...
 
\ No newline at end of file
Data/template/ingredients.txt
Show inline comments
...
 
@@ -9,7 +9,7 @@ Carrot		| 224, 112, 32   | 8      | Y | 
 
Clay		| 128, 96, 32    | 10     | Y | bulk    | 20
 
DeadTongue	| 112, 64, 64    | 500    | Y | normal  | 4
 
ToadSkin	| 48, 96, 48     | 500    | Y | normal  | 4
 
FalconsBait	| 128, 240, 224  | 10000  | Y | normal  | 4
 
FalconBait	| 128, 240, 224  | 10000  | Y | normal  | 4
 
RedSand		| 144, 16, 24    | 4      | Y | bulk    | 20
 
Lead		| 80, 80, 96     | 50     | Y | normal  | 6
 
Silver		| 16, 16, 32     | 50     | Y | normal  | 6
DesertPaintCodex.csproj
Show inline comments
...
 
@@ -17,6 +17,9 @@
 
    <Folder Include="Models\" />
 
    <AvaloniaResource Include="Assets\**" />
 
    <None Remove="CodexTheme.xaml" />
 
    <None Update="Data\renames.txt">
 
      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
 
    </None>
 
  </ItemGroup>
 
  <ItemGroup>
 
    <PackageReference Include="Avalonia" Version="0.10.999-cibuild0014036-beta" />
Models/Reagent.cs
Show inline comments
...
 
@@ -13,6 +13,8 @@ namespace DesertPaintCodex.Models
 
        public string PracticalPaintName { get; }
 
        public bool Enabled { get; set; }
 

	
 
        public int Index { get; private set; }
 

	
 
        public uint Cost
 
        {
 
            get => _cost;
...
 
@@ -32,7 +34,7 @@ namespace DesertPaintCodex.Models
 
        }
 

	
 
        // catalyst
 
        public Reagent(string name, string ppName)
 
        public Reagent(string name, string ppName, int index)
 
        {
 
            Name = name;
 
            PracticalPaintName = ppName;
...
 
@@ -40,9 +42,10 @@ namespace DesertPaintCodex.Models
 
            Enabled = true;
 
            RecipeMax = 1;
 
            IsCatalyst = true;
 
            Index = index;
 
        }
 

	
 
        public Reagent(string name, string ppName, byte red, byte green, byte blue)
 
        public Reagent(string name, string ppName, byte red, byte green, byte blue, int index)
 
        {
 
            Color = new PaintColor(red, green, blue);
 
            Name = name;
...
 
@@ -51,6 +54,7 @@ namespace DesertPaintCodex.Models
 
            RecipeMax = 10;
 
            Enabled = true;
 
            IsCatalyst = false;
 
            Index = index;
 
        }
 

	
 
        public override string ToString()
Services/ReagentService.cs
Show inline comments
...
 
@@ -10,23 +10,24 @@ namespace DesertPaintCodex.Services
 
{
 
    internal static class ReagentService
 
    {
 
        // PP format
 
        private static readonly Regex _reagentRegex          = new(@"(?<name>\w+)\s*\|\s*(?<red>\d+),\s*(?<green>\d+),\s*(?<blue>\d+)\s*\|\s*(?<cost>\d+)\s*\|\s*(?<enabled>[YN])\s*\|\s*(?<bulk>(bulk|normal))\s*\|\s*(?<max>\d+).*");
 
        private static readonly Regex _catalystRegex         = new(@"(?<name>\w+)\s*\|\s*catalyst\s*\|\s*(?<cost>\d+)\s*\|\s*(?<enabled>[YN])\s*\|\s*(?<bulk>(bulk|normal)).*");
 
        private static readonly Regex _ppReagentRegex        = new(@"(?<name>\w+)\s*\|\s*(?<red>\d+),\s*(?<green>\d+),\s*(?<blue>\d+)\s*\|\s*(?<cost>\d+)\s*\|\s*(?<enabled>[YN])\s*\|\s*(?<bulk>(bulk|normal))\s*\|\s*(?<max>\d+).*");
 
        private static readonly Regex _ppCatalystRegex       = new(@"(?<name>(\w*\s)*\w+\w+)\s*\|\s*catalyst\s*\|\s*(?<cost>\d+)\s*\|\s*(?<enabled>[YN])\s*\|\s*(?<bulk>(bulk|normal)).*");
 
        private static readonly Regex _internalReagentRegex  = new(@"(?<name>(\w*\s)*\w+)\s*\|\s*(?<ppname>\w+)\s*\|\s*(?<red>\d+),\s*(?<green>\d+),\s*(?<blue>\d+).*");
 
        private static readonly Regex _internalCatalystRegex = new(@"(?<name>(\w+\s)*\w+)\s*\|\s*(?<ppname>\w+)\s*\|\s*catalyst.*");
 
        private static readonly Regex _renameHeader          = new(@"#\s*(?<header>\w*)");
 
        private static readonly Regex _renameRegex           = new(@"(?<oldName>(\w+\s)*\w+)\s*\|\s*(?<newName>(\w+\s)*\w+)");
 

	
 
        private static readonly Dictionary<string, Reagent> _reagents   = new();
 
        private static readonly Dictionary<string, string>  _nameLookup = new(); // pp name to our name
 

	
 
        private static string _lastReagentsFile = string.Empty;
 
        private static readonly Dictionary<string, string>  _ppToInternalName = new(); // pp name to our name
 
        private static readonly Dictionary<string, string> _ppRenames = new();
 
        private static readonly Dictionary<string, string> _internalRenames = new();
 

	
 
        private static bool _initialized = false;
 

	
 
        public static List<string> Names { get; } = new();
 

	
 

	
 
        public static void Initialize()
 
        private static void Initialize()
 
        {
 
            if (_initialized) return;
 

	
...
 
@@ -34,17 +35,25 @@ namespace DesertPaintCodex.Services
 
            Debug.Assert(reagentsPath != null);
 
    
 
            Load(reagentsPath);
 

	
 
            string? renamesPath = FileUtils.FindApplicationResourceFile("renames.txt");
 

	
 
            if (renamesPath != null)
 
            {
 
                LoadAliases(renamesPath);
 
            }
 

	
 
            _initialized = true;
 
        }
 
        
 
        // Loads reagent name/colors
 
        public static void Load(string file)
 
        private static void Load(string file)
 
        {
 
            if (_lastReagentsFile == file) return;
 
            _lastReagentsFile = file;
 
            _reagents.Clear();
 
            _ppToInternalName.Clear();
 
            Names.Clear();
 
            
 
            _reagents.Clear();
 
            _nameLookup.Clear();
 
            Names.Clear();
 
            int index = 0;
 

	
 
            using StreamReader reader = new(file);
 
            string? line;
...
 
@@ -55,14 +64,16 @@ namespace DesertPaintCodex.Services
 
                {
 
                    string name   = match.Groups["name"].Value;
 
                    string ppname = match.Groups["ppname"].Value;
 
                    Debug.WriteLine("Adding reagent[" + name + "] from line: [" + line + "]");
 
                    _reagents.Add(name,
 
                        new Reagent(name, ppname,
 
                            byte.Parse(match.Groups["red"].Value),
 
                            byte.Parse(match.Groups["green"].Value),
 
                            byte.Parse(match.Groups["blue"].Value)));
 
                            byte.Parse(match.Groups["blue"].Value),
 
                            index++));
 
                    // nameStore.AppendValues(name);
 
                    Names.Add(name);
 
                    _nameLookup.Add(ppname, name);
 
                    _ppToInternalName.Add(ppname, name);
 
                }
 
                else
 
                {
...
 
@@ -72,11 +83,46 @@ namespace DesertPaintCodex.Services
 
                    
 
                    string name   = match.Groups["name"].Value;
 
                    string ppname = match.Groups["ppname"].Value;
 
                    _reagents.Add(name, new Reagent(ppname, ppname));
 
                    _reagents.Add(name, new Reagent(ppname, ppname, index++));
 
                    // nameStore.AppendValues(name);
 
                    Names.Add(name);
 
                    _nameLookup.Add(ppname, name);
 
                    _ppToInternalName.Add(ppname, name);
 
                }
 
            }
 
                }
 

	
 
        private static void LoadAliases(string file)
 
        {
 
            _ppRenames.Clear();
 
            _internalRenames.Clear();
 

	
 
            Dictionary<string, string>? renameDict = null;
 
            
 
            using StreamReader reader = new(file);
 
            string? line;
 
            while ((line = reader.ReadLine()) != null)
 
            {
 
                Match match = _renameHeader.Match(line);
 
                if (match.Success)
 
                {
 
                    string header = match.Groups["header"].Value;
 
                    renameDict = header switch
 
                    {
 
                        "PracticalPaint" => _ppRenames,
 
                        "Internal"       => _internalRenames,
 
                        _                => renameDict
 
                    };
 
                }
 

	
 
                if (renameDict == null) continue;
 
                
 
                match = _renameRegex.Match(line);
 
                
 
                if (!match.Success) continue;
 
                
 
                string oldName = match.Groups["oldName"].Value;
 
                string newName = match.Groups["newName"].Value;
 
                renameDict.Add(oldName, newName);
 
            }
 
        }
 

	
...
 
@@ -88,11 +134,16 @@ namespace DesertPaintCodex.Services
 
            string? line;
 
            while ((line = reader.ReadLine()) != null)
 
            {
 
                Match match = _reagentRegex.Match(line);
 
                Match match = _ppReagentRegex.Match(line);
 
                if (match.Success)
 
                {
 
                    string ppname = match.Groups["name"].Value;
 
                    if (_nameLookup.TryGetValue(ppname, out string? name))
 
                    if (_ppRenames.TryGetValue(ppname, out string? rename))
 
                    {
 
                        ppname = rename; // Apply rename.
 
                    }
 
                    
 
                    if (_ppToInternalName.TryGetValue(ppname, out string? name))
 
                    {
 
                        Reagent reagent = GetReagent(name);
 
                        if (reagent.IsCatalyst) continue;
...
 
@@ -100,19 +151,20 @@ namespace DesertPaintCodex.Services
 
                        reagent.Cost      = uint.Parse(match.Groups["cost"].Value);
 
                        reagent.RecipeMax = uint.Parse(match.Groups["max"].Value);
 
                    }
 
                    else
 
                    {
 
                        // bad name?
 
                    }
 
                }
 
                else
 
                {
 
                    match = _catalystRegex.Match(line);
 
                    match = _ppCatalystRegex.Match(line);
 
                    
 
                    if (!match.Success) continue;
 
                    
 
                    string ppname = match.Groups["name"].Value;
 
                    if (_nameLookup.TryGetValue(ppname, out string? name))
 
                    if (_ppRenames.TryGetValue(ppname, out string? rename))
 
                    {
 
                        ppname = rename; // Apply rename.
 
                    }
 
                    
 
                    if (_ppToInternalName.TryGetValue(ppname, out string? name))
 
                    {
 
                        Reagent reagent = GetReagent(name);
 
                        
...
 
@@ -121,10 +173,6 @@ namespace DesertPaintCodex.Services
 
                        reagent.Enabled = match.Groups["enabled"].Value.Equals("Y");
 
                        reagent.Cost    = uint.Parse(match.Groups["cost"].Value);
 
                    }
 
                    else
 
                    {
 
                        // bad name?
 
                    }
 
                }
 
            }
 
        }
...
 
@@ -189,16 +237,42 @@ namespace DesertPaintCodex.Services
 
        {
 
            Initialize();
 
            
 
            if (_reagents.TryGetValue(reagentName, out Reagent? returnVal)) return returnVal;
 
            // convert pp name to our internal name
 
            if (_nameLookup.TryGetValue(reagentName, out string? otherName))
 
            Reagent? returnVal = GetReagentByInternalName(reagentName);
 

	
 
            if (returnVal != null) return returnVal;
 

	
 
            returnVal = GetReagentByPPName(reagentName);
 

	
 
            if (returnVal != null) return returnVal;
 
            
 
            string? rename = null;
 
            if (_internalRenames.TryGetValue(reagentName, out rename))
 
            {
 
                _reagents.TryGetValue(otherName, out returnVal);
 
                returnVal = GetReagentByInternalName(rename);
 
                if (returnVal != null) return returnVal;
 
            }
 

	
 
            if (_ppRenames.TryGetValue(reagentName, out rename))
 
            {
 
                returnVal = GetReagentByPPName(rename);
 
                if (returnVal != null) return returnVal;
 
            }
 
            
 
            Debug.Assert(returnVal != null);
 

	
 
            return returnVal;
 
        }
 

	
 
        private static Reagent? GetReagentByInternalName(string internalName)
 
        {
 
            return _reagents.TryGetValue(internalName, out Reagent? returnVal) ? returnVal : null;
 
        }
 

	
 
        private static Reagent? GetReagentByPPName(string ppName)
 
        {
 
            if (!_ppToInternalName.TryGetValue(ppName, out string? otherName)) return null;
 
            
 
            return _reagents.TryGetValue(otherName, out Reagent? returnVal) ? returnVal : null;
 
    }
 
}
 
}
Util/Constants.cs
Show inline comments
...
 
@@ -18,9 +18,9 @@ namespace DesertPaintCodex.Util
 
            {
 
                if (_stubReactionTest == null)
 
                {
 
                    _stubReactionTest = new(
 
                        new Reagent("Toad Skin", "ToadSkin", 48, 96, 48), 
 
                        new Reagent("Falcons Bait", "FalconsBait", 128, 240, 224), 
 
                    _stubReactionTest = new ReactionTest(
 
                        new Reagent("Toad Skin", "ToadSkin", 48, 96, 48, 0), 
 
                        new Reagent("Falcons Bait", "FalconBait", 128, 240, 224, 1), 
 
                        null, ClipType.None, true);
 
                }
 

	
0 comments (0 inline, 0 general)