corriger moi ce code svp

Version imprimable

ce code m'envoie des erreurs

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
$resultat= $_GET['titre'];
$sql =  $bdd-> query ("SELECT id,titre,resume,article,image,auteur,date "
        . "FROM politique "
        . "WHERE titre ='$resultat'"
        . "union"
        . "SELECT id,titre,resume,article,image,auteur,date "
        . "FROM diplomatie "
        . "WHERE titre ='$resultat'"
        . "union"
        . "SELECT id,titre,resume,article,image,auteur,date "
        . "FROM finance "
        . "WHERE titre ='$resultat'"
        . "union");
while ($donnees = $sql->fetch()){

Bonjour,

Quelque chose dans ce genre devrait faire l'affaire :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
$titre = isset($_GET['titre']) ? $_GET['titre'] : '';
 
if ($titre !== '')
{
    $cols  = 'id, titre, resume, image, auteur, `date`';
    $titre = $bdd->quote($titre);
    $sql = <<<sql
SELECT {$cols} FROM politique  WHERE titre = '{$titre}'
UNION
SELECT {$cols} FROM diplomatie WHERE titre = '{$titre}'
UNION
SELECT {$cols} FROM finance    WHERE titre = '{$titre}'
sql;
 
    $qry = $bdd->query($sql);
    while ($row = $qry->fetch(PDO::FETCH_ASSOC))
    {
        // ...
    }
}

13/09/2014, 18h44
japhettchabao

il me renvoit erreur

il me renvoi

Citation:

( ! ) Fatal error: in C:\wamp\www\togoportail\lire_article.php on line 40
( ! ) PDOException: in C:\wamp\www\togoportail\lire_article.php on line 40

. voici la ligne 40

Code:

$qry = $bdd->query($sql);

Ben il faut chercher un peu... et faire des essais autour du sql

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
$titre = isset($_GET['titre']) ? $_GET['titre'] : '';
 
if ($titre !== '')
{
    $cols  = '`id`, `titre`, `resume`, `image`, `auteur`, `date`';
    $sql = <<<sql
(SELECT {$cols} FROM politique  WHERE titre = :titre1)
UNION
(SELECT {$cols} FROM diplomatie WHERE titre = :titre2)
UNION
(SELECT {$cols} FROM finance    WHERE titre = :titre3)
sql;
 
    $stmt = $bdd->prepare($sql); // je suppose que tu utilises PDO
    $stmt->bindParam(':titre1', $titre);
    $stmt->bindParam(':titre2', $titre);
    $stmt->bindParam(':titre3', $titre);
    $qry = $stmt->execute($sql);
    while ($row = $qry->fetch(PDO::FETCH_ASSOC))
    {
        // ...
    }
}

13/09/2014, 19h23
japhettchabao

il me renvoit erreur

meme erreur
13/09/2014, 19h26
rawsrc

Bon je ne vais pas jouer aux devinettes :bug:, si tu ne files pas plus d'infos sur ton erreur et environnement voire plus de code, tu devras te dépatouiller tout seul.

Est-ce que tu comprends au moins mon code ?
14/09/2014, 10h27
japhettchabao

je comprends pas ton code

sur la page d'acceuil, il y a plusieurs articles. quand l'utilisateur clique sur le titre d'un article, je recupère le lien de l'article d'ou le get. maintenant la base de données comprends les tables politque, economie et autre qui ont un champ titre. je veux faire une recherche dans les tables et trouver le titre correspondant puis afficher l'article de ce titre. merci
14/09/2014, 10h51
rawsrc

Donne le code derrière $bdd

Code:

1
2
3
4
5
6
7
8
9
10
11
while ($donnees = $sql->fetch()){
              echo ' '
                  . '<fieldset id="affichearticle">'
                      . '<center><span id="titrearticle">'. htmlspecialchars($donnees['titre']).'</span></center> <br /><br />'
                      . '<fieldset id="afficheimagearticle"><img src="articles/'.($donnees['image']) .' " width="300px" height="400px"/></fieldset>'
                      . '<br /><br /><p>'. htmlspecialchars($donnees['article']).' </p>'
                      . '<fieldset id="auteur"><span>'. htmlspecialchars($donnees['auteur']).'<br />'
                      . ''. htmlspecialchars($donnees['date']).'</span></fieldset><br /><br />'
                      . '</fieldset>';
              }
              ?>

14/09/2014, 11h00
rawsrc

non, je voudrais voir l'initialisation de $bdd (je suppose que c'est du PDO, mais je voudrais en être sûr)

Tiens essaie avec ça et poste le message d'erreur :

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
$titre = isset($_GET['titre']) ? $_GET['titre'] : '';
 
if ($titre !== '')
{
    $cols  = '`id`, `titre`, `resume`, `image`, `auteur`, `date`';
    $sql = <<<sql
(SELECT {$cols} FROM politique  WHERE titre = :titre1)
UNION
(SELECT {$cols} FROM diplomatie WHERE titre = :titre2)
UNION
(SELECT {$cols} FROM finance    WHERE titre = :titre3)
sql;
 
     try {
        $stmt = $bdd->prepare($sql); // je suppose que tu utilises PDO
        $stmt->bindParam(':titre1', $titre);
        $stmt->bindParam(':titre2', $titre);
        $stmt->bindParam(':titre3', $titre);
        $qry = $stmt->execute($sql);
     }
     catch (PDOException $e)
     {
         echo $e->getMessage();
         exit;
     }
 
    while ($row = $qry->fetch(PDO::FETCH_ASSOC))
    {
        // ...
    }
}

j'ai fusionner nos code, sa marche merci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<?php
              $resultat= $_GET['titre'];
               $cols  = '`article`, `titre`, `resume`, `image`, `auteur`, `date`';
$sql =  $bdd-> query ("SELECT {$cols} FROM politique WHERE titre = '$resultat'UNION SELECT {$cols} FROM finance WHERE titre = '$resultat'");
while ($donnees = $sql->fetch()){
              echo ' '
                  . '<fieldset id="affichearticle">'
                      . '<center><span id="titrearticle">'. htmlspecialchars($donnees['titre']).'</span></center> <br /><br />'
                      . '<fieldset id="afficheimagearticle"><img src="articles/'.($donnees['image']) .' " width="300px" height="400px"/></fieldset>'
                      . '<br /><br /><p>'. htmlspecialchars($donnees['article']).' </p>'
                      . '<fieldset id="auteur"><span>'. htmlspecialchars($donnees['auteur']).'<br />'
                      . ''. htmlspecialchars($donnees['date']).'</span></fieldset><br /><br />'
                      . '</fieldset>';
              }
              ?>

14/09/2014, 11h55
rawsrc

Ne gardes surtout pas ton code SQL, il n'est pas du tout sécurisé.
Gardes plutôt le mien...
14/09/2014, 12h00
japhettchabao

le tien ne marche pas

je vais faire de l'url rewritting pour securiser. t'as un tuto, stp?
14/09/2014, 12h23
rawsrc

l'url rewriting n'a strictement rien à voir avec la sécurité relative à SQL :aie:, c'est juste de la cosmétique.

ce code est à proscrire :

Code:

$sql = $bdd-> query ("SELECT {$cols} FROM politique WHERE titre = '$resultat'UNION SELECT {$cols} FROM finance WHERE titre = '$resultat'");

Tu dois le remplacer par le mien : syntaxe ->prepare()... ->bindParam()...
14/09/2014, 13h46
japhettchabao

je le change comment?