Mercurial > repos > IBDev-IBBoard.WarFoundry.API
annotate api/Util/UnitEquipmentUtil.cs @ 201:4d7ff70bb109
Re #208: equipmentslot limit issues
* Fix numeric slot issues by using "amount taken excluding this item" method
* Rename "without this item" to "excluding this item" to clarify purpose
* Restructure percentage limits and break some tests (some of which are only broken by rounding errors)
author | IBBoard <dev@ibboard.co.uk> |
---|---|
date | Thu, 05 Nov 2009 21:09:03 +0000 |
parents | 4bbf5624ced6 |
children | 37ad50823531 |
rev | line source |
---|---|
161
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
1 // This file (UnitEquipmentUtil.cs) is a part of the IBBoard.WarFoundry.API project and is copyright 2009 IBBoard. |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
2 // |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
3 // The file and the library/program it is in are licensed and distributed, without warranty, under the GNU Affero GPL license, either version 3 of the License or (at your option) any later version. Please see COPYING for more information and the full license. |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
4 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
5 using System; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
6 using System.Collections.Generic; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
7 using System.Text; |
183
36adabb1c3ea
Re #198: Add slots with counts to units
IBBoard <dev@ibboard.co.uk>
parents:
182
diff
changeset
|
8 using IBBoard.Limits; |
161
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
9 using IBBoard.WarFoundry.API.Objects; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
10 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
11 namespace IBBoard.WarFoundry.API.Util |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
12 { |
162
624422e91a1c
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
161
diff
changeset
|
13 public class UnitEquipmentUtil |
161
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
14 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
15 /// <summary> |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
16 /// Gets an array of allowed <see cref="UnitEquipmentItem"/>s based on the current selections of the unit, taking in to account Mutex groups and other limits. |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
17 /// </summary> |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
18 /// <param name="unit">The <see cref="Unit"/> to get equipment items for</param> |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
19 /// <returns>The array of allowed <see cref="UnitEquipmentItem"/>s</returns> |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
20 public static UnitEquipmentItem[] GetAllowedEquipmentItems(Unit unit) |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
21 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
22 List<UnitEquipmentItem> list = new List<UnitEquipmentItem>(); |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
23 UnitEquipmentItem[] currItems = unit.GetEquipment(); |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
24 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
25 foreach (UnitEquipmentItem item in GetAllEquipmentItems(unit)) |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
26 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
27 bool allowed = IsAllowedByMutex(item, currItems); |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
28 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
29 if (allowed) |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
30 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
31 list.Add(item); |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
32 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
33 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
34 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
35 return list.ToArray(); |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
36 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
37 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
38 private static bool IsAllowedByMutex(UnitEquipmentItem item, UnitEquipmentItem[] currItems) |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
39 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
40 bool allowed = true; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
41 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
42 foreach (UnitEquipmentItem currItem in currItems) |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
43 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
44 if (ItemsAreMutuallyExclusive(currItem, item)) |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
45 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
46 allowed = false; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
47 break; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
48 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
49 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
50 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
51 return allowed; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
52 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
53 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
54 /// <summary> |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
55 /// Gets a list of all <see cref="UnitEquipmentItem"/>s that would stop the unit taking <code>item</code> because of mutex groups. |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
56 /// </summary> |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
57 /// <param name="unit">The unit that wants to take the equipment item</param> |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
58 /// <param name="item">The item to check blocking items for</param> |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
59 /// <returns>a list of all <see cref="UnitEquipmentItem"/>s that would stop the unit taking <code>item</code></returns> |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
60 public static List<UnitEquipmentItem> GetBlockingEquipmentItems(Unit unit, UnitEquipmentItem item) |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
61 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
62 List<UnitEquipmentItem> items = new List<UnitEquipmentItem>(); |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
63 UnitEquipmentItem[] currItems = unit.GetEquipment(); |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
64 |
174 | 65 foreach (UnitEquipmentItem unitItem in currItems) |
161
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
66 { |
194
1412a42190a1
* Fix mutex group clash checking
IBBoard <dev@ibboard.co.uk>
parents:
190
diff
changeset
|
67 if (ItemsAreMutuallyExclusive(unitItem, item)) |
161
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
68 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
69 items.Add(unitItem); |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
70 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
71 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
72 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
73 return items; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
74 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
75 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
76 public static UnitEquipmentItem[] GetAllEquipmentItems(Unit unit) |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
77 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
78 return unit.UnitType.GetEquipmentItems(); |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
79 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
80 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
81 public static bool ItemsAreMutuallyExclusive(UnitEquipmentItem item1, UnitEquipmentItem item2) |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
82 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
83 bool areMutex = false; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
84 string[] item1mutex = item1.MutexGroups; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
85 string[] item2mutex = item2.MutexGroups; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
86 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
87 foreach (string mutex in item1mutex) |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
88 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
89 foreach (string otherMutex in item2mutex) |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
90 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
91 if (mutex.Equals(otherMutex)) |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
92 { |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
93 areMutex = true; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
94 goto postLoop; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
95 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
96 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
97 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
98 postLoop: |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
99 |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
100 return areMutex; |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
101 } |
182
6fe336109128
Re #198: Add slots with count to unit
IBBoard <dev@ibboard.co.uk>
parents:
174
diff
changeset
|
102 |
6fe336109128
Re #198: Add slots with count to unit
IBBoard <dev@ibboard.co.uk>
parents:
174
diff
changeset
|
103 public static int GetMaxEquipmentCount (Unit unit, UnitEquipmentItem equip) |
187 | 104 { |
105 int unitSize = unit.Size; | |
106 int max = Math.Max(equip.MinLimit.GetLimit(unitSize), equip.MaxLimit.GetLimit (unitSize)); | |
107 return GetEquipmentCountLimit (unit, max, equip); | |
185 | 108 } |
109 | |
187 | 110 private static int GetEquipmentCountLimit (Unit unit, int currLimit, UnitEquipmentItem equip) |
185 | 111 { |
187 | 112 int newLimit = currLimit; |
189 | 113 AbstractLimit limit = GetSlotLimitForItem(unit, equip); |
183
36adabb1c3ea
Re #198: Add slots with counts to units
IBBoard <dev@ibboard.co.uk>
parents:
182
diff
changeset
|
114 |
36adabb1c3ea
Re #198: Add slots with counts to units
IBBoard <dev@ibboard.co.uk>
parents:
182
diff
changeset
|
115 if (!(limit is UnlimitedLimit)) |
36adabb1c3ea
Re #198: Add slots with counts to units
IBBoard <dev@ibboard.co.uk>
parents:
182
diff
changeset
|
116 { |
201
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
117 int slotMax = limit.GetLimit (unit.Size) - unit.GetEquipmentAmountInSlotExcludingItem(equip); |
187 | 118 newLimit = Math.Min (slotMax, newLimit); |
183
36adabb1c3ea
Re #198: Add slots with counts to units
IBBoard <dev@ibboard.co.uk>
parents:
182
diff
changeset
|
119 } |
36adabb1c3ea
Re #198: Add slots with counts to units
IBBoard <dev@ibboard.co.uk>
parents:
182
diff
changeset
|
120 |
187 | 121 return newLimit; |
182
6fe336109128
Re #198: Add slots with count to unit
IBBoard <dev@ibboard.co.uk>
parents:
174
diff
changeset
|
122 } |
189 | 123 |
124 private static AbstractLimit GetSlotLimitForItem(Unit unit, UnitEquipmentItem equip) | |
125 { | |
126 return unit.UnitType.GetEquipmentSlotLimit(equip.SlotName); | |
127 } | |
128 | |
186 | 129 |
185 | 130 public static int GetMinEquipmentCount (Unit unit, UnitEquipmentItem equip) |
187 | 131 { |
132 int unitSize = unit.Size; | |
133 int min = Math.Min (equip.MinLimit.GetLimit (unitSize), equip.MaxLimit.GetLimit (unitSize)); | |
134 return GetEquipmentCountLimit (unit, min, equip); | |
185 | 135 } |
186 | 136 |
137 public static bool IsEquipmentRatioLimited(Unit unit, UnitEquipmentItem equip) | |
138 { | |
189 | 139 AbstractLimit limit = GetSlotLimitForItem(unit, equip); |
186 | 140 return equip.IsRatioLimit && (limit is IPercentageLimit || limit is UnlimitedLimit); |
141 } | |
187 | 142 |
143 public static double GetMaxEquipmentPercentage(Unit unit, UnitEquipmentItem equip) | |
144 { | |
145 double limit = 0; | |
189 | 146 AbstractLimit slotLimit = GetSlotLimitForItem(unit, equip); |
201
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
147 |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
148 if (slotLimit is IPercentageLimit) |
187 | 149 { |
201
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
150 limit = ((IPercentageLimit)slotLimit).Percentage; |
187 | 151 } |
188 | 152 else |
153 { | |
201
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
154 limit = GetPercentageOfUnitSize(slotLimit.GetLimit(unit.Size), unit); |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
155 } |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
156 |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
157 limit = limit - GetPercentageOfUnitSize(unit.GetEquipmentAmountInSlotExcludingItem(equip), unit); |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
158 |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
159 return GetMinOfSlotLimitAndEquipmentLimit(equip.IsRatioLimit, limit, equip.MaxLimit, unit); |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
160 } |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
161 |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
162 private static double GetPercentageOfUnitSize(int number, Unit unit) |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
163 { |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
164 return (number / (double)unit.Size) * 100; |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
165 } |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
166 |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
167 private static double GetMinOfSlotLimitAndEquipmentLimit(bool equipIsRatio, double limit, AbstractLimit equipLimit, Unit unit) |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
168 { |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
169 if (equipIsRatio) |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
170 { |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
171 limit = Math.Min(limit, ((IPercentageLimit)equipLimit).Percentage); |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
172 } |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
173 else |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
174 { |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
175 limit = Math.Min(limit, GetPercentageOfUnitSize(equipLimit.GetLimit(unit.Size), unit)); |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
176 } |
187 | 177 |
178 return limit; | |
179 } | |
190 | 180 |
181 public static double GetMinEquipmentPercentage(Unit unit, UnitEquipmentItem equip) | |
182 { | |
183 double limit = 0; | |
184 AbstractLimit slotLimit = GetSlotLimitForItem(unit, equip); | |
185 | |
201
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
186 if (slotLimit is IPercentageLimit) |
190 | 187 { |
201
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
188 limit = ((IPercentageLimit)slotLimit).Percentage; |
190 | 189 } |
190 else | |
191 { | |
201
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
192 limit = GetPercentageOfUnitSize(slotLimit.GetLimit(unit.Size), unit); |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
193 } |
190 | 194 |
201
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
195 limit = limit - GetPercentageOfUnitSize(unit.GetEquipmentAmountInSlotExcludingItem(equip), unit); |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
196 |
4d7ff70bb109
Re #208: equipmentslot limit issues
IBBoard <dev@ibboard.co.uk>
parents:
196
diff
changeset
|
197 return GetMinOfSlotLimitAndEquipmentLimit(equip.IsRatioLimit, limit, equip.MinLimit, unit); |
190 | 198 } |
161
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
199 } |
81abc04b3dbe
Re #192: Improve "clean coding" practice
IBBoard <dev@ibboard.co.uk>
parents:
diff
changeset
|
200 } |