Лучший способ «вернуть истину» на основе условий от вложенных функций

Question

Это может быть новичок, но я пытаюсь создать функцию, которая возвращает true. Однако это основано на том, что происходит внутри нескольких других функций внутри.

function checkGeo(){ 

    // CHECK FOR GEOLOCATION 
    if("geolocation" in navigator) { 

     navigator.geolocation.getCurrentPosition(function(position){ 
      sessionStorage.pinlat = position.coords.latitude; 
      sessionStorage.pinlon = position.coords.longitude; 
      // position object is set! 
     }); 

     // position is not defined 
     if (position.coords.latitude && position.coords.longitude){ 
      return true; 
     } 

    } 

} 

Это порядок, я хочу, чтобы случиться с моей геолокации проверки, но я немного удивлен, что вложенные если тестируется перед заканчивает метод getCurrentPosition.

Ввод этого условия в функцию успеха getCurrentPosition и возврат истины оттуда не делает checkGeo возвратом true. Как проверить, завершилась ли эта асинхронная функция и, следовательно, проверить ее результаты для return true?

Answer 1

У вашей функции есть finished переменной

function checkGeo(){ 
    var self = this; 
    this.ready = function() {} 
    this.result = false; 
    if("geolocation" in navigator) { 
     navigator.geolocation.getCurrentPosition(function(position) { 
      sessionStorage.pinlat = position.coords.latitude; 
      sessionStorage.pinlon = position.coords.longitude; 
      self.result = (position.coords.latitude && position.coords.longitude); 
      self.ready.call(self); 
     }); 
    } 
} 

Теперь вы может использовать эту функцию:

var run = new checkGeo(); 
run.ready = function() { 
    alert(this.result); //Both work 
    alert(run.result); //Both work 
}; 

Немного сложнее, но лучше программировать, на мой взгляд.

Answer 2

position в анонимной функции не совпадает с position в заявлении if после него. Область применения JavaScript (игнорирование ключевого слова ES6 let для простоты) осуществляется по функциям.

Кроме того, если getCurrentPosition() является асинхронным, вы не можете полагаться на функцию анонимного обратного вызова, которая запускается раньше всего.

Если все, что вы хотите return true, чтобы показать, что вы пытаетесь получить геолокации информацию без каких-либо гарантий, что вы будете успешны, использовать что-то более, как это:

function checkGeo(){ 
    var hasGeolocation = false; 

    // CHECK FOR GEOLOCATION 
    if("geolocation" in navigator) { 
     hasGeolocation = true; 

     navigator.geolocation.getCurrentPosition(function(position){ 
      sessionStorage.pinlat = position.coords.latitude; 
      sessionStorage.pinlon = position.coords.longitude; 
      // position object is set! but only inside this function. 
     }); 

     return hasGeolocation; 
    } 
} 

С другой стороны, если вы пытаются указать return true, что геолокация была успешно установлена, тогда вам нужно указать ее каким-то другим способом, чем возвращаемое значение синхронной функции, потому что вы не будете знать, что она будет установлена ​​(может произойти ошибка, пользователь может запретить геолокацию для вашего сайта и т. д.), пока асинхронная функция не вызовет обратный вызов.

Answer 3

Вызов геолокации асинхронный, поэтому вы не можете вернуть результат из функции. Когда функция заканчивается, вы еще не знаете результат асинхронного вызова. Возвращение чего-либо из обратного вызова для асинхронного вызова не приведет к возврату значения функции, поскольку функция уже вернулась.

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

function checkGeo(callback){ 
    if("geolocation" in navigator) { 
    navigator.geolocation.getCurrentPosition(function(position){ 
     sessionStorage.pinlat = position.coords.latitude; 
     sessionStorage.pinlon = position.coords.longitude; 
     callback(position.coords.latitude && position.coords.longitude); 
    }); 
    } else { 
    callback(false); 
    } 
} 

Использование:

checkGeo(function(exists){ 
    // here you can use the result 
    if (exists) { 
    // ... 
    } 
});