2

Есть код

var znak = parseFloat($('td').eq(2).html()) || 0,
    lng0 = znak;
var znac = setInterval(function (){
   znak = lng0 = parseFloat($('td').eq(2).html());
},2000);
var znak1 = parseFloat($('td').eq(3).html()) || 0,
    lat1 = znak1;
var znac = setInterval(function (){
   znak1 = lat1 = parseFloat($('td').eq(3).html());
},2000);

C помощью которого получаю данные, и, если данные еще не получены, то я устанавливаю первоначальное значение 0.

Код мне очень помог в проекте, но у меня будет около 50-100 таких маркеров, и находить каждом из них значения - не вариант...

Как можно улучшить этот код ???

7
  • 2
    в прошлом вопросе я уже писал как нужно эти интервалы поменять, и заменить их вообще на один интервал
    – Grundy
    12 янв 2017 в 8:24
  • Убрать жиквери и использовать ванильку
    – Arthur
    12 янв 2017 в 8:26
  • можешь ссылку дать ?
    – elik
    12 янв 2017 в 8:26
  • можно ссылку на вашу ваньльку ?)
    – elik
    12 янв 2017 в 8:27
  • 1
    ваниль не в плане фреймворк, а чистый джс
    – Arthur
    12 янв 2017 в 8:32

1 ответ 1

2

Ну, по порядку:

Сперва, заменить этот транслит на что-нибудь менее травмирующее зрение и психику.

Затем, объединить все действия под интервалами в ОДИН интервал, в котором будут оббегаться все эти "маркеры", как было неоднократно предложено ранее.

После неплохо было бы вынести МНОГОКРАТНО используемый функционал в, соответственно, фукнцию. Не очень понимаю суть всего происходящего, но если так будет угодно, все что делается до интервала в одну функцию, все, что после - в другую.

Ну и как опять же указали ранее, пользоваться VanillaJS - инструмент хоть и сложный в освоении, но как только познаешь - откроется невообразимая мощь этого гиганта программирования.

Ваш ответ

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

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