Új hozzászólás Aktív témák

• #7032 Sk8erPeter nagyúr Speeedfire #7024

  Új Válasz 2013-02-08 15:15:13 #7032

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  Sk8erPeter

  nagyúr

  válasz Speeedfire #7024 üzenetére

  Azért van ott a "JSHint" gomb, hogy használd...
  Először is "infowindow" néven hoztad létre a függvényt, majd infowindow névvel használtál egy változót, írja is a JSHint megnyomására, hogy "infowindow is already defined". Ugyanezt a hibát követted el a "map"-nél.

  Aztán egy lényegesebb dolog:
  infowindow(marker, address[i]), map;
  kissé rossz helyre tetted a zárójelet.
  De mondom, mindez egyből kiderült volna a JSHint gomb megnyomása után.
  Aztán még az is, hogy függvénydefiníció loopon belül nem a legjobb.
  Ez a while (address[i]) sem túl szép szerintem, beszédesebb lenne helyette egy tisztességes for loop:
  for(var i=0; i < address.length; i++) {...}

  Egy gyors console.log debuggolással az is gyorsan kiderül, hogy az infowindow nevű függvényben az info minden alkalommal undefined, tehát ez a kulcs.

  A geocode callbackben ha kiírattad volna az address[i]-t, gyorsan kiderült volna, hogy az ott undefined, és az is egyértelmű, hogy mivel a geocode-nak már átadtad ezt az address[i]-t, ez itt felesleges, viszont ha kiíratod a results paramétert, akkor látható, hogy működőképes a results[0].formatted_address. Egyszerűen tehát ezt kell átadni a függvénynek az address[i] helyett.

  http://jsfiddle.net/Sk8erPeter/HaZGA/9/
  Tádámm, működik.
  A console.log()-okat direkt bennehagytam.

  Egyébként erre való a JavaScript topic.

  Szerk.: látom közben megírták mások is a fő problémát, de azért a fentebbieket is olvasd el, és használd a JSHint gombot ezentúl (meg sztem ezeket tényleg inkább JS topicba).

  [ Szerkesztve ]

  Sk8erPeter

Új hozzászólás Aktív témák