Variables de session, messages de bienvenue

Question

Pourquoi cela ne fonctionne-t-il pas? Mon message de bienvenue, ça ne se présente pas:

<p>Welcome <cfoutput>#Recordset1.UserID#</cfoutput>.</p> 

La variable de session sur la page de connexion que j'ai créé est:

<cflock timeout=999 scope="Session" type="Exclusive"> 
    <cfset Session.IDUsers =''> 
</cflock> 

est ce incorrect? Sur la page d'index où je suis en train d'afficher mon message de bienvenue je:

<cfquery name="Recordset1" datasource="cfGossip"> 
    SELECT * 
    FROM users 
    WHERE users.IDUsers = <cfqueryparam value="#Session.IDUsers#"> 
</cfquery> 

Je ne sais pas si cela fonctionne, ou est-il nécessaire?

Answer 1

Si vous définissez l'ID utilisateur stocké dans la session comme chaîne vide, lorsque vous l'interrogez, seuls les utilisateurs dont l'ID est la chaîne vide ne doivent pas figurer parmi les utilisateurs. Par conséquent, la requête renvoie un ensemble vide et votre page n'affiche (correctement) pas un ID utilisateur.

Comment identifiez-vous initialement l'utilisateur? Demandez-vous une base de données lors de la connexion? Conservez-vous un cookie? Lire des cartes de Tarot? Pour que cela fonctionne, à un certain moment, vous devez stocker l'ID utilisateur correct, probablement dans la session. Pour ce faire, vous devez d'abord identifier l'utilisateur. En outre, si vous utilisez CF6 +, vous n'avez probablement pas besoin de la commande cflock. Il est maintenant utilisé pour empêcher les conditions de course, CF étant désormais sans danger pour les threads.

Answer 2

On dirait que vous ne faites que commencer avec les FC, bienvenue dans la communauté.

Ma compréhension de votre code rend le regard de la structure comme ce qui suit, si je vous comprends bien:

<cfset session.idUsers = '' />
<cfquery datasource = "cfgossip" name = "recordset1">
SELECT * FROM USERS WHERE USERS.ID_USERS = <cfqueryparam cfsqltype = "cf_sql_integer" value = "#session.idUsers# />
</cfquery>
<cfoutput>Welcome #recordset1.userID#</cfoutput>

La raison de cette ne fonctionne pas parce que votre valeur session.idUsers est vide. En supposant que vous ayez un utilisateur dans votre base de données avec une valeur ID_USERS égale à 1, vous pouvez modifier le CFSET et afficher les résultats. De plus, bien que je trouve génial de vous voir utiliser CFQUERYPARAM, je vous recommande d'inclure un attribut CFSQLTYPE dans la balise chaque fois que possible pour fournir une ligne de défense supplémentaire contre les attaques par injection. Vous pouvez consulter http://livedocs.adobe.com/coldfusion/8/htmldocs/help.html?content=Tags_p-q_18.html pour voir la liste des types disponibles.

Answer 3

Y a-t-il quelque part dans votre code où vous définissez votre session.IDUsers? Vous l'initialisez comme un blanc ''. Coldfusion ne le peuplent pas pour vous. La portée de session est un endroit qui se rappellera des choses pour cet utilisateur que vous mettez là pour une période d'inactivité spécifiée, habituellement 20 minutes. Donc, j'espère, quelque part avant d'exécuter votre requête, vous avez une logique supplémentaire qui remplit cela, sinon vous demandez à la base de données d'un utilisateur nommé ''.

Ceci est juste un point de style, mais ce qui suit peut travailler mieux pour vous:

<cfset Session.IDUsers =''> 

<!--- Do something here to populate Session.IDUsers ---> 

<!--- Creates a blank query - not necessary, but can reduce errors later ---> 
<cfset Recordset1 = queryNew("UserID")> 

<!--- Populate the query from the database ---> 
<cfquery name="Recordset1" datasource="cfGossip"> 
    SELECT * 
    FROM users 
    WHERE users.IDUsers = <cfqueryparam value="#Session.IDUsers#"> 
</cfquery> 

<!--- If the query has data, use it, otherwise show generic message ---> 
<cfoutput> 
<cfif Recordset1.recordcount> 
    <p>Welcome #Recordset1.UserID#.</p> 
<cfelse> 
    <p>Welcome new user!</p> 
</cfif> 
</cfoutput> 

<!--- OR since we used queryNew("userID"), we can simplify without throwing an error. ----> 
<cfoutput> 
    <p>Welcome <cfif len(Recordset1.userID)>#Recordset1.userID#.<cfelse>new user!</cfif></p> 
</cfoutput> 

Mettre le cfoutput en dehors du bloc de paragraphe sera plus facile si vous avez des variables supplémentaires à insérer dans le texte.Quoi qu'il en soit, à moins que vous n'ayez oublié de partager un peu plus du code, je pense que le problème est que le fichier session.IDUsers est vide et doit être rempli avant la requête. J'espère que ça aide!