Function con redirect all'interno

[master87tm]

Ciao a tutti,

vi posto qui una function di esempio:

Function rs_viewuser(desField)
'Creo una nuova stringa SQL
If Replace(Request.QueryString("username"), "'", "''") <> "" Then
viewuserSQL = "SELECT * FROM tbl_utenti WHERE name='" & Replace(Request.QueryString("username"), "'", "''")& "'"
End If
	
Set RS2 = Server.CreateObject ("ADODB.RecordSet")
RS2.Open viewuserSQL,cString
dim icountz
 icountz  =  0
if not RS2.eof then

if RS2("stato") = "Disabled" then
icountz = 1
disable = RS2("name")
end if
retField2 = RS2(desField)
end if
RS2.Close
Set RS2 = Nothing
	
if icountz = 1 then
adoCon.Close
Set adoCon = Nothing
response.redirect("login.asp") 	
end if
	
rs_viewuser = retField2

End Function

Per poi richiamare la funzione così:

Ciao <%=rs_viewuser("name")%>!

Avere un redirect all'interno di una function non è nulla di problematico per la stabilità del codice?

Grazie

ciao Roby

[Alethesnake]

Se distruggi correttamente gli oggetti che crei non è destabilizzante. La questione che si potrebbe sollevare (ma qui pesa anche il mero gusto personale) è che avrebbe più senso che la funzione ritornasse sempre un valore, e che il redirect lo facessi sulla base del valore ritornato (che potresti impostare a Null nel caso in cui icountz = 1 sempre per esempio).
Ripeto comunque che questa è solo un'opinione personale.

[master87tm]

Perchè all'inizio facevo i redirect dopo aver richiamato la funzione però mi ero accorto di aver lasciato in diverse pagine del sito (almeno 50) le connessioni aperte prima del redirect e per far prima avevo impostato la function così...

ti ringrazio per l'aiuto,