Perlas del código on PsiloMan

¿Ha dicho usted factorización?

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

Creo que le pagaban por línea de código.

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 '%''%' )

Código de la función 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;
}

Yo hago TRIM, tú haces TRIM, él hace TRIM

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

Nunca se sabe si el primero no ha funcionado. Un TRIM advertido vale por dos.

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

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

Una comprobación inútil

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

Tras recuperar un carácter de la cadena contenida en el array, comprueba si el resultado puede ser igual a una palabra.

$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';

Código de la función remove_everything_but_letter_space:

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

La concatenación es optimización

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 concatenación de “\r\n” con ‘;MR;’ optimiza la ejecución del script PHP. Yo habría escrito:

$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)

Pero solo soy autodidacta.

Una confirmación del usuario que nunca se mostrará

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

Tras leer el código, le pregunté si lo había probado. La respuesta fue, por supuesto, que sí. Mi respuesta fue que creo que hay un pequeño error hacia la línea 67 de su 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;
	}
}

Un script un poco voraz

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

15 minutos de ejecución con 2 GB de memoria.

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

Una condición siempre verdadera

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

Esta pequeña perla la escribió alguien que dice haber estudiado en el mejor IUT (instituto universitario de tecnología francés) de informática, en Orsay. Creo que se debió de saltar algunas clases.

if (1) {
 // 400 líneas de código
 ...
}

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