effacer l'image avant d'enregistrer la nouvelle

[artichaudd]

Bonjour!

Voila avant d'enregistrer mon image, que je renomme 1.son_extension
je dois vérifier si il n'en existe pas déjà une dans le dossier, qui pourrait s'apeller genre 1.png, ou 1.jpg, ou 1.gif,
si oui il faut que je l'efface, avec un unlink()
avant d'enregistrer la nouvelle, pour ne pas que cohabites deux images commençant pas 1.quelque chose

la ou je suis embeter c'est pour la manip a faire pour vérifier si il existe déjà une image commençant par "1."

Ensuite j'enregistre la nouvelle, ca c'est bon
et enfin je l'affiche en
récupérant son extension indispensable ( je pense) pour l'afficher :
exemple :

voici ou j'en suis

Code :

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
<?php
if(isset($_FILES['monfichier1']) AND $_FILES['monfichier1']['error'] == 0 AND $_FILES['monfichier1']['size'] <= 1000000)
{
 
	$dossier = '../images/uploads-atelier/normal/'.$nomDossier;
 
	if(!is_dir($dossier)) //si le dossier n'existe pas
	mkdir($dossier); //Tente de créer un dossier.
 
	// Testons si l'extension est autorisée
	$infosfichier = pathinfo($_FILES['monfichier1']['name']);
	$extension_upload = $infosfichier['extension'];
	$extensions_autorisees = array('jpg', 'jpeg', 'gif', 'png');
	if (in_array($extension_upload, $extensions_autorisees))
	{
		$open=opendir($dossier);
 
		while (($search = readdir($open)) !== false)
		{
			//là je dois effacer l'image commancant par "1."
		}
		closedir($open);
		//une fois l'image effacée , j'enregistre la nouvelle
		if(move_uploaded_file($_FILES['monfichier1']['tmp_name'], $dossier.'/1.'.$extension_upload))
		  echo '<br />L\'envoi du fichier photo1 à bien été effectué !';
		else
		  echo '<br />Erreur d\'enregistrement du fichier.';
		 // j'affiche la nouvelle image 
		$extension = pathinfo($path.'/1.', PATHINFO_EXTENSION);//je récupère l'extention
		echo '<img src="1.'.$extension.'" alt="image"/>';// j'affiche la nouvelle image 
	}	
}
?>

[gwinyam]

Ok dans l'idée, il ne te reste donc plus qu'à vérifier si l'image existe, ou plus précisément le fichier existe, et ça se passe par là: http://php.net/manual/fr/function.file-exists.php
J'espère que ça sera bon avec ça

[artichaudd]

voila mon code mis ajour grace a tes conseils
pour le file_exists et le unlink, est'il obligatoire de présiser l'extention?

et a la fin ma méthode pour retrouver l'extention de mon image afin de l'afficher est'elle correcte?

merci

Code :

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
 
<?php
if(isset($_FILES['monfichier1']) AND $_FILES['monfichier1']['error'] == 0 AND $_FILES['monfichier1']['size'] <= 1000000)
{
 
	$dossier = '../images/uploads-atelier/normal/'.$nomDossier;
 
	if(!is_dir($dossier)) //si le dossier n'existe pas
	mkdir($dossier); //Tente de créer un dossier.
 
	// Testons si l'extension est autorisée
	$infosfichier = pathinfo($_FILES['monfichier1']['name']);
	$extension_upload = $infosfichier['extension'];
	$extensions_autorisees = array('jpg', 'jpeg', 'gif', 'png');
	if (in_array($extension_upload, $extensions_autorisees))
	{
		$open=opendir($dossier);
 
		while (($search = readdir($open)) !== false)
		{
			//là je dois effacer l'image commancant par "1."
			if(file_exists ( $dossier.'/1.' )) // est'il obligatoire de présiser l'extention
			{
				unlink($dossier.'/1.'); // la aussi est'il obligatoire de présiser l'extention
			}
		}
		closedir($open);
		//une fois l'image effacée , j'enregistre la nouvelle
		if(move_uploaded_file($_FILES['monfichier1']['tmp_name'], $dossier.'/1.'.$extension_upload))
		  echo '<br />L\'envoi du fichier photo1 à bien été effectué !';
		else
		  echo '<br />Erreur d\'enregistrement du fichier.';
		 // j'affiche la nouvelle image 
		$extension = pathinfo($path.'/1.', PATHINFO_EXTENSION);//je récupère l'extention
		echo '<img src="1.'.$extension.'" alt="image"/>';// j'affiche la nouvelle image 
	}	
}
?>

