Discussion :

[Invité]

Bonjour,
Pour mon premier post, je n'ai aucun soucis. Voir même, que des possibilités.
Dans le cadre de mon apprentissage solitaire au PHP, j'ai écrits plusieurs petits trucs/machins.
Maintenant que je suis un peu plus à l'aise, je souhaite améliorer mes programmes.
Par améliorations, j'entends :

• amélioration du code
• simplication du code
• sécurisation du code

Ne sachant pas trop comment m'y prendre, je lance un appel à ceux qui ont deux minutes, à ceux qui flânent...

Le fichier sert à coder/décoder une phrase/mot/lettre par le système cassis (k=6).

Je ne mets pas le fichier complet... Je reçois à partir d'un input la phrase à coder.

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
 
<?php 
$tableaucode = array ("-","f","g","h","i","j","k","l","m","n","o","p","q","r","s","t","u","v","w","x","y","z","a","b","c","d","e");
if(!empty($_GET['bouton']))
{
	//premier travail verifier si la chaine est vide
 
	if((!empty($_GET['texte'])))
	{
		$codeerreur=0;
		$code = $_GET['texte'];
		$tabMot = explode(" " , $code);
		$nbreTabMot = count($tabMot);
		$i = 0;
 
		$txt ="";
		while ($i < $nbreTabMot) 
		{
			$tabLettre = explode("-",$tabMot[$i]);
			$nbreTabLettre = count($tabLettre);
			$a = 0;
			while ($a < $nbreTabLettre) 
			{
				$txt .= $tableaucode[$tabLettre[$a]];
				$a++;
			}
			$txt .= " ";
			$i++;
		}
 
	}
	else 
	{
		$codeerreur=1;
		$resultat="le texte est vide";
	}
 
 
 
}
if(!empty($_GET['bouton1']))
{
	//premier travail verifier si la chaine est vide
 
	if((!empty($_GET['texte1'])))
	{
		$codeerreur=0;
		$code = $_GET['texte1'];
		$tabMot = explode(" " , $code);
		$nbreTabMot = count($tabMot);
		$i = 0;
 
		$txt ="";
		while ($i < $nbreTabMot) 
		{
			$tabLettre = str_split($tabMot[$i],1);
			$nbreTabLettre = count($tabLettre);
			$a = 0;
			while ($a < $nbreTabLettre) 
			{
				echo $tabLettre[$a];
				$txt .= array_search($tabLettre[$a], $tableaucode);
				$a++;
				if($a < $nbreTabLettre )
				{
					$txt .= "-";
				}
			}
			$txt .= "  ";
			$i++;
		}
 
	}
	else 
	{
		$codeerreur=1;
		$resultat="le texte est vide";
	}
 
 
 
}
 
?>

La première idée qui me vient serait de n'avoir qu'un seul input. C'est le PHP qui viendrait déterminer si il faut coder ou decoder.
Le seconde point serait de sécuriser la variable via un preg_match ?

Qu'en pensez-vous ? Avez-vous d'autres suggestions ?

Merci et bon surf !

[imikado]

Question: "bouton" et "bouton1" peuvent être éxécutée en même temps ?

Code :

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

1
2
3
if(!empty($_GET['bouton']))
(...)
if(!empty($_GET['bouton1']))

oui pour le

Code :

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

preg_match('/^([a-z-]*)$/',$_GET['texte'])

Plusieurs pistes d'amélioration:
1. utiliser des conventions de nommage des variable aCode pour variable code de type array, iNbMot (pour variable NbMot de type integer)
2. factoriser: voir ce qui peut être mutualisé , le principe de DRY (don't repeat yourself )

[Invité]

utiliser des conventions de nommage des variable aCode pour variable code de type array, iNbMot (pour variable NbMot de type integer)

L'idée c'est de nommer les variables pour que l'ont sachent ce qu'elles contiennent et leurs types ?

Code :

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

1
2
3
4
5
6
7
8
if((!empty($_GET['texte'])) AND preg_match("/^[0-9\s\-]+$/", $_GET['texte']))
	{
....
	}
	elseif((!empty($_GET['texte'])) AND preg_match("/^[a-zA-Z\s\-]+$/", $_GET['texte']))
        {
        }
         else{}

En effet j'ai ajouté "preg_match" ce qui me permet de n'avoir plus qu'un input. Ca me permet de sécuriser et d'améliorer.

Je continue à chercher des améliorations.

Merci pour ce premier retour.

[imikado]

L'idée c'est de nommer les variables pour que l'ont sachent ce qu'elles contiennent et leurs types ?

Exact

Code :

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

1
2
3
4
5
6
7
8
if((!empty($_GET['texte'])) AND preg_match("/^[0-9\s\-]+$/", $_GET['texte']))
	{
....
	}
	elseif((!empty($_GET['texte'])) AND preg_match("/^[a-zA-Z\s\-]+$/", $_GET['texte']))
        {
        }
         else{}

En effet j'ai ajouté "preg_match" ce qui me permet de n'avoir plus qu'un input. Ca me permet de sécuriser et d'améliorer.

Je continue à chercher des améliorations.

Merci pour ce premier retour.

Bien, mais vous n'avez pas répondu à ma question: vous avez deux conditions if pour "bouton" et "bouton1" : peuvent-t-ils etre éxécutée en même temps ?

Une chose toute bête peut être également de migrer vers une classe

[Invité]

Les deux boutons s'expliquent. Le premier etait pour un input "décodage". Le second pour un input "codage".
Maintenant je n'ai donc plus qu'un seul input. Je répond a votre question ?

Concernant la classe, cela necessite de la POO ? C'est pour moi une très grande inconnue la POO.

[imikado]

Les deux boutons s'expliquent. Le premier etait pour un input "décodage". Le second pour un input "codage".
Maintenant je n'ai donc plus qu'un seul input. Je répond a votre question ?

Oui c'était ma question

Concernant la classe, cela necessite de la POO ? C'est pour moi une très grande inconnue la POO.

C'est le moment et la bonne base pour s'y mettre vous y arriverez facilement je pense