Discussion :

[longplaying]

Bonjour,

Pour mes cours je dois passer par PHP pour ajouter, supprimer et modifier une base de données créée antérieurement.
Je suis bloquée pour la syntaxe de mon script qui est la suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
<?php 
	$connexion =  mysql_connect('localhost','root','');
			if(!$connexion) {
				echo'erreur connexion my sql : ', mysql_error();
			}
	
	$res = mysql_query('use tp_hopital');
			if(!$res) {
				echo'erreur res my sql : ', mysql_error();
			}
	$res = mysql_query('SET NAME utf8');
	
	$nom = $_REQUEST ['Nom'];
	$numP = $_REQUEST ['NumP'];
	$chambre = $_REQUEST ['Chambre'];
	$dateA = $_REQUEST['dateA'];
	
	$sql=mysql_query("INSERT INTO patient(nomP, numP, numCH, dateA) VALUES (\''.$nom.'\',\''.$numP.'\',\''.$chambre.'\',\''.$dateA.';)"); 
		  
	$requete=mysql_query($sql); 
			if(!$requete){
				echo'erreur requete my sql : ', mysql_error();
			}

?>

Lorsque je fais le test dans le formulaire, j'ai une erreur en retour qui me dit que j'ai une erreur par rapport au Query qui serait vide ??
J'ai essayé de modifier pour voir ce que ça faisait (mettre une virgule au lieu du point virgule, ne rien mettre... C'est toujours le même problème... )
Merci de m'éclairer :)

[Séb.]

Ton script est mauvais :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$sql=mysql_query("INSERT INTO patient(nomP, numP, numCH, dateA) VALUES (\''.$nom.'\',\''.$numP.'\',\''.$chambre.'\',\''.$dateA.';)");

$requete=mysql_query($sql);

Tu envoies le résultat de mysql_query( ) (ressource/booléen, cf. doc), à un autre mysql_query( ). mysql_query( ) n'accepte qu'une chaîne de caractères représentant une requête SQL.


Astuce : pour un script plus clair et facilement débugable voilà comment je procède :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
$sql = 'INSERT INTO patient(nomP, numP, numCH, dateA) '
     . 'VALUES("%s", "%s", "%s", "%s")' ; // Types à adapter selon besoins
 
$sql = sprintf($sql, $nom, $numP, $chambre, $dateA) ; // Données à protéger avec mysql_real_escape_string( )
 
$rs = mysql_query($sql) ;
 
if ( $rs === FALSE ) { // Erreur ?
    echo '<p>', mysql_error( ), "</p><p>$sql</p>" ;
    // ...
}
 
// ...

[RunCodePhp]

Salut

Une autre solution, qui n'a besoin d'aucune astuce, c'est utiliser PDO (ou MySQLi, Objet si possible) qui est tournée vers le futur alors que les fonctions mysql_truc_machin n'est plus maintenue, donc qu'on peu qualifier de pièce de musée.

En utilisant PDO et en faisant des requêtes préparées, il n'y a plus à se prendre la tête à échapper les données, c'est géré automatiquement.

Exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
<?php
// Connexion
try {
    $options = array(
        PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
        PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8'
    );
    $dbh = new PDO('mysql:host=localhost;dbname=nom_de_la_bdd', 'root', '', $options);
}
catch (PDOException $e) {
    echo 'Erreur : '.$e->getMessage();
    exit();
}
 
 
 
try {
    $sth = $dbh->prepare('INSERT INTO patient(nomP, numP, numCH, dateA)
    VALUES(:nomP, :numP, :numCH, :dateA)');
 
    $sth->bindValue(':nomP', $nom, PDO::PARAM_STR);
    $sth->bindValue(':numP', $numP, PDO::PARAM_STR);
    $sth->bindValue(':numCH', $chambre, PDO::PARAM_STR);
    $sth->bindValue(':dateA', $dateA, PDO::PARAM_STR);
    $sth->execute();
}
catch (PDOException $e) {
    echo 'Erreur : '.$e->getMessage();
}
?>

Si tu préfères rester avec les fonctions mysql_*, apparemment il manque l'appel à la fonction mysql_select_db() pour sélectionner la Bdd où sera exécuté les requêtes.

[Séb.]

il manque l'appel à la fonction mysql_select_db()

Bien vu.

[ABCIWEB]

Une autre solution, qui n'a besoin d'aucune astuce, c'est utiliser PDO (ou MySQLi, Objet si possible)

Oui enfin le mode procédural de mysqli possède les même propriétés que le mode objet. C'est plus par homogénéité avec le reste du code qu'on peut préférer le mode objet.
Je dis cela pour ne pas décourager d'utiliser mysqli en mode procédural car c'est parfois un excellent moyen de migrer de mysql vers mysqli à peu de frais puisque la syntaxe est presque identique (faut juste parfois modifier l'ordre des paramètres et certains deviennent obligatoires).

[phicarre]

Justement existe t'il une marche à suivre pour passer de mysql à mysqli:
- au niveau de code PHP
- au niveau de mysql

[ericd69]

salut,

pour le passage de mysql à mysqli procédural... aucun traumatisme, c'est quasi ment la même syntaxe c'est d'ailleurs pensé pour

pareil pour passer de mysqli objet à PDO...

perso je te conseille de directement passer à PDO qui est multi SGBD et langage vu que tu le retrouves implémenté en C/C++ notamment...

la programmation objet te laisse le choix de te créer un connecteur sur mesure laissant apparaitre plus ou moins de méthodes selon que tu utilises l'héritage ou une encapsulation dans une classe maison

déjà tu utilises PDO directement pour faire ce que tu as besoin et te faire la main... ensuite selon tes connaissances en programmation objet tu encapsules ou dérive les appels à PDO dans une classe qui simplifiera l'écriture de ton code ensuite...