Septima / qgis-addresstoolsdk

QGIS Processing plugin for working with Danish addresses
GNU General Public License v2.0
4 stars 1 forks source link

Når der ikke er adresseoplysninger i rækken, returneres en tilfældig adresse (husnummer) #7

Closed drlind closed 3 years ago

drlind commented 3 years ago

Når der ikke er adresseoplysninger i rækken, returneres en tilfældig adresse (husnummer)

I denne csv-fil geokodes der på en fuld adresselinje der frembringes sådan: _concat("field_2", ', ', "field_3", ' ', "field4")

To af de nederste rækker er tomme (datasjusk, men det er vel hvad geokoderen skal kunne klare) - alligevel foreslås en tilfældig adresse i det pågældende postnummer.

image

CW-RED_AdresserTST02.txt

AsgerPetersen commented 3 years ago

Tak for issuet.

Det, der sker her, er, at udtrykket concat("field_2", ', ', "field_3", ' ', "field_4") ikke bliver null, hvis et eller flere fields er null. I stedet genereres en streng a la , Holbæk eller ,. Pluginet prøver ikke at være klogt, det vil vi gerne overlade til DAWA, så det sender din tekst afsted, og så gætter DAWA altså på et-eller-andet.

Hvis man har felter, der er null, og ikke ønsker at DAWA skal gætte i det tilfælde, så kan man feks bruge udtrykket: "field_2"||', '|| "field_3"|| ' '|| "field_4", da resultatet af dette er null, hvis blot et af felterne er null: image

Det er nok lidt afhængigt af case, om man ønsker den ene eller anden virkemåde. Så jeg ved ikke, om vi kan ændre i vejledningen uden at gøre det unødigt kompliceret for flertallet.

drlind commented 3 years ago

1) Det burde nok fremgå af vores vejledning at plugin'en ikke er egnet til "snavsede" adresser

2) Dette tip burde måske nævnes i vores vejledning Hvis man har felter, der er null, og ikke ønsker at DAWA skal gætte i det tilfælde, så kan man feks bruge udtrykket: "field_2"||', '|| "field_3"|| ' '|| "field_4", da resultatet af dette er null, hvis blot et af felterne er null: