google maps kort med fejl som jeg ik kan finde eller forstå

Kan det være adresser, der ikke er helt præcise, eller ikke findes i deres tabeller.

<ole>

Der kan være flere årsager. Nogle af adresserne kender krak.dk heller ikke - og en adresse på Jylandsvej er det jo f.eks. ret oplagt, Google ikke kan finde  ;o)

Hvis du bygger søgningen på adresser, du ikke selv har indtastet, men i stedet bruger en form for tilmelding, kunne du validere adressen lige efter tilmeldingen (onchange på tekstfeltet/-felterne).

Du kunne begynde med at sætte en variabel 'bTriedOnce' til false i begyndelsen af dokumentet.
Kan adressen ikke findes efter indtastning - og bTriedOnce er false - alert'es brugeren: "Tjek venligst, om adressen er korrekt." - og bTriedOnce sættes til true.
Hvis adressen ikke kan findes - og bTriedOnce er true - indsættes alligevel i databasen og der sendes en mail til dig selv om manuelt at tjekke adressen.

På den måde alert'es brugeren kun én gang, så det ikke bliver irriterende  =)

/mvh
</bole>

jep det var en god ide.
Jeg har faktisk startet på at lave noget lignende det du beskriver :o)

Men jeg har også fundet ud af at mange af de adresser som bliver afvist som "ugyldige" eller "umulige at finde" faktisk sagtens kan findes på google maps. Jeg bruger akkurat samme adresse og mange af dem findes uden problemer. Så der må være en fejl. Bare lige med at finde den.

Kunne det i de tilfælde tænkes at være et tegnsæt problem?

nej for der er også nogen med alm tegne som glipper. og det ser mere ud til at forekomme i grupper. Og oftere jo længere nede man kommer... Underligt :)

Erik >> Måske, men der findes adresser med specialtegn. Kommer alle adresser fra samme kilde, er det ikke sandsynligt - kommer de fra forskellige, er det absolut en mulighed  =)

(28/11-2008 22:11:09) >> aha ... interessant! Måske, der går ged i noget, fordi du sender formange queries på for kort tid - eller serveren har for travlt i dét splitsekund(?)

Prøv at samle de ikke fundne op i et array og løb det derefter igennem og send de samme queries én gang mere. Hvad sker der ved det?

- og tjek lige alle adresser. Der var som sagt et par af dem, jeg testede på Krak, som heller ikke kunne findes dér. Oplagt er naturligvis sdavøfajl, som den jeg viste før, men der kan sikkert også være andre grunde  =)

hmmm jeg har prøvet at opdatere siden http://www.huggernet.dk/pgs/links.php?vis_firmaer=1 nogle gange og kigge nøje efter hvilke som har ugyldig adresse. og det overrasker mig at det er forskelligt fra gang til gang hvilke adresser som bliver afvist. hehe :o) dete r dæleme en underlig fejl :o)

ahhh ja da :) rigtig genial ide :)

Så er jeg helt sikker på, jeg har ret i min seneste antagelse  =)

genial ide, men hvor mange gange skal den prøve hver adresse? :o)

og hvordan implementerer jeg det på en nem og smart måde i det kode jeg har? Skal det ind som her:

function AddressPoint(address,txt_str,firmanavn_title) {
    if (geocoder) {
        geocoder.getLatLng( address,
            function(point){
                if (point){
                    map.addOverlay(createMarker(point, txt_str, firmanavn_title));
                } else {
                    HER ???
                }
            }
        );
    }
}

Du kunne evt. sætte en variabel til 5. Så kunne du løbe alle adresser igennem én gang og samle de ikke-fundne op i et array.

Derefter løber du array'et igennem og tæller variablen ned med én. Når variablen er 0, løber du ikke array'et igennem mere.

Det kan aldrig give 100% garanti for, at alle bliver fundet, men mon ikke det bliver tilfældet?  =)

aha okay. god løsning. jeg må lige prøve om jeg kan finde ud af det. javascript er ikke min stærke side, men det kan vel ik være så svært :o) Bliver spændende at se om det virker :o)

Du kan gøre noget à la:

var aNotFound = [];
var nMaxQueries = 5;
function AddressPoint(address,txt_str,firmanavn_title) {
    if (geocoder) {
        geocoder.getLatLng( address,
            function(point){
                if (point){
                    map.addOverlay(createMarker(point, txt_str, firmanavn_title));
                } else {
                    aNotFound.push([point, txt_str, firmanavn_title])
                }
            }
        );
    }
}

function CheckAgain() {
    var a = aNotFound.concat("");
    aNotFound = [];
    for (var i=0,j=a.length; i<j; i++) {
        AddressPoint(a[0], a[1], a[2]);
    }
    if (aNotFound.length>0 && --nMaxQueries>0) CheckAgain();
}

Når du har løbet alle adresser igennem første gang, kan du kalde funktionen 'CheckAgain', hvis aNotFound.length er større end 0

Udtrykket:
    var a = aNotFound.concat("");

- som måske kan forekomme lidt kryptisk, lægger en klon af 'aNotFound' i 'a'  ;o)

Og så kunne man også overveje at slå adressen op een eneste gang og finde længde-/breddegrad, putte det i databasen, og pludselig, som ren magi, ren Harry Potter, få en hurtig applikation.

Hehe det kom lige som jeg sad og var ved at begynde at ryge ud af håret :o) jeg vil lige sidde og regne ud hvordan det virker :o) Mange tak

hehe jeg blir helt stolt af alt det jeg har lavet i forhold til hvor lidt jeg egentligt ved om at kode :o) hehe

okay hvad synes du om denne løsning:

var aNotFound = [];
var nMaxQueries = 6;
function AddressPoint(address,txt_str,firmanavn_title,brugers_id) {
    if (geocoder) {
        geocoder.getLatLng( address,
            function(point){
                if (point){
                    map.addOverlay(createMarker(point, txt_str, firmanavn_title));
                } else {
                    aNotFound.push([address, txt_str, firmanavn_title, brugers_id])
                }
            }
        );
    }
}

function CheckAgain() {
    var a = aNotFound.concat("");
    aNotFound = [];
    for (var i=0,j=a.length; i<j; i++) {
        if (nMaxQueries==1){
            closeobj("driving_" + a[3]); // fjerner kørselsvejledning
            openobj_inline("confirm_" + a[3]); // viser link til 'ret adressen' i stedet for kørselsvejledning
            <?php if (!$_GET["med_alert"]){echo"//";} ?>return alert(a[0] + " ikke fundet"); // så webmaster kan teste
        } else {
            AddressPoint(a[0], a[1], a[2], a[3]);
        }
    }
    if (aNotFound.length>0 && --nMaxQueries>0) CheckAgain();
}

var point = null;
AddressPoint("Vejnavn 111, postnr og by, land", "Firmanavn og adresse osv", "Firmanavn","282");
og mange flere.......

// Den tester lige de adresser som umiddelbart ikke gav pote
if (aNotFound.length > 0){
    CheckAgain();
}

måske skulle man indsætte en delay ??

Hvad sker der, hvis du skriver:

if (aNotFound.length > 0){
    CheckAgain();
}
alert(aNotFound.length);

Hvor mange er der så, du ikke kunne finde?

