Discussion :

[Oprichnik]

Voici une portion de code que je viens de découvrir :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
if (isset($changeemail) && !isset($password) ) {
		$sql .= " email = '".Database::escape_string($changeemail)."' ";
	} elseif (isset($password) && isset($changeemail)) {
		$sql .= " email = '".Database::escape_string($changeemail)."', ";
		$password = api_get_encrypted_password($password);
		$sql .= " password = '".Database::escape_string($password)."'";
	} elseif (isset($password) && !isset($changeemail)) {
		$password = api_get_encrypted_password($password);
		$sql .= " password = '".Database::escape_string($password)."'";
	} else {
		// remove trailing , from the query we have so far
		$sql = rtrim($sql, ',');
	}

Heureusement qu'il ne fait pas ça pour tout le formulaire

[pmithrandir]

A par le fait qu'il aurait pu regrouper ces tests 2 a 2 pour alleger de quelques instructions son code, et peut être utiliser des variables pour rendre le tout plus lisible, ou est le problème ?

2 variable avec 2 valeurs possible(enfin, 2 existance possible) ca fait bien 4 cas a traiter non ?

[Oprichnik]

Imagine un formulaire avec 5 champs, ça fait 2^5, ça fait 32 test.

Il faut faire un test par champ :

Code :

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

1
2
3
4
5
6
7
8
9
10

if (isset($changeemail) ) {
		$sql .= " email = '".Database::escape_string($changeemail)."' ";
	}

if (isset($password)) {
		$password = api_get_encrypted_password($password);
		$sql .= " password = '".Database::escape_string($password)."'";
	}