échec insertion, pas message erreur

Version imprimable

Bonjour à tous, je n'arrive pas à faire une insertion c'est bête :roll:, en gros voilà mon code.

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
 
 
class add_film extends Film{
 
 
	public function insert($nom, $type){
 
		$pdo = $this->getPDO();
		if($type == 'aventure')
			$type = '2';
		else 
			$type = '1';
 
		$add = $pdo->prepare('INSERT INTO FILM (id_film,nom_film,id_cat) VALUES (:id_film,:nom_film,:id_cat)');
		$id = $pdo->lastInsertId();
		echo $id.$nom.$type;
		$add->execute(array
				("id_film"=>$id,":nom_film"=>$nom,":id_cat"=>$type)
			     );
		if($add){
			echo 'ok';
 
		}
		else 
			echo 'non';
 
 
		$add->closeCursor();		
 
	}
 
 
 
}
 
 
$a = new add_film();
 
if (!(empty($_POST['nom']) && !(empty($_POST['type'])))){
	$nom = $_POST['nom'];
	$type = $_POST['type'];
	$a->insert($nom,$type);
}

Salut,
Tu ne le sais peut-être pas encore mais crois-moi, je te garanti une galère "royale" vu comment tu es parti...
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 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 // tu comptes créer une classe par action sur la base de données ? // sans compter que tu fais de l'héritage fonctionnel ce qui est un très // fortement déconseillé : add_film n'est pas un Film (au sens sémantique) // donc pas d'héritage possible class add_film extends Film { // si on suit ta logique : tu as une classe qui s'appelle add_film // dans laquelle tu as une fonction insert... // add_film <=> insert, non ? public function insert($nom, $type) { $pdo = $this->getPDO(); // $cat = ($type === 'aventure' ? 2 : 1; if ($type == 'aventure') $type = '2'; else $type = '1'; $add = $pdo->prepare('INSERT INTO FILM (id_film,nom_film,id_cat) VALUES (:id_film,:nom_film,:id_cat)'); // tu récupères l'id créé avant même que le nouveau film ne soit créé ! $id = $pdo->lastInsertId(); echo $id.$nom.$type; // $add est un PDOStatement et ->execute() renvoie un boolean // qui ici n'est pas réucpéré : cela serait plutôt : $exec = $add->execute(); $add->execute(array( ":id_film" => $id, ":nom_film" => $nom, ":id_cat" => $type )); // echo ($exec) ? 'ok' : 'non'; if ($add) { echo 'ok'; } else echo 'non'; // pas besoin de clore le curseur, c'est automatique et plus rapide quand c'est géré par le système $add->closeCursor(); } } $a = new add_film(); if (!(empty($_POST['nom']) && !(empty($_POST['type'])))){ $nom = $_POST['nom']; $type = $_POST['type']; $a->insert($nom,$type); }
Un peu de lecture pour toi ici

Tu t'attaques à trop de choses de front : problèmes d'architecture et maîtrise du langage PHP.
Personnellement, avant de me pencher sur l'architecture de mes développements, j'ai commencé par bétonner la connaissance du langage.
Réalise déjà simplement tes projets d'étudiant, fais en sorte que cela fonctionne et soit sécurisé un minimum et ensuite seulement tu te pencheras sur l'architecture.
Pourquoi cet ordre, c'est simple : il faut d'abord connaître un minimum ce qui est possible de faire avec le langage et ensuite il faut être solide sur ses fondamentaux pour avoir une approche suffisamment abstraite afin de pouvoir s'attaquer aux (complexes) problèmes architecturaux. Quand tu débutes et si tu veux y arriver, il ne faut pas aller trop vite et s'éparpiller dans tous les sens (ou il faut un super prof).

Allez bon courage :ccool:

Citation:
Envoyé par rawsrc
Salut,
Tu ne le sais peut-être pas encore mais crois-moi, je te garanti une galère "royale" vu comment tu es parti...
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 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 // tu comptes créer une classe par action sur la base de données ? // sans compter que tu fais de l'héritage fonctionnel ce qui est un très // fortement déconseillé : add_film n'est pas un Film (au sens sémantique) // donc pas d'héritage possible class add_film extends Film { // si on suit ta logique : tu as une classe qui s'appelle add_film // dans laquelle tu as une fonction insert... // add_film <=> insert, non ? public function insert($nom, $type) { $pdo = $this->getPDO(); // $cat = ($type === 'aventure' ? 2 : 1; if ($type == 'aventure') $type = '2'; else $type = '1'; $add = $pdo->prepare('INSERT INTO FILM (id_film,nom_film,id_cat) VALUES (:id_film,:nom_film,:id_cat)'); // tu récupères l'id créé avant même que le nouveau film ne soit créé ! $id = $pdo->lastInsertId(); echo $id.$nom.$type; // $add est un PDOStatement et ->execute() renvoie un boolean // qui ici n'est pas réucpéré : cela serait plutôt : $exec = $add->execute(); $add->execute(array( ":id_film" => $id, ":nom_film" => $nom, ":id_cat" => $type )); // echo ($exec) ? 'ok' : 'non'; if ($add) { echo 'ok'; } else echo 'non'; // pas besoin de clore le curseur, c'est automatique et plus rapide quand c'est géré par le système $add->closeCursor(); } } $a = new add_film(); if (!(empty($_POST['nom']) && !(empty($_POST['type'])))){ $nom = $_POST['nom']; $type = $_POST['type']; $a->insert($nom,$type); }
Un peu de lecture pour toi ici

Tu t'attaques à trop de choses de front : problèmes d'architecture et maîtrise du langage PHP.
Personnellement, avant de me pencher sur l'architecture de mes développements, j'ai commencé par bétonner la connaissance du langage.
Réalise déjà simplement tes projets d'étudiant, fais en sorte que cela fonctionne et soit sécurisé un minimum et ensuite seulement tu te pencheras sur l'architecture.
Pourquoi cet ordre, c'est simple : il faut d'abord connaître un minimum ce qui est possible de faire avec le langage et ensuite il faut être solide sur ses fondamentaux pour avoir une approche suffisamment abstraite afin de pouvoir s'attaquer aux (complexes) problèmes architecturaux. Quand tu débutes et si tu veux y arriver, il ne faut pas aller trop vite et s'éparpiller dans tous les sens (ou il faut un super prof).

Allez bon courage :ccool:
Merci pour le conseil, mais là c'est plus tôt une critique de la conception dont je suis tout à fait d'accord avec vous, mais mon problème c'est que je n'arrive pas à trouver pourquoi ma requête ne marche pas ?

08/09/2012, 20h49
stade13

Citation:

Envoyé par stade13

Merci pour le conseil, mais là c'est plus tôt une critique de la conception dont je suis tout à fait d'accord avec vous, mais mon problème c'est que je n'arrive pas à trouver pourquoi ma requête ne marche pas ?

Bon j'ai su pourquoi, problème de base de donnée :?