Discussion :

[laurentSc]

Bonjour,

j'ai une table qui a été créée comme ça (export sous phpMyadmin) :

Code SQL :

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

1
2
3
4
5
6
7
8
CREATE TABLE IF NOT EXISTS `news` (
  `id` smallint(6) NOT NULL AUTO_INCREMENT,
  `titre` varchar(255) COLLATE utf8_unicode_ci NOT NULL,
  `auteur` varchar(255) COLLATE utf8_unicode_ci NOT NULL,
  `date` datetime NOT NULL,
  `contenu` text COLLATE utf8_unicode_ci NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci AUTO_INCREMENT=3 ;

Pour en lire le contenu, je fais :

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
 
$news = array();
$resultats=$connexion->query("SELECT id, auteur, titre,
DATE_FORMAT('date', '%d/%m/%Y %H') AS date_formatee, contenu
FROM news
ORDER BY date DESC");
$result = $resultats->fetch(PDO::FETCH_ASSOC);
 
foreach($result as $champ) {
	 $news[] = $champ;
	 }
.....
foreach($news as $n)
{
echo "titre=".$n['titre']."<br>";
echo '
<div class="news">
<h2>'.$n['titre'].'</h2>
<p>News postée le '.str_replace(' ', ' à ', $n['date_formatee']).'
par '.$n['auteur'].'</p>
<p>'.$n['contenu'].'</p>
</div>';
}

et alors qu'un var_dump de $news me donne (pour le 1er enregistrement)

0 => string '1' (length=1)
1 => string 'vincent1870' (length=11)
2 => string 'Une premi�re news' (length=17)
3 => null
4 => string 'Bienvenue � tous sur ce beau site !<br /> <br /> Bon surf ! ' (length=62)

(déjà problème des caractères accentués alors que la bdd est en utf8), le code ci-dessus me donne

1
News postée le 1 par 1

1

v
News postée le v par v

v

etc

donc au lieu de prendre les champs complets de la table, ça ne prend que le premier caractère ; pourquoi ?

[Invité]

GRRRRRRRRRRR !!
Je t'ai déjà dit :

Code sql :

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

1
2
3
4
5
6
7
8
CREATE TABLE IF NOT EXISTS `news` (
  `news_id` smallint(6) NOT NULL AUTO_INCREMENT,
  `news_titre` varchar(255) COLLATE utf8_unicode_ci NOT NULL,
  `news_auteur` varchar(255) COLLATE utf8_unicode_ci NOT NULL,
  `news_date` datetime NOT NULL,
  `news_contenu` text COLLATE utf8_unicode_ci NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci AUTO_INCREMENT=3 ;

Arrête d'utiliser date comme nom de champ, car il est RESERVE !!
Tu vas inutilement te COMPLIQUER la vie !
Et générer des confusions/erreurs !

LIS LES TUTOS !!! ET SUIS LES RECOMMANDATIONS !
=> Passez à l'UTF-8 sans manquer une étape
=> Comprendre PDO

MON TUTO est un BON EXERCICE : Système de Gestion-Affichage de Nouvelles
TOUTES les étapes sont abordées :

• connexion
• INSERT
• UPDATE
• DELETE
• affichage de la liste
• affichage de la fiche

"A quoi ça sert que Ducros jreaux62 se décarcasse ?"

[laurentSc]

Pitié, pas la tête !

A vrai dire, ce code, c'est surtout du copier-coller...et le tuto sur PDO (l'autre, je le connais bien) il explique bien comment faire des requêtes préparées, mais rien sur le fetch...

Bon, j'ai modifié le nom des champs, mais avec

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
<?php
function recuperer_news($connexion)
{
$news = array();
$resultats=$connexion->query("SELECT news_id, news_auteur, news_titre,
DATE_FORMAT('news_date', '%d/%m/%Y %H') AS news_date_formatee, news_contenu
FROM news
ORDER BY date DESC");
 
$result = $resultats->fetch(PDO::FETCH_ASSOC);
 
foreach($result as $champ) {
	 $news[] = $champ;
	 }
 
return $news;
}
 
?>

j'ai

Fatal error: Call to a member function fetch() on a non-object in C:\wamp\www\news\modeles\news.php on line 10

[Invité]

utilise les requetes préparées :
->prepare(...);
->execute();

tu as le choix entre bind_param(...) et array(...)

utiles aussi :
->rowcount();
->fetch();
->fetchAll();

[laurentSc]

Je te tiendrai au courant quand j'aurai essayé, mais aujourd'hui, pas beaucoup de temps Je savais pas qu'il y avait une méthode "fetch" ; et pourquoi "rowcount" me serait utile ?

[laurentSc]

Bonsoir,

j'ai utilisé les requêtes préparées et la méthode "fetch" :

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
function recuperer_news($connexion)
{
$news = array();
 
$query = "SELECT news_id, news_auteur, news_titre,
DATE_FORMAT('news_date', '%d/%m/%Y %H') AS news_date_formatee, news_contenu
FROM news
ORDER BY news_date DESC;";
$prep = $connexion->prepare($query);
$prep->execute();
$arr = $prep->fetch();
 
 
foreach($arr as $champ) {
	 $news[] = $champ;
	 }
//echo "news=";var_dump($news);
return $news;
}

mais cet affichage :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
foreach($news as $n)
{
echo "titre=".$n['titre']."<br>";
echo '
<div class="news">
<h2>'.$n['news_titre'].'</h2>
<p>News postée le '.str_replace(' ', ' à ', $n['news_date_formatee']).'
par '.$n['news_auteur'].'</p>
<p>'.$n['news_contenu'].'</p>
</div>';
}

donne toujours ça :

titre=1


1

News postée le 1 par 1

1
titre=1


1

News postée le 1 par 1

1
titre=v


v

News postée le v par v

v
titre=v


v

News postée le v par v

v

etc

alors qu'un var_dump de $n donne :

array
0 => string '1' (length=1)
1 => string '1' (length=1)
2 => string 'vincent1870' (length=11)
3 => string 'vincent1870' (length=11)
4 => string 'Une premi�re news' (length=17)
5 => string 'Une premi�re news' (length=17)
6 => null
7 => null
8 => string 'Bienvenue � tous sur ce beau site !<br /> <br /> Bon surf ! ' (length=62)
9 => string 'Bienvenue � tous sur ce beau site !<br /> <br /> Bon surf ! ' (length=62)

Quel est le problème ?

[sabotage]

aie aie aie il faut que tu t'entraines a manipuler les tableaux PHP.

Mais bon pour PDO il existe déjà une méthode :

Code :

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

1
2
3
4
5
6
7
8
9
10
function recuperer_news($connexion)
{
$query = "SELECT news_id, news_auteur, news_titre,
DATE_FORMAT('news_date', '%d/%m/%Y %H') AS news_date_formatee, news_contenu
FROM news
ORDER BY news_date DESC;";
$prep = $connexion->query($query);
$news = $prep->fetchAll(PDO::FETCH_ASSOC);
return $news;
}

[laurentSc]

Merci pour ta réponse, par contre, comme je souhaite progresser, peux-tu me dire quelle est mon erreur ?

Sinon, jreaux62 m'avait conseillé d'utiliser les requêtes préparées et tu ne le fais pas ; pourquoi ?

Pour finir, 2 problèmes :
- les champs de la table concernés sont en utf8 et pourtant, ils s'affichent mal ; pourquoi ?

- le champ "news_date_formatee" est à null ; pourquoi ?

[sabotage]

Pour la date, tu as mis des apostrophes sur le nom de la colonne.

Voila ce que faisais ton code :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
// recupère une ligne
$arr = $prep->fetch();
 
// parcours les colonnes de cette ligne
foreach($arr as $champ) {
         // stocke chaque valeur dans le tableau $news
	 $news[] = $champ;
	 }
// fin

[Invité]

Sinon, jreaux62 m'avait conseillé d'utiliser les requêtes préparées et tu ne le fais pas ; pourquoi ?

La requête "SELECT... " ici présente NE contient PAS de variables "à protéger".
Une requête "préparée" n'a donc pas de réel intérêt ici.
C'est néanmoins une bonne habitude à prendre !

Il faut savoir ce qu'on fait, et prendre les mesures adéquates et adaptées à la situation.

- les champs de la table concernés sont en utf8 et pourtant, ils s'affichent mal ; pourquoi ?

Tant que tu écriras ton code "au petit bonheur la chance", sans prendre en compte ce que tu as appris dans les différents tutos (que tu as lu et/ou qu'on t'a conseillé de lire), tes scripts ne "réagiront" pas comme tu l'espères.

Il faut de la rigueur !

->fetch()
->fetchAll()
Renseigne-toi sur ces instructions ! Ce n'est pas la même chose...

[laurentSc]

je sus très content car c'est résolu :

- j'ai enlevé les apostrophes pour la date :

Code :

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

1
2
3
4
$query = "SELECT news_id, news_auteur, news_titre,
DATE_FORMAT(news_date, '%d/%m/%Y %H') AS news_date_formatee, news_contenu
FROM news
ORDER BY news_date DESC;";

- comme fetch ne récupère qu'une ligne, je fais une boucle :

Code :

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

1
2
while (($arr = $prep->fetch())!= null)
	$news[]=$arr;

- même si ici une requête préparée n'a pas d'intérêt, je prends une bonne habitude :

Code :

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

1
2
$prep = $connexion->prepare($query);
$prep->execute();

- j'ai pris en compte ce qui est dans le tuto sur l'utf8 :

Code :

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

$connexion->exec("SET CHARACTER SET utf8");

(et un coup de Google car le tuto ne prend pas en compte pdo)

Tant que tu écriras ton code "au petit bonheur la chance"

sauf que ce code n'est pas de moi...

[Invité]

Tu progresses !

comme fetch ne récupère qu'une ligne, je fais une boucle :

Code :

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

1
2
while (($arr = $prep->fetch())!= null)
	$news[]=$arr;

C'est (à peu près) équivalent au code donné par Sabotage :

Code :

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

1
2
3
4
$arrAll = $prep->fetchAll(); // on récupère toutes les lignes
foreach($arrAll as $arr) {
	$news[]=$arr;
}

Et si tu réfléchis bien : $news EST EGAL A $arrAll !
Donc, ceci suffit !

Code :

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

$news = $prep->fetchAll(); // on récupère toutes les lignes

...sauf que ce code n'est pas de moi...

C'est bien ce que j'ai dit : faire des copier-coller sans comprendre...

[laurentSc]

C'est (à peu près) équivalent au code donné par Sabotage

bien sûr, mais je voulais m'approprier le fonctionnement de $prep->fetch()...

Et si tu réfléchis bien : $news EST EGAL A $arrAll !

C'est vrai que ici l'emploi de fetchAll simplifie beaucoup par rapport à fetch.