From 2f81b826524599859ce4c8bf6164bab83debf662 Mon Sep 17 00:00:00 2001
From: Ben Culkin <scorpress@gmail.com>
Date: Thu, 2 Jul 2020 16:01:39 -0400
Subject: Implement affix grouping by contents

Implements a basic way to 'group' affixes together.

An affix group is a collection of affixes which generally provide the
same benefits, but at varying levels. For example, '+2 strength' and '+4
strength' would be considered grouped affixes, assuming that they were
spawning on the same items and such
---
 src/main/java/tlIItools/Affix.java | 57 +++++++++++++++++++++++++++++++++++---
 1 file changed, 53 insertions(+), 4 deletions(-)

(limited to 'src/main/java/tlIItools/Affix.java')

diff --git a/src/main/java/tlIItools/Affix.java b/src/main/java/tlIItools/Affix.java
index 7afb32b6..ffec66d9 100644
--- a/src/main/java/tlIItools/Affix.java
+++ b/src/main/java/tlIItools/Affix.java
@@ -1,6 +1,7 @@
 package tlIItools;
 
 import java.util.ArrayList;
+import java.util.Comparator;
 import java.util.List;
 import java.util.Scanner;
 
@@ -112,7 +113,7 @@ public class Affix {
 	 * 
 	 * For instance, an affix that granted +2 strength, and one that granted +4
 	 * strength would be considered to be in the same affix group (assuming that
-	 * both of those strength bonuses had the same effect name.
+	 * both of those strength bonuses had the same effect name).
 	 * 
 	 * @param afx
 	 *            The affix to check if we are in an affix group with.
@@ -137,8 +138,6 @@ public class Affix {
 				return false;
 		} else if (!equipTypes.equals(afx.equipTypes))
 			return false;
-		if (inNonEquip != afx.inNonEquip)
-			return false;
 		if (isEnchantment != afx.isEnchantment)
 			return false;
 		if (isPerson != afx.isPerson)
@@ -158,8 +157,51 @@ public class Affix {
 		return true;
 	}
 
+	/**
+	 * Gets the name of the 'affix group' that this affix is in.
+	 * 
+	 * By 'affix group', what we mean is that it is essentially the same affix, with
+	 * generally just differing levels/values.
+	 * 
+	 * For instance, an affix that granted +2 strength, and one that granted +4
+	 * strength would be considered to be in the same affix group (assuming that
+	 * both of those strength bonuses had the same effect name, and applied to the
+	 * same sorts of items).
+	 * 
+	 * @return The name of the affix group
+	 */
+	public String getAffixGroupName() {
+		StringBuilder sb = new StringBuilder();
+
+		for (Effect eft : effects) {
+			sb.append(eft.getEffectGroup());
+		}
+		
+		for (String enchantSource : enchantSources) {
+			sb.append(enchantSource);
+		}
+		
+		for (String equipType : equipTypes) {
+			sb.append(equipType);
+		}
+		
+		for (String nonEquipType : nonequipTypes) {
+			sb.append(nonEquipType);
+		}
+		
+		for (String socketableType : socketableTypes) {
+			sb.append(socketableType);
+		}
+
+		return sb.toString();
+	}
+
 	/*
 	 * Are invalid equip types being added?
+	 * 
+	 * NOTE: This is kinda bad practice. It should really be handled via two
+	 * separate affix methods, one to add a valid affix, one to add an invalid, w/
+	 * the caller keeping tracking/calling the right one.
 	 */
 	private boolean inNonEquip;
 
@@ -319,7 +361,11 @@ public class Affix {
 			}
 			sb.append("\n");
 		}
-
+		
+		sb.append("Affix Group: ");
+		sb.append(getAffixGroupName());
+		sb.append("\n");
+		
 		return sb.toString();
 	}
 
@@ -409,6 +455,9 @@ public class Affix {
 				afx.effects.add(eft);
 			}
 		}
+		// Sort effects, so that they are in a stable order, even if specified out of
+		// order
+		afx.effects.sort(Comparator.comparingInt((val) -> val.hashCode()));
 
 		long endTime = System.nanoTime();
 		if (doTiming) {
-- 
cgit v1.2.3