Сортировка изображений по разрешению с помощью Python и PIL

А есть ли что-то подобное, но только чтоб упорядочить фотографии по дате создания?

Вот, попробуйте, подправил код выше ( правда сортирует все файлы а не только картинки :) ):

Copy Source | Copy HTML

1. #!/usr/bin/env python
2. # -*- coding: UTF-8 -*-
3. """ sorts images by resolution"""
4.  
5.  
6. import os,sys
7. import time
8.  
9. dirname = os.path.abspath(sys.argv[1])
10. try:
11.     newdir = os.path.abspath(sys.argv[2])
12. except:
13.     newdir = dirname
14.  
15.  
16. def image_sort(dirname, newdir, recur= 0):
17.     if not recur:print 'sorting started ...'
18.     else: print 'sorting started in %s...'%dirname
19.     if not newdir: newdir = dirname
20.  
21.     imagelist = []
22.  
23.     if os.path.isdir(dirname):
24.         for x in os.listdir(dirname):
25.             absx = dirname+os.sep+x
26.             if os.path.isfile(absx):imagelist.append(absx)
27.             else:
28.                 image_sort(absx, newdir+os.sep+x,recur=1)
29.         for name in imagelist:
30.             try:
31.                 file_date = time.localtime(os.stat(name).st_mtime)
32.             except EnvironmentError, error:
33.                 print 'seems error: %s with '%error, name, '/n'
34.                 continue
35.             imdir = '%s--%s--%s'%(file_date.tm_year,file_date.tm_mon,file_date.tm_mday)
36.             imdir = os.path.join(newdir,imdir)
37.             if os.path.split(dirname)[-1] == os.path.split(imdir)[-1]:
38.                 continue
39.             elif not os.path.exists(imdir):
40.                 print 'making dir %s'%imdir
41.                 os.mkdir(imdir)
42.             try:
43.                 os.system('move "%s" "%s"'%(name,imdir))
44.             except EnvironmentError:
45.                 print 'error with '+ name, '/n'
46.     if not recur:print 'sorting completed!'
47.  
48. if __name__ == '__main__':
49.     image_sort(dirname, newdir)

Поправьте меня, если я не прав, но может лучше вместо вызова внешней утилиты move использовать функцию move из библиотеки shutil?

я бы на вашем месте эту пренебрежительно оторванную от сердца фразу про быдлокод убрал бы, ну или хотябы изменил, никто не идеален

Достаточно чистенький код, но есть пара ньюансов.

Во-первых, я бы советовал использовать вместо указания конкретных исключений (IOError, WindowsError) — Exception. Смысл состоит в том, что все остальные исключения так или иначе охватываются им, так как наследуют его. Кроме того, вы можете получить и текстовое сообщение об этом исключении. По вашему текущему коду, если произошла ошибка, совершенно невозможно узнать, что же случилось. А если возникнет исключение, не описанное в ветке exception, то код просто мирно пойдет дальше, работая с фактически неинициализированными (или инициализированными мусором) переменными.

Во-вторых, старайтесь использовать как можно меньше операции конкатенации (это которая "+") в строках, пользуйтесь операцией подстановки. Так ваш код будет более читабельным, и кроме того, более гибким (не придется переписывать десяток плюсиков при смене строки).

В-третьих, мне не совсем понятно, что вы хотели сказать символами '/n'. Если вы имели в виду символ перевода строки, то он пишется с обратным слешем: '\n'. И в print его использовать не обязательно, эта инструкция автоматически переводит строку за собой.

Вот пример, как лучше писать:
try:
os.system('move "%s" "%s"'%(name,imdir))
except Exception, err:
print 'error with \"%s\": %s' (name, str(err))

Этот код преобразует в читабельный текст информацию о возникшей ошибке, а так же перехватывает все исключения, которые могут образоваться в контексте try.

Использование конкретных исключений делают только в разборе ошибок, когда на определенный тип исключения приходится конкретное, уникальное действие. Например:
try:
some_function()
except Exception1:
exception1_handler()
except Exception2:
exceptuon2_handler()
except:
unknown_exception_handler()

> исключения возбуждаемые sys.exit() тоже будут перехватываться

Программа у вас не многопоточна, поэтому можно этим не заботиться. Но даже если и так, то перехват исключения от sys.exit() — хорошо, поскольку это сигнал к завершению работы программы. Он прервет выполнение функции, что, в принципе, правильно.
Если хочется обрабатывать такое исключение отдельно, то можно использовать ту конструкцию с несколькими ветками эксцептов, сначала поставить эксцепт на исключение SystemExit, а потом уже на Exception. Но опять же, это надо только при многопоточности.

> А насчет кавычек, не пойму зачем их экранировать ведь они разные.

Ну, это как-бы отчасти правила хорошего тона, а отчасти — переносимость. Некоторые версии могут неверно истолковать наличие кавычек внутри других кавычек, так как, по сути, это спецсимволы. У меня, например, глаз сразу зацепился, я подумал, что строка прерывается. В C, скажем, так же. Кавычки и все спецсимволы внутри строк следует экранировать во избежании ошибок компиляции или двойного толкования.

Благодаря таким как вы… — я потихоньку изучаю питон.

absx = dirname+os.sep+x

Вы же ниже используете os.path.join, зачем тут такой изворот? :)

молодца

А можно модернизировать скрипт: задавать 2 директории(откуда и куда) а также смотреть соотнощение сторон и сувать это на первый уровень (16:9/1280x720)?