PHP

Call to a member function execute() on a non-object

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Call to a member function execute() on a non-object

Gavrinis

Open your mind

Bonjour,

J'ai un petit problème avec un bout de code pompé :whistle: . Bizarrement, il passe sous WAMP mais pas sous Apache/PHP, en bref en dev Ok, en prod pas OK. Je cherche depuis un moment ce qui bloque mais je ne vois pas très bien.

Code :

$desired_theme_id = NULL;
if (!empty($_GET['theme']) and is_numeric($_GET['theme']))
$desired_theme_id = intval($_GET['theme']);
else
{
if (empty($_SESSION['current_theme']))
$desired_theme_id = DEFAULT_THEME_ID;
}
if($desired_theme_id)
{
$sql = 'SELECT id, nom, repertoire FROM themes WHERE id = :id_theme';
$statement = $db->prepare($sql);
$statement->execute(array(':id_theme' => $desired_theme_id));
if (!($theme = $statement->fetch()) or !is_dir(THEMES_ROOT_FOLDER.'/'.$theme['repertoire']))
{
$statement->execute(array(':id_theme' => DEFAULT_THEME_ID));
$theme = $statement->fetch();
}
$_SESSION['current_theme'] = $theme['repertoire'];
}

Erreur : "Fatal error: Call to a member function execute() on a non-object in /var/www/common.php on line 62", la ligne 62 correspond à la 16 ici, le tableau GET ne contient rien car c'est la homepage (domaine.net), les constantes sont bien initialisées :

Code :

define('DEFAULT_THEME_ID', 1);
define('THEMES_ROOT_FOLDER', 'themes');

Merci d'éclairer ma lanterne.

Publicité

macgawel

Bonjour.

Apparemment, ce qui ne lui plait pas c'est ça :

Code :

$statement->execute(array(':id_theme' => $desired_theme_id));

A priori, je dirais que $statement n'est pas défini comme un objet - en prod'.

Essaye de faire un

Code :

var_dump($statement);

avant le execute, ça devrait te donner des informations.
A voir :
Créer une page php avec juste

Code :

<?php
phpinfo();
?>

Et regarde s'il n'y a pas des différences entre l'environnement de dév' et celui de prod'...

Gavrinis

Open your mind

Le "var_dump($statement);" retourne ceci :

Code :

object(PDOStatement)#2 (1) { ["queryString"]=> string(59) "SELECT id, nom, repertoire FROM themes WHERE id = :id_theme" }

Pour phpinfo(), je regarde tout de suite.

Message cité 1 fois

macgawel

Le var_dump te donne la même chose dans les deux environnements ?

Gavrinis

Open your mind

Oui c'est identiquement la même chose. Du côté de phpinfo aussi mise à part que le prod est configuré pour être sécurisé.

rufo

Pas me confondre avec Lycos!

par sécurisé, faut comprendre "safe_mode" à On? Parce que je sais que ce genre de mode pose pas mal de pb à de nombreux scripts php...

---------------
Astres, outil de help-desk GPL : http://sourceforge.net/projects/astres, ICARE, gestion de conf : http://sourceforge.net/projects/icare, Outil Planeta Calandreta : https://framalibre.org/content/planeta-calandreta

Gavrinis

Open your mind

Non, il est désactivé aussi en prod.

rufo

Pas me confondre avec Lycos!

