Mes erreurs, prendre de bonnes habitudes

**Goodgame** · 07/08/2008, 11h11

Bonjour.

J'avais essayé de me mettre au php il y a quelques mois, mais ma fainéantise m'a poussé a arreter :-°
Donc la je m'y remet plus sérieusement (enfin j'espere cette fois ci) et j'essai d'apprendre a me servir d'emacs et a indenter d'une facon correcte par la meme occasion.

Je ressort donc le tp sur le minichat que j'avais fait dans le temps, pour que vous m'aidiez a corriger la merde que j'ai fait, pour arriver a un truc correct et a prendre de bonnes habitudes.

J'ai aussi essayé (sans grand succès) d'indenter mon texte pour qu'il tienne en 70 (72 je crois en fait) lignes.

Donc voila le code

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
 
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.
org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" lang="fr">
    <head>
        <title>Mini-chat</title>
        <meta http-equiv="Content-Type" content="text/html; charset=iso-
	8859-1" />
    </head>
    <style type="text/css">
    	form
    	{
    	text-align:center;
    	}
    </style>
    <body>
		<?php
		if (isset($_POST['pseudo']) AND isset($_POST['message'])
		   ) // si les variables existent
		{
			if ($_POST['pseudo'] != NULL AND
                           $_POST['message'] != NULL)
			{
		   $pseudo = mysql_real_escape_string
                   (html_ specialchars($_POST['pseudo']));
				$message = mysql_real_escape_string
                               (html_specialchars($_POST['message']));
				mysql_connect("localhost", "root");
				mysql_select_db("test");
				mysql_query("INSERT INTO minichat VALUES
				("","$pseudo","$message")");
			}	
		}	
 
		$reponse = mysql_query ("SELECT * FROM minichat");
		while ($affiche = myslq_fetch_aray($reponse) ;
		{
			?> <strong> <?php echo "$affiche['pseudo'] :"; ?
			> </strong> <?php echo "$affiche['message']"; ?>
			<form method="post" action="tp2.php">
				<p>
					<label>Pseudo: </label>
					<input type="text" name="pseudo"
					          id="pseudo">
					<label>Message: </label>
					<input type="text" name="message
							" id="message">
				</p>	
			</form>	
		}
                        ?>

Merci a ceux qui m'aideront.

**MaitrePylos** · 07/08/2008, 11h33

Bonjour, personnellement, j'aurais écrit ton code 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
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
 
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.
org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" lang="fr">
    <head>
        <title>Mini-chat</title>
        <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
    </head>
    <style type="text/css">
    	form
    	{
			text-align:center;
    	}
    </style>
    <body>
		<?php
		if (isset($_POST['pseudo']) AND isset($_POST['message'])) // si les variables existent
		{
			if ($_POST['pseudo'] != NULL AND $_POST['message'] != NULL)
			{
				$pseudo = mysql_real_escape_string(htmlspecialchars($_POST['pseudo']));
				$message = mysql_real_escape_string(htmlspecialchars($_POST['message']));
				mysql_connect("localhost", "root");
				mysql_select_db("test");
				mysql_query("INSERT INTO minichat VALUES('".$pseudo."','".$message."')");
			}
		}
 
		$reponse = mysql_query ("SELECT * FROM minichat");
		while ($affiche = myslq_fetch_aray($reponse)) 
		{
			?>  <strong> <?php echo $affiche['pseudo'] ; ?>
			 	</strong> <?php echo $affiche['message']; ?>
			<form method="post" action="tp2.php">
				<p>
					<label>Pseudo: </label>
					<input type="text" name="pseudo" id="pseudo">
					<label>Message: </label>
					<input type="text" name="message" id="message">
				</p>	
			</form>	
		<?php } ?>

Maintenant, il est clair qu'on peut l'améliorer, notamment en sortant de cette page tout ce qui ne devrais pas s'y trouver:

Tel que ta connexion à la base de données.
Le fait de couper tes lignes, un peu n'importe où.
Tu dois faire attention à tes quotes et double-quotes.
Faire attention au fermeture de balise(dans ton while notamment)

Je te conseillerais au départ d'utiliser un IDE, qui repère ce genre d'erreurs, cela facilite beaucoup l'apprentissage.

**Goodgame** · 07/08/2008, 14h50

Merci beaucoup.

Je prefere rester sous emacs, c'est toujours mieux de prendre les bonnes habitudes des le début.

Sinon, pour bien (re)commencer, est ce que ce cours est conseillé http://g-rossolini.developpez.com/tu...e=introduction ?

**MaitrePylos** · 07/08/2008, 15h01

Je pense que effectivement tu trouveras de très bonne pratiques dans ce cours, qui est au demeurant très complet.

N'hésite jamais non plus de faire un tour dans la bibliothèque des cours de PHP.

Bon courage.

**Goodgame** · 08/08/2008, 19h43

Ok, merci beaucoup.
Je vais donc me lancer dans ce cours tout doucement.

Mes erreurs, prendre de bonnes habitudes

Langage PHP

Discussions similaires

Partager

Partager