Vill inte räkna

SQL-syntaxen är korrekt och kolumnen borde vara integer(i alla fall numerisk), men det jag sitter och undrar på är att du använder smclrs när du gör execute. När du bygger din SQL så använder du ju den som om den vore ett recordset.

Hur skulle du skriva?

Jag har bara kopierat annan text från scriptet.

När man gör en execute så behöver man ett connectionobject (eller commandobject går också bra)

  set connection = server.createobject("adodb.connection")

  connection.open "dsn=..."
  set smclrs = ...

  sqlStr = SQL = "UPDATE smclads SET visad_ggr=visad_ggr+1 WHERE id=" & smclrs("id")
  connection.execute sqlStr

ungefär så

Jag har skrivit det innan, men ville posta för mycket här bara... :-)

Men den plussar ändå inte?

Nu är kolumnen int, men det händer ändå inget.

Ok, först kan du lägga in en koll på att någon post verkligen uppdateras

connection.execute sqlStr,rows,128
if  rows = 0 then
    response.write "Inga rader uppdaterades" & <br>
end if

Ett annat problem kan vara att det ligger null och inte 0 i kolumnen visad_ggr och i så fall så kommer det att förbli null då null + 1 blir null.

Man kan kontrollera om ett fält är null i SQL med villkoret

columnName is null

jag använder Enterprise Manager för att kolla på databasen. De gamla annonserna har ju fått värdet null, men den nya som jag gjort, och bara kollar har värdet 0. (Alla nya som sätts in får det)

Jag har även lagt in samma script på om man svarar på annonsen. Det funkar perfekt. Den visar hur många som svarat på annonsen.

Hur är det nu? Uppdateras det någon rad? Om inte så kanske du kan lägga upp lite mer kod så ...

Jag använder precis samma kod på båda scripten, på det ena stället så funkar det.

Kan det ha nånting att göra med att jag lägger koden i ett case? I det fall det inte fungerar alltså.

Tja, då låter det ju mer som om man aldrig utför den kod där man gör update. Dvs att det är någon glitch i logiken i din case.

Hur ska man bära sig åt då?

Kan man inte ha
<%
kod
%>

var man än vill ha den?

Typ t.om i en tabell? Nu har jag ju inte det, men som exempel...

Jo, det går ju bra att stoppa in kod inline så där. Det man bör tänka på är att VB är känsligt för radbrytningar, dvs man kan inte dela upp statement på flera rader utan att ha fortsättningstecken och inte heller kan man ha flera statements på samma rad utan att ha ett : emellan.

Men vad är det som inte funkar då?

lite tradigt när det inte går som man vill! :-/

Du verkar vara ovilliig att visa någon kod . Har lite svårt att något vettigt annars.

Det är bara det att det är fetmycket kod! Det är ett köpt script som jag blivit ombedd att lägga till lite nya funktioner i.

ska jag maila dig det istället? Det är för mycket att visa här.

Ok, men jag hinner nog inte titta så mycket i kväll. Måste ju jobba lite också.
Min mailadress finns i min profil,
klicka på ikonen med ansiktet och frågetecknet.

Okej, jag fick ditt mail. Efter det så har jag inte lyckats starta min mailklient (jag hann dock ta ut en papperskopia) så jag får lägga ett svar här.

Det som jag kan säga är ju att i början på din sub så har du satt upp
on error resume next
men du har ju ingen egen felhantering vid dina anrop där du gör din update. Det kan ju gå fel där så du kan ju lägga in en on error goto 0 innan.

Sen så undrar jag varför du öppnar en ny connection, du har ju redan en (smcldataconn).

Annars så får du väl försöka lägga in lite traceutskrifter på lämpliga ställen.

Samamnfattningsvis, så har jag inte sett något definitivt fel.

Hmmm. Skickades mailet som HTML? Sorry! Tänkte inte på det!!

Öppnade en ny connection senare för att prova. Som jag sa.

Kan du rekommendera hur man ska skriva för att det ska bli rätt? Kan man lägga det innan in error resume next?