jeg tester det lige... i mellemtiden kan du måske sige mig om jeg skal skrive :
if (nMaxQueries=1){
eller
if (nMaxQueries==1){

if (nMaxQueries==1){

den resulterer i alert('0') altså skriver den "0" til mig

Men du kan vel ligeså godt skrive:
    if (aNotFound.length > 0){

Så tester den én runde mere, men det kan du jo sætte i 'nMaxQueries '

og det har ik rigtig hjulpet :(

"den resulterer i alert('0') altså skriver den "0" til mig" >> Fint, så er der jo ikke nogen, den ikke kan finde  =)

Hvad har ikke hjulpet?

jo men jeg gør noget andet. jeg sætter 'nMaxQueries  til 6 og så skal den sidste runde fjerne en kørselsvejledning ved at hide link til kørselsvejledning. for så er adressen jo nok ugyldig

det har ikke hjulpet for jeg ved at der er ugyldige adresser som skulle resultere i at aNotFound.length ender med at være mere end 0. ellers er der noget den gør forkert.

jeg bruger nu:


// funktion til geocoder
var aNotFound = [];
var nMaxQueries = 6;
function AddressPoint(address,txt_str,firmanavn_title,brugers_id) {
    if (geocoder) {
        geocoder.getLatLng( address,
            function(point){
                if (point){
                    map.addOverlay(createMarker(point, txt_str, firmanavn_title));
                } else {
                    aNotFound.push([address, txt_str, firmanavn_title, brugers_id])
                }
            }
        );
    }
}
// Denne tjekker endnu en gang de mislykkedes forsøg på at finde adresserne
function CheckAgain() {
    var a = aNotFound.concat("");
    aNotFound = [];
    for (var i=0,j=a.length; i<j; i++) {
        if (nMaxQueries==1){
            closeobj("driving_" + a[3]); // fjerner kørselsvejledning
            openobj_inline("confirm_" + a[3]); // viser link til 'ret adressen' i stedet for kørselsvejledning
            <?php if (!$_GET["med_alert"]){echo"//";} ?>return alert(a[0] + " ikke fundet"); // så webmaster kan teste
        } else {
            AddressPoint(a[0], a[1], a[2], a[3]);
        }
    }
    if (aNotFound.length>0 && --nMaxQueries>0) CheckAgain();
}

Ja, sorry! Du skal skrive:

if (aNotFound.length>0 && --nMaxQueries>0) CheckAgain();
else alert("Kunne ikke finde disse "+a.length+" adresser:\n---------------------\n"+a.join("\n\n"));

- og vi prøver ige  :o|

if (aNotFound.length>0 && --nMaxQueries>0) CheckAgain();
else alert("Kunne ikke finde disse "+aNotFound.length+" adresser:\n---------------------\n"+aNotFound.join("\n\n"));

som du kan se har jeg lavet en lille ændring (der følger brugers_id med) men det så den kan fjerne kørselsvejledning linket ved siden af brugerens adresse

ok prøver jeg lige :o)

den siger stadig 0

er det fordi jeg har rettet point til address i :
aNotFound.push([address, txt_str, firmanavn_title, brugers_id])

??

jeg synes det så mere korrekt ud

Nej, det er rigtig nok. Prøv lige at flytte:
    var aNotFound = [];

- op til:
    var geocoder = null; // noget til geocoder
    var aNotFound = [];

- også:
    var nMaxQueries = 6;

still the same ;O)

den er lidt tricky :)

Du tester et forkert sted. Det er ved sidste linje i funktionen 'CheckAgain', du skal alert'e  ;o)

ahhh :)

ok det er flyttet ind i bunden af funktionen 'CheckAgain'
men det gør ikke nogen forskel...

skal jeg ik efter alle adresserne skrive ca således:

var point = null;
AddressPoint("Vejnavn 111, postnr og by, land", "Firmanavn og adresse osv", "Firmanavn","282");
og mange flere.......

// DENNE SKAL VÆRE HERefter IK????
if (aNotFound.length>0) CheckAgain();

er det ik heri problemet er:
function AddressPoint(address,txt_str,firmanavn_title,brugers_id) {
    if (geocoder) {
        geocoder.getLatLng( address,
            function(point){
                if (point){
                    map.addOverlay(createMarker(point, txt_str, firmanavn_title));
                } else {
                    aNotFound.push([address, txt_str, firmanavn_title, brugers_id])
                }
            }
        );
    }
}

for der er jo ingenting i aNotFound arrayet og det burde der være for der er ugyldige adresser som 100% sikkert burde ende deri

Der er masser i aNotFound og nMaxQueries er 6, så CheckAgain synes slet ikke at blive kaldt  :o|

I stedet for:
    if (aNotFound.length>0) CheckAgain();

- så prøv bare at skrive:
    CheckAgain();

har jeg prøvet ;O) virker ik, men jeg kan lige prøve igen

resultat: kan ikke finde disse 0 adresser:

Dooooohhhh ...!!!  :o(

AddressPoint(a[i][0], a[i][1], a[i][2], a[i][3]);

- og det skal du selvfølgelig også bruge de andre steder, hvor du bruger a-array'et  ;o)

hehehe :) ja jeg har lidt svært ved at gennemskue js men det giver mening  :)

a er et array af arrays  ;o)

skal
var a = aNotFound.concat("");
og
aNotFound = [];

ik os flyttes op

29/11-2008 00:10:19

