Insert de plusieurs entrée avec Classe CRUD

**nico21000** · 06/02/2011, 19h10

Bonjour,

J'utilise cette classe CRUD et j'aimerai faire une requête insert de plusieurs entrées.

Voici ce que j'obtient lorsque j'affiche le tableau $values passé en paramètre :

Array
(
[0] => Array
(
[nom] => dupont
[prenom] => julien
[societe] => fichier
[email] => julien.dupont@hotmail.com
)

[1] => Array
(
[nom] => Durant
[prenom] => lkuh
[societe] => fichier
[email] => durant@gmail.com
)

)

Et voici l'erreur :

Exception levée dans l'application.
Message SQLSTATE[HY093]: Invalid parameter number: parameter was not defined

Si quelqu'un comprend comment envoyer plusieurs entrées en une requête...

Merci d'avance.

**MaitrePylos** · 07/02/2011, 09h18

peux-t-on voir ton code, ainsi que la définition de ta table ?

**Benjamin Delespierre** · 07/02/2011, 12h19

Message SQLSTATE[HY093]: Invalid parameter number: parameter was not defined

C'est probablement un problème dû au requêtes préparées. Il doit de manquer des valeurs dans le tableau de remplacement, regarde par ici: http://www.php.net/manual/en/pdostatement.execute.php

Vérifie et reviens nous voir

**nico21000** · 08/02/2011, 08h32

Voici mon code :

Controller :

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
 
$this->add($contact->Contact);
 
   public function add($tab = null) {
        if(is_null($tab)){
 
			}else{
				$tab = array_values($tab);
				if(count($tab) > 1){  //Si j'ai plus de une entré à envoyer
					$values = array();
 
                    foreach($tab as $key=>$value){
                    	array_push($values, array('nom'=>$tab[$key]->nom,'prenom'=>$tab[$key]->prenom,'societe'=>$tab[$key]->societe,'email'=>$tab[$key]->email));
                    }
 
					$this->data->AddEmail($values);
				}else{    // Si je n'ai qu'une entré à envoyer ça marche...
                    $values = array('nom'=>$tab[0]->nom,'prenom'=>$tab[0]->prenom,'societe'=>$tab[0]->societe,'email'=>$tab[0]->email);
 
					$this->data->AddEmail($values);
                }
 
			}
 
 
    }

Model :

Code :

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

1
2
3
4
5
 
public function AddEmail($values){
		$this->dbInsert('emails', $values);
		return $r;	
	}

Classe CRUD :

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
 
 public function dbInsert($table, $values)
        {
            /*** snarg the field names from the first array member ***/
            $fieldnames = array_keys($values);
            /*** now build the query ***/
            $size = sizeof($fieldnames);
            $i = 1;
            $sql = "INSERT INTO $table";
            /*** set the field names ***/
            $fields = '( ' . implode(' ,', $fieldnames) . ' )';
            /*** set the placeholders ***/
            $bound = '(:' . implode(', :', $fieldnames) . ' )';
            /*** put the query together ***/
            $sql .= $fields.' VALUES '.$bound;
            /*** prepare and execute ***/
			$stmt = BDinstance::getInstance()->prepare($sql);
 
			/* echo '<pre>';
             print_r($sql);
             echo '</pre>';
			*/
			return $stmt->execute($values);
        }

et voici ma table :

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
 
CREATE TABLE `emails` (
  `id` int(11) NOT NULL auto_increment,
  `email` varchar(255) NOT NULL,
  `nom` varchar(255) NOT NULL,
  `societe` varchar(255) NOT NULL,
  `prenom` varchar(255) NOT NULL,
  `adresse` text NOT NULL,
  `cp` char(5) NOT NULL,
  `ville` varchar(255) NOT NULL,
  `tel` char(10) NOT NULL,
  `url` tinytext NOT NULL,
  `commentaire` text NOT NULL,
  `date_inscri` date NOT NULL,
  `valid` tinyint(1) NOT NULL,
  PRIMARY KEY  (`id`),
  UNIQUE KEY `email` (`email`)
) ENGINE=MyISAM AUTO_INCREMENT=24 DEFAULT CHARSET=latin1 AUTO_INCREMENT=24 ;

**Madfrix** · 08/02/2011, 10h34

Bonjour,

tu mets à jour ta table emails avec 4 colonnes (nom, prenom, société, email) mais cette table contient d'autres colonnes qui ne peuvent pas être nulles (en fait toutes tes colonnes). Vu qu'elles n'ont pas de valeurs par défaut, tu as une erreur d'insertion

**nico21000** · 08/02/2011, 10h55

