Convertir en PDO

**dancom5** · 09/03/2013, 05h44

Bonjour.

J'ai besoin d'aide pour convertir la parti mysql en version PDO de ceci :

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
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
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
<?php
if($_POST['page'])
{
$page = $_POST['page'];
$cur_page = $page;
$page -= 1;
$per_page = 1;
$previous_btn = true;
$next_btn = true;
$first_btn = true;
$last_btn = true;
$start = $page * $per_page;
include"db.php";
 
$query_pag_data = "SELECT * from crud_categorie LIMIT $start, $per_page";
$result_pag_data = mysql_query($query_pag_data) or die('MySql Error' . mysql_error());
$msg = "";
while ($row = mysql_fetch_array($result_pag_data)) {
$htmlmsg=htmlentities($row['categorie']);
    $msg .= "<li><b>" . $row['id'] . "</b> " . $htmlmsg . "</li>";
}
$msg = "<div class='data'><ul>" . $msg . "</ul></div>"; // Content for Data
 
 
/* --------------------------------------------- */
$query_pag_num = "SELECT COUNT(*) AS count FROM crud_categorie";
$result_pag_num = mysql_query($query_pag_num);
$row = mysql_fetch_array($result_pag_num);
$count = $row['count'];
$no_of_paginations = ceil($count / $per_page);
 
/* ---------------Calculating the starting and endign values for the loop----------------------------------- */
if ($cur_page >= 7) {
    $start_loop = $cur_page - 3;
    if ($no_of_paginations > $cur_page + 3)
        $end_loop = $cur_page + 3;
    else if ($cur_page <= $no_of_paginations && $cur_page > $no_of_paginations - 6) {
        $start_loop = $no_of_paginations - 6;
        $end_loop = $no_of_paginations;
    } else {
        $end_loop = $no_of_paginations;
    }
} else {
    $start_loop = 1;
    if ($no_of_paginations > 7)
        $end_loop = 7;
    else
        $end_loop = $no_of_paginations;
}
/* ----------------------------------------------------------------------------------------------------------- */
$msg .= "<div class='pagination'><ul>";
 
// FOR ENABLING THE FIRST BUTTON
if ($first_btn && $cur_page > 1) {
    $msg .= "<li p='1' class='active'> << </li>";
} else if ($first_btn) {
    $msg .= "<li p='1' class='inactive'> << </li>";
}
 
// FOR ENABLING THE PREVIOUS BUTTON
if ($previous_btn && $cur_page > 1) {
    $pre = $cur_page - 1;
    $msg .= "<li p='$pre' class='active'> < </li>";
} else if ($previous_btn) {
    $msg .= "<li class='inactive'> < </li>";
}
for ($i = $start_loop; $i <= $end_loop; $i++) {
 
    if ($cur_page == $i)
        $msg .= "<li p='$i' style='color:#fff;background-color:#000;' class='active'>{$i}</li>";
    else
        $msg .= "<li p='$i' class='active'>{$i}</li>";
}
 
// TO ENABLE THE NEXT BUTTON
if ($next_btn && $cur_page < $no_of_paginations) {
    $nex = $cur_page + 1;
    $msg .= "<li p='$nex' class='active'> > </li>";
} else if ($next_btn) {
    $msg .= "<li class='inactive'> > </li>";
}
 
// TO ENABLE THE END BUTTON
if ($last_btn && $cur_page < $no_of_paginations) {
    $msg .= "<li p='$no_of_paginations' class='active'> >> </li>";
} else if ($last_btn) {
    $msg .= "<li p='$no_of_paginations' class='inactive'> > </li>";
}
$goto = "<input type='text' class='goto' size='1' style='margin-top:-1px;margin-left:60px;'/><input type='button' id='go_btn' class='go_button' value='Go'/>";
$total_string = "<span class='total' a='$no_of_paginations'>Page <b>" . $cur_page . "</b> / <b>$no_of_paginations</b></span>";
$msg = $msg . "</ul>" . $goto . $total_string . "</div>";  // Content for pagination
echo $msg;
}

Ce qui m'intéresse particulièrement, c'est la partie MYSQL pour avoir en PDO. J'apprécie.

**sabotage** · 09/03/2013, 08h27

Montre nous ce que tu as fait et on pourra t'aider a le corriger.

**dancom5** · 10/03/2013, 02h03

ok. Tien, j'ai converti en PDO mais je ne suis pas sur que
c'est tout correcte:

Y'a peut-être place à amélioration au niveau PDO.

Le résultat est la et fonctionne.

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
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
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
<?php
if($_POST['page'])
{
$page = $_POST['page'];
$cur_page = $page;
$page -= 1;
$per_page = 1;
$previous_btn = true;
$next_btn = true;
$first_btn = true;
$last_btn = true;
$start = $page * $per_page;
include"db.php";
 
 
// TUTO suivi sur : query http://www.phpeveryday.com/articles/PDO-Prepared-Statement-and-Bound-Values-P553.html
 
$query_pag_data = "SELECT * from crud_categorie LIMIT $start, $per_page";
$result_pag_data = $con->prepare($query_pag_data );
$result_pag_data->execute();
$result_pag_data->bindColumn(1, $id);
$result_pag_data->bindColumn(2, $categorie);
$result_pag_data->bindColumn(3, $description);
 
$msg = "";
 
while($result_pag_data->fetch()) {
 
    $msg .= "<li><b>" . $id . "</b> " . $categorie . "</li>";
 
}
$msg = "<div class='data'><ul>" . $msg . "</ul></div>"; // Content for Data
 
/* --------------------------------------------- */
 
$query_pag_num = "SELECT * FROM crud_categorie";
$result_pag_num = $con->prepare( $query_pag_num );
$result_pag_num->execute();
$count = $result_pag_num->rowCount();
echo $count;
 
$no_of_paginations = ceil($count / $per_page);
 
/* ---------------Calculating the starting and endign values for the loop----------------------------------- */
if ($cur_page >= 7) {
    $start_loop = $cur_page - 3;
    if ($no_of_paginations > $cur_page + 3)
        $end_loop = $cur_page + 3;
    else if ($cur_page <= $no_of_paginations && $cur_page > $no_of_paginations - 6) {
        $start_loop = $no_of_paginations - 6;
        $end_loop = $no_of_paginations;
    } else {
        $end_loop = $no_of_paginations;
    }
} else {
    $start_loop = 1;
    if ($no_of_paginations > 7)
        $end_loop = 7;
    else
        $end_loop = $no_of_paginations;
}
/* ----------------------------------------------------------------------------------------------------------- */
$msg .= "<div class='pagination'><ul>";
 
// FOR ENABLING THE FIRST BUTTON
if ($first_btn && $cur_page > 1) {
    $msg .= "<li p='1' class='active'> << </li>";
} else if ($first_btn) {
    $msg .= "<li p='1' class='inactive'> << </li>";
}
 
// FOR ENABLING THE PREVIOUS BUTTON
if ($previous_btn && $cur_page > 1) {
    $pre = $cur_page - 1;
    $msg .= "<li p='$pre' class='active'> < </li>";
} else if ($previous_btn) {
    $msg .= "<li class='inactive'> < </li>";
}
for ($i = $start_loop; $i <= $end_loop; $i++) {
 
    if ($cur_page == $i)
        $msg .= "<li p='$i' style='color:#fff;background-color:#000;' class='active'>{$i}</li>";
    else
        $msg .= "<li p='$i' class='active'>{$i}</li>";
}
 
// TO ENABLE THE NEXT BUTTON
if ($next_btn && $cur_page < $no_of_paginations) {
    $nex = $cur_page + 1;
    $msg .= "<li p='$nex' class='active'> > </li>";
} else if ($next_btn) {
    $msg .= "<li class='inactive'> > </li>";
}
 
// TO ENABLE THE END BUTTON
if ($last_btn && $cur_page < $no_of_paginations) {
    $msg .= "<li p='$no_of_paginations' class='active'> >> </li>";
} else if ($last_btn) {
    $msg .= "<li p='$no_of_paginations' class='inactive'> > </li>";
}
$goto = "<input type='text' class='goto' size='1' style='margin-top:-1px;margin-left:60px;'/><input type='button' id='go_btn' class='go_button' value='Go'/>";
$total_string = "<span class='total' a='$no_of_paginations'>Page <b>" . $cur_page . "</b> / <b>$no_of_paginations</b></span>";
$msg = $msg . "</ul>" . $goto . $total_string . "</div>";  // Content for pagination
echo $msg;
}

**rawsrc** · 10/03/2013, 10h38

Salut,

As-tu un peu potassé la doc PDO parce que vu le code que tu montres, je ne le crois pas.

Dans ce bout de code

Code :

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

1
2
3
4
5
6
$query_pag_data = "SELECT * from crud_categorie LIMIT $start, $per_page";
$result_pag_data = $con->prepare($query_pag_data );
$result_pag_data->execute();
$result_pag_data->bindColumn(1, $id);
$result_pag_data->bindColumn(2, $categorie);
$result_pag_data->bindColumn(3, $description);

Je voudrais savoir où sont présentes dans le SQL les colonnes auxquelles tu rattaches des valeurs ?

**sabotage** · 10/03/2013, 16h37

Ca ne sert à rien d'utiliser la préparation des requêtes quand elles n'ont pas de paramètres

Code :

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

mysql_query($query_pag_data)

=>

Code :

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

$con->query($query_pag_data)

**ABCIWEB** · 10/03/2013, 17h07

Code :

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

1
2
3
4
5
$query_pag_num = "SELECT * FROM crud_categorie";
$result_pag_num = $con->prepare( $query_pag_num );
$result_pag_num->execute();
$count = $result_pag_num->rowCount();
echo $count;

c'est pas optimisé, pour faire un simple comptage

=>

Code :

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

1
2
3
4
5
 
$query_pag_num = "SELECT count(*) FROM crud_categorie";
$result_pag_num = $con->query($query_pag_num);
$count = $result_pag_num->fetchColumn();
echo $count;

**dancom5** · 10/03/2013, 19h36

Merci pour vos réponses.

rawsrc, je me base toujours en premier sur php.net pour tout ce qui
touche PHP et MYSQL. Mais, en essayant de convertir, j'ai fais des
erreurs et j'ai pas tout vu non plus. Je suis débutant en PDO.

rawsrc, j'ai mal complété:

sabotage, quand on parle de paramètres, c'est ceci?

Code :

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

ici->$result_pag_data->bindColumn(1, $id);

là où j'ai mis ici?

Les données sortait quand même. Mais, je veux faire les choses correcte.

Le code faisant object de ce topic c'est un exemple de pagination que
je tentais de comprendre pour convertir en PDO. C'est ajax, jquery.

Mon code à moi, c'est celui la que je vais adapter pour ajouter
la pagination.

Je vois que ma page de pagination est directement sur la page.
Alors qu'il faut que je fasse une page: load_data.php.

Je vais avoir fort à faire.

**ABCIWEB** · 10/03/2013, 20h26

Envoyé par dancom5

je me base toujours en premier sur php.net

Pas assez, tu n'as pas compris les requêtes préparées. Elles s'appuient sur des marqueurs nommés ou des marqueurs interrogatifs et il n'y en a pas dans tes requêtes, c'est ce que t'a dit rawsrc. Et sabotage disait qu'il ne faut pas toujours faire des requêtes préparées car c'est inutile et je t'ai donné un exemple dans mon dernier message.

**dancom5** · 21/03/2013, 04h03

Je pense que c'est bon comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
$query_pag_data = "SELECT * from crud_categorie LIMIT $start, $per_page";
 
$msg = "";
 
foreach ($con->query($query_pag_data) as $row) {
    $msg .= "<li><b>" . $row['id'] . "</b> " . $row['categorie'] . "</li>";
}
$msg = "<div class='data'><ul>" . $msg . "</ul></div>"; // Content for Data
 
/* --------------------------------------------- */
 
$query_pag_num = "SELECT count(*) FROM crud_categorie";
$result_pag_num = $con->query($query_pag_num);
$count = $result_pag_num->fetchColumn();

**ABCIWEB** · 21/03/2013, 04h52

Oui tant que les variables $start et $per_page ne sont pas des variables utilisateur (sinon vaudrait mieux les protéger ou les caster préalablement en entiers).

Au passage si ça t'intéresse y'a un topic ici assez documenté sur un sujet très similaire au tiens (migration de mysql vers pdo). Le 11ème message de ce topic pourrait t'intéresser notamment si tu avais à faire des requêtes préparées avec des paramètres utilisateurs dans la clause limit.

**dancom5** · 21/03/2013, 05h25

C'est en parti utilisateur:

Code :

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

1
2
3
4
5
6
7
8
9
$page = $_POST['page'];
$cur_page = $page;
$page -= 1;
$per_page = 1;
$previous_btn = true;
$next_btn = true;
$first_btn = true;
$last_btn = true;
$start = $page * $per_page;

$start est en parti utilisateur à cause de $page.

Merci pour la suggestion du topic ABCIWEB, je vais voir.
Merci aussi pour vos intervention rawsrc et sabotage.

**ABCIWEB** · 21/03/2013, 06h40

Oui ben dans ces cas là si tu attends un entier dans $_POST['page'] tu peux simplement le caster en entier et lui donner une valeur par défaut si la valeur fournie n'est pas numérique donc je ferais :

Code :

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

$page = isset($_POST['page']) && is_numeric($_POST['page']) ? intval($_POST['page']) : 0 ;

avec ça $page est suffisamment contrôlée.

**dancom5** · 21/03/2013, 06h45

Merci

Et dans le cas, où j'aurais un second paramétré comme ceci :

Code :

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

?page=1&section=2

**ABCIWEB** · 21/03/2013, 07h24

Ben là apparemment c'est des variables GET et si tu attends des entiers tu peux faire le même principe que pour l'exemple avec POST['page']. Où vois-tu un problème ?

Code :

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

1
2
$page = isset($_GET['page']) && is_numeric($_GET['page']) ? intval($_GET['page']) : 0 ;
$section = isset($_GET['section']) && is_numeric($_GET['section']) ? intval($_GET['section']) : 0 ;

(faire attention quand même que la valeur 0 par défaut soit correcte)
C'était pas bien difficile à trouver, faudrait que tu bosse un peu ...

Après si tu veux utiliser les dernières fonctions à la mode tu peux utiliser filter_input mais dans ce cas de figure tu n'y gagnera rien.

**dancom5** · 18/06/2013, 13h26

Merci pour l'aide.

J'ai approfondie mes connaissances générale en PDO.

C'est vrai que c'est plutôt agréable son utilisation.

Convertir en PDO [PDO]

PHP & Base de données

Discussions similaires

Partager

Partager