Insert de plusieurs entrée avec Classe CRUD [Résolu]

[nico21000]

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]

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

[Benjamin Delespierre]

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]

Voici mon code :

Controller :

Code :

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 :

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

Classe CRUD :

Code :

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 :

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]

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]

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]

J'ai changer quelques petites choses :

Controller :

Code :

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]

Essaies de modifier comme ceci :

Code php :

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

[nico21000]

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]

teste cela je pense que ca devrait être bon :

Code php :

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]

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 :

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

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

[Madfrix]

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 :

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]

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 :

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]

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 :

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

[nico21000]

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]

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

Code php :

1
2
 
return $stmt->execute();

[nico21000]

Bonjour,

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

Code :

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();
		}