PKHeX/PKHeX.Core/Legality/Areas/EncounterArea7.cs

using System.Collections.Generic;

namespace PKHeX.Core
{
    /// <inheritdoc />
    /// <summary>
    /// <see cref="GameVersion.Gen7"/> encounter area
    /// </summary>
    public sealed class EncounterArea7 : EncounterArea32
    {
        public override IEnumerable<EncounterSlot> GetMatchingSlots(PKM pkm, IReadOnlyList<EvoCriteria> chain)
        {
            foreach (var slot in Slots)
            {
                foreach (var evo in chain)
                {
                    if (slot.Species != evo.Species)
                        continue;

                    if (!slot.IsLevelWithinRange(pkm.Met_Level))
                        break;

                    if (slot.Form != evo.Form && !Legal.WildChangeFormAfter.Contains(slot.Species))
                    {
                        if (slot.Species != (int)Species.Minior) // Random Color, edge case
                            break;
                    }

                    yield return slot;
                    break;
                }
            }
        }
    }
}
-												Split encounter areas, relocate slot finding to obj

Now logic is reasonably split, and each format of area has its own way
of yielding slots

Too much junk with checking flute boosts or catch combo applicability;
just let the area dictate how slots match.

											
										
										
											2019-09-12 23:20:52 -07:00
+								using System.Collections.Generic;
 								namespace PKHeX.Core
 								{
-												Minor reorganization

Move the ea32->array creation to the actual class

											
										
										
											2019-09-13 09:08:12 -07:00
+								    /// <inheritdoc />
-												Split encounter areas, relocate slot finding to obj

Now logic is reasonably split, and each format of area has its own way
of yielding slots

Too much junk with checking flute boosts or catch combo applicability;
just let the area dictate how slots match.

											
										
										
											2019-09-12 23:20:52 -07:00
+								    /// <summary>
 								    /// <see cref="GameVersion.Gen7"/> encounter area
 								    /// </summary>
 								    public sealed class EncounterArea7 : EncounterArea32
 								    {
-												Refactoring some IEncounterable-type'd object initialization and original-generation evo chain (#2974)


											
										
										
											2020-08-21 16:35:49 -07:00
+								        public override IEnumerable<EncounterSlot> GetMatchingSlots(PKM pkm, IReadOnlyList<EvoCriteria> chain)
-												Split encounter areas, relocate slot finding to obj

Now logic is reasonably split, and each format of area has its own way
of yielding slots

Too much junk with checking flute boosts or catch combo applicability;
just let the area dictate how slots match.

											
										
										
											2019-09-12 23:20:52 -07:00
+								        {
-												Refactoring some IEncounterable-type'd object initialization and original-generation evo chain (#2974)


											
										
										
											2020-08-21 16:35:49 -07:00
+								            foreach (var slot in Slots)
-												Split encounter areas, relocate slot finding to obj

Now logic is reasonably split, and each format of area has its own way
of yielding slots

Too much junk with checking flute boosts or catch combo applicability;
just let the area dictate how slots match.

											
										
										
											2019-09-12 23:20:52 -07:00
+								            {
-												Refactoring some IEncounterable-type'd object initialization and original-generation evo chain (#2974)


											
										
										
											2020-08-21 16:35:49 -07:00
+								                foreach (var evo in chain)
-												Split encounter areas, relocate slot finding to obj

Now logic is reasonably split, and each format of area has its own way
of yielding slots

Too much junk with checking flute boosts or catch combo applicability;
just let the area dictate how slots match.

											
										
										
											2019-09-12 23:20:52 -07:00
+								                {
-												Refactoring some IEncounterable-type'd object initialization and original-generation evo chain (#2974)


											
										
										
											2020-08-21 16:35:49 -07:00
+								                    if (slot.Species != evo.Species)
 								                        continue;
-												Split encounter areas, relocate slot finding to obj

Now logic is reasonably split, and each format of area has its own way
of yielding slots

Too much junk with checking flute boosts or catch combo applicability;
just let the area dictate how slots match.

											
										
										
											2019-09-12 23:20:52 -07:00
-												Refactoring some IEncounterable-type'd object initialization and original-generation evo chain (#2974)


											
										
										
											2020-08-21 16:35:49 -07:00
+								                    if (!slot.IsLevelWithinRange(pkm.Met_Level))
 								                        break;
-												Split encounter areas, relocate slot finding to obj

Now logic is reasonably split, and each format of area has its own way
of yielding slots

Too much junk with checking flute boosts or catch combo applicability;
just let the area dictate how slots match.

											
										
										
											2019-09-12 23:20:52 -07:00
-												Refactoring some IEncounterable-type'd object initialization and original-generation evo chain (#2974)


											
										
										
											2020-08-21 16:35:49 -07:00
+								                    if (slot.Form != evo.Form && !Legal.WildChangeFormAfter.Contains(slot.Species))
 								                    {
 								                        if (slot.Species != (int)Species.Minior) // Random Color, edge case
 								                            break;
 								                    }
-												Split encounter areas, relocate slot finding to obj

Now logic is reasonably split, and each format of area has its own way
of yielding slots

Too much junk with checking flute boosts or catch combo applicability;
just let the area dictate how slots match.

											
										
										
											2019-09-12 23:20:52 -07:00
-												Refactoring some IEncounterable-type'd object initialization and original-generation evo chain (#2974)


											
										
										
											2020-08-21 16:35:49 -07:00
+								                    yield return slot;
 								                    break;
 								                }
 								            }
-												Split encounter areas, relocate slot finding to obj

Now logic is reasonably split, and each format of area has its own way
of yielding slots

Too much junk with checking flute boosts or catch combo applicability;
just let the area dictate how slots match.

											
										
										
											2019-09-12 23:20:52 -07:00
+								        }
 								    }
-												Refactoring some IEncounterable-type'd object initialization and original-generation evo chain (#2974)


											
										
										
											2020-08-21 16:35:49 -07:00
+								}