Discussion :

[dodineau]

Bonjour,
je cherche simplement à afficher les photos d'un répertoire pour pouvoir en sélectionner une.
Pour l'instant j'ai programmé l'affichage du fond avec pygame mais maintenant je voudrais afficher chaque photo du répertoire une par une et pouvoir cliquer pour passer a la suivante.
Merci pour votre aide, j'ai vainement cherché sur le web mais je ne trouve rien de tout simple.
A+

[wiztricks]

Salut,

je voudrais afficher chaque photo du répertoire une par une et pouvoir cliquer pour passer a la suivante.

Il faut procéder par étapes: d'abord afficher une photo, puis fabriquer une liste de photos et naviguer de l'une à l'autre, puis construire cette liste à partir du contenu du répertoire...

- W

[dodineau]

Merci pour cette réponse.
J'ai commencé à lister le répertoire contenant les photos mais je n'arrive pas à isoler les fichiers JPG.

Code :

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

1
2
3
4
5
6
7
 
import os
path_images = 'Images/'
list = os.listdir(path_images)
number_files = len(list)
for nb in range(number_files):
    print (list[nb])

en exécutant ce code j'obtiens bien la liste des fichiers de mon répertoire mais je voudrais isoler les JPG.

A+

[marco056]

Tu peux regarder du côté de "endswith()" ou bien en comparant les 4 derniers caractères ".jpg", donc chaine[-4:]

[VinsS]

Salut,

Utilise glob dans ce cas

Code :

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

1
2
3
 
import glob
images = glob.glob("Images/" + "*.JPG")

Mais tu devras peut-être rajouter les ".jpg", ".jpeg", et ".JPEG" ...

[dodineau]

VinsS : Je ne voulais pas utiliser la fonction glob.glob car j'ai lu sur beaucoup de sites que oslistdir était plus fiable.

marco056 : C'est ce que j'ai commencé à faire mais je pensais qu'il existait une fonction directe du style de glob.glob.

A+

[lg_53]

VinsS : Je ne voulais pas utiliser la fonction glob.glob car j'ai lu sur beaucoup de sites que oslistdir était plus fiable.

Tu peux citer tes sources ? Car chez moi glob.glob a toujours bien fonctionné ...

[dodineau]

Tu peux citer tes sources ? Car chez moi glob.glob a toujours bien fonctionné ...

Bonsoir,
je ne retrouve plus où j'avais lu cette remarque.
Mais bon comme j'utilise os.listdir et que ça fonctionne je n'ai pas non plus cherché plus loin.

[Sve@r]

Bonsoir

je ne retrouve plus où j'avais lu cette remarque.
Mais bon comme j'utilise os.listdir et que ça fonctionne je n'ai pas non plus cherché plus loin.

Je ne suis pas certain qu'on puisse dire que l'une est plus ou moins fiable que l'autre. En général, je considère que les outils Python sont tous aussi fiables les uns que les autres.

Tout ce qu'on peut dire c'est que les deux fonctions n'ont pas le même mode d'action. glob.glob() va te permettre d'utiliser des méta-caractères tandis que os.listdir() va se contenter de lister le dossier cible. Donc ça t'offre deux points d'entrée différents pour un travail de listing de dossier.
Et bien entendu chaque mode d'action doit avoir en contrepartie des inconvénients variés et différents selon qu'on va choisir l'un ou l'autre de ces deux outils.

[dodineau]

Merci pour votre aide.
J'ai fait cette galerie simple.
Je vous mets le fichier en téléchargement pour ceux que ça intéresse.
Pour tester il vous suffit d'appuyer sur A pour voir l'image précédente ou P pour voir l'image suivante.

Pièce jointe 487359

Merci pour votre retour.
A+

[Sve@r]

Je vous mets le fichier en téléchargement pour ceux que ça intéresse.

Pas mal. Tes lettres stylisées sont très belles.

Je te propose maintenant les modifications suivantes
Déjà remplacer list=truc puis list.sort() par un simple list=sorted(truc). Eventuellement ne pas utiliser un nom de type comme variable serait un plus.

Ensuite remplacer

Code python :

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

1
2
3
4
5
6
7
8
9
10
11
image='Lettres/A.jpg'
img = pygame.image.load(image)
img = img.convert() 
img = pygame.transform.scale(img, (100,100))
screen.blit(img,(225,(monitor_v-100)/2))
 
image='Lettres/P.jpg'
img = pygame.image.load(image)
img = img.convert() 
img = pygame.transform.scale(img, (100,100))
screen.blit(img,(1500,(monitor_v-100)/2))

par

Code python :

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

1
2
3
4
5
6
7
8
9
10
for (lettre, factor) in (
	("A", 225),
	("P", 1500),
):
	img = pygame.image.load(os.path.join("Lettres", "%s.jpg" % lettre)).convert()
	screen.blit(
		pygame.transform.scale(img, (100,100)),
		(factor, (monitor_v-100)/2),
	)
# for

On peut même supprimer la variable "img" mais ça risque de devenir moins lisible. Accessoirement chez-moi, 1500 c'est trop à droite (moi j'ai mis 800).

On peut supprimer la variable "stop1" par l'utilisation judicieuse d'un break et remplacer if event.type==KEYDOWN and (event.key==K_q or event.key==K_SPACE) par if event.type == KEYDOWN and event.key in (K_q, K_SPACE) ce qui t'offre ainsi la possibilité de rajouter facilement d'autres valeurs éventuelles. On peut même factoriser la gestion du "KEYDOWN" (testé 3 fois) associé au "K_a" et du "K_p" en jouant avec les modulos mais je ne suis pas certain que ce soit vraiment mieux.