Bonjour et merci pour ta réponse,

Je me suis posé la question en écrivant le message. Je test ce soir et je vous redis demain.

Encore merci et bonne journée

**nico21000** · 08/02/2011, 13h56

J'ai changer quelques petites choses :

Controller :

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
 
public function add($tab = null) {
        if(is_null($tab)){
 
			}else{
				$tab = array_values($tab);
				if(count($tab) > 1){  //Si j'ai plus de une entré à envoyer
					$values = array();
 
                /*    foreach($tab as $key=>$value){
                    	array_push($values, array('nom'=>$tab[$key]->nom,'prenom'=>$tab[$key]->prenom,'societe'=>$tab[$key]->societe,'email'=>$tab[$key]->email));
                    }
                  */
 
				   array_push($values, array('nom'=>'nom1','prenom'=>'prenom1','societe'=>'societe1','email'=>'email1@test.fr'));
                   array_push($values, array('nom'=>'nom2','prenom'=>'prenom2','societe'=>'societe2','email'=>'email2@test.fr'));
 
                   $this->data->AddEmail($values);
				}else{    // Si je n'ai qu'une entré à envoyer ça marche...
                    $values = array('nom'=>$tab[0]->nom,'prenom'=>$tab[0]->prenom,'societe'=>$tab[0]->societe,'email'=>$tab[0]->email);
 
					$this->data->AddEmail($values);
                }
 
			}
 
 
    }

La structure de la table :

CREATE TABLE `emails` (
`id` int(11) NOT NULL auto_increment,
`email` varchar(255) NOT NULL,
`nom` varchar(255) NOT NULL,
`societe` varchar(255) NOT NULL,
`prenom` varchar(255) NOT NULL,
`adresse` text,
`cp` char(5) default NULL,
`ville` varchar(255) default NULL,
`tel` char(10) default NULL,
`url` tinytext,
`commentaire` text,
`date_inscri` date default NULL,
`valid` tinyint(1) default NULL,
PRIMARY KEY (`id`),
UNIQUE KEY `email` (`email`)
) ENGINE=MyISAM AUTO_INCREMENT=24 DEFAULT CHARSET=latin1 AUTO_INCREMENT=24 ;

Et j'ai toujours la même erreur, et lorsque je fais le test avec ce tableau :

Array
(
[nom] => dupont
[prenom] => julien
[societe] => fichier
[email] => julien.dupont@hotmail.com
)

Cela fonctionne alors qu'avec ce tableau :

Array
(
[0] => Array
(
[nom] => dupont
[prenom] => julien
[societe] => fichier
[email] => julien.dupont@hotmail.com
)

[1] => Array
(
[nom] => Durant
[prenom] => lkuh
[societe] => fichier
[email] => durant@gmail.com
)

)

Cela de fonctionne pas.

**Madfrix** · 08/02/2011, 14h33

Essaies de modifier comme ceci :

Code php :

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

1
2
3
4
5
 
public function AddEmail($values){
    foreach($values as $value)
        $this->dbInsert('emails', $value);
}

**nico21000** · 08/02/2011, 19h52

Bonjour,

Le problème avec cette méthode c'est que je fais une requête pour chaque email a insérer dans la table et la but de mon application est de faire de l'import à partir d'un fichier .csv qui peux être très volumineux (3000 entrées par exemple donc 3000 requêtes). Je pense que comme ça, ça sera pas optimisé.
Il doit forcément y avoir un moyen de tous envoyer en une requête.
Merci quand même.

**Madfrix** · 08/02/2011, 23h05

teste cela je pense que ca devrait être bon :

Code php :

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
31
32
33
 
public function dbInsert($table, $values) {
 
	$bound = '';
	$bindParams = array();
	$fieldnames = array_keys($values[0]);
 
    $sql = 'INSERT INTO '. $table;
 
    $fields = '( ' . implode(', ', $fieldnames) . ' )';
 
	foreach($values as $key=>$value) {
		$bound .= ' (';
		foreach($fieldnames as $fieldname) {
			$param = ':' . $fieldname.$key . ', ';
			$bound .= $param;
			$bindParams[] = array('param'=>$param, 'value'=>$value[$fieldname]);
		}
		$bound = substr($bound, 0, -2);
		$bound .= '), ';
	}
 
	$bound = substr($bound, 0, -2);
 
    $sql .= $fields.' VALUES '.$bound;	
 
	$stmt = BDinstance::getInstance()->prepare($sql);
 
	foreach($bindParams as $value)
		$stmt->bindParam($value['param'], $value['value']);
 
	return $stmt->execute($values);
}

