Request Review: Multiple orders spawning when limited to 1 order

void createTrade(const string symbol, int direction, double lot_size, int magicNumber, string tradeComment,
                  datetime expiration=0, color colour=clrNONE, int openBuys = 1, int openSells = 1)
{
 int buyCount = 0; //How many Longs are open
 int sellCount = 0; //How many Shorts are open
 int count = OrdersTotal(); //Total amount of alive positions
 
 for(int i = count; i > 0; i--) //Checks the internal order manager for open positions...
   {
    if(OrderSelect(i, SELECT_BY_POS))
      {
       if(OrderSymbol() == symbol && OrderMagicNumber() == magicNumber) // ^ From this bot...
         {
          if(OrderType() == ORDER_TYPE_BUY) buyCount++; // ^ and their type
          if(OrderType() == ORDER_TYPE_SELL) sellCount++;
         }
      }
   }
   RefreshRates();
   if(buyCount < openBuys && direction == 1) //If less orders are open than the limit, place a new order of the assigned type
      {
       if(OrderSend(symbol, OP_BUYSTOP, lot_size, Bid,3,0,0,tradeComment,magicNumber,expiration,colour)){} //if statement hides a compiler warning
      }
   if(sellCount < openSells && direction == -1)
      {
       if(OrderSend(symbol, OP_SELLLIMIT, lot_size, Ask,3,0,0,tradeComment,magicNumber,expiration,colour)){} //if statement hides a compiler warning\
      }
}

bool isNewCandle(const string symbol)
{
 datetime currCandle = iTime(symbol, 0, 0);
 static datetime prevCandle = currCandle;
 
 if(prevCandle < currCandle)
   {
    prevCandle = currCandle;
    return(true);
   }
  else
   {
    return(false);
   }
}

Quoting Cunaohznfes2

Disliked

Thanks @FerruFX It was actually just the for loop, I was using it properly from a "loop through all the orders" standpoint but I forgot I was using it as an index as well, which requires the modification to count and the GreaterThanEquivalence(>=) operator. In case anyone reads this looking for more answers: The original loop went from the maximum total ACTIVE orders and decremented to 0 but quit the loop at 0, thus meaning I counted every single order in the pool. (Order n to Order 1) However when using it as an index such as in my call to OrderSelect(),...

Ignored