주시
1
0
포크
이미 Warrior_EA을(를) 포크했습니다
0
원본 프로젝트 MrBaro75/Warrior_EA
Warrior_EA/Money/MoneyRiskBase.mqh
AnimateDread 15827a6b77 refactor(trade-mgmt): remove all confidence-scaled trade management
Five modes went, all of them staking real risk on the model's confidence:
Intelligent entry (ENTRY_INTELLIGENT), stop (SL_INTELLIGENT), target
(TP_INTELLIGENT), trailing (CTrailingIntelligent) and lot size
(CMoneyIntelligent's quarter-Kelly). With them, the Confidence_Source
input and the CONFIDENCE_SOURCE enum, whose only job was choosing which
number those five read.

The reason is calibration, not correctness: the confidence magnitude is
known to be miscalibrated against the label prior, so every one of these
modes multiplied money by a quantity whose units were never established.
The DB arm had a second, independent defect - since the tester DB guard
(SignalDatabaseActive) it reads 0 in tester and optimizer but non-zero
live, so any backtest of CONF_DB/CONF_BLENDED could not reproduce live
trading. And what the DB produces is a filter-RANKING win rate, not a
per-trade win probability.

Both confidence numbers are still recorded per trade (aiConfidence /
dbConfidence) and still bucketed against outcome in TradeJournalReport.
Recording is what keeps the question answerable; acting on it was the
part with no evidence behind it. ConfidenceBridge.mqh now carries an
explicit telemetry-only rule at the top.

ENUM ORDINALS PINNED. Removing a member vacated a value in four enums at
once and MT5 does not validate an enum input replayed from a saved .set
or a stored optimization pass. TRAILING_STRATEGY and
MONEY_MANAGEMENT_STRATEGY now carry explicit values so the survivors keep
the numbers they were saved as, and ValidateBarrierInputs is widened into
ValidateTradeManagementInputs covering SL_Mode, TP_Mode,
Entry_Multiplier, TrailingStrategy and MM_STRATEGY. Without that gate a
chart saved with the Intelligent stop would feed SL_Mode = -1 into a
multiplier now used verbatim, placing the stop on the wrong side of entry.

RETRAIN-NEUTRAL: neither SL_Mode nor TP_Mode appears in
BuildModelFingerprint() or ComputeDbConfigFingerprint() since the
swing-pivot target replaced the barrier labels. No .nnw, .cfg or .db
re-keys. Also drops the now-dead g_TradeRewardRiskRatio bridge, the
CMoneyRiskBase::AdjustRiskAmount hook and the unsigned AIConfidence().

Compile-verified in _claude_stage: 0 errors, 0 warnings.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
2026-08-25 10:10:20 -04:00

156 라인
8.9 KiB
MQL5

//+------------------------------------------------------------------+
//| MoneyRiskBase.mqh |
//| AnimateDread |
//| https://www.mql5.com |
//+------------------------------------------------------------------+
#include "..\Expert\ExpertMoneyCustom.mqh"
#include "..\Variables\RiskBudget.mqh"
//+------------------------------------------------------------------+
//| Class CMoneyRiskBase. |
//| Shared risk-based lot-sizing core for every money-management |
//| strategy that sizes a trade off a fixed account-risk percentage. |
//| It was extracted when CMoneyFixedRisk and CMoneyIntelligent had |
//| CalculatePotentialLoss()/CheckOpenLong()/CheckOpenShort()/ |
//| CalculateLotSize() duplicated near-verbatim between them; |
//| CMoneyIntelligent was removed 2026-08-25, so CMoneyFixedRisk is |
//| currently the only subclass. Kept as a base anyway - what lives |
//| here is the account-risk contract (including the RiskBudget |
//| clamp), not one strategy's arithmetic. |
//| AdjustLotSize() is the one remaining divergence point; the |
//| AdjustRiskAmount() hook beside it went with its only overrider. |
//+------------------------------------------------------------------+
class CMoneyRiskBase : public CExpertMoneyCustom
{
public:
virtual double CheckOpenLong(double price, double sl);
virtual double CheckOpenShort(double price, double sl);
virtual double CheckClose(CPositionInfo *position) { return(0.0); }
protected:
double CalculatePotentialLoss(ENUM_ORDER_TYPE orderType, double price, double sl);
double CalculateLotSize(double loss);
private:
//--- CheckOpenLong()/CheckOpenShort() share this whole body, differing only in the order type
//--- passed through to CalculatePotentialLoss()/ValidateLotForTrade(); see CheckTrailingStop()
//--- in Trailing\TrailingATR.mqh for the same isLong-parameter unification pattern.
double CheckOpen(ENUM_ORDER_TYPE type, double price, double sl);
//--- Reserved extension point for a future lot-size-level adjustment (e.g. equity-curve-
//--- based scaling); no current subclass overrides this - default is a no-op.
virtual double AdjustLotSize(double lot) { return lot; }
};
//+------------------------------------------------------------------+
//| Getting lot size for open long position. |
//+------------------------------------------------------------------+
double CMoneyRiskBase::CheckOpenLong(double price, double sl)
{
return CheckOpen(ORDER_TYPE_BUY, price, sl);
}
//+------------------------------------------------------------------+
//| Getting lot size for open short position. |
//+------------------------------------------------------------------+
double CMoneyRiskBase::CheckOpenShort(double price, double sl)
{
return CheckOpen(ORDER_TYPE_SELL, price, sl);
}
//+------------------------------------------------------------------+
//| Shared CheckOpenLong()/CheckOpenShort() body - see class header. |
//+------------------------------------------------------------------+
double CMoneyRiskBase::CheckOpen(ENUM_ORDER_TYPE type, double price, double sl)
{
if(m_symbol == NULL)
return 0.0;
double loss = CalculatePotentialLoss(type, price, sl);
if(loss <= 0.0)
{
// loss<=0 means sl landed on the wrong side of price (upstream signal bug, stale/gapped
// quote, or a mid-tick price move) - reject the trade instead of silently opening at
// LotsMin(), which used to bypass Money_Risk_Percent entirely with no trace of why.
PrintFormat("%s: rejected - non-positive potential loss (%.5f) for price=%.5f sl=%.5f on %s",
__FUNCTION__, loss, price, sl, m_symbol.Name());
return 0.0;
}
double lot = AdjustLotSize(CalculateLotSize(loss));
if(lot <= 0.0 || !MathIsValidNumber(lot))
{
PrintFormat("%s: rejected - invalid computed lot size (%.5f) for loss=%.5f on %s", __FUNCTION__, lot, loss, m_symbol.Name());
return 0.0;
}
string description;
// Article 2555 #14/#3/#5/#2 in one gate: tradeable symbol, legal volume on the SYMBOL_VOLUME_STEP
// grid, inside SYMBOL_VOLUME_LIMIT for this direction, and covered by ACCOUNT_MARGIN_FREE. See
// CExpertMoneyCustom::ValidateLotForTrade() - it logs its own (throttled) rejection reason.
if(!ValidateLotForTrade(m_symbol.Name(), lot, type, description))
return 0.0;
return lot;
}
//+------------------------------------------------------------------+
//| Calculate potential loss |
//+------------------------------------------------------------------+
double CMoneyRiskBase::CalculatePotentialLoss(ENUM_ORDER_TYPE orderType, double price, double sl)
{
if(price == 0.0)
price = (orderType == ORDER_TYPE_BUY) ? m_symbol.Ask() : m_symbol.Bid();
if(price == 0.0)
{
// SymbolInfoDouble-backed Ask()/Bid() returns 0 when no quote is available yet
// (disconnected/freshly-selected symbol) - a 0 price makes OrderProfitCheck's result
// meaningless, so fail loudly instead of feeding it into the lot-size calculation.
PrintFormat("%s: no valid price available for %s (Ask/Bid returned 0)", __FUNCTION__, m_symbol.Name());
return 0.0;
}
return -m_account.OrderProfitCheck(m_symbol.Name(), orderType, 1.0, price, sl);
}
//+------------------------------------------------------------------+
//| Calculate the lot size based on potential loss and account balance|
//+------------------------------------------------------------------+
double CMoneyRiskBase::CalculateLotSize(double loss)
{
if(loss <= 0.0 || !MathIsValidNumber(loss))
{
// Both current callers reject a non-positive loss before reaching here, but this is where the
// division happens, so this is where the invariant belongs - CalculatePotentialLoss() signals
// "no usable quote" by returning exactly 0.0, and any future caller that forwards that value
// straight through would otherwise divide by it and hand back an inf lot size.
PrintFormat("%s: rejected - potential loss must be positive and finite, got %.5f", __FUNCTION__, loss);
return 0.0;
}
double riskAmount = m_account.Balance() * m_percent / 100.0;
//--- ACCOUNT-LEVEL CLAMP. Money_Risk_Percent alone answers "what is my usual risk per trade", which
//--- is a different question from "how much am I still ALLOWED to lose today". Sizing off balance
//--- with no reference to the second is how a routine stop-out breaches a hard daily limit: at 3.2%
//--- into a 4% day, a full-size trade is already over the line before it is placed. CapRiskAmount()
//--- reduces this to a fraction of what genuinely remains after every open position's loss-to-stop,
//--- and returns 0 when the trade must not be taken at all. See Variables\RiskBudget.mqh.
double allowed = g_riskBudget.CapRiskAmount(riskAmount);
if(allowed <= 0.0)
return 0.0; // budget exhausted / halted - CapRiskAmount has logged the reason
riskAmount = allowed;
double stepvol = m_symbol.LotsStep();
if(stepvol <= 0.0)
{
// A 0 SYMBOL_VOLUME_STEP (not yet synced by the broker) would otherwise divide by zero and
// propagate inf/NaN downstream into volume/margin checks with no diagnostic.
PrintFormat("%s: rejected - LotsStep() returned %.5f for %s (broker volume data not ready?)", __FUNCTION__, stepvol, m_symbol.Name());
return 0.0;
}
double lot = MathFloor(riskAmount / loss / stepvol) * stepvol;
//--- BELOW-MINIMUM MEANS NO TRADE, NOT A BIGGER TRADE. Downstream, TCNormalizeVolume() bumps any
//--- sub-minimum volume UP to SYMBOL_VOLUME_MIN (correct for a user-entered fixed lot), which on a
//--- symbol where min > step (indices/metals: min 0.10, step 0.01) would turn a risk-derived 0.05
//--- into 0.10 - double the intended risk, AFTER CapRiskAmount() already clamped it. That is the
//--- exact "routine stop-out breaches the daily limit" path the budget exists to close, and the
//--- opposite of RISK_BUDGET_MIN_SIZE_FRACTION's near-binary rule (see Variables\RiskBudget.mqh):
//--- when the affordable size is not at least the broker's minimum, the trade is refused here, in
//--- the risk-sizing path, before the normalizer can inflate it. (2026-08-11)
double minvol = m_symbol.LotsMin();
if(minvol > 0.0 && lot < minvol)
{
PrintFormat("%s: rejected - risk-derived lot %.2f is below the broker minimum %.2f on %s; "
"opening at the minimum would exceed the intended risk (budget-capped risk %.2f, "
"1-lot loss %.2f)", __FUNCTION__, lot, minvol, m_symbol.Name(), riskAmount, loss);
return 0.0;
}
return lot;
}
//+------------------------------------------------------------------+