Tom's Guide > Forum > Programmation > Requête SQL

Requête SQL

Forum Programmation : Requête SQL

TomsGuide.com : 800 000 inscrits répondent à toutes vos questions high-tech et informatique. Pour obtenir de l'aide, inscrivez-vous gratuitement !
Créer un nouveau sujet Répondre
Mot :    Pseudo :           
 
ZoX31   11-05-2005 à 16:50:29

Salut,

j'ai un petit problème dans mon code ci-dessous :


if (isset($_GET['nom2']))
{
if (isset($_GET['prenom2']))
{
if (isset($_GET['groupe2']))
{
if (isset($_GET['sg2']))
{
$nom2 = $_GET['nom2'] ;
$prenom2 = $_GET['prenom2'] ;
$groupe2 = $_GET['groupe2'] ;
$sg2 = $_GET['sg2'] ;
$id2 = $_GET['id'] ;


$host = 'localhost' ;
$user = 'root' ;
$pass = '' ;
$nombd = '1c23html' ;
$db = mysql_connect ($host,$user,$pass) ;
mysql_select_db($nombd, $db) ;
mysql_query("UPDATE `etudiant` SET `Sous_groupe` = $sg2 , `Groupe` = $groupe2 , `Nom` = $nom2 , `Prenom` = $prenom2 WHERE `Id_etu` = $id2" ) ;
mysql_close ($db);
}
}
}
}

en fait la requête SQL ne semble pas être éxecuté et aucune erreur n'est généré si quelqu'un peut m'aider merci .


Répondre
Liens sponsorisés
Inscrivez-vous ou connectez-vous pour masquer ceci.
ZoX31   11-05-2005 à 16:53:37
- 0 +

C'est bon j'ai trouvé la solution en fait merci .

Répondre à ZoX31
Teoteed   11-05-2005 à 17:32:27
- 0 +

Ben donne là pour ce qui feront une recherche (oui ils sont rare :-D ) ;-)

@++ ;-)

Répondre à Teoteed
mr_keyser   11-05-2005 à 17:38:10
- 0 +

au passage :