29/11-2008 00:18:37 >> Nej

og skal der stå :
if (aNotFound.length>0) CheckAgain();
eller hvad er det du henviser til?

Nje, prøv at fjerne betingelsen

men det virker stadig ikke.... sikker på at :
var a = aNotFound.concat("");
og
aNotFound = [];

skal stå i starten af funktionen 'CheckAgain' ??

Ja, helt sikker  =)

okay :o)
Jamen sikke dog en tricky bæst vi har gang i her. du skal have mange tak fordi du gider hjælpe med det. Uden ville jeg være lidt vingskudt :)

okay jeg opsummerer lige hvad vi har nu:

// funktion til geocoder
function AddressPoint(address,txt_str,firmanavn_title,brugers_id) {
    if (geocoder) {
        geocoder.getLatLng( address,
            function(point){
                if (point){
                    map.addOverlay(createMarker(point, txt_str, firmanavn_title));
                } else {
                    aNotFound.push([address, txt_str, firmanavn_title, brugers_id])
                }
            }
        );
    }
}
// Denne tjekker endnu en gang de mislykkedes forsøg på at finde adresserne
function CheckAgain() {
    var a = aNotFound.concat("");
    aNotFound = [];
    for (var i=0,j=a.length; i<j; i++) {
        if (nMaxQueries==1){
            closeobj("driving_" + a[i][3]); // fjerner kørselsvejledning
            openobj_inline("confirm_" + a[i][3]); // viser link til 'ret adressen' i stedet for kørselsvejledning
            <?php if (!$_GET["med_alert"]){echo"//";} ?>return alert(a[i][0] + " ikke fundet"); // så webmaster kan teste
        } else {
            AddressPoint(a[i][0], a[i][1], a[i][2], a[i][3]);
        }
    }
    if (aNotFound.length>0 && --nMaxQueries>0) CheckAgain();
    else alert("Kunne ikke finde disse "+aNotFound.length+" adresser:\n---------------------\n"+aNotFound.join("\n\n"));
}


//og så adresserne...
var point = null;
AddressPoint("Vejnavn 111, postnr og by, land", "Firmanavn og adresse link til kørselsvejledning (med id self) osv...", "Firmanavn","282");
//og mange flere.......


CheckAgain();

Jeg giver op. Jeg kan ikke komme videre uden selv at sidde og teste  =)

og disse sættes længere oppe:
var geocoder = null; // noget til geocoder
var aNotFound = [];
var nMaxQueries = 6;


Sikker på denne er korrekt? :
aNotFound.push([address, txt_str, firmanavn_title, brugers_id])

http://www.huggernet.dk/pgs/links.php?vis_firmaer=1&med_alert=1

Kaffe! :D

hmm min fejlkonsol i firefox siger at a ikke er defineret. Kan det være fejlen?

Lad mig reiterere 28/11-2008 22:43:52 - det kan være vældig sjovt at bruge den bedste aften på hele ugen, fredag, med et smovse sig i lækker javascript, men

1) Det er spild af ressourcer at slå adresser op på XXX virksomheder, og få koordinaterne, hver evig eneste gang en besøgende går ind på siden.

2) Desuden tager det tid, for den besøgende, og er afhængig af den besøgendes maskiner og tykkelse af kabler.

Derfor mit forslag om at gøre det een gang for alle, for en adresse. Det kan helt sikkert gøres serverside (siger en østjyde, der ikke lige har prøvet dét, men som har erfaring fra andre Google-API-er, og som en gang har stået på toppen af det ene World Trade Center tårn).

nej det er nok ik det for det har noget med google at gøre

Du må ikke skrive return her:
    <?php if (!$_GET["med_alert"]){echo"//";} ?>return alert(a[i][0] + " ikke fundet"); // så webmaster kan teste

Så stopper alt jo der ...

At a ikke er defineret er bare en advarsel. Ligemeget.

Jeg skulle tilføje fra før: Det er ok at cache koordinater for Hovedvej 117. Med de aktuelle tektoniske pladeforskydning under Danmark, skal den ikke genberegnes før om ca. 100 år.

Erik >> Nej, det er ikke en advarsel, men en fejl  ;o)

- sjovt nok siger IE, at 'ta' ikke er defineret  :D

