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