Code :
  1. if (isset($_GET['nom2']))
  2. {
  3. if (isset($_GET['prenom2']))
  4. {
  5. if (isset($_GET['groupe2']))
  6. {
  7. if (isset($_GET['sg2']))
  8. {



Tu pourrai mettre aussi :

Code :
  1. if ((isset($_GET['nom2']))&&
  2.     (isset($_GET['prenom2']))&&
  3.     (isset($_GET['groupe2']))&&
  4.     (isset($_GET['sg2'])))


Trouve ca plus propre et moins long surtout...enfin c'est un détail ;-)

Répondre à mr_keyser
SiM07   11-05-2005 à 18:29:32
- 0 +

Citation :


Mr_Keyser a écrit :
au passage :

Code :
  1. if (isset($_GET['nom2']))
  2. {
  3. if (isset($_GET['prenom2']))
  4. {
  5. if (isset($_GET['groupe2']))
  6. {
  7. if (isset($_GET['sg2']))
  8. {



Tu pourrai mettre aussi :

Code :
  1. if ((isset($_GET['nom2']))&&
  2.     (isset($_GET['prenom2']))&&
  3.     (isset($_GET['groupe2']))&&
  4.     (isset($_GET['sg2'])))


Trouve ca plus propre et moins long surtout...enfin c'est un détail ;-)



C'est sur que c'est plus propre

Répondre à SiM07
mr_keyser   11-05-2005 à 20:38:13
- 0 +

Citation :


SiM07 a écrit :
[quote]
Mr_Keyser a écrit :
au passage :

Code :
  1. if (isset($_GET['nom2']))
  2. {
  3. if (isset($_GET['prenom2']))
  4. {
  5. if (isset($_GET['groupe2']))
  6. {
  7. if (isset($_GET['sg2']))
  8. {



Tu pourrai mettre aussi :

Code :
  1. if ((isset($_GET['nom2']))&&
  2.     (isset($_GET['prenom2']))&&
  3.     (isset($_GET['groupe2']))&&
  4.     (isset($_GET['sg2'])))


Trouve ca plus propre et moins long surtout...enfin c'est un détail ;-)



C'est sur que c'est plus propre
[/quote]

Moins long en tout cas! Et plus....professionel peut etre! ;-)

Répondre à mr_keyser
CRicky   11-05-2005 à 22:25:29
- 0 +

C'est plus pro et peut-être plus performant lors de l'analyse sémantique (1 seul contexte au lieu de 4) :-D

EDIT: mouaif en fait je sais pas si c'est plus rapide lol

Répondre à CRicky
Cool@IDN   11-05-2005 à 22:35:43
- 0 +

Si on parle professionalisme, securité, et lisibilité, j'aurais plutôt fais ca :

Code :
  1. <?php
  2. if( isset($_GET['nom2']) && isset($_GET['prenom2']) && isset($_GET['groupe2']) && isset($_GET['sg2']) && is_numeric($_GET["id"]) ){
  3.     $nom2 = $_GET['nom2'];
  4.     $prenom2 = addslashes($_GET['prenom2']);
  5.     $groupe2 = addslashes($_GET['groupe2']);
  6.     $sg2 = addslashes($_GET['sg2']);
  7.     $id2 = $_GET['id'];
  9.     $host = 'localhost';
  10.     $user = 'root';
  11.     $pass = '';
  12.     $nombd = '1c23html';
  13.     $db = mysql_connect ($host,$user,$pass) or die("Impossible de se connecter" );
  14.     mysql_select_db($nombd, $db) or die("Impossible d'ouvrir la base de données" );
  15.     mysql_query("UPDATE `etudiant` SET `Sous_groupe` = '".$sg2."' , `Groupe` = '".$groupe2."' , `Nom` = '".$nom2."' , `Prenom` = '".$prenom2."' WHERE `Id_etu` = '".$id2."'" ) or die("Erreur dans la requêtte SQL" );
  16.     mysql_close ($db);
  17. }
  18. ?>

Répondre à Cool@IDN
CRicky   11-05-2005 à 22:42:50
- 0 +

Il manque un ou deux commentaires ;-)

Répondre à CRicky
mr_keyser   11-05-2005 à 22:47:45
- 0 +

Citation :


Cool a écrit :
Si on parle professionalisme, securité, et lisibilité, j'aurais plutôt fais ca :

Code :
  1. <?php
  2. if( isset($_GET['nom2']) && isset($_GET['prenom2']) && isset($_GET['groupe2']) && isset($_GET['sg2']) && is_numeric($_GET["id"]) ){
  3.     $nom2 = $_GET['nom2'];
  4.     $prenom2 = addslashes($_GET['prenom2']);
  5.     $groupe2 = addslashes($_GET['groupe2']);
  6.     $sg2 = addslashes($_GET['sg2']);
  7.     $id2 = $_GET['id'];
  9.     $host = 'localhost';
  10.     $user = 'root';
  11.     $pass = '';
  12.     $nombd = '1c23html';
  13.     $db = mysql_connect ($host,$user,$pass) or die("Impossible de se connecter" );
  14.     mysql_select_db($nombd, $db) or die("Impossible d'ouvrir la base de données" );
  15.     mysql_query("UPDATE `etudiant` SET `Sous_groupe` = '".$sg2."' , `Groupe` = '".$groupe2."' , `Nom` = '".$nom2."' , `Prenom` = '".$prenom2."' WHERE `Id_etu` = '".$id2."'" ) or die("Erreur dans la requêtte SQL" );
  16.     mysql_close ($db);
  17. }
  18. ?>





Hum....on va pas etre trop pro non plus :biggrin:

Répondre à mr_keyser
okin@IDN   11-05-2005 à 22:58:22
- 0 +

Il n'y a meme pas besoin de faire :

$prenom2 = addslashes($_GET['prenom2']);
$groupe2 = addslashes($_GET['groupe2']);
$sg2 = addslashes($_GET['sg2']);
$id2 = $_GET['id'];


Mais d'inserer addslashes($_GET['sg2']); dans le insert , ca revient au meme, ligne de code en moins.

Répondre à okin@IDN
Créer un nouveau sujet Répondre
Tom's Guide > Forum > Programmation > Requête SQL
Aller à :

Il y a 2012 utilisateurs connus et inconnus. Pour voir la liste des connectés connus, cliquez ici.

Plan du forum
Forum Bestofmedia ©
2000-2009 Bestofmedia Group
Page générée en 0,438 secondes
Attention

Vous allez répondre sur un sujet resté inactif pendant plus de 6 mois.
Assurez-vous d'apporter des éléments nouveaux à la discussion avant de poursuivre.

Répondre Annuler
Liens