mirror of
https://github.com/kwsch/PKHeX
synced 2024-12-20 17:33:25 +00:00
1e86fdcea8
## Issue We want to discard-but-remember any slots that aren't a perfect fit, on the off chance that a better one exists later in the search space. If there's no better match, then we gotta go with what we got. ## Example: Wurmple exists in area `X`, and also has a more rare slot for Silcoon, with the same level for both slots. * We have a Silcoon that we've leveled up a few times. Was our Silcoon originally a Wurmple, or was it caught as a Silcoon? * To be sure, we have to check the EC/PID if the Wurmple wouldn't evolve into Cascoon instead. * We don't want to wholly reject that Wurmple slot, as maybe the Met Level isn't within Silcoon's slot range. --- Existing implementation would store "deferred" matches in a list; we only need to keep 1 of these matches around (less allocation!). We also want to differentiate between a "good" deferral and a "bad" deferral; I don't think this is necessary but it's currently used by Mystery Gift matching (implemented for the Eeveelution mystery gifts which matter for evolution moves). The existing logic didn't use inheritance, and instead had static methods being reused across generations. Quite kludgy. Also, the existing logic was a pain to modify the master encounter yield methods, as one generation's quirks had to not impact all other generations that used the method. --- The new implementation splits out the encounter yielding methods to be separate for each generation / subset. Now, things don't have to check `WasLink` for Gen7 origin, because Pokémon Link wasn't a thing in Gen7. --- ## Future Maybe refactoring yielders into "GameCores" that expose yielding behaviors / properties, rather than the static logic. As more generations and side-gamegroups get added (thanks LGPE/GO/GameCube), all this switch stuff gets annoying to maintain instead of just overriding/inheritance. ## Conclusion This shouldn't impact any legality results negatively; if you notice any regressions, report them! This should reduce false flags where we didn't defer-discard an encounter when we should have (wild area mons being confused with raids).
136 lines
4.3 KiB
C#
136 lines
4.3 KiB
C#
using System.Linq;
|
|
|
|
namespace PKHeX.Core
|
|
{
|
|
/// <summary>
|
|
/// Trade Encounter data with a fixed Catch Rate
|
|
/// </summary>
|
|
/// <remarks>
|
|
/// Generation 1 specific value used in detecting unmodified/un-traded Generation 1 Trade Encounter data.
|
|
/// Species & Minimum level (legal) possible to acquire at.
|
|
/// </remarks>
|
|
public sealed record EncounterTrade1 : EncounterTradeGB
|
|
{
|
|
public override int Generation => 1;
|
|
public override int LevelMin => CanObtainMinGSC() ? LevelMinGSC : LevelMinRBY;
|
|
|
|
private readonly int LevelMinRBY;
|
|
private readonly int LevelMinGSC;
|
|
|
|
public EncounterTrade1(int species, GameVersion game, int rby, int gsc) : base(species, gsc, game)
|
|
{
|
|
TrainerNames = StringConverter12.G1TradeOTName;
|
|
|
|
LevelMinRBY = rby;
|
|
LevelMinGSC = gsc;
|
|
}
|
|
|
|
public EncounterTrade1(int species, GameVersion game, int rby) : this(species, game, rby, rby) { }
|
|
|
|
public byte GetInitialCatchRate()
|
|
{
|
|
var pt = Version == GameVersion.YW ? PersonalTable.Y : PersonalTable.RB;
|
|
return (byte)pt[Species].CatchRate;
|
|
}
|
|
|
|
protected override void ApplyDetails(ITrainerInfo sav, EncounterCriteria criteria, PKM pk)
|
|
{
|
|
base.ApplyDetails(sav, criteria, pk);
|
|
var pk1 = (PK1)pk;
|
|
pk1.Catch_Rate = GetInitialCatchRate();
|
|
}
|
|
|
|
internal bool IsNicknameValid(PKM pkm)
|
|
{
|
|
var nick = pkm.Nickname;
|
|
if (pkm.Format <= 2)
|
|
return Nicknames.Contains(nick);
|
|
|
|
// Converted string 1/2->7 to language specific value
|
|
// Nicknames can be from any of the languages it can trade between.
|
|
int lang = pkm.Language;
|
|
if (lang == 1)
|
|
{
|
|
// Special consideration for Hiragana strings that are transferred
|
|
if (Version == GameVersion.YW && Species == (int)Core.Species.Dugtrio)
|
|
return nick == "ぐりお";
|
|
return nick == Nicknames[1];
|
|
}
|
|
|
|
return GetNicknameIndex(nick) >= 2;
|
|
}
|
|
|
|
internal bool IsTrainerNameValid(PKM pkm)
|
|
{
|
|
string ot = pkm.OT_Name;
|
|
if (pkm.Format <= 2)
|
|
return ot == StringConverter12.G1TradeOTStr;
|
|
|
|
// Converted string 1/2->7 to language specific value
|
|
int lang = pkm.Language;
|
|
var tr = GetOT(lang);
|
|
return ot == tr;
|
|
}
|
|
|
|
private int GetNicknameIndex(string nickname)
|
|
{
|
|
var nn = Nicknames;
|
|
for (int i = 0; i < nn.Count; i++)
|
|
{
|
|
if (nn[i] == nickname)
|
|
return i;
|
|
}
|
|
return -1;
|
|
}
|
|
|
|
private bool CanObtainMinGSC()
|
|
{
|
|
if (!ParseSettings.AllowGen1Tradeback)
|
|
return false;
|
|
if (Version == GameVersion.BU && EvolveOnTrade)
|
|
return ParseSettings.AllowGBCartEra;
|
|
return true;
|
|
}
|
|
|
|
private bool IsMatchLevel(PKM pkm, int lvl)
|
|
{
|
|
if (pkm is not PK1)
|
|
return lvl >= LevelMinGSC;
|
|
return lvl >= LevelMin;
|
|
}
|
|
|
|
public override bool IsMatchExact(PKM pkm, DexLevel dl)
|
|
{
|
|
if (!IsMatchLevel(pkm, pkm.CurrentLevel)) // minimum required level
|
|
return false;
|
|
|
|
if (Version == GameVersion.BU)
|
|
{
|
|
// Encounters with this version have to originate from the Japanese Blue game.
|
|
if (!pkm.Japanese)
|
|
return false;
|
|
// Stadium 2 can transfer from GSC->RBY without a "Trade", thus allowing un-evolved outsiders
|
|
if (EvolveOnTrade && !ParseSettings.AllowGBCartEra && pkm.CurrentLevel < LevelMinRBY)
|
|
return false;
|
|
}
|
|
|
|
return true;
|
|
}
|
|
|
|
protected override bool IsMatchPartial(PKM pkm)
|
|
{
|
|
if (!IsTrainerNameValid(pkm))
|
|
return true;
|
|
if (!IsNicknameValid(pkm))
|
|
return true;
|
|
|
|
if (ParseSettings.AllowGen1Tradeback)
|
|
return false;
|
|
if (pkm is not PK1 pk1)
|
|
return false;
|
|
|
|
var req = GetInitialCatchRate();
|
|
return req != pk1.Catch_Rate;
|
|
}
|
|
}
|
|
}
|