---
title: "Loop fel?"
type: "forum-thread"
url: "https://www.webforum.nu/amne/asp/13552-loop-fel"
topic: "ASP"
topic_url: "https://www.webforum.nu/amne/asp"
author: "sven"
published: "2002-01-04T19:09:00.000Z"
updated: "2002-01-04T19:38:00.000Z"
replies: 5
views: 165
page: 1
pages: 1
language: "sv-SE"
site: "webForum — webforum.nu"
rights: "Upphovsrätten till varje inlägg tillhör dess författare."
attribution: "Citera som: webForum, https://www.webforum.nu/amne/asp/13552-loop-fel"
---

# Loop fel?

## #1 — sven, 2002-01-04T19:09Z

Hjälp...

Får inte detta att funka. Den loopar inte genom RS utan kollar bara den första. Förmodligen bara ett tankefel från mig (stort tankefel)... med vad?

```
<%
SQL = "SELECT user2 FROM tbl_user"
set RS = db.Execute(SQL)
Do until RS.EOF
	
if Request.Form("user2") = RS("user2") then
session("finns") = "Det finns redan en användare med det namnet. Välj ett annat"
Response.Redirect("new.asp")
	
else if request.form("user2") = "" then
session("finns") = "Du måste välja ett användarnamn"
Response.Redirect("new.asp")
	
else
Response.Redirect("pass.asp")
		
end if
end if
RS.MoveNext
Loop	
%>
```

Permalänk: https://www.webforum.nu/p/13552

## #2 — LarsG, 2002-01-04T19:14Z

Det är bättre att lägga in villkoret i sql-satsen.

```
<%
if request.form("user2") = "" then
session("finns") = "Du måste välja ett användarnamn"
Response.Redirect("new.asp")
end if

SQL = "SELECT user2 FROM tbl_user where user2 = '" & Request.Form("user2") & "'"

set RS = db.Execute(SQL)
exists = not rs.eof
rs.close
db.close
if exists then
session("finns") = "Det finns redan en användare med det namnet. Välj ett annat"
Response.Redirect("new.asp")
end if

Response.Redirect("pass.asp")

%>
```

Permalänk: https://www.webforum.nu/p/302642

## #3 — jorgen, 2002-01-04T19:15Z

Hej, ställ vilkoret i SQL-frågan istället. Det är bättre.

mvh Jörgen :)

```
<%
SQL = "SELECT user2 FROM tbl_user " & _
" WHERE user2 = '"& request.form("user2") &"' "
set RS = db.Execute(SQL)

If RS.EOF Then

'#-- användanamnet fanns inte --

else

Do until RS.EOF

'#-- användanamnet fanns --

RS.MoveNext
Loop 
End If
%>
```

Permalänk: https://www.webforum.nu/p/302643

## #4 — sven, 2002-01-04T19:26Z

Tackar och bugar....

Undrar bara. Jörgen. Varför är det bättre så som du föreslår?

Permalänk: https://www.webforum.nu/p/302644

## #5 — jorgen, 2002-01-04T19:31Z

Hej, du behöver hämta mindre information från databasen om du ställer villkoret i SQL-frågan. Då får du nämligen bara den information du behöver.

Observera att jag menade inte att min kod var bättre än LarsG:s, (vi skrev våra inlägg samtidigt) utan av den anledning jag skrev ovan. Precis som LarsG också påpekade.

mvh Jörgen :)

Permalänk: https://www.webforum.nu/p/302645

## #6 — sven, 2002-01-04T19:38Z

Oppssss...

Såg inte att LarsG hade samma förslag :r

Men nu fattar jag i alla fall varför det är bättre att ställa vilkoret i SQL satsen.

Permalänk: https://www.webforum.nu/p/302646

---

Tråden på webben: https://www.webforum.nu/amne/asp/13552-loop-fel