Else-sats som vägrar att _inte_ köras

Byt querystring mot form.

Varför? Jag vill ju kolla om det är form-värden som skickas, eller om det är querystrings. Man kan skicka informationen på två olika sätt nämligen.

If cstr(Request.Form ( "minKnapp" )) = "knappValue" And len(Request.Form ( "fld1" )) > 0 And len(Request.Form ( "fld2" )) > 0 Then
  ' Gör nåt.
ElseIf len(Request.QueryString ( "fld1" )) > 0 And len(Request.QueryString ( "fld2" )) > 0 Then
  ' Gör nåt annat.
Else
  Response.Redirect "Felhantering.asp"
End If

Har varit med om att det blivit knas och skriva "" om det är fel subtyp. Eftersom fältet antagligen kan vara null kan man inte använda cstr, för då blir det fel, det är därför säkrare med len.

mvh Jörgen

Nej, det kanske inte var så relevant svar.

Om du gör en utskrift på allt då

response.write "k:" & Request.Form ( "minKnapp" ) & ":k"
response.write "f1:" & Request.Form ( "fld1" ) & ":1f"
response.write "f2:" & Request.Form ( "fld2" ) & ":2f"

response.write "q1:" & Request.querystring ( "fld1" ) & ":1q"
response.write "q2:" & Request.querystring ( "fld2" ) & ":2q"

eller du kanske har gjort det?

Redan försökt att omvandla till andra typer och att använda Len (). Hjälper inte ett jota, tyvärr!

Grejjen är den, att om man tar bort hela Else-blocket, så stämmer allt. Kan inte köra så dock, eftersom att det blir en säkerhetslucka...

Jag har aldrig varit med om något liknande tidigare, det här är helskumt!

Red.} LarsG> Japp, det har gjorts... Alla värden stämmer, dom finns där.

Det är något riktigt allvarligt fel på någonting!

Följande kod skriver ut "ID:et finns":

If Trim ( cStr ( Request.Form ( "replyThread" ) ) ) = "Skicka!" And Len ( Request.Form ( "threadID" ) ) > 0 And Len ( Request.Form ( "forumID" ) ) > 0 Then
    id = Request.Form ( "threadID" )
ElseIf Request.QueryString ( "threadID" ) <> "" And Request.QueryString ( "forumID" ) <> "" Then
&nbsp; &nbsp; id = Request.QueryString ( "threadID" )
End If

If id <> "" Then
&nbsp; &nbsp; Response.Write "ID:et finns"
Else
&nbsp; &nbsp; Response.Write "ID:et finns INTE!"
End If
Response.End

Följande kod redirectar till Felhantering.asp:

If Trim ( cStr ( Request.Form ( "replyThread" ) ) ) = "Skicka!" And Len ( Request.Form ( "threadID" ) ) > 0 And Len ( Request.Form ( "forumID" ) ) > 0 Then
    id = Request.Form ( "threadID" )
ElseIf Request.QueryString ( "threadID" ) <> "" And Request.QueryString ( "forumID" ) <> "" Then
&nbsp; &nbsp; id = Request.QueryString ( "threadID" )
End If

If Trim ( cStr ( id ) ) = "" Then
&nbsp; &nbsp; Response.Redirect "Felhantering.asp"
End If

Det är helt sjukt!
Villkoren stämmer, om man går in på sidan med värden i querystringen, men inte via formulär med POST!

Snart ger jag upp ASP:en helt... "#%¤#&%/&/%.

Verkar vara ett väldigt krångligt tillvägagångsätt du har valt om man kan posta från två olika fält. Lägg till en "action" variabel för varje forumlär i så fall:

<INPUT TYPE="hidden" NAME="action" VALUE="namn_för_querystrings">

<INPUT TYPE="hidden" NAME="action" VALUE="namn_för_form">

Sedan tar du emot det på följande vis:

If Request("action") = "namn_för_XXX" Then
 ' Blä
Else
 ' Bla
End If

Grejjen är den att man inte kan posta information på två olika sätt, utan endast med POST. GET används när man klickar på en länk från en annan sida.

Självklart skulle jag kunna göra som du visar ändå, men jag tycker inte att det blir lättare. Mindre koll på vad som händer, känns det som... Min kod BÖR ju fungera, det är något klurigt med ASP-tolken/motorn bara.

Nä, nu gör du som jag säger!

Så här är väl inte så krångligt:

Select Request.Method
Case "POST"
 ' Gör vad du nu ska göra med det postade forumläret.
Case "GET"
 ' Gör nåt med länken.
End Select

Då måste jag göra om en massa, ju!

Jag väntar på LarsG, Vide, emission, @nders eller ett smartare svar av dig.

Ska se vad jag kan göra... Finns många olika sätt att göra på.

Du har prövat att startat om systemet? Tycker det verkar väldigt konstigt att det inte funger ar om du verkligen har skrivit som du skriver.

Jadå, jar har provat ALLT! Nästan.

Jag får väl bygga om skiten ändå...

<%
If Len(Trim(Request.Form( "fld1" ))) > 0 And Len(Trim(Request.Form( "fld2" ))) > 0 Then
  ' Gör nåt!!!
Else
  Response.Redirect "Felhantering.asp"
End If
%>

???

r/ Funkar i resten av världen vad jag vet i alla fall

Då måste jag göra om en massa, ju!

Och jag som trodde det var bättre att ha en bra stuktur i min kod...

Jag väntar på LarsG, Vide, emission, @nders eller ett smartare svar av dig.

Ska se vad jag kan göra... Finns många olika sätt att göra på.

Och det bästa är återigen att göra det så strukturerat och enkelt som möjligt. Med andra ord är det bäst att skriva om koden.

Hm, du vet ju inte hur resten av min kod ser ut, så det där håller egentligen inte.

Men självklart är det bra med strukturerad kod, det har jag alltid. Däremot har jag ovanan att försöka hålla ned på antalet olika dokument, och använda samma dokument till flera saker. Det är därför det här problement uppstod...

Jag som verkligen inte vill använda Pace's alternativ blir väl tvungen till det nu. Jäbbrans.

Är du säker på att det inte är något annat på sin sida spm gör att du hamnar på Felhantering.asp då?
Kanske en dum fråga men ofta brukar ju felen vara enkla men man ser dom inte.

100% säker. Har testat att skicka till en annan sida i else-blocket, och då skickas jag dit.

SPiN: Läs och begrunda!

Bah. Det där är för sissys! Jag använder egna COM-objekt och templates! Det är grejjer, man får riktigt bra koll på sina sidor, dokumenten som man skriver är 100% pure ASP ( nästan ) och vilken kontroll!