Perles du code on PsiloMan

Je TRIM, tu TRIM, il TRIM

Fri, 24 Apr 2015 20:49:00 +0200

On ne sait jamais si le premier n’a pas fonctionné. un TRIM averti en vaut deux.

if (isset($_GET['id_coll'])) $id_coll = trim($_GET['id_coll']);
else $id_coll = '';

if (trim($id_coll)!='' && is_numeric($id_coll))
{
 ......
}

Vous avez dit : Factorisation

Fri, 24 Apr 2015 20:49:00 +0200

Je pense qu’il devait être payé a la ligne de code.

OR (
UPPER('".remove_everything_but_letter_plus($agent_firstname)."') LIKE '%.%'
OR UPPER('".remove_everything_but_letter_plus($agent_firstname)."') LIKE '% %'
OR UPPER('".remove_everything_but_letter_plus($agent_firstname)."') LIKE '%-%'
OR UPPER('".remove_everything_but_letter_plus($agent_firstname)."') LIKE '%''%'
OR UPPER('".remove_everything_but_letter_plus($agent_lastname)."') LIKE '%.%'
OR UPPER('".remove_everything_but_letter_plus($agent_lastname)."') LIKE '% %'
OR UPPER('".remove_everything_but_letter_plus($agent_lastname)."') LIKE '%-%'
OR UPPER('".remove_everything_but_letter_plus($agent_lastname)."') LIKE '%''%' )

Code de la function remove_everything_but_letter_plus :

function remove_everything_but_letter_plus($string)
{
	$string = preg_replace('/[^A-Za-z \'\.\-]/', '', $string);
	$string = sql_escape($string);
	return $string;
}

Un test inutile

Mon, 04 Feb 2013 15:00:00 +0100

Après avoir récupérer un caractère de la chaîne de caractères contenu dans le tableau, il teste si le résultat peu être égal à un mot.

$user_regime = trim(strtoupper(remove_everything_but_letter_space(substr($tabinfouser[17], 0, 1))));
if ($user_regime == 'ALPHA') $user_regime = 'A';
else if ($user_regime == 'BETA') $user_regime = 'B';
else if ($user_regime == 'OMEGA') $user_regime = 'O';
else if ($user_regime == 'EPSILON') $user_regime = 'E';

Code de la function remove_everything_but_letter_space :

function remove_everything_but_letter_space($string)
{
	$string = preg_replace('/[^A-Za-z \-]/', '', $string);
	return $string;
}

La concaténation c'est de l'optimisation

Sun, 03 Feb 2013 15:15:00 +0100

$content = str_replace('{#(Pr@uT[};MR;', "\r\n".';MR;', $content);
$content = str_replace('{#(Pr@uT[};MME;', "\r\n".';MME;', $content);
$content = str_replace('{#(Pr@uT[};MLLE;', "\r\n".';MLLE;', $content)

La concaténation de “\r\n” avec ‘;MR;’ optimise l’éxécution du script php. Moi j’aurai écrit :

$content = str_replace('{#(Pr@uT[};MR;', "\r\n;MR;", $content);
$content = str_replace('{#(Pr@uT[};MME;', "\r\n;MME;", $content);
$content = str_replace('{#(Pr@uT[};MLLE;', "\r\n;MLLE;", $content)

Mais je ne suis qu’un autodidacte.

Une confirmation par l'utilisateur que l'on n'affichera jamais

Sat, 02 Feb 2013 17:00:00 +0100

Après avoir lu le code, je lui demande si il l’a testé. La réponse étant bien sur que oui. Ma réponse étant je pense que tu as un petit bug vers la ligne 67 de ton script.

function confirmation() {
	if (trim(document.getElementById('ANIMATION').value) == '' ) {
		alert('Merci de remplir les champs du formulaire avant validation');
		return false;
	} else {
		return true;
	}

	var answer = confirm('En cliquant sur le bouton "Valider définitivement" les informations que vous avez saisie, cliquez sur OK, sinon, cliquez sur Annuler');
	if (answer) {
		return true;
	} else {
		return false;
	}
}

Script un peu gourmand

Fri, 01 Feb 2013 14:30:00 +0100

15 minutes d’exécution avec 2 Go de mémoire.

set_time_limit(900);
ini_set('memory_limit','2048M');
error_reporting(E_ALL);

Une condition vrai pour toujours

Thu, 31 Jan 2013 15:00:00 +0100

Cette petite perle a été écrite par une personne qui dit avoir étudié dans le meilleur IUT d’informatique (ORSAY). Je pense qu’il a du raté certains cours.

if (1) {
 // 400 lignes de code
 ...
}

if (isset($details_csv)) return $details_csv;
else return '';