Erreur entre PHP et BD mysql

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Erreur entre PHP et BD mysql

lofo7

Bonjour,
j'ai créé ma base de données et un formulaire d'inscription à un espace membre. Alors lorsque je remplis le formulaire et que je le soumets, j'obtiens cette erreur :

Erreur : Column count doesn't match value count at row 1

Mon problème je pense vient de l'insertion de données, dans ma base, de mon formulaire de type "radio". Je demande si les membres veulent s'inscrire à la newletter donc ils ont une case à cocher.

voici ma base :

Code :

CREATE TABLE `membres` (
`id` mediumint(9) NOT NULL auto_increment,
`session` text NOT NULL,
`pseudo` text NOT NULL,
`passe` text NOT NULL,
`email` text NOT NULL,
`ville` text NOT NULL,
`infolettre` varchar(255) NOT NULL default '0',
KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=1 ;

le script pour l'inscription :

Code :

<?php
session_start();
require_once('config.php');
$titre_page='Inscription';
include_once('haut.php');
?><form method='post' action='verification.php' enctype='multipart/form-data'>
<table width="400" border="0" align="center">
<tr>
<td colspan="2" class="menu"><div align="center">Inscription</div></td>
</tr>
<tr>
<td width="195"><label for="pseudo">Pseudo : </label></td>
<td width="195"><input name="pseudo" type="text" id="pseudo" /></td>
</tr>
<tr>
<td><label for="email">E-mail : </label></td>
<td><input name="email" type="text" id="email" /></td>
</tr>
<tr>
<td><label for="mdp">Mot de passe : </label></td>
<td><input name="mdp" type="password" id="mdp" /></td>
</tr>
<tr>
<td><label for="mdp2">Confirmation du mot de passe : </label></td>
<td><input name="mdp2" type="password" id="mdp2" /></td>
</tr>
<tr>
<td><label for="ville">Ville : </label></td>
<td><input name="ville" type="text" id="ville" /></td>
</tr>
<tr>
<td><label for="infolettre">S'inscrire à l'infolettre ? : </label></td>
<td>Oui<input name="infolettre" type="radio" value="1" />
         <input name="infolettre" type="radio" value="0" />Non</td>
</tr>
<tr><td> </td></tr>
<tr>
<td><div align="right">
<input name="Envoyer" type="submit" value="Valider" />
</div></td>
<td><input name="Effacer" type="reset" id="Effacer" value="Effacer" /></td>
</tr>
</table>
</form>
</div>
<center><a href="index.php">Retour</a></center>
<?php
include_once('bas.php');
?>

et le script de vérification :

Code :

<?php
session_start();
require_once('config.php');
$titre_page='Inscription';
include_once('haut.php');
$email=formulaires($_POST['email']);
$pseudo=formulaires($_POST['pseudo']);
$mdp=formulaires($_POST['mdp']);
$mdp2=formulaires($_POST['mdp2']);
$ville=formulaires($_POST['ville']);
$infolettre=formulaires($_POST['infolettre']);
//// VERIFICATIONS BANALES ////
if(!$mdp || !$mdp2 || strlen($mdp) < 5)
{
echo'Votre mot de passe est invalide. Il doit faire plus de 5 caractères et ne contenir que des caractères alphanumériques (A-Z, a-z, 0-9)<br /><a href="inscription.php" onClick="history.back()">Retour</a>';
include_once('bas.php');
return FALSE;
}
if($mdp!=$mdp2)
{
echo'Les deux mots de passe rentrés ne sont pas identiques<br /><a href="inscription.php" onClick="history.back()">Retour</a>';
include_once('bas.php');
return FALSE;
}
if(!$pseudo || strlen($pseudo) > 15)
{
echo'Votre pseudo est invalide. Il doit faire moins de 15 caractères et ne contenir que des caractères alphanumériques (A-Z, a-z, 0-9)<br /><a href="inscription.php" onClick="history.back()">Retour</a>';
include_once('bas.php');
return FALSE;
}
if(!$email)
{
echo'Veuillez insérer un email valide<br /><a href="inscription.php" onClick="history.back()">Retour</a>';
include_once('bas.php');
return FALSE;
}
//// VERIFICATIONS DES EXISTANCES ////
$reponse_mail=mysql_query("SELECT email FROM membres WHERE email='$email'" ) or die ('Erreur : '.mysql_error()); //verification si e-mail existe déjà
$count_mail=mysql_num_rows($reponse_mail);
if($count_mail == 1)
{
echo'Cet e-mail existe déjà.<br /><a href="inscription.php" onClick="history.back()">Retour</a>';
include_once('bas.php');
return FALSE;
}
$reponse_pseudo=mysql_query("SELECT pseudo FROM membres WHERE pseudo='$pseudo'" ) or die ('Erreur : '.mysql_error()); //verification si pseudo existe déjà
$count_pseudo=mysql_num_rows($reponse_pseudo);
if($count_pseudo == 1)
{
echo 'Ce pseudo existe déjà.<br /><a href="inscription.php" onClick="history.back()">Retour</a>';
include_once('bas.php');
return FALSE;
}
for ($ligne=0;$ligne<30;$ligne++) //Création d'un identifiant aléatoire
{
@$session.=substr('0123456789AZERTYUIOPMLKJHGFDSQWXCVBN',(rand()%(strlen('0123456789AZERTYUIOPMLKJHGFDSQWXCVBN'))),1);
}
$mdp=md5($mdp); //Codage du mot de passe
mysql_query("INSERT INTO membres VALUES ('', '$session', '$pseudo', '$mdp', '$email')" ) or die ('Erreur : '.mysql_error()); //insertion dans la bdd
echo'Merci de vous vous etre inscrit sur '.$nom_du_site.'<br /><a href="index.php">Identifiez vous !</a>';
include_once('bas.php');
?>

Je vois pas d'où peut venir l'erreur. Si vous avez une idée...

Merci
Max

Publicité

MagicBuzz

ouais, moi j'ai ça comme email :

'; drop table membres; --

Message cité 1 fois

lofo7

ok je te remercie, j'ai modifié. Pour le moment je fais des tests avant de l'insérer concrètement sur le site donc ça ne risque pas grand chose.

dwogsi

Défaillance cérébrale...

MagicBuzz a écrit :

ouais, moi j'ai ça comme email :

'; drop table membres; --

Je reconnais qu'il n'est pas pruduent de laisser ce genre d'ouverture mais arrêtez de toujours répondre ça!

Citation :

mysql_query() envoie une seule requête (les requêtes multiples ne sont pas supportées) à la base de données courante sur le serveur associé avec le link_identifier spécifié.

http://fr.php.net/mysql_query

Message cité 1 fois

---------------
-- Debian -- Le système d'exploitation universel | Le gras c'est la vie! | /(bb|[^b]{2})/

MagicBuzz

dwogsi a écrit :

Je reconnais qu'il n'est pas pruduent de laisser ce genre d'ouverture mais arrêtez de toujours répondre ça!

Citation :

mysql_query() envoie une seule requête (les requêtes multiples ne sont pas supportées) à la base de données courante sur le serveur associé avec le link_identifier spécifié.

http://fr.php.net/mysql_query

ben à ce moment, dans le login, le truc qui marche dans 99,9% des cas...

' or 1 = 1 or '

Y'as d'autres mécanismes que xxxxx_query(), qui permettent de :
1/ ganger en performances
2/ gagner en lisibilité
3/ ne pas polluer son code à grand coups d'échappement automatiques des quotes (que ce soit real_escape() ou magicquotes)
4/ permettre des contrôles de type en amont de l'exécution
5/ gagner en portabilité (pour rappel, \' N'EST PAS LE CARACTERE D'ECHAPPEMENT DE LA QUOTE DANS LA NORME SQL !)

Donc UTILISEZ LES !

mysql_query() ça date d'un autre âge, à l'époque où MySQL ne savait même pas faire une jointure "inner join" sans planter. Il serait temps d'évoluer un peu. Quandun gars demande de l'aide sur le forum pour du HTML et se pointe avec un <table> ou un <font color="red">, tout le monde lui vole dans les plumes en disant que c'est de la merde, qu'il faut plus l'utiliser et tout ça.

Alors je ne cèderai pas, quand je vois un accès à une base écrit comme une merde des années 70, ça me fait hurler, et vous pourrez bien dire tout ce que vous voudrez, je ne changerai pas ma façon de dire ce que je pense

Message cité 1 fois
Message édité par MagicBuzz le 21-11-2007 à 09:41:36

soulmanto

Chat Noir replica

là ça m'intéresse... En l'occurence, j'utilise un wrapper pour tout ce qui est accès à la base requêtes et tout le tintouin, mais qu'est-ce qui peut être significativement plus rapide qu'un mysql(i)_query() ou mysql(i)_real_query()?

MagicBuzz

http://maximilian.developpez.com/m [...] tatements/

Y'a peut-être plus de détails dans la doc PHP officielle.
Je sais qu'il y a aussi une lib (standard ?) qui permet d'ajouter un iveau d'abstraction supplémentaire, et ainsi permet de faire abstraction du SGBD utilisé, ce qui est assez pratique pour un développement dans une optique de portabilité.

Message cité 1 fois

omega2

MagicBuzz > Le "zend framework" permet d'avoir un bon niveau d'abstraction et il doit surement y en avoir d'autres.

soulmanto

Chat Noir replica

Ha, oui, les requêtes préparées... Plus que le côté perfs, c'est le côté sécurité qui m'a intéressé là-dedans. J'en avais parlé y'a assez longtemps sur le topic des développeurs de forum, mais ça n'avait pas tenté grand monde!

MagicBuzz

moi je ne jure que par ça.

la syntaxe est à peine plus lourde, mais en revanche tu es sûr de ce qui est exécuté.
et avec MySQL > 5, des optimisations supplémentaires ont été apportées, ce qui améliore considérablement les perfs.

la doc de SQL Server est bien plus verbeuse que cet article, et s'applique en gros à tous les SGBD qui supportent ce type de requête (c'est à dire, à ma connaissance, tous).

à noter par contre que je ne sais pas si mysql support le paramètres nommés. ça améliore considérablement la lisibilité des requêtes (ça donne un peu comme dans le post de lofo7, sauf que tu trouves des "@toto" ou ":toto" au lieu de "$toto" dans tes requêtes. c'est plus lisible que des "?" )

Publicité

casimimir

y doit bien exister des tests quelque part, mais a priori une requete préparée pourrait même être plus rapide car si elle est exécutée de multiple fois le sgbd ne doit pas la recompiler a chaque fois.
Perso je pense qu'il n'y a que des avantages

Message cité 1 fois

dwogsi

Défaillance cérébrale...

MagicBuzz a écrit :

Alors je ne cèderai pas, quand je vois un accès à une base écrit comme une merde des années 70, ça me fait hurler, et vous pourrez bien dire tout ce que vous voudrez, je ne changerai pas ma façon de dire ce que je pense

Non mais je suis tout à fait d'accord avec toi. Je signalait juste que l'exemple qui est souvent donné ne fonctionne pas.

---------------
-- Debian -- Le système d'exploitation universel | Le gras c'est la vie! | /(bb|[^b]{2})/

anapajari

s/travail/glanding on hfr/gs;

MagicBuzz a écrit :

http://maximilian.developpez.com/m [...] tatements/

Perso, je préfère largement PDO à mysqli...
A noter également pour ceux qui ne pourraient installer ces extensions qu'il existe de nombreuses librairies d'abstaction qui permettent de se servir de prepared statement ( ma favorite étant adodb)

casimimir a écrit :

Nan mais un sgbd ça "compile" pas les requêtes hein On parle de "plan d'éxécution".
Mais tu as raison sur le fond. Au "prepare", le plan d'éxécution de la requête est memorisé pour pouvoir être ré-utilisé lors de chaque execute.

casimimir

c'est ce que je voulais dire, compiler était un résumé, mais oui je voulais dire que la requete était passée par le parser et l'optimizer, encore que la requete stockée sera une version hashée je crois, donc allez, moitié moitié ^^

FORUM HardWare.fr

Programmation

SQL/NoSQL

Erreur entre PHP et BD mysql

Sujets relatifs
Problème : Appli delphi sur base mysql whitespace	[PHP][6] - Meeting notes
[PHP] Problème déconnection sous IE	[Mysql] Droit sur les tables ??
Aide pour la création d'un flux RSS	[Crypt::CBC] RandomIV, Rijndael et PHP
SQL oracle erreur nombre invalide	[MySQL] Trigger mettant à jour une autre table
[PHP] Upload de fichier en plusieurs parties	privilèges MYSQL
Plus de sujets relatifs à : Erreur entre PHP et BD mysql

Page générée en 0.196 secondes