Et bien évidemment rajouter # coding: utf-8 en seconde ligne ce qui te permettra d'inclure dans ton code des instructions, strings et commentaires avec des caractères accentués.

[wiztricks]

Salut,

Outre les remarques de VinsS, en écrivant:

Code :

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

1
2
3
while stop1==0:
    event=pygame.event.poll()
    ...

pygame.event.poll retourne toujours un event qui s'il n'y a rien sera pygame.NOEVENT. Dit autrement, çà bouffe du CPU pour pas grand chose d'autant qu'existe pygame.event.wait qui attend qu'il y ait un event à retourner.

- W

[dodineau]

Bonjour et merci pour vos retours.

wiztricks: J'ai changé avec un wait au lieu de poll

Sve@r : J'ai fait les transformations que tu proposes et j'ai ajouté import os .
Tout fonctionne.
Je pensais que utf8 était d'office avec python 3, non ?

PS : les lettres et les photos sont trouvées sur internet

J'ai mis le fichier modifié en pièce jointe Pièce jointe 487440.

A vos avis...

[Sve@r]

Je pensais que utf8 était d'office avec python 3, non ?

Oui, pardon j'avais pas fait attention. Ceci dit, ça n'empêche pas de le mettre quand-même pour les nostalgiques de P2...

A vos avis...

Tu as le droit, pour plus de lisibilité, de découper tes lignes. Exemple avec screen.blit(pygame.transform.scale(pygame.image.load(os.path.join("Lettres", "%s.jpg" % lettre)).convert(),(100,100)),(factor, (monitor_v-100)/2),) que j'écrirai

Code python :

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

1
2
3
4
5
6
7
screen.blit(
	pygame.transform.scale(
		pygame.image.load(os.path.join("Lettres", "%s.jpg" % lettre)).convert(),
		(100, 100),
	),
	(factor, (monitor_v-100)/2),
)

Reste la variable "list" qui va obfusquer le type list(). Et ensuite tu mets tout ton code dans une fonction (par exemple "diaporama()") dans laquelle tu passes le dossier contenant les images en paramètre et tu clôtures ton programme de cette façon

Code python :

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

1
2
if __name__ == "__main__":
	diaporama(sys.argv[1])

Ainsi ton programme pourra soit être appelé depuis ton shell (tu lui passes le dossier à traiter en paramètre), soit être importé comme module dans un autre programme.

[dodineau]

Et ensuite tu mets tout ton code dans une fonction (par exemple "diaporama()") dans laquelle tu passes le dossier contenant les images en paramètre et tu clôtures ton programme de cette façon

Code python :

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

1
2
if __name__ == "__main__":
	diaporama(sys.argv[1])

Ainsi ton programme pourra soit être appelé depuis ton shell (tu lui passes le dossier à traiter en paramètre), soit être importé comme module dans un autre programme.

J'ai ajouté ces 2 lignes en fin de programme mais ça me donne une erreur en quittant.

Code :

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

1
2
    diaporama(sys.argv[1])
NameError: name 'diaporama' is not defined

[wiztricks]

J'ai ajouté ces 2 lignes en fin de programme mais ça me donne une erreur en quittant.

Code :

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

1
2
    diaporama(sys.argv[1])
NameError: name 'diaporama' is not defined

Probable que vous ayez omis préalable disant "mets tout ton code dans une fonction (par exemple "diaporama()")"

- W

[dodineau]

Voila ma structure

• def diaporama()
• code
• if __name__ == "__main__":
  diaporama(sys.argv[1])

résultat :

Code :

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

1
2
    diaporama(sys.argv[1])
IndexError: list index out of range

[wiztricks]

Avez vous compris l'intérêt de ces changements?

Ainsi ton programme pourra soit être appelé depuis ton shell (tu lui passes le dossier à traiter en paramètre), soit être importé comme module dans un autre programme.

- W

[dodineau]

Tout d'abord je travaille avec un raspberry Pi3 et Thonny python pour écrire le programme.
Je suis assez débutant et je ne connais pas encore toutes les subtilités de Python
L'intérêt d'exécuter depuis le shell c'est que l'on peut dire au programme avec quel répertoire de photos travailler. Çà peut être intéressant si l'on a plusieurs répertoires différents. (c'est effectivement mon cas).
Moi pour l'instant je veux juste que ça fonctionne depuis Thonny mais bien sur par la suite je l'intégrerai comme module quand j'en aurai besoin donc effectivement cette modification m'intéresse.

[wiztricks]

L'intérêt d'exécuter depuis le shell c'est que l'on peut dire au programme avec quel répertoire de photos travailler. Çà peut être intéressant si l'on a plusieurs répertoires différents. (c'est effectivement mon cas).

Et bien l'erreur traduit que vous avez probablement omis de passer le répertoire de photos en argument à la ligne de commande lançant le programme.

Je suis assez débutant et je ne connais pas encore toutes les subtilités de Python

Lancer un programme en ligne de commande avec des arguments genre ls -all ne dépend pas du langage avec lequel a été écrit le programme. Ce qui est spécifique à Python, c'est comment récupérer ces arguments via sys.argv.

- W