Redimensionner photo php

Bon, en l'atat, je ne peux pas prendre ton code et le tester car il me manque des fichier inclus.

Ce que je vois comme améliorations pour la taille, la lisibilité et la maintenabilité de ton code :

ne pas faire 40 fois les mêmes opérations sur des variables différentes, mais faire un tableau contenant les clés du tableau $_GET ou $_POST à traiter.

Exemple, remplacer :

$postcode = $_POST['postcode'];
$postcode = strip_tags($postcode);
$postcode = strtoupper($postcode);
$price = $_POST['price'];
$price = strip_tags($price);
$branch = $_POST['branch'];
$branch = strip_tags($branch);
$bedrooms = $_POST['bedrooms'];
$bedrooms = strip_tags($bedrooms);
$address1 = $_POST['address1'];
$address1 = strip_tags($address1);
$address2 = $_POST['address2'];
$address2 = strip_tags($address2);
$address3 = $_POST['address3'];
$address3 = strip_tags($address3); 

Par

$post_vars = array('postcode', 'price', 'bedrooms'.....);

$variable = array();
foreach ($post_vars as $cle => $val) {
	$variable[$cle] = strip_tags($val);
}
$variable['postcode'] = strtoupper($variable['postcode']);

Et ainsi de suite.

Autre truc : toujours utiliser «mysql_escape_strings» pour les chaînes de caractères à mettre dans les requêtes à la base :

Donc :

$Query = "INSERT INTO $table1 VALUES ('0', '$postcode', '$price', '$branch', '$bedrooms', '$address1', '$address2', '$address3', '$agentDescription', '$propWeek', '$agentID', '$imgpath', '$imgpath2')"; 

devient :

$Query = "INSERT INTO $table1 VALUES ('0', 
	'".mysql_escape_strings('$postcode')."', 
	'".mysql_escape_strings('$price')."', 
	'".mysql_escape_strings('$branch')."', 
	'".mysql_escape_strings('$bedrooms')."', 
	'".mysql_escape_strings('$address1')."', 
	'".mysql_escape_strings('$address2')."', 
	'".mysql_escape_strings('$address3')."', 
	'".mysql_escape_strings('$agentDescription')."', 
	'".mysql_escape_strings('$propWeek')."', 
	'".mysql_escape_strings('$agentID')."', 
	'".mysql_escape_strings('$imgpath')."', 
	'".mysql_escape_strings('$imgpath2')."')"; 

Sinon, après une commande de redirection : header("location: $PHP_SELF?message=$message");, toujours ajouter un exit(); (rien ne sert de continuer le script, le client est déjà parti !).


