Suppression plusieurs fichiers à partir de bdd

Version imprimable

Bonjour,
Cela fait un petit moment que je cherche, pouvez-vous m'aider ?
J'ai des rubriques et des fichiers pdf stockés en bdd.
Je souhaiterai pouvoir supprimer ma rubrique avec tous ses fichiers associés :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
//Recuperer dossier
$query ='SELECT * FROM dossier WHERE RubriqueID=\'' . $_GET['supprimer_rub'] . '\'';
$retour = mysql_query($query) OR die("<div class='erreur'>Dossiers non trouv&eacute;s.</div>");			
 
while($data = mysql_fetch_array($retour))
{
	$id = $data['id'];	
	$pdfname = $data['pdfname'];
 
	if($data['pdfname'] !=""){					
		// on récupère le nombre d'éléments
		$fichiers = $data['pdfname'];
		$nb_fichiers = count($fichiers);
		// on fait la boucle pour supprimer
		for($k=0; $k<= $nb_fichiers; $k++){
			$dir = 'chemin/pdf/'.$fichiers[$k];
			if ( is_file ($dir) ) {
   				unlink ($dir) ;
			}
		}
	}
}

Mais cela ne fonctionne pas...
Merci d'avance pour votre aide.

22/02/2013, 17h48
sabotage

$data['pdfname'] n'est pas un tableau PHP.
Si c'est une liste séparée par un caractère, il faut utiliser explode().
24/02/2013, 13h14
noceb
Merci Sabotage mais ce n'est pas une liste séparée par un caractère...
et je n'ai pas réussi à récupérer le nom de mon fichier par explode().
En fait là, j'ai cherché différentes solutions mais débutante, je tourne et retourne.
Celle qui me semble intéressante est celle-ci :
Faire un tableau
Code:

1 2 3 4 5 6 7 $tab = array(); while($data = mysql_fetch_array($retour)) { if (isset($tab[$data['pdfname']])) { explode($tab[$data['pdfname']]); } echo $tab[$data['pdfname']];
mais là j'ai

Citation:

" Notice: Undefined index: fichier1.pdf in "
" Notice: Undefined index: fichier2.pdf in "
" Notice: Undefined index: fichier3.pdf in "

Pouvez-vous me donner une piste pour avancer ?
Merci d'avance.
24/02/2013, 13h18
sabotage

Citation:

mais ce n'est pas une liste séparée par un caractère...

Qu'est ce que c'est alors ?

Bonjour Sabotage,
Suite à ton dernier message, j'ai repris calmement :roll: ta piste...
et tous mes fichiers se suppriment bien maintenant, il me reste juste encore une erreur :

Citation:

Notice: Undefined offset: 2 in ... on line 86 Call Stack

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 //Recuperer dossier $query ='SELECT * FROM dossier WHERE RubriqueID=\'' . $_GET['supprimer_rub'] . '\''; $retour = mysql_query($query) OR die("<div class='erreur'>Dossiers non trouvés</div>"); while($data = mysql_fetch_array($retour)) { $pdfname =$data['pdfname']; if($pdfname !=""){ $fichiers = $pdfname." "; $f = explode(" ", $fichiers); if(isset($f)) { $nb_fichiers = count($f); if(isset($nb_fichiers)) { // on fait la boucle pour supprimer for($i=0; $i<= $nb_fichiers; $i++){ $dir = 'chemin/pdf/'.$f[$i]; if ( is_file ($dir) ) { unlink ($dir) ; } } } } } }
La ligne concernée est :

Code:

$dir = 'chemin/pdf/'.$f[$i];

J'ai vu que ce type d'erreur correspondait en général à un manque d'isset mais je ne vois vraiment pas où il manquerait...
En tout cas, merci bcp pour ton attention et ta patience...

27/02/2013, 00h44
Meyfarth

Salut,

Plusieurs remarques sur ton code :
1) Le isset($nb_fichiers) ne sert à rien, count() te renvoyant forcément une valeur, la variable est forcément définie. Si tu veux vérifier qu'il y ait au moins un fichier, il faut vérifier que $nb_fichiers soit supérieur à 0.

Code:

if($nb_fichiers > 0)

OU

Code:

if(!empty($nb_fichiers))

Note : la fonction empty renvoie "true" si la valeur testée est null, une chaine vide, un entier égale à 0, ... Voir la doc sur la fonction pour plus de détail, mais à utiliser avec précautions !

2) N'oublies pas que ton tableau de données commence à l'index 0. Si ton tableau contient 3 fichiers, count() te renverra 3 et ton tableau ira de l'index 0 à l'index 2.
Dans ton for, tu boucles de 0 jusqu'à $nb_fichiers (3 dans mon exemple), mais $tab[3] n'est pas défini, car il ferait alors référence au 4ème élément du tableau. Or ici, tu n'en as que 3 ! Il faut faire un "<" au lieu d'un "<="

Code:

for($i = 0; $i < $nb_fichiers; $i++)

3) Même s'il y a peu de différence de performance entre un mysql_fetch_array() et un mysql_fetch_assoc(), le premier prend deux fois plus de mémoire que le second. En effet, le mysql_fetch_array stock les résultats en associatif ET en index. Pour t'en rendre compte, fais un print_r() sur un $data

Code:

print_r($data)

Il est préférable d'utiliser mysql_fetch_assoc() si tu n'utilises que les noms de champs et pas leur index.

4) Quelque chose me surprend dans ton code : ta gestion du nom de fichier. Tu vérifies que le fichier n'est pas vide, si c'est le cas, tu le récupère, ajoutes un espace à la fin et fais un explode sur cet espace. Pourquoi ?

A+