Использование Promises для операторов if - PullRequest
Новая
       32

Использование Promises для операторов if

1 голос
/ 15 апреля 2019

Я пытаюсь использовать Promises при условии if. Проблема, с которой я сталкиваюсь, заключается в том, что даже если выражение if ложно, код под ним все еще выполняется. Как бы я исправить эту ошибку?

Пример здесь, но полный код в ссылке: https://jsbin.com/qezimeyopo/edit?js

//MAIN TRAILING STOPLOSS
async function binanceTrailingSLOrder(symbol, orderId, quantity, oldPrice, percentage, active) {

  const clean_trade = client.ws.trades([symbol], async trade => { //run websocket
    var livePrice = parseFloat(binance_symbols[symbol]["close"]); //set new price to live price
    console.log("1) order ID: " + orderId + " active: " + active);

    if (active == true) {
      try {
        const orderStatus = await binanceCheckOrderStatus(symbol, orderId);

        console.log("2) order ID: " + orderId + " active: " + active + ", new SL: " + (oldPrice * ((100 + percentage) / 100)));

        switch (orderStatus.status) {
          case "NEW":
          case "PENDING":
            console.log("Still running ...");

            if (livePrice >= (oldPrice * ((100 + percentage) / 100)) && active == true) {
              active = false;
              const cancelOrder = await binanceCancelOrder(symbol, orderId);

              if (cancelOrder) {
                console.log("Old SL cancelled");
                var newSL = livePrice * ((100 - percentage) / 100);
                newSL = binanceNormalizePrice(symbol, newSL);

                try {
                  const newStopLoss = await binanceStopOrder(symbol, 'SELL', quantity, newSL, newSL);

                  if (newStopLoss) {
                    orderId = newStopLoss.orderId;
                    quantity = newStopLoss.origQty;
                    oldPrice = livePrice;
                    active = true;
                  }
                } catch (err) {
                  console.log(err);
                }
              }
            }
            break;
          default:
            console.log("Final case: " + orderStatus.status);
            break;
        }

      } catch (err) {
        console.log(err);
      }
    }
  });
}
/*
Algorithm:

 1. Run websocket to recieve price in realtime
 2. Check if the order is still active
 3. If yes, and price is higher x%:
    a. cancel old order
    b. update new order
*/

Ошибка, с которой я сталкиваюсь, даже если active равен false, console.log("2) order ID: ... все еще выполняется, как показано на этом рисунке: enter image description here

1 Ответ

1 голос
/ 15 апреля 2019

Скорее всего, ваша проблема связана с несколькими асинхронными функциями, совместно использующими одну переменную active.Некоторый псевдокод с существом проблемы: 

function processAllOrders(symbol, active) {
  client.doForAllOrdersWithSymbol(symbol, async (order) => {
    console.log("Active before await: ", active)

    if (active) {
      const status = await order.getStatus() // execution waits here

      // So active could have changed in the meantime...
      console.log("Active after await: ", active)
      if (status === 'whatever') {
        active = false // <- changes active variable
      }
    }
  })
}

Поскольку функция выполняется для нескольких заказов, первый случайным образом поступающий в устав изменит активный параметр для всех заказов.Поскольку у вас есть await после первого файла console.log, значение active может измениться во время ожидания выполнения.

...