La gestion du <select id="branch" name="branch" size="1" class="branch"> n'est pas très élégante. On pourrait remplacer :

 <?php if(empty($branchSave)){
echo"<option selected='selected'>Yarm</option>
<option>Eaglescliffe</option>
<option>Stokesley</option>
<option>Ingleby Barwick</option>
<option>Middlesbrough</option>
<option>Other Villages</option></select> ";
}else{
$j=0;
$Town = Array('Yarm','Eaglescliffe','Stokesley','Ingleby Barwick','Middlesbrough','Other Villages');

echo "<option selected='selected'>".$branchSave."</option>";


$j=0;
while($j<=5){
if($branchSave==$Town[$j]){
$j=$j+1;
}else{
$name=$Town[$j];
echo "<option>".$name."</option>";
$j=$j+1;
}
}

?> 

Par :

 <?php 
$Town = Array('Yarm','Eaglescliffe','Stokesley','Ingleby Barwick','Middlesbrough','Other Villages');

$chaine_option = ''.
foreach ($Town as $laville) {
	if ($laville == $branchSave) 
		$selected = ' selected="selected"';
	else 
		$selected = '';
	$laville = htmlentities($laville);
	$chaine_option .= "<option$selected>$laville</option>\n"
}

?> 

Au lieu de passer les chaînes de message telles quelles en GET, les mettre en session et faire attention pour les redirections (Location au lieu de location même si généralement ça passe) :

 $message= "Please check the form for missing data in hightlighted fields";
header("location: $PHP_SELF?message=$message&$errorParse"); 

Par exemple :

 $_SESSION['message'] = "Please check the form for missing data in hightlighted fields";
header("Location: $PHP_SELF"); 

Mais en fait, pourquoi se rediriger sur soi-même ?
Il faudrait structurer la page différemment de manière à détecter plus tôt ce types d'erreurs...



Et puis au lieu d'imbriquer 2 requêtes, en faire une puis une autre et c'est tout :

 //Sets the Flag to NO for all properties if the current property is SET to YES
$Query_propWeek= "SELECT id, propWeek FROM $table1 WHERE propWeek = 'Yes'";

$Result_propWeek = mysql_db_query ($DBName, $Query_propWeek, $Link);


while($Row_propWeek = mysql_fetch_array($Result_propWeek)){

$Row = $Row_propWeek[id] ;

$Query_updatePropWeek = "UPDATE $table1 SET propWeek = 'No' WHERE id = $Row";
if (mysql_db_query ($DBName, $Query_updatePropWeek, $Link)){

//echo "Properties Propweek changed";
} 

Deviendrait :

 //Sets the Flag to NO for all properties if the current property is SET to YES
$Query_propWeek= "SELECT id, propWeek FROM $table1 WHERE propWeek = 'Yes'";

$Result_propWeek = mysql_db_query ($DBName, $Query_propWeek, $Link);

$Row = array();
while($Row_propWeek = mysql_fetch_array($Result_propWeek)){

	$Row[] = $Row_propWeek[id] ;
}

$Query_updatePropWeek = "UPDATE $table1 SET propWeek = 'No' WHERE id = ". implode(' or id = ', $Row);
if (mysql_db_query ($DBName, $Query_updatePropWeek, $Link)){

//echo "Properties Propweek changed";
 

Bon, tout ce que j'ai suggéré ici est a priori bon, mais je n'ai pas pu faire l'essai. Il y aurait encore pas mal de trucs à cleaner, notamment pour la gestion des messages d'erreurs...

A mon avis, ce problème d'image est l'arbre qui cache la forêt. Ton code est trop touffu inutilement et difficilement debuggable/modifiable/comprehensible.

Si ton truc tourne lorsque tu l'extrais de son contexte (c'est ce que j'ai cru comprendre) alors c'est que le pb vient d'ailleurs...

En espérant que ça serve.

Raph

Disons que passer par des variables indépendantes '$postcode' pour $_POST['postcode'] ne sert pas beaucoup. Dans mon exemple, je préférerais utiliser $variable['postcode'] plutôt que de l'affecter à '$postcode'.

Les boucles sont à utiliser à plein d'autres endroits:
- Par exemple, là où tu collestes les variables GET vers //Collect GET vars
- Pour constituer ta vairable $savedData
- et peut-être d'autres

De manière générale, lorsque tu fais 2 fois la même chose tu peux te demander si
- tu ne peux pas mettre ce traitement dans une boucle en passant par un tableau
- tu ne peux pas faire une fonction (dans ton code, ça manque)


Raph

C'est pas tout à fait ça mais presque :

foreach ($_POST as $cle => $valeur) {
$variable[$cle] = strip_tags($valeur);
} 

Là, ça devrait marcher. HTTP_POST_VARS c'est de l'ancien temps ;-)

Raph

Ah oui, je m'étais trompé, il ne faut pas mettre entre guillemets :

mysql_escape_string($formValue["postcode"])

Là, ça devrait aller mieux...



Tu sais, ton code, tu devrais le mettre entre <code> et </code> , ça serait plus lisible que juste collé comme ça.


Raph

Bon et maintenant ?

Ok, alors tu peux me filer le code qui marche maintenant ? (n'oublie pas <code> et </code>)

Raph

Il faut faire attention car le serveur Web Apache tourne en tant qu'un certain utilisateur qui n'est pas le même que celui sous lequel tu fais tourner tes applications et sous lequel tu crées les répertoires...

Il faudrait remplacer copy par move_uploaded_file.