Problème avec getmtime

**Ahsoka** · 31/10/2022, 04h18

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 :

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

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

**MPython Alaplancha** · 31/10/2022, 06h12

Bonjour,

Code :

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

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'
>>>

**tyrtamos** · 31/10/2022, 09h39

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.

**Sve@r** · 31/10/2022, 10h10

Bonjour

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

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 python :

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

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

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()