[gwinyam]

PATHINFO_EXTENSION est spécifiquement voué à faire ce que tu souhaites, donc ok pour l'extension.

Par contre, oui, tu dois récupérer l'extension systématiquement. Vu que tu ne sais pas toujours si ça sera un JPEG, un GIF, un truc, un bidule ou encore une carotte, récupérer l'extension est ta seule façon de le savoir.

[artichaudd]

je n'ai pas tres bien compris ce que tu m'as dis
1) tu veux dire que je dois d'abord récupérer l'extension de mon ancienne image (avec pathinfo() ) avant de pouvoir l'effacer
car le unlink a besoin de l'extension pour fonctionner

2) le pathinfo() lui n'a pas besoin de l'extension , il lira et renverra les infos fichier commençant par "1."?

voir les modifs de mon code a l'effacement et a l'affichage de mes images :

Code :

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
<?php
if(isset($_FILES['monfichier1']) AND $_FILES['monfichier1']['error'] == 0 AND $_FILES['monfichier1']['size'] <= 1000000)
{
 
	$dossier = '../images/uploads-atelier/normal/'.$nomDossier;
 
	if(!is_dir($dossier)) //si le dossier n'existe pas
	mkdir($dossier); //Tente de créer un dossier.
 
	// Testons si l'extension est autorisée
	$infosfichier = pathinfo($_FILES['monfichier1']['name']);
	$extension_upload = $infosfichier['extension'];
	$extensions_autorisees = array('jpg', 'jpeg', 'gif', 'png');
	if (in_array($extension_upload, $extensions_autorisees))
	{
		$open=opendir($dossier);
 
		while (($search = readdir($open)) !== false)
		{
			//là je dois effacer l'image commancant par "1."
			if(file_exists ( $dossier.'/1.' )) // est'il obligatoire de présiser l'extention
			{
				$extension = pathinfo($dossier.'/1.', PATHINFO_EXTENSION);//je récupère l'extention
				unlink($dossier.'/1.'.$extension); // la aussi est'il obligatoire de présiser l'extention
			}
		}
		closedir($open);
		//une fois l'image effacée , j'enregistre la nouvelle
		if(move_uploaded_file($_FILES['monfichier1']['tmp_name'], $dossier.'/1.'.$extension_upload))
		  echo '<br />L\'envoi du fichier photo1 à bien été effectué !';
		else
		  echo '<br />Erreur d\'enregistrement du fichier.';
		 // j'affiche la nouvelle image 
		$extension = pathinfo($dossier.'/1.', PATHINFO_EXTENSION);//je récupère l'extention
		echo '<img src="'.$dossier.'1.'.$extension.'" alt="image"/>';// j'affiche la nouvelle image 
	}	
}
?>

[gwinyam]

Ah euh non, t'as pas forcément de récupérer spécifiquement plus l'extension qu'autre chose.
L'extension fait partie du nom de fichier.

Pour effacer un fichier, il faut que tu connaisses le nom complet de ce fichier. Si il a une extension, il te faut cette extension, tout comme si il n'est pas à la racine, il te faut l'adresse complète. Qui l'un et l'autre normalement te sont fournis de façon générale par le path.

[artichaudd]

donc le pathinfo() lui n'a pas besoin de l'extension , il lira et renverra les infos fichier commençant par "1."?

le file_exists() n'a pas non plus besoin de l'extention

mais le unlink en a besoin, c'est ca

mon code te semble t'il logique?

