0

Есть некий запрос к IndexedDB

function GetData(storename,index,callback) 
    {
        var openReq = indexedDB.open("catalogs");
        openReq.onsuccess = function() 
        {


            var db = openReq.result;
            var transaction = db.transaction([storename], "readwrite");
            var objectStore = transaction.objectStore(storename);
            var objectStoreRequest = objectStore.get(index);
            var store=null;  
              objectStoreRequest.onsuccess =function(event)
              {
              store =objectStoreRequest.result;


              return store;
              }
              transaction.oncomplete = function(event)
              {
                db.close();
                if(callback)
                    callback(store);
              }
        }
    }

Я хочу записать значение в какуюто переменную вот так:

var result;
GetData('pesticides',i,function(res){result=res;  });  
console.log(result);

Но значение result вне области видимости и в консоль возвращается undefined.

2 ответа 2

2

Ужасное форматирование кода, никакого codestyle это раз. Во вторых нет проверки что typeof callback === 'function' это два. Используйте промисы и promises api. Придется переписать код функции под промисы.

function GetData (storename, index) {
  return new Promise(function (resolve, reject) {
    var openReq = indexedDB.open("catalogs")

    openReq.onsuccess = function () {
      var db = openReq.result
      var transaction = db.transaction([storename], "readwrite")
      var objectStore = transaction.objectStore (storename)
      var objectStoreRequest = objectStore.get (index)
      var store = null

      objectStoreRequest.onsuccess = function (event) {
        store = objectStoreRequest.result
        resolve(store)
      }
      transaction.oncomplete = function (event) {
        db.close()
        resolve(store)
      }
    }
  }
}

Потом в коде где надо получить результат пишите

GetData('sss', 2).then(function (data) {
  //в data лежит результат resolve(store)
}, function () {
  //Обработка ошибки
})

Не забывайте что код асинхронный

п.с. в коде возможно ошибки есть, кажется не закрыл скобку
п.с.с Я не знаю как работает ваш код, но мне кажется там очень много лишних переменных

3
  • еще можно добавить события типа onfail и там вызвать reject() который вызвовет фейловый коллбэк промиса
    – IC_
    24 мая 2017 в 4:07
  • Я знаю что код ужасный, но нет времени на красоту. Что такое codestyle? Создавал переменные как в примере в документации к IndexedDB, можно запрос и в одну строчку прописать. Спасибо за критику и советы, сейчас буду изучать промисы.
    – l1s4un
    24 мая 2017 в 4:22
  • @lischuk установите eslint npm i -g eslint потом инициализируйте eslint init выберите standart. Он будет жестко ругать за несоответсвие кода стилю
    – IC_
    24 мая 2017 в 4:32
0

Так не получится.

Чтобы работало или метод должен быть синхронным (плохо, плохо, плохо!) или result должен быть глобальным, относительно callback'а, а еще лучше перенести по возможности обработку в колбэк.

var result; 
function processResult(resultData) {
// ..
}

GetData('pesticides', i, processResult);

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.