Переменная не хочет переопределяться. Почему?

Question

[Даниил Боровков]

var lat = 0, lon = 0;
function getLoc(){
    if(navigator.geolocation){
        navigator.geolocation.getCurrentPosition(function (pos) {
            lat = pos.coords.latitude;
            lon = pos.coords.longitude;
        });
    }
}
getLoc();

В результате переменные lat и lon не переопределяются и так и остаются нулями.

Answer 1

[HoHsi]

Если хотите использовать асинхронные операции в синхронном стиле, придется прикрутить промисы

/** Переписываем с промисами */
function getLoc(options) {
  if (navigator.geolocation ){
    // Запускаем новый промис
    return new Promise((resolve, reject) => {  
      navigator.geolocation.getCurrentPosition((pos) => {
        resolve(pos);
      }, (err) => {
        reject(err);
      }, options);
    });
  } else {
    // Если нет, возвращаем заглушку, или можете сделать "reject"
    return Promise.resolve({
      pos: {
        coords: {
          latitude: 0,
          longitude: 0
        }
      }
    });
  }
}

getLoc()
.then((pos) => {
  const {
    latitude: lat,
    longitude: lon
  } = pos.coords;

  console.log("Wow");
  console.log("So promissed");
  console.log(lat, lon);
})
.catch((err) => {
  // Пользователь отказался давать координаты
  console.log(":(");
  console.error(err);
});

Comments

[Алексей]

Поясните, пожалуйста, вместо того чтобы обработать полученные координаты внутри
navigator.geolocation.getCurrentPosition(function (pos) {
lat = pos.coords.latitude;
lon = pos.coords.longitude;
console.log(lat, lon);
});
мы пишем код, приведенный выше, чтобы в итоге получить то же самое? Серьезно?

[HoHsi]

Алексей: Абсолютно серьезно. 22 строчки, и у нас есть промисифированная функция, которую можно переиспользовать.

Что касается "внутри", то это верная дорога в Callback hell

Answer 2

[Алексей]

Вероятно вы ожидаете что сразу после getLoc(); у вас будут получены координаты? Это не так.
Посмотрите пример - https://developer.mozilla.org/en-US/docs/Web/API/G...
Вам нужно обработать полученные координаты или тут:

var lat = 0, lon = 0;
function getLoc(){
    if(navigator.geolocation){
        navigator.geolocation.getCurrentPosition(function (pos) {
            lat = pos.coords.latitude;
            lon = pos.coords.longitude;
            // тут нужно обработать полученные координаты
        });
    }
}
getLoc();

или как-то иначе, например через setTimeout периодически проверять, появились ли в переменных lat и lon координаты.

Comments

[HoHsi]

setTimeout? серьезно?

[Алексей]

Прочитайте ответ полностью, пожалуйста.
Вариант с setTimeout был дан как альтернативное решение асинхронной обработке, а не как рекомендованное решение.

[HoHsi]

Алексей: осмелюсь предположить, что спрашивающий - новичок. И сейчас он воспринимает любой более менее работающий ответ как верный, и фиксирует его у себя в голове. А потом, когда такой джун приходит под ваше крыло начинается веселый рефакторинг и PR'ы.

Так-что, все что, является костыльной альтернативой, лучше либо вообще не рассказывать, либо уточнять что это ужасный вариант.

[Даниил Боровков]

HoHsi: Не новичок. Решил мысль думать, а почему уже не первый проект для получения IP адреса используем сервисы аля ip-info.com и есть ли альтернативы. Хотя вариант костыльный и с Tor не работает, все же что, да и сделал. Но на данном этапе ip-info делает все значительно лучше и быстрее на несколько сотен ms.

[HoHsi]

Даниил Боровков: кастыльность не в geolocation, а в использовании setTimeout для проверки апрува от пользователя