Code :

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
<?php
if(isset($_FILES['monfichier1']) AND $_FILES['monfichier1']['error'] == 0 AND $_FILES['monfichier1']['size'] <= 1000000)
{
 
	$dossier = '../images/uploads-atelier/normal/'.$nomDossier;
 
	if(!is_dir($dossier)) //si le dossier n'existe pas
	mkdir($dossier); //Tente de créer un dossier.
 
	// Testons si l'extension est autorisée
	$infosfichier = pathinfo($_FILES['monfichier1']['name']);
	$extension_upload = $infosfichier['extension'];
	$extensions_autorisees = array('jpg', 'jpeg', 'gif', 'png');
	if (in_array($extension_upload, $extensions_autorisees))
	{
		$open=opendir($dossier);
 
		while (($search = readdir($open)) !== false)
		{
			//là je dois effacer l'image commancant par "1."
			if(file_exists ( $dossier.'/1.' )) // est'il obligatoire de présiser l'extention
			{
				$extension = pathinfo($dossier.'/1.', PATHINFO_EXTENSION);//je récupère l'extention
				unlink($dossier.'/1.'.$extension); // la aussi est'il obligatoire de présiser l'extention
			}
		}
		closedir($open);
		//une fois l'image effacée , j'enregistre la nouvelle
		if(move_uploaded_file($_FILES['monfichier1']['tmp_name'], $dossier.'/1.'.$extension_upload))
		  echo '<br />L\'envoi du fichier photo1 à bien été effectué !';
		else
		  echo '<br />Erreur d\'enregistrement du fichier.';
		 // j'affiche la nouvelle image 
		$extension = pathinfo($dossier.'/1.', PATHINFO_EXTENSION);//je récupère l'extention
		echo '<img src="1.'.$extension.'" alt="image"/>';// j'affiche la nouvelle image 
	}	
}
?>

[gwinyam]

C'est tous la même chose (PHP même si des fois semble bizarre sur sa syntaxe est quand même un langage relativement bien conçu).
Si l'un a besoin de l'extension, tous en ont besoin. Mais tu n'as pas besoin systématiquement de choper "PATHINFO_EXTENSION" pour la mettre à la suite de ton nom de fichier vu que l'extension fait partie intégrante de base de ton nom de fichier.
Bon ceci dit, dans ton script, vu que tu forces le nom court du fichier à "1.", là effectivement, tu as besoin de préciser l'extension.
Pour le côté logique, je ne l'ai pas exécuté, donc à vue d'oeil, ça m'a l'air bon comme ça, le mieux est que tu testes, si ça roule, c'est que c'est logique donc bon.

[artichaudd]

ok je vais le tester
une derniere observation
et vrai je n'ai pas besoin de mon opendir() et readdir()?
je peux mettre directement if(file_exists.....

[gwinyam]

en effet, file_exists contrôle si le fichier existe selon l'arborescence fournie. L'open_dir et read_dir sont donc une perte de temps mais en plus et surtout une source potentielle inutile d'erreurs supplémentaires à traiter.
N'oublie pas le bouton si ça marche

[artichaudd]

encore une dernière précision car je suis un peu dur de la comprenette
tu me confirme aussi que pour le file_exists() et le path info(), je n'ai pas besoin de préciser l'extension?

[gwinyam]

En effet.

[artichaudd]

je dois avoir une erreur
ca m'enregistre bien mon image
mais ca ne me supprime pas l'ancienne image nommer "1."
ce qui fait que dans mon dossier cohabite
1.png et 1.jpeg

voici mon code

Code :

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
<?php
// Testons si le fichier n'est pas trop gros
if (isset($_FILES['monfichier1']) AND $_FILES['monfichier1']['error'] == 0 AND ($_FILES['monfichier1']['size'] <= 1000000))
{
	$dossier = '../images/uploads-atelier/normal/'.$nomDossier;
 
	if(!is_dir($dossier)) //si le dossier n'existe pas
		mkdir ($dossier); //Tente de créer un dossier.
 
	// Testons si l'extension est autorisée
	$infosfichier = pathinfo($_FILES['monfichier1']['name']);
	$extension_upload = $infosfichier['extension'];
	$extensions_autorisees = array('jpg', 'jpeg', 'gif', 'png');
	if (in_array($extension_upload, $extensions_autorisees))
	{
		if(file_exists ( $dossier.'/1.' )) // si il existe un fichier commencant par "1."
		{
			$extension = pathinfo($dossier.'/1.', PATHINFO_EXTENSION);//je récupère son extention indispensable pour unlink()
			unlink($dossier.'/1.'.$extension); // et je supprime le précèdant fichier nommé "1."
		}
 
		if(move_uploaded_file($_FILES['monfichier1']['tmp_name'], $dossier.'/1.'.$extension_upload))
		  echo '<br />L\'envoi du fichier photo1 à bien été effectué !';
		else
		  echo '<br />Erreur d\'enregistrement du fichier.';
	}	
}
?>

[gwinyam]

Normal, t'essaie d'effacer le fichier "1."+extension du fichier que tu viens de charger et pas fichier que tu as déjà

[artichaudd]

non je ne crois pas
regardes bien:
1) je teste le fichier charger enregistré temporairement (qui viens d'etre chargé via formulaire), si son extension ($extension_upload) est autorisée alors
2) je teste si il y a un fichier nommé "1." existe déjà, dans $dossier, si oui je le supprime
3) je transfere le fichier charger du dossier temporaire au definitif dans $dossier

Code :

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
<?php
// Testons si le fichier n'est pas trop gros
if (isset($_FILES['monfichier1']) AND $_FILES['monfichier1']['error'] == 0 AND ($_FILES['monfichier1']['size'] <= 1000000))
{
//1) je teste le fichier charger enregistré temporairement (qui viens d'etre chargé via formulaire), si son extension ($extension_upload) est autorisée alors
	$dossier = '../images/uploads-atelier/normal/'.$nomDossier;
 
	if(!is_dir($dossier)) //si le dossier n'existe pas
		mkdir ($dossier); //Tente de créer un dossier.
 
	// Testons si l'extension est autorisée
	$infosfichier = pathinfo($_FILES['monfichier1']['name']);
	$extension_upload = $infosfichier['extension'];
	$extensions_autorisees = array('jpg', 'jpeg', 'gif', 'png');
	if (in_array($extension_upload, $extensions_autorisees))
	{
//2) je teste si il y a un fichier nommé "1." existe, dans $dossier, si oui je le supprime
		if(file_exists ( $dossier.'/1.' )) // si il existe un fichier commencant par "1."
		{
			$extension = pathinfo($dossier.'/1.', PATHINFO_EXTENSION);//je récupère son extention indispensable pour unlink()
			unlink($dossier.'/1.'.$extension); // et je supprime le précèdant fichier nommé "1."
		}
//3) je transfere le fichier charger du dossier temporaire au definitif dans $dossier
 
		if(move_uploaded_file($_FILES['monfichier1']['tmp_name'], $dossier.'/1.'.$extension_upload))
		  echo '<br />L\'envoi du fichier photo1 à bien été effectué !';
		else
		  echo '<br />Erreur d\'enregistrement du fichier.';
	}	
}
?>

[gwinyam]

Ouais mais à l'étape 2, ton fichier s'appelle "1.", pas "1.jpg" ou "1.png". Normal qu'il efface rien.

[artichaudd]

tu veux dire dans le file_exists , le pathinfo ou unlink()
si c'est pour le unlink($dossier.'/1.'.$extension);
donc il doit effacer 1.'.$extension donc l'extension est bien précisée?

[gwinyam]

Pour le coup oui (pour le unlink)

[artichaudd]

je ne comprend pas bien ta réponse , peux tu développer
qu'est ce que tu as remarquer , peux tu etre plus précis

merci

[gwinyam]

Tu ne peux pas effacer un fichier qui s'appelle "1." puisqu'il n'existe pas.
Donc oui, tu es obligé de préciser l'extension dans ton appel à unlink.