Jo men det har jeg sådan set lavet det du omtaler. Jeg har lavet således at brugerne som endnu ikke har bekræftet deres længde og breddegrad er korrekt, får en månedlig email hvor de får et direkte link til den side hvor de skal bekræfte at nålen er korrekt placeret. Så det er kun her den første måned der skal slåes så mange adresser op. derefter så er det måske 10 adresser + nogle enkelte nye brugere, som ikke har en LATitude og LONgtitude eller hvad det hedder.

>> 29/11-2008 00:41:54 ahhhhhh :)

Stadigvæk learner: Det sker dog alligevel et pænt antal gange fra den besøgendes maskine. Disse kunne også stå i din tabel, men en status: "ubekræftet".

Ja, Ole. Det siger vi bare.

jeps det var nok bedre at gøre det sådan

Men bestem selv - tag dig ikke af mig. Jeg sidder bare et sted hvor jeg bruger mine kræfter på at undgå redundante udregning. "Your milage may vary", som man sige på spansk.

I øjeblikket ser det ud til, du har fundet vejen til den evige 'lykke'  ;D

men no matter what så burde det da virke gør det ik?

Det kan ikke 100% udelukkes, at det burde virke - men ret meget tyder på, vi laver en fejl. At man ikke kan finde sin fejl, bør ikke lægges til grund for at konkludere, at tingene bør virke  ;o)

Det er for lang tid siden, jeg sidst har rodet med GMaps API'et - så jeg skal som sagt selv sidde og teste, hvis jeg skal finde fejlen ... det gør jeg ikke  =)

okay
Jeg aner bare ik hvad push og concat betyder og hvad der sker.
og jeg forstår heller ik hvad der sker i denne:
for (var i=0,j=a.length; i<j; i++) {
jeg er vant til at bruge foreach () i php

Jeg ved godt, der er en uvane blandt PHP'ere til altid at bruge foreach-løkker - selvom der ligeså godt kunne bruges en alm. for-løkke (som er væsentligt bedre performende):
    http://dk2.php.net/manual/en/control-structures.for.php
    https://developer.mozilla.org/en/Core_JavaScript_1.5_Reference/Statements/for

Push gør det samme som array_push: tilføjer et element i slutningen af et array:
    http://dk2.php.net/manual/en/function.array-push.php
    https://developer.mozilla.org/en/Core_JavaScript_1.5_Reference/Global_Objects/Array/push

- og når man konkatenerer (samler) et array med en tom streng, oprettes en klon af array'et. Normalt bruges concat til at samle to arrays:
    https://developer.mozilla.org/en/Core_JavaScript_1.5_Reference/Global_Objects/Array/concat

i php ville jeg nok skrive noget lign

if (der var nogen adresser som ikke blev fundet)
  $count=false
  while ($count < 7){
    if ($count==6){
      så findes den bare ik og kørselsvejl skal fjernes og evt en besked til brugeren første gang
    } else {
      fjern den fra array med de mislykkede
      prøv igen om den findes
    }
    $count++;
  }
}

Ja, men nu er det jo hverken Perl, Phyton, PHP eller C++  =)

hehe :)

kan fejlen være i denne? :
var aNotFound = [];

jeg fandt en lille fejl,

der manglede et semikolon i enden af :
aNotFound.push([address, txt_str, firmanavn_title, brugers_id])

men jeg har ik oplevet at det løste problemet desværre.

Måske kan denne tutorial http://code.google.com/support/bin/answer.py?answer=80200&topic=11369&ctx=sibling hjælpe dig, men du skal være klar over at den IKKE viser nær så korrekt som http://maps.google.com/, hvilken heller IKKE er særlig god. Forklaring på hvorfor disse 2 er forskellige http://code.google.com/support/bin/answer.py?answer=60738&topic=12266&fromgroup=true.

Denne http://www.findvej.dk/ er og bliver den bedste.

Sorry jeg ik har svaret. har lagt syg et par dage.

Det er jo kanon nemt med den php tutorial. TUSIND TAK :D

Jeg vil lave det i morgen.

Smid et svar allesammen og jeg vil fordele point.

Tusind tak for alt hjælpen :D

;o)

Jeg forventer os svar fra tinem og måske erik?