question bête : la connexion à la bd est ok? En gros que dans le ficheir de conf, t'as pas laissé le nom de la bd de test? Parce qu'à aucun moment dans ton script tu ne testes ce que contient $statement
(la ligne $db->prepare($sql) pourrait très bien renvoyer une erreur, mais n'étant pas familier avec PDO...)

Gavrinis

Open your mind

Le nom de la base de donnée est la même, login et mot de passe sont adaptés.

Profil supprimé

Gavrinis a écrit :

Le "var_dump($statement);" retourne ceci :

Code :

object(PDOStatement)#2 (1) { ["queryString"]=> string(59) "SELECT id, nom, repertoire FROM themes WHERE id = :id_theme" }

Pour phpinfo(), je regarde tout de suite.

à la ligne 61 ?

Message cité 1 fois

Publicité

Gavrinis

Open your mind

Bon c'est résolu, il y avait bien une erreur (merci Rufo ).
Maintenant j'ai droit à une autre ... "Warning: PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number: parameter was not defined in /var/www/includes/functions.php on line 72 Warning"

Code :

$desired_lang_id = NULL;
if (!empty($_GET['lang']) && ($_GET['lang'] == 'fr' || $_GET['lang'] == 'en'))
$desired_lang_id = $_GET['lang'];
else
{
if (empty($_SESSION['language_id']) or empty($_SESSION['language_short']))
$desired_lang_id = DEFAULT_LANG_ID;
}

La ligne 72 correspond ici à la ligne 3, je lui mets une condition "si le paramètre est vide va au else", bien sûr que le paramètre est vide car c'est comme au dessus nous sommes sur la homepage, mais seulement il parcourt quand même le if et après vient se plaindre :sweat:

Message cité 2 fois

Gavrinis

Open your mind

Non mon "var_dump($statement);" était à la ligne 60.

Gavrinis

Open your mind

J'ai parlé trop vite, il me sort toujours l'erreur précédente mais cette fois en ligne 60 (ligne 14 de mon premier post) en plus de la nouvelle erreur

macgawel

Gavrinis a écrit :

Code :

$desired_lang_id = NULL;
if (!empty($_GET['lang']) && ($_GET['lang'] == 'fr' || $_GET['lang'] == 'en'))
$desired_lang_id = $_GET['lang'];
else
{
if (empty($_SESSION['language_id']) or empty($_SESSION['language_short']))
$desired_lang_id = DEFAULT_LANG_ID;
}

Si le script passe dans ta condition IF, c'est qu'elle est validée...

Tu as testé voir s'il y passe effectivement (echo par exemple) ?
Essaye de fractionner ton test. C'est peut-être moins optimisé, mais ce sera plus simple à maintenir :

Code :

// Tableau contenant les langues utilisables
$Tlang = array ( "fr", "en" );
// Langage par défaut : la première du tableau des langues
$desired_lang_id = $Tlang[0];
// On commmence par regarder si un langage est demandé dans la session
// Je n'ai pas tout suivi, dans ton script donc je fais le minimum...
if ( isset ($_SESSION['language_id'] ) && ( ! empty ($_SESSION['language_id'] ) {
// Je pars du principe que le contenu de SESSION est correct. Sinon, faire une vérif.
$desired_lang_id = $_SESSION['language_id'];
}
// ensuite on regarde si un langage est demandé par le GET
if (isset ( $_GET['lang'] ) ) && ( in_array ( $_GET['lang'], $Tlang ) ) {
$desired_lang_id = $_GET['lang'];
}

Ceci-dit, je ne pense pas que ce soit cette ligne qui te génére un Warning: PDOStatement...

Sinon, vu que tu utilises PDO - ou une variante :

Code :

$sql = 'SELECT id, nom, repertoire FROM themes WHERE id = :id_theme';
//En cas d'échec du PREPARE, $statement vaudra FALSE...
// Du coup, on teste sa valeur avant de continuer...
if ( $statement = $db -> prepare ( $sql ) ) {
// La fonction execute renvoie TRUE si elle est bien passée, FALSE sinon
$statement->execute(array('id_theme' => $desired_theme_id)) or die ( "Erreur à l'execution du statement " );
}
else {
die ( "Erreur de statement avec la requête ".$sql );
}

Apparemment, d'après la doc PHP, tu devrais préciser dans ton prepare un array...

Message cité 1 fois

rufo

Pas me confondre avec Lycos!

Gavrinis a écrit :

Ca me paraît plus simple à résoudre. il te dit que le paramètrre passé n'est pas défini. Soit $_GET['lang'] contient rien (auquel cas voir pourquoi) soit il contient une valeur qui n'est pas un nombre. Attention, en général, ce qui est passé dans du GET ou POST est vu comme un type string. PDO fait sans doute la distinction entre la valeur 1 et "1". Si $_GET['lang'] n'est aps vide, essayes de faire (integer)$_GET['lang'] pour forcer le type.

Message cité 1 fois

Gavrinis

Open your mind

macgawel a écrit :

Si le script passe dans ta condition IF, c'est qu'elle est validée...

Tu as testé voir s'il y passe effectivement (echo par exemple) ?
Essaye de fractionner ton test. C'est peut-être moins optimisé, mais ce sera plus simple à maintenir :

Code :

// Tableau contenant les langues utilisables
$Tlang = array ( "fr", "en" );
// Langage par défaut : la première du tableau des langues
$desired_lang_id = $Tlang[0];
// On commmence par regarder si un langage est demandé dans la session
// Je n'ai pas tout suivi, dans ton script donc je fais le minimum...
if ( isset ($_SESSION['language_id'] ) && ( ! empty ($_SESSION['language_id'] ) {
// Je pars du principe que le contenu de SESSION est correct. Sinon, faire une vérif.
$desired_lang_id = $_SESSION['language_id'];
}
// ensuite on regarde si un langage est demandé par le GET
if (isset ( $_GET['lang'] ) ) && ( in_array ( $_GET['lang'], $Tlang ) ) {
$desired_lang_id = $_GET['lang'];
}

Ceci-dit, je ne pense pas que ce soit cette ligne qui te génére un Warning: PDOStatement...

Sinon, vu que tu utilises PDO - ou une variante :

Code :

$sql = 'SELECT id, nom, repertoire FROM themes WHERE id = :id_theme';
//En cas d'échec du PREPARE, $statement vaudra FALSE...
// Du coup, on teste sa valeur avant de continuer...
if ( $statement = $db -> prepare ( $sql ) ) {
// La fonction execute renvoie TRUE si elle est bien passée, FALSE sinon
$statement->execute(array('id_theme' => $desired_theme_id)) or die ( "Erreur à l'execution du statement " );
}
else {
die ( "Erreur de statement avec la requête ".$sql );
}

Apparemment, d'après la doc PHP, tu devrais préciser dans ton prepare un array...

Le problème s'est qu'un echo ne changera rien car le $_GET['lang'] est sensé ne rien contenir lors de l'arrivé sur la page, le echo n'affichera donc rien.
J'ai mis ton test mais ça n'a rien changé .
Ta solution pour le Warning me donne ceci "Erreur de statement avec la requ�te SELECT id, nom, repertoire FROM themes WHERE id = :id_theme"
Pour le prepare Ok mais je ne vois pas quel array mettre :??:

Message édité par Gavrinis le 17-01-2009 à 11:58:55

Gavrinis

Open your mind

rufo a écrit :

Ca me paraît plus simple à résoudre. il te dit que le paramètrre passé n'est pas défini. Soit $_GET['lang'] contient rien (auquel cas voir pourquoi) soit il contient une valeur qui n'est pas un nombre. Attention, en général, ce qui est passé dans du GET ou POST est vu comme un type string. PDO fait sans doute la distinction entre la valeur 1 et "1". Si $_GET['lang'] n'est pas vide, essayes de faire (integer)$_GET['lang'] pour forcer le type.

Il ne contient rien car quand le visiteur arrive sur la homepage, il n'y a pas encore de paramètre dans l'url, c'est pour cela qu'il y a une alternative avec les sessions.
Sa valeur contenue n'est effectivement pas un nombre étant donné que c'est soit "fr" soit "en".

Gavrinis

Open your mind

[:undertaker666]

macgawel

Citation :

Exemple #1 Prépare une requête SQL avec des paramètres nommés
<?php
/* Exécute une requête préparée en passant un tableau de valeurs */
$sql = 'SELECT nom, couleur, calories
FROM fruit
WHERE calories < :calories AND couleur = :couleur';
$sth = $dbh->prepare($sql, array(PDO::ATTR_CURSOR => PDO::CURSOR_FWDONLY));
$sth->execute(array(':calories' => 150, ':couleur' => 'red'));
$red = $sth->fetchAll();
$sth->execute(array('calories' => 175, 'couleur' => 'yellow'));
$yellow = $sth->fetchAll();
?>

Si tu lis la doc de prepare et execute :
Le problème, c'est que tu demande d'exécuter une requête en lui passant des paramètres, mais que tu la prepare sans paramètres.
Du coup, forcément ça coince !

Essaye d'adapter un des exemples de la doc à ton cas, et vois ce que ça donne...

Gavrinis

Open your mind

C'est encore pire si j'ajoute cela à mon prepare car la page ne s'affiche plus du tout, une page blanche c'est tout.

macgawel

Code :

$sth->execute(array(':calories' => 150, ':couleur' => 'red'));

Tu as bien mis les ":" dans le execute ?

Gavrinis

Open your mind

Oui

Code :

$statement->execute(array(':id_theme' => $desired_theme_id));

Gavrinis

Open your mind

Pas d'idées ?

Publicité

FORUM HardWare.fr

Programmation

PHP

Call to a member function execute() on a non-object

Sujets relatifs
Balise Object probleme IE	vector contenant object utilisant polymorphisme, help
[C++][resolu]error: no matching function for call to...	SimpleXML Object vers Array ?
std::map associer des index et une classe, comparison function	Fatal error: Can't use function return value in write context in
[AJAX] Problème avec innerHTML=xhr_object.responseText	from c function func(double (g(char))) to cpp cool object design
Plus de sujets relatifs à : Call to a member function execute() on a non-object

Page générée en 0.085 secondes