Supprimer photos qui ne sont pas dans la base

Version imprimable

bonjour
je n'arrive pas a afficher tous les noms de photos à virer... il m'indique que le 1er et pourtant il y en a plus de 10.

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
 
if($opdossier = opendir('./photos'))
{
  while(false !== ($fichier = readdir($opdossier)))
  {
   if($fichier != '.' && $fichier != '..' && $fichier != 'index.html' && $fichier != 'nofoto.png')
   {
   	$photo = $bdd->query("SELECT photo FROM annonce WHERE photo != 'a:0:{}'");
	$photos = $photo->fetchall();
	foreach ($photos AS $visufoto)
	{
		if (!empty($visufoto['photo']))
		{
		$listfoto = unserialize($visufoto['photo']);
 
	if (in_array($fichier, $listfoto))
	{
	$okfoto = 1;
	}
		}
	} 
	if (!isset($okfoto) AND !empty($visufoto['photo'])){
	echo "a virer :".$fichier."<br>";
	//unlink ("./photos/$fichier");
	unset($okfoto);
	}
 
   }
  }
} 
 
closedir($opdossier);

je ne sais pas si on peut vérifier directement en sql comme mon champ photo et serializé

C'est pas pratique du tout ces listes de photos sérialisées, je dirais mauvais design de la base de données, mais on va faire avec. Pour éviter trop d'imbrications d'accolades et surtout pour éviter de faire la même requête SQL à chaque fichier, j'écrirais le code de cette manière:

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
if ( $handle = opendir('./photos') )
{
    $fichiers = [];
 
    while ( false !== $fichier = readdir($handle) )
    {
        if ( !in_array($fichier, ['.', '..', 'index.html', 'nofoto.png']) )
        {
            $fichiers[] = $fichier;
        }
    }
 
    $qres = $bdd->query("SELECT photo FROM annonce WHERE photo != 'a:0:{}'");
 
    $photos = array_column($qres->fetchall(), 'photo');
    $photos = array_map('unserialize', $photos);
    $photos = array_merge([], ...$photos);
 
    $toRemove = array_diff($fichiers, $photos);
 
    foreach ( $toRemove as $fichier ) {
        echo "À virer: ", $fichier, "<br />", PHP_EOL;
    }
    closedir($handle);
}

Bien sûr, je n'ai pas pu tester!

11/10/2021, 11h44
maxtrident

merci, je ne savais pas qu'on pouvait faire comme ça.
j'avais pensé à récupérer les photos comme sur le sujet: extraire les noms des photos d'une variable
mais ça va être super lourd.
mon champ photo me sert uniquement pour contrôler les photos existante... car si l'utilisateur modifie ou supprime avant de validé, la photo est quand même enregistré dans le serveur.
A voir sérialisées n'est pas la bonne façon... c'est quoi la meilleur façon pour enregistrer les noms de photos pour chaque page ?

Citation:

A voir sérialisées n'est pas la bonne façon... c'est quoi la meilleur façon pour enregistrer les noms de photos pour chaque page ?

C'est un design classique avec 3 tables (une pour les pages, une pour les photos, une qui fait la liaison entre les pages et les photos), un truc comme ça:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 t_page --------------- pag_id (primary key int autoincrement) pag_date pag_title pag_content t_photo --------------- pho_id (primary key int autoincrement) pho_desc pho_path (uniq) t_page_photo --------------- fk_pag_id fk_pho_id
Avec un tel schéma, dans la table t_photo, une ligne correspond à une photo, on peut ajouter une contrainte d'unicité sur le chemin (deux photos ne peuvent pas avoir le même chemin). Du coup on peut avoir l'ensemble des photos directement en faisant: SELECT pho_path FROM t_photo.

Tu remarqueras que t_page ne contient aucune colonne relative à une quelconque photo, de même que t_photo n'a aucune colonne relative à une quelconque page. C'est t_page_photo qui a pour rôle de faire la liaison entre une page et une photo.

Disons que la page ayant pour id 21, contienne les photos ayant pour id 12, 13 et 14. La table t_page_photo aura donc les lignes:
Code:

1 2 3 4 5 fk_pag_id | fk_pho_id ----------+----------- 21 | 12 21 | 13 21 | 14
Le préfixe fk_ est une convention de nommage que j'utilise pour signifier que fk_pag_id et fk_pho_id sont des clefs étrangères (Foreign Key) qui proviennent respectivement pag_id et pho_id (les clefs primaires de t_page et t_photo). Ce sont des contraintes qu'il faut déclarer à la création de la table, elles empêcheront notamment d'insérer une ligne avec une id de photo ou de page qui n'existe pas.

Pour récupérer les chemins des photos d'une page, il faudra alors faire une jointure, par exemple:
Code:

1 2 3 4 5 SELECT ph.pho_path FROM t_page_photo pa LEFT JOIN t_photo ph ON ph.pho_id = pa.fk_pho_id WHERE pa.fk_pag_id = 21
Ça peut paraître compliqué, mais en fait c'est très performant. Il faut bien voir qu'une base de données relationnelle c'est une machine à faire des opérations ensemblistes sur les tables et dont les rouages sont les clefs numériques. Le but de ce genre de design est notamment de respecter un des principes fondamentaux dans la construction d'une base de données: l'atomicité des données (ce qui n'est pas le cas avec une liste de photos).

Le site est particulièrement bien fourni en tutoriels sur les bases de données et la modélisation si tu veux t'aguerrir sur le sujet. N'hésite pas à jeter un œil dans la section modélisation/schéma du forum et à te procurer l'outil looping.

11/10/2021, 17h54
maxtrident

merci pour l'explication bien détaillé.
je pensais qu' avoir une seul table avec tous dedans été plus rapide et prend moins de ressource pour le serveur.
je vais essayer cette méthode par la suite, mais beaucoup plus dur à comprendre.