learner er det = Anne-Mette Jørsfeldt? Det er en rigtig spændende side du er ved at lave http://www.huggernet.dk. Har du brugt et CMS-system eller har du selv opbygget siden?

Var det tutorial'en som fik det til at fungere? Der er mange andre gode tutorials med phpmysql lavet af Pamela. Jeg kunne ikke have lavet mine maps uden.

*LoL* Tines spørgsmål foranledigede mig til at kikke i kildekoden for spor af et CMS. Der fandt jeg denne her, som - undskyld jeg morer mig over koden - er fuldstændig ubetalelig morsom:

<script type='text/javascript'>    document.write('<!'+'-- <br><br><center>Denne side virker kun korrekt hvis din browser kan læse JavaScript og det kan din browser tilsyneladende ikke.<br><br>Du har muligvis slået JavaScript fra i din browsers indstillinger.<br><br>Du kan ændre din browsers indstillinger eller downloade en ny browser.<br>Vi foreslår at du bruger Microsoft Internet Explorer eller Mozilla Firefox - Disse giver dig fuld adgang til denne side.<br><br><br><br>Beklager.<center><br><br><br><br><br><br><br><br><br><br> -->');
</script>

For det første forsøger man at udskrive en besked til brugeren om, at hans browser ikke understøtter JavaScript ... men hvordan er det nu lige, man får en browser, som ikke understøtter JavaScript til at skrive noget ud med JavaScript?

Hvis det et kort øjeblik lykkes os at se bort fra dén 'detalje', så er beskeden til brugeren ovenikøbet konstrueret som en HTML-kommentar - der jo som bekendt er usynlig for brugeren!

Jeg ved godt, det måske ikke er så sjovt at se folk grine ad ens kode, men denne her er som sagt grænsende til det hysterisk humoristiske. Hvis en kode skal have Nobelprisen for kode-jokes, må det være denne her  ;D

Godt jeg kan bidrage med lidt underholdning herinde :o)
Ja det er jo ikke alle knive der er lige skarpe...
Men en sløv kniv er dog bedre end ingen kniv ;o)

Lidt ramt er man vel...
Jeg kan godt se at det er nogle latterlige linier og har da os fjernet dem. Men bemærk dog det pæne sprog og den gode intention hvor jeg altid prøver at hjælpe mine brugere med at få en god oplevelse.

Når sandheden skal frem...
I virkeligheden kommer "trickset" du omtaler faktisk herindefra en tråd jeg deltog i for laaaaaaang tid siden. Så man kan evt grine lidt videre af ham eller hende :o)

Tak for hjælpen :o)

God Jul

Nu blev du helt slået ud af olebole. :-)

Men hvad med mine spørgsmål?

tinem:

Anne Mette Jørsfeldt har designet sidens udseende for mig. Hun er dygtig og har rimelige priser så brug hende endelig hvis du får brug for en grafiker eller en fotograf. Derfor jeg anbefaler hende og linker til hende i bunden af siden.

Huggernet er faktisk ved at være en gammel dame. Ca 7-8 år gammel og under stadig udvikling/forbedring. Det er IKKE bygget på et CMS-system, men kodet af mig selv i PHP/MYSQL og lidt Javascript. Og siden er faktisk lige blevet sat sådan halvt til salg da jeg har nogle andre projekter som jeg har mere lyst til at arbejde med.

Jeg er lidt nysgerrig efter at finde ud af, hvordan du bruger geocoder i dit system så folk selv kan indtaste deres oplysninger/adresser/koordinater. Jeg regner med at folk inkl. admin, så også selv har mulighed for at rette deres oplysninger?

Har prøvet at finde ud af hvem "learner" er uden held, for evt. at kontakte dig direkte om disse ting?

Det er ellers rimelig let at gå ind under informationer og så læse om hvem der står bag siden

learner >> Jeg gik også udfra, det var noget, du havde sakset. Du må som sagt undskylde, jeg hyggede mig højt, men jeg kunne ikke lade være. Det var ikke dig, men koden jeg morede mig over.

- og så lang tid er det nu heller ikke siden, at jeg selv kunne have fundet på at sakse den slags til eget brug efter devisen: "Nåja, JavaScript er jo JavaScript - og det her ser da vældig pænt ud!"

God Jul  *<|;o)

Helt i orden du ;o)