**MaitrePylos** · 09/02/2011, 09h20

C'est choux blancs et blancs choux, de plus imagine qu'il ne passe pas le même nombres de paramètres dans ses tableaux

le code suivants, va lui poser soucis

Code :

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

1
2
 
$fieldnames = array_keys($values[0]);

Et cela lui fera quand même 3000 requêtes.

**Madfrix** · 09/02/2011, 09h59

Envoyé par MaitrePylos

de plus imagine qu'il ne passe pas le même nombres de paramètres dans ses tableaux

Là je suis d'accord, libre à lui de modifier un peu le code pour adapter à ses besoins

Envoyé par MaitrePylos

Et cela lui fera quand même 3000 requêtes

Là je suis pas d'accord, cela lui fera une seule requête de ce type :

Code sql :

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

1
2
3
4
5
 
INSERT INTO saTable (C1, C2)
VALUES (val1, val2),
(val3, val4),
(val5, val6)

EDIT: de toute manière s'il ne passe pas le même nombres de paramètres dans ses tableaux, il sera oblié de faire des requêtes d'insertions unitaires donc de passer par la méthode que je lui ai proposé dans le post #8

**nico21000** · 09/02/2011, 17h56

Bonjour et merci pour l'intérêt que vous portez sur mon problème.

Ma requête semble être correcte :

INSERT INTO emails( nom, prenom, societe, email ) VALUES (:nom0, :prenom0, :societe0, :email0), (:nom1, :prenom1, :societe1, :email1)

Mais j'ai toujour la même erreur :

Exception levée dans l'application.
Message SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens
Fichier /homepages/0/d341517557/htdocs/mailer/application/core/crud.phpLigne 145

voici mon tableau de test :

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
Array
(
    [0] => Array
        (
            [nom] => nom1
            [prenom] => prenom1
            [societe] => societe1
            [email] => email1@test.fr
        )
 
    [1] => Array
        (
            [nom] => nom2
            [prenom] => prenom2
            [societe] => societe2
            [email] => email2@test.fr
        )
 
)

**Madfrix** · 09/02/2011, 19h11

Bonjour,

écris cela juste avant le dernier foreach de ta fonction et controle si tu as autant de paramètres que de valeurs avec la requête SQL (j'ai pas testé le code que je t'ai proposé) :

Code php :

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

1
2
3
4
 
echo '<pre>';
print_r($bindParams);
echo '</pre>';

**nico21000** · 09/02/2011, 19h27

Voici le résultat de mon echo :

Array
(
[0] => Array
(
[param] => :nom0,
[value] => nom1
)

[1] => Array
(
[param] => :prenom0,
[value] => prenom1
)

[2] => Array
(
[param] => :societe0,
[value] => societe1
)

[3] => Array
(
[param] => :email0,
[value] => email1@test.fr
)

[4] => Array
(
[param] => :nom1,
[value] => nom2
)

[5] => Array
(
[param] => :prenom1,
[value] => prenom2
)

[6] => Array
(
[param] => :societe1,
[value] => societe2
)

[7] => Array
(
[param] => :email1,
[value] => email2@test.fr
)

)

**Madfrix** · 09/02/2011, 19h32

Désolé, j'ai pas corrigé le execute dans la fonction. Mets ceci à la place pour voir :

Code php :

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

1
2
 
return $stmt->execute();

**nico21000** · 16/02/2011, 17h18

Bonjour,

Finalement cela fonctionne avec la fonction de Madfrix un tout petit peu modifiée :

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
 
public function dbMultiInsert($table, $values) {
		    $bound = '';
		    $bindParams = array();
		    $fieldnames = array_keys($values[0]);   
		    $sql = 'INSERT INTO '. $table;
 
		    $fields = '( ' . implode(', ', $fieldnames) . ' )';
 
		    foreach($values as $key=>$value) {
		        $bound .= ' (';
		        foreach($fieldnames as $fieldname) {
					$param = '"' . $value[$fieldname] . '", ';
		            $bound .= $param;
		            $bindParams[] = array('param'=>$param, 'value'=>$value[$fieldname]);
		        }
		        $bound = substr($bound, 0, -2);
		        $bound .= '), ';
		    }		        
		    $bound = substr($bound, 0, -2);
		    $sql .= $fields.' VALUES '.$bound;  
		    $stmt = BDinstance::getInstance()->prepare($sql);
			foreach($bindParams as $value)
		        $stmt->bindParam($value['param'], $value['value']);
 
			return $stmt->execute();
		}

Insert de plusieurs entrée avec Classe CRUD

PHP & Base de données

Discussions similaires

Partager

Partager