Problème avec getmtime

Bonjour :)

Je débute en langage Python. Désolée si je fais des erreurs qui peuvent paraitre évidente pour certain.

Je bute sur un problème avec une partie de mon code.

Je voudrais récupérer mon fichier le plus ancien pour le transférer dans un autre dossier,
et ainsi de suite jusqu'à atteindre une limite de taille pour mon dossier source, mais mon programme me renvoi des fichiers de date random.

Pourtant la fonction getmtime n'est-elle pas sensée me donner la dernière modification d'un fichier et min(glob.glob du plus ancien au plus récent?

Merci pour toute aide !

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

import os import time import glob import shutil   files = os.listdir('dossier source') for file in files: if file.endswith(('.jpg','.png','.gif')): files = min(glob.glob('dossier source/*'), key=os.path.getmtime) file_size = os.path.getsize('dossier source/{file}') shutil.move('dossier source/{file}', 'dossier destination') print(f"Moved file...: {(file)}") QL = QL - file_size if QL < 1048576: break

---

Bonjour,

Code:

---

1 2 3 4 5 6

>>> d = { file: os.path.getmtime(file) for file in glob.glob('*')} >>> d {'systemd-private-89fd67f5d0324ac1a5d0f8d5add7834f-geoclue.service-B8r4Ng': 1667189863.1469831, 'systemd-private-89fd67f5d0324ac1a5d0f8d5add7834f-upower.service-oJK1Df': 1667189856.734983, 'Temp-a01b6dd3-4227-4a4f-848d-fa75e789240e': 1667189931.693228, 'ssh-BDiMbvD8TBum': 1667189860.010983, 'systemd-private-89fd67f5d0324ac1a5d0f8d5add7834f-colord.service-F6GGrg': 1667189856.8149831, 'systemd-private-89fd67f5d0324ac1a5d0f8d5add7834f-systemd-logind.service-te9Q7f': 1667189856.7229831, 'systemd-private-89fd67f5d0324ac1a5d0f8d5add7834f-systemd-resolved.service-J6rU9g': 1667189856.542983, 'bid.py': 1667190746.4828832, 'systemd-private-89fd67f5d0324ac1a5d0f8d5add7834f-systemd-timesyncd.service-DGWtai': 1667189856.546983, 'config-err-3XKa49': 1667189859.4629831} >>> min(d, key=d.get) 'systemd-private-89fd67f5d0324ac1a5d0f8d5add7834f-systemd-resolved.service-J6rU9g' >>>

---

Bonjour

Il y a des choses pas logiques dans le code. Par exemple, avoir l'ensemble des fichiers avec os.listdir, et utiliser glob.glob pour chacun des fichiers, ce qui va relire l'ensemble des fichiers... Par ailleurs, glob.glob n'a pas de paramètre "key". Voir la doc:https://docs.python.org/3/library/glob.html#glob.glob.

Dans le principe:

- glob.glob est très bien pour trouver les fichiers souhaités. Attention cependant, il peut trouver aussi des répertoires, et il faut donc ne retenir que les fichiers avec un test os.path.isfile(...).

- une fois qu'on a la liste des fichiers, il suffit de les trier. On utilise la méthode .sort(...), qui a une clé "key" qui permet de dire sur quoi doit porter le tri: on prend alors getmtime. Si le tri n'est pas dans le bon sens, on utilise le paramètre "reverse" de sort. Voir la doc: https://docs.python.org/fr/3/library...sort#list.sort

- on a alors la liste des fichiers triés par date de modif, et on en fait ce qu'on en veut.

Bonjour

Citation:

---

Envoyé par Ahsoka

Pourtant la fonction getmtime n'est-elle pas sensée me donner la dernière modification d'un fichier

---

Bien entendu... mais faut lui donner le fichier à ta fonction getmtime pour qu'elle sache quoi chercher :P
Suffit pas de simplement mettre "key=fonction" pour que la fonction qui utilise cette clef (sort, min, max, etc) "sache" qu'il faut appliquer ladite fonction aux différents éléments de l'itérable qu'elle reçoit
Un exemple d'utilisation

Code:

---

1 2	tab=["azerty", "qwerty", "zaza"] print(min(tab, key=lambda x: x[1]))

---

Dans le cas présent, j'indique à min() que la clef sera une fonction dite "lambda" (fonction créée à la volée), fonction qui, pour chaque élément traité (donc éléments de "tab"), devra utiliser son second caractère => ce sera donc le min de chaque "second caractère" => zaza

Fais attention aussi aux f-strings. os.path.getsize('dossier source/{file}') ça ne fonctionnera pas !!! Il faut bien écrire un "f" devant la string pour que ça fonctionne => os.path.getsize(f'dossier source/{file}') (avis perso moi je n'aime pas les f-strings => os.path.getsize('dossier source/%s' % file) ou même os.path.getsize('dossier source/{0}'.format(file)) je trouve ça bien moins sujet aux erreurs mais je le redis c'est un avis perso qui n'a pas pour but de supplanter les opinions de chacun sur le sujet)

D'ailleurs pour créer un chemin il y a os.path.join qui a l'avantage supplémentaire d'adapter la syntaxe du chemin à l'OS dans lequel il est invoqué => os.path.getsize(os.path.join('dossier source', file)) ce qui a au-moins l'avantage de fermer immédiatement ce débat qui aurait pu s'ouvrir :oops:

Citation:

---

Envoyé par tyrtamos

Par ailleurs, glob.glob n'a pas de paramètre "key"

---

Attention, tu as fait une erreur de parenthèses, le paramètre "key" est placé sur la fonction "min()" qui, elle, possède bien ce paramètre, pas sur la fonction glob.glob() :lol:

attention aussi dans ton code files est une liste des fichier à la ligne 6, mais devient autre chose en ligne 9

Merci beaucoup pour tout vos précieux conseils.