Help! - How to make trailing stop loss take losses?

I get negative profit when running trailing stop loss because the code does not account for the commission and swap,
I tried the followings which don't work
need help, thanks!



void LongTrailingStop()
{
int lcnt;
int ltotal = OrdersTotal();
double BuyProfit;

for(lcnt=ltotal-1; lcnt >= 0; lcnt--)
{
OrderSelect(lcnt, SELECT_BY_POS, MODE_TRADES);
if(OrderType()<=OP_SELL && OrderSymbol()==Symbol())
{
if(OrderType()==OP_BUY) // long position is opened
{
if(TrailingStop>0)
{
if(Bid-OrderOpenPrice()>(Point*TrailingStop))
{
if(OrderStopLoss()<Bid-(Point*TrailingStop))
{
BuyProfit = (OrderProfit() + OrderCommission() + OrderSwap());
OrderModify(OrderTicket(),OrderOpenPrice(),Bid-(Point*TrailingStop+BuyProfit),OrderTakeProfit(),0,Green);
return(0);
}
}
}

}
}
}
}

hi johnny,

From your code, it seems that you assume OrderProfit() has one point unit price.
OrderProfit() doesn't equal to Bid-OrderOpenPrice(). if OrderProfit() return a value of 1, it doesn't mean your profit is 1 point.
OrderProfit() depends on your lot size, calculated as follows:
1 pips = 10 point = $10
for buy order, OrderProfit() = lotsize*(Bid-OrderOpenPrice)/(10*point)*$10

hope this help

Thanks for your reply,

OrderModify(OrderTicket(),OrderOpen Price(),Bid-(Point*TrailingStop+BuyProfit),Orde rTakeProfit(),0,Green);
return(0);

doesn't work, error 130

for trailing stop, I use OrderModify, and
bool OrderModify( int ticket, double price, double stoploss, double takeprofit, datetime expiration, color arrow_color=CLR_NONE)

and by default the double stoploss does not take into account the cost of trading.

stop loss that I coded "Bid-(Point*TrailingStop+BuyProfit)" will no work.

Maybe I should ask, how do I code a trailing stop not that will take into account the cost of trading when stopping losses?