Führen Sie SQL Server-Funktion für jede Zeile in der Tabelle

Ich habe eine Funktion, die iteriert "für jede Zeile" in der Tabelle. Wenn es ausgeführt wird, es sollte in jeder Zeile, ziehen Sie die zugehörigen Werte für die Zeile und führen die Funktion aus, die in Kurven, gibt ein Ergebnis zurück und aktualisiert die richtige Zeile mit dem richtigen Wert. Was geschieht, ist, dass es ausgeführt wird, und die Rückgabe der Wert der letzten Zeile und aktualisiert die Zeile mit diesem Wert. Irgendwelche Ideen was ich falsch mache?

SELECT 
     RowNum = ROW_NUMBER() OVER(ORDER BY ID)
     ,*
INTO #Geo
FROM DDDG

DECLARE @MaxRownum INT
SET @MaxRownum = (SELECT MAX(RowNum) FROM #Geo)

DECLARE @Iter INT
SET @Iter = (SELECT MIN(RowNum) FROM #Geo)

WHILE @Iter <= @MaxRownum
BEGIN
     SELECT *
     FROM #Geo
     WHERE RowNum = @Iter

    DECLARE @address nvarchar (100);
    Select @address =  Ad1 from DDDG where id = @Iter;

    DECLARE @state nvarchar (100);
    SET @state = 'FL';

    DECLARE @zip nvarchar (100);
    SET @zip = '33142';

    DECLARE @city nvarchar (100);
    SET @city = 'Miami';

    DECLARE @nation nvarchar (2);
    SET @nation = 'us';

    DECLARE @g geography;

    WAITFOR DELAY '00:00:00.050'
    SET @g = dbo.Geocode(@nation, @state, @city, @zip, @address);

    Update DDDG 
    Set Lat = @g.Lat 
    where id = @Iter;

    Update DDDG 
    Set long = @g.Long 
    where id = @Iter;

     SET @Iter = @Iter + 1
END

DROP TABLE #Geo

Update .....

danke an alle... mich hat es fast funktioniert.. Nach eine Hölle der eine Menge von Versuch und Irrtum habe ich herausgefunden, dass ich brauchte, um zu ändern Sie die folgenden

Update DDDG Set Lat = @g.Lat where id = id;
Update DDDG Set long = @g.Long where id = id;

Ich habe nur ein problem jetzt. Wenn ich die Zeilen löschen die RowNum-und die @Iter nicht mehr überein. gibt es eine Möglichkeit, dies zu beheben problem andere als die Neuerstellung der Tabelle jedes mal?

Sollten Sie überarbeiten die Logik zu beseitigen die Schleife...SQL ist eine set-basierte Sprache und führt sehr schlecht wie diese. Es Griffe viel besser, wenn Sie Locker die lineare Linie des Denkens, ein Skript zu aktualisieren, werden alle Zeilen, keine Schleife.
Sind Sie sicher, dass der Geocode-Funktion nicht einfach wieder die gleiche Breite und Länge für jede Adresse?
Ich bin sicher, (Dr. Wily Frage)und da für einen besseren Weg habe ich auf diese für nur 6 Monate oder so.. wenn Sie mir zeigen, einen besseren Weg.... im all für Sie

InformationsquelleAutor user3266908 | 2014-02-04

1
```
WITH CTE_DDD(ID,Ad1)
AS
(
  SELECT ID, Ad1 FROM DDD
)
UPDATE d 
SET d.lat = dbo.Geocode(@nation, @state, @city, @zip,Ad1).lat, long = dbo.Geocode(@nation, @state, @city, @zip,Ad1).long 
FROM DDD d 
WHERE d.id = ID
```
Geänderte Abfrage nach immer downvotes von Kollegen für das oben genannte:
```
WITH CTE_DDD AS
(
  SELECT * FROM DDD
)
UPDATE CTE_DDD SET lat = (SELECT lat from dbo.Geocode(@nation, @state, @city, @zip,Ad)), long = (SELECT long from    dbo.Geocode(@nation, @state, @city, @zip,Ad))
```
Ich würde Zustimmen, dass die CROSS APPLY-ist eine bessere Lösung als würde es Sie rufen die Funktion einmal pro Zeile.
- OMG, Ihr die größten. Dies ist das größte Stück code, das ich je gesehen habe in meinem ganzen Leben.. ich würde Ihnen 5 Sterne, ich könnte... das bringt eine Wochen lange Suche ein Ende.... Im neu um all dies, die Websites, die Sie empfehlen, ein noob, oder Bücher?
- Vielen Dank und froh, dass es geholfen hat! W3Schools-website ist gut zu lernen Sie die grundlegenden Stoffe aus; und natürlich würden Sie schöne Artikel über das internet geschrieben von SQL Gurus. Beginnen Sie mit Joins Select -, Insert -, Delete -, Update-Gruppierung, CTE, row_number()...ich habe in letzter Zeit entdeckt, dass aktiv auf Foren wie diesem hier, ist guter Weg, um zu lernen, so würde ich empfehlen, es auch zu Euch.
- Sorry, aber wie kommt dieser auch arbeiten?, Sie sind nicht die Bereitstellung einer Tabelle alias für Ad1, so sollte es eine Fehlermeldung zu geben. Und warum würden Sie brauchen, um machen einen JOIN mit sich selbst, es ist viel viel mehr einfach nur update der Tabelle
- er ist nicht in der Tabelle zu sich selbst (es sei denn, die Antwort bearbeitet wurde). Er ist die Definition eines Allgemeinen Tabellenausdrucks, die überhaupt nicht genutzt. Und die WHERE d.id = ID ist Total überflüssig...
- Sie haben Recht, natürlich. Die id = ID warf mich ab und ich nahm an, es war ein JOIN
- So, nach der Lektüre Ihrer Bearbeiten, warum brauchen Sie einen CTE, wenn die CTE hat eigentlich die gleichen Daten wie die Tabelle?
- du hast Recht, my bad, ich habe versucht, CTE-für-Zeile-Weise update. Aber ich bekam zu wissen, dass auch direkt ein update auf die Tabelle hätte das gleiche Ergebnis wie @Lamak die erste Lösung. Ich werde die Lösung hier für andere zu wissen, wie Sie nicht missbrauchen, CTE. Vielen Dank an Euch. Eine Klarstellung von Ihnen: was ist Rückgabetyp der Geocode-Funktion, ist es eine Tabelle? update-Anweisung nicht akzeptieren diese Art von Zugriff auf lat Feld: dbo.GeoCode('us','FL','Miami','33142',Ad1).Lat
InformationsquelleAutor Dipendu Paul
7

Ich kann etwas fehlt, aber warum brauchen Sie eine Schleife um alle?:
```
UPDATE DDDG
SET Lat     = dbo.GeoCode('us','FL','Miami','33142',Ad1).Lat,
    Long    = dbo.GeoCode('us','FL','Miami','33142',Ad1).Long;
```
Ist es auch möglich zur Verbesserung die Abfrage weiter reduzieren die Anzahl der Anrufe zu dbo.GeoCode nur einen pro Zeile:
```
UPDATE t
SET Lat     = g.GC.Lat,
    Long    = g.GC.Long
FROM DDDG AS t
CROSS APPLY (
    SELECT dbo.GeoCode('us','FL','Miami','33142',Ad1) AS GC
) AS g;
```
- Ich habe es ausprobiert, aber beim laufen so aktualisiert er alle Datensätze mit der letzten Adresse in der Tabelle
- gut, Sie sind nicht die Angabe einer Weise zu erfahren, was die letzten - Adresse für eine Zeile
- diese Abfrage würde aktualisieren Sie die gesamte Tabelle als WHERE-Bedingung fehlt, und nur die Letzte Ad1 lat und long würde aktualisiert werden, werden alle Datensätze.
- gibt es nicht eine WHERE Bedingung in der Frage, und op ist das Durchlaufen der gesamten Tabelle
- user3266908 verwendet hat where id = @Iter in der Abfrage.
- Das ist, weil er gerade eine Schleife. Er ist mit diesem durch zu gehen, jeder einzelne von den ids der Tabelle. Ich stellte eine set-basierte Lösung, daher auch der Grund, warum ich nicht mit einem WHERE
- vielen Dank an alle... mich hat es fast funktioniert.. Nach eine Hölle der eine Menge von Versuch und Irrtum habe ich herausgefunden, dass ich brauchte, um zu ändern, Folgen Sie
- einfach nur neugierig, was ist der return-Typ der Geocode. Ich habe vermutet, es werden Tisch, hab ich Recht?
InformationsquelleAutor Lamak

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.