Issue where looking at Result.Position after creating an order is null?

amusleh
12 Apr 2022, 10:16

Hi,

After you call execute market order method you should check result.IsSuccessful property before trying to access the Position property of result, example:

            var result = ExecuteMarketOrder(TradeType.Buy, SymbolName, Symbol.VolumeInUnitsMin);

            if (result.IsSuccessful)
            {
                Print("Order was placed successfully, now you can use result.Position");

                var position = result.Position;
            }
            else
            {
                Print("Order was not placed because of: {0}", result.Error);
                Print("In this case result.Position is null");
            }

The above code is the right way for using result of an ExecuteMarketOrder non async call.

In case you just want to store a position on a collection when it opens you can use Positions.Opened event, example:

using System;
using System.Collections.Generic;
using cAlgo.API;

namespace NewcBot
{
    [Robot(AccessRights = AccessRights.None)]
    public class NewcBot : Robot
    {
        private readonly List<Position> _positions = new List<Position>();

        [Parameter(DefaultValue = "MyBot")]
        public string Label { get; set; }

        protected override void OnStart()
        {
            Positions.Opened += Positions_Opened;
        }

        // This will be called whenever a new position is opened
        // on trading account
        private void Positions_Opened(PositionOpenedEventArgs obj)
        {
            if (obj.Position.Label.Equals(Label, StringComparison.Ordinal))
            {
                _positions.Add(obj.Position);
            }
        }
    }
}

For ExecuteMarketOrderAsync you should use the callback delegate:

using System.Collections.Generic;
using cAlgo.API;

namespace NewcBot
{
    [Robot(AccessRights = AccessRights.None)]
    public class NewcBot : Robot
    {
        private readonly List<Position> _positions = new List<Position>();

        [Parameter(DefaultValue = "MyBot")]
        public string Label { get; set; }

        protected override void OnStart()
        {
            ExecuteMarketOrderAsync(TradeType.Buy, SymbolName, Symbol.VolumeInUnitsMin, ExecuteMarketOrderCallback);
        }

        private void ExecuteMarketOrderCallback(TradeResult result)
        {
            if (result.IsSuccessful)
            {
                var position = result.Position;

                _positions.Add(position);

                Print("Position opened at {0}", position.EntryPrice);
            }
            else
            {
                Print("Error: {0}", result.Error);
            }
        }
    }
}

manoj.clsd.kumar@gmail.com
12 Apr 2022, 11:55

RE:

Thank you.