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 !