correction d'un code php

**ayoubo** · 01/03/2012, 22h45

Bonsoir à tous !

Je suis débutant en PHP et je souhaite réaliser un premier site dynamique de l'inscription d'un personne . et je veux un(e) personne de corriger les erreurs dans ce code la bas et merci d'avance :
pour la classe personne contient les variables de personne et fonction ajouter supprimer modifier.
la classe afficher les données enregistrer dans la base de donnée
et avoir deux button modifier et supprimer par la récupération de id.

classe personne :

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
106
107
108
109
110
111
112
113
114
<?php
class personne
{
private $id;
private $nom;
private $prenom;
private $adresse;
private $cd;
private $telephone;
 
// les acesseurs id
 
public function getid()
{
return $this->id;
}
public function setid($id)
{
 
        $this->id=$id;
 
}
 
//les acesseurs nom
public function getnom()
{
return $this->nom;
}
public function setnom($nom)
{
return $this->nom=$nom;
}
 
//les acesseurs de prenom
public function getprenom()
{
return $this->preonm=prenom;
}
public function setprenom($prenom)
{
$this->prenom=$prenom;
}
 
//les acesseurs de adresse
public function getadress()
{
return $this->adress;
}
public function setadress($adress)
{
$this->adress=$adress;
}
// les acesseurs codepostal
public function getcd()
{
return $this->cd;
}
public function setcd($cd)
{
$this->cd=$cd;
}
//les acesseurs de tel
public function gettelephone()
{
return $this->telephone;
}
public function settelephone($telephone)
{
$this->telephone=telephone;
}
 
public function ajouter($id,$nom,$prenom,$adress,$cd,$telephone)
{
$con=mysql_select_db('localhost','root');
mysql_select_db('infos',$con);
mysql_query('insert into personnes values("'$this->nom'","'$this->prenom'","'$this->adresse'","'$this->cp'",'$this->telephone')');
echo'<script type="text/javascript">
alert("ajout effectuer");
</script>';
}
//function modifier
 
public function modifier()
{
$con=mysql_select_db('localhost','root');
mysql_select_db('infos',$con);
mysql_query(UPDATE personnes set nom=$this->setnom(),prenom=$this->setprenom(),adress=$this->setadress(),cd=$this->setcd(),
telephone=$this->settelephone() where id='$id');
echo'<script type="text/javascript"> alert ("opération effectuer ");</script>';}
 
 
 
 
//fonction supprimer
public function supprimer($id)
{
$con=mysql_connect('localhost','root');
mysql_select_db('infos',$con);
$sql=("Delete from personnes where id="$id);
echo $sql;
$requete=mysql_query($sql,$con);
 
if($requete)
  {
    echo("La suppression à été correctement effectuée") ;
  }
  else
  {
    echo("La suppression à échouée") ;
  }
 
}
 
?>

page d'inscription :

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
<?php
include("connexion.class.php");
$con=new connexion;
include("exception.class.php");
include("personne.class.php");
 
if(isset($_post['ok']))
{
if(!empty($_POST['nom'] and !empty($_POST['prenom'] and !empty($_POST['adresse'] and !empty($_POST['codepostal'] and !empty($_POST['telephone'] ))
{
$p=new personne();
$p->setnom=$_POST['nom'];
$p->setprenom=$_POST['prenom'];
$p->setadresse=$_POST['adresse'];
$p->setcd=$_POST['codepostal'];
$p->settelephone=$_POST['telephone'];
$p->ajouter();
}
}
 
?>

classe afficher

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<?php
class afficher()
 
{
public function affichertout
{
$con=mysql_connect('localhost','root');
mysql_select_db('infos',$con);
mysql_query("select * from personnes");
while ($data=mysql_fetch_array($cm)) 
{echo(" id est :".$data['id']."  nom ".$data['nom']."  prenom  : ".$data['prenom']."adresse est :".$data['adresse']."codepostal est :".$data['cd']."telephone est :".$data['telephone']"<br>";}
}
}
?>

page inscription

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
<?php
include("connexion.class.php");
include("exception.class.php");
include("cafficher.class.php");
include("personne.class.php");
 
// debut de tabelau
echo '<table bgcolor="#FFFFFF">'."\n";
        // premiere ligne on affiche les titres de tables personnes
        echo '<tr>';
		  echo '<td bgcolor="#669999"><b><u>id</u></b></td>';
        echo '<td bgcolor="#669999"><b><u>Nom</u></b></td>';
        echo '<td bgcolor="#669999"><b><u>Prenom</u></b></td>';
      echo '<td bgcolor="#669999"><b><u>Adresse</u></b></td>';
        echo '<td bgcolor="#669999"><b><u>Code Postal</u></b></td>';
      echo '<td bgcolor="#669999"><b><u>Telephone</u></b></td>' ;
	  echo '<td bgcolor="#669999"></td>' ;
 
      echo '</tr>'."\n";
	  // lecture et affiche   
    while($data = mysql_fetch_array($cm)) {
        echo '<tr>';
        echo '<td bgcolor="#CCCCCC">'.$data["nom"].'</td>';
        echo '<td bgcolor="#CCCCCC">'.$data["prenom"].'</td>';
      echo '<td bgcolor="#CCCCCC">'.$data["adresse"].'</td>';
        echo '<td bgcolor="#CCCCCC">'.$data["cp"].'</td>';
      echo '<td bgcolor="#CCCCCC">'.$data["ville"].'</td>';
	  echo'<td bgcolor="#CCCCCC"><input name="supprimer" type="button" value="supprimer" ><input name="modifier" type="button" value="modifier" onclick="window.location.href="'modifier.php'"></td>'
      echo '</tr>'."\n";
	  if(isset($_post['supprimer'] && $id=$_get["id"])
	  {
	  $pr=new personne()
	  $pr->supprimer($id);
	  }
    }
    echo '</table>'."\n";
    // fin du tableau.
}
 
 
?>

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
<?php
include("connexion.class.php");
include("exception.class.php");
include("personne.class.php");
 
$id = $_GET["id"] ;
$result->nom;
$result->prenom;
$result->adresse;
$result->cd;
$result->telephone;
 
&requete=personne.modifier($id)
if( $result = mysql_fetch_object( $requete ) )
  {
  ?>
<form name="insertion" action="modification3.php" method="POST">
  <input type="hidden" name="id" value="<?php echo($id) ;?>">
  <table border="0" align="center" cellspacing="2" cellpadding="2">
    <tr align="center">
      <td>nom</td>
      <td><input type="text" name="nom" value="<?php echo($result->nom) ;?>"></td>
    </tr>
    <tr align="center">
      <td>prenom</td>
      <td><input type="text" name="prenom" value="<?php echo($result->prenom) ;?>"></td>
    </tr>
    <tr align="center">
      <td>adresse</td>
      <td><input type="text" name="adresse" value="<?php echo($result->adresse) ;?>"></td>
    </tr>
    <tr align="center">
      <td>code postal</td>
      <td><input type="text" name="codePostal" value="<?php echo($result->cp) ;?>"></td>
    </tr>
    <tr align="center">
      <td>numero de telephone</td>
                                        <td><input type="text" name="telephone" value="<?php echo($result->telephone) ;?>"></td>
    </tr>
    <tr align="center">
      <td colspan="2"><input type="submit" value="modifier"></td>
    </tr>
  </table>
</form>
 
?>
}
?>

classe de exception

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
 
<?php
//gestion des erreurs avec les exceptions 
class erreur extends exception 
{
public function __construct($msg)
{
parent::__construct($msg);
}
public function retourneerreur()
{
$msg='<div><strong>'.$this->getmessage().'<strong>'';
var $msg='ligne:' '.$this->getline()''</div>';
return $msg;
}
}
?>

et merci d'avance

**Inazo** · 02/03/2012, 09h29

Bonjour,

Je comprend pas trop votre démarche mais bon

. Vous avez des bugs ou autre dans votre code ?

Dans tous les cas j'ai vu ceci qu'il faudrait éviter dans la page inscription :

Code :

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

1
2
3
4
 if(isset($_post['supprimer'] && $id=$_get["id"])
	  {
	  $pr=new personne()
	  $pr->supprimer($id);

Vous faire X fois un test que ne peut être vrai qu'une seule fois et si il est vrai vous allez le lancer autant de fois qu'il y a de ligne dans votre requête et vous allez vous génerer des erreurs car l'enregistrement aura déjà été supprimer. Ensuite vous avez oubliez un "=" pour votre test ce qui signifie que $id prendra toujours la valeur du GET...

De plus on écrit $_POST et non pas $_post idem pour $_GET et non pas $_get

Cordialement,

correction d'un code php

PHP & Base de données

Discussions similaires

Partager

Partager