How can i code 2 different take profits with different conditions ?

sodastodas

I have made a cBot based on an oscillator with a lot volume of 0.1 and i want it to take 2 different take profits with the following conditions:
1- a fixed take profit of 100 pips with a lot volume of 0.08
2- a take profit based on the oscillator's value. I need it to take profit of 0.02 lot (remainder of the initial 0.1 lot volume) whenever the oscillator crosses down the zero line (if it was an open buy position) and whenever it crosses above the zero line (if it was an open sell position)

This idea is similar to the Advanced take profit option , i just don't know how to code it

This is the code i am using and the cbotset file is also attached :

using System;
using cAlgo.API;
using cAlgo.API.Collections;
using cAlgo.API.Indicators;
using cAlgo.API.Internals;

namespace cAlgo.Robots
{
    [Robot(AccessRights = AccessRights.None)]
    public class Momentumhighopenlow : Robot
    {
        [Parameter(DefaultValue = 0.1)]
        public double Volume { get; set; }

        [Parameter(DefaultValue = 9)]
        public int Periods { get; set; }
        
        [Parameter(DefaultValue = 100)]
        public double TakeProfit { get; set; }

        private MovingAverage Hullopen;
        private MovingAverage HullHigh;
        private MovingAverage HullLow;

        protected override void OnStart()
        {
            // Initialize Hull Moving Averages for Open, High, and Low prices
            Hullopen = Indicators.MovingAverage(Bars.OpenPrices, Periods, MovingAverageType.Hull);
            HullHigh = Indicators.MovingAverage(Bars.HighPrices, Periods, MovingAverageType.Hull);
            HullLow = Indicators.MovingAverage(Bars.LowPrices, Periods, MovingAverageType.Hull);
        }

        protected override void OnTick()
        {
            // Calculate Momentum value
double hullolastvalue = Hullopen.Result.LastValue;
double hullhlastvalue = HullHigh.Result.LastValue;
double hullllastvalue = HullLow.Result.LastValue;
double hullopreviousbar = Hullopen.Result.Last(1);
double hullhpreviousbar = HullHigh.Result.Last(1);
double hulllpreviousbar = HullLow.Result.Last(1);
double momo = hullolastvalue - hullopreviousbar;
double momh = hullhlastvalue - hullhpreviousbar;
double moml = hulllpreviousbar - hullllastvalue;
double mom = momh-(momo+moml);
double hullopreviousbar2 = Hullopen.Result.Last(2);
double hullhpreviousbar2 = HullHigh.Result.Last(2);
double hulllpreviousbar2 = HullLow.Result.Last(2);
double momo2 = hullopreviousbar - hullopreviousbar2;
double momh2 = hullhpreviousbar - hullhpreviousbar2;
double moml2 = hulllpreviousbar2 - hullllastvalue;
double previousmom = momh2-(momo2+moml2);

            // Trading logic
            if (mom > 0 && previousmom <= 0)
            {
                Open(TradeType.Buy);
                Close(TradeType.Sell);
            }
            else if (mom < 0 && previousmom >= 0)
            {
                Close(TradeType.Buy);
                Open(TradeType.Sell);
            }
        }

        private void Open(TradeType tradeType)
        {
            // Open a trade only if there are no open positions
            if(Positions.Count == 0)
            ExecuteMarketOrder(tradeType, SymbolName, Volume);
        }

        private void Close(TradeType tradeType)
        {
            // Close all positions of the specified trade type
            Print($"Attempting to close {tradeType} positions...");
            foreach (var position in Positions)
            {
                if (position.TradeType == tradeType && position.SymbolName == SymbolName)
                {
                    Print($"Closing position {position.Id} ({position.TradeType})");
                    position.Close();
                }
            }
        }

        protected override void OnStop()
        {
            // Handle cBot stop event
        }
    }
}

Pipster

a fixed take profit of 100 pips with a lot volume of 0.08

This is not possible, instead you can have a position size calculated based on a stop loss of 100

a take profit based on the oscillator's value. I need it to take profit of 0.02 lot (remainder of the initial 0.1 lot volume) whenever the oscillator crosses down the zero line (if it was an open buy position) and whenever it crosses above the zero line (if it was an open sell position)

I would contact a developer if no one can help you with this.

sodastodas

@Pipster , Regarding condition 1 : why not ? isn't this similar to the advanced take profit option (as in the attached photo) ?

Regarding condition 2: the oscillator can already do that but using the entire 0.1 lot size. If condition 1 is not applicable , then there is no need to modify the code I can just reset the volume lot size to 0.02

what I am trying to do here is divide the 0.1 lot : 0.08 for the fixed take profit and 0.02 for the oscillator when it crosses the zero line

Pipster

Sorry, I misunderstood your question; the cTrader ExecuteMarketOrder has 6 overrides in the signature; one is take profit.

ExecuteMarketOrder(tradeType, SymbolName, Volume, string.Empty, 0, TakeProfit);

Attempt to implement your code for your 2nd issue; I will see if I can help.

what I am trying to do here is divide the 0.1 lot : 0.08 for the fixed take profit and 0.02 for the oscillator when it crosses the zero line

This does not make much sense to me, can you explain clearly your formula and a use case.

sodastodas

@pipster , this is the current code i am using. This code opens or buys based on the oscillator's value oscillating back and forth across the zero line

i want to edit the code so it can keep using this same condition with a 0.08 lot volume AND a 0.02 lot volume with a take profit of 100 pips

using System;
using cAlgo.API;
using cAlgo.API.Collections;
using cAlgo.API.Indicators;
using cAlgo.API.Internals;

namespace cAlgo.Robots
{
    [Robot(AccessRights = AccessRights.None)]
    public class Momentumhighopenlow : Robot
    {
        [Parameter(DefaultValue = 0.1)]
        public double Volume { get; set; }

        [Parameter(DefaultValue = 9)]
        public int Periods { get; set; }
        
        [Parameter(DefaultValue = 100)]
        public double TakeProfit { get; set; }

        private MovingAverage Hullopen;
        private MovingAverage HullHigh;
        private MovingAverage HullLow;

        protected override void OnStart()
        {
            // Initialize Hull Moving Averages for Open, High, and Low prices
            Hullopen = Indicators.MovingAverage(Bars.OpenPrices, Periods, MovingAverageType.Hull);
            HullHigh = Indicators.MovingAverage(Bars.HighPrices, Periods, MovingAverageType.Hull);
            HullLow = Indicators.MovingAverage(Bars.LowPrices, Periods, MovingAverageType.Hull);
        }

        protected override void OnTick()
        {
            // Calculate Momentum value
double hullolastvalue = Hullopen.Result.LastValue;
double hullhlastvalue = HullHigh.Result.LastValue;
double hullllastvalue = HullLow.Result.LastValue;
double hullopreviousbar = Hullopen.Result.Last(1);
double hullhpreviousbar = HullHigh.Result.Last(1);
double hulllpreviousbar = HullLow.Result.Last(1);
double momo = hullolastvalue - hullopreviousbar;
double momh = hullhlastvalue - hullhpreviousbar;
double moml = hulllpreviousbar - hullllastvalue;
double mom = momh-(momo+moml);
double hullopreviousbar2 = Hullopen.Result.Last(2);
double hullhpreviousbar2 = HullHigh.Result.Last(2);
double hulllpreviousbar2 = HullLow.Result.Last(2);
double momo2 = hullopreviousbar - hullopreviousbar2;
double momh2 = hullhpreviousbar - hullhpreviousbar2;
double moml2 = hulllpreviousbar2 - hullllastvalue;
double previousmom = momh2-(momo2+moml2);

            // Trading logic
            if (mom > 0 && previousmom <= 0)
            {
                Open(TradeType.Buy);
                Close(TradeType.Sell);
            }
            else if (mom < 0 && previousmom >= 0)
            {
                Close(TradeType.Buy);
                Open(TradeType.Sell);
            }
        }

        private void Open(TradeType tradeType)
        {
            // Open a trade only if there are no open positions
            if(Positions.Count == 0)
            ExecuteMarketOrder(tradeType, SymbolName, Volume);
        }

        private void Close(TradeType tradeType)
        {
            // Close all positions of the specified trade type
            Print($"Attempting to close {tradeType} positions...");
            foreach (var position in Positions)
            {
                if (position.TradeType == tradeType && position.SymbolName == SymbolName)
                {
                    Print($"Closing position {position.Id} ({position.TradeType})");
                    position.Close();
                }
            }
        }

        protected override void OnStop()
        {
            // Handle cBot stop event
        }
    }
}

algoman

The sentence below does not make any sense, can you explain in more detail.

i want to edit the code so it can keep using this same condition with a 0.08 lot volume AND a 0.02 lot volume with a take profit of 100 pips

sodastodas

@ algoman , what i am trying to do here is to have 2 takeprofits for each position opened
1- First takeprofit is 100 pips with a 0.08 lot volume
2- Second takeprofit occurs only when the oscillator crosses the zero line with a volume of 0.02 lots (the remainder of the 0.1 lot)

algoman

You need to contact a developer to help you implement this or at least make an attempt to code this yourself. If you are lucky, someone might help you.

Read the forum guide