Как правильно настроить пропуск?

Question

[Elbro]

Всем доброго времени суток. Есть скрипт, который качает изображения с определенной частотой и:

Дает им имена 1, 2, 3, 4
Создает папку temp
Скидывает в папку temp
По окончанию переименовывает папку

Бывает такое, что работа обрывается и файл валяется в исходной папке и скрипт сначала закидывает файл 1 в temp, качает новый и выдает ошибку (я все это симулировал ниже в скриншотах).
Вопрос, как удалить файл если два с одинаковым именем, либо же научить переписывать поверх.

(Если папка temp есть, то переместить, если нет, то создать и переместить)

Сам скрипт

import os 
import shutil
from os import path
filename = os.listdir(os.getcwd())
frames = ['.jpeg', '.jpg', '.svg', '.png', '.PNG']
framesLocation = "Snapshots_temp"
for file in filename:
    if os.path.splitext(file)[1] in frames:
        if(path.exists(framesLocation)):
            shutil.move(file, framesLocation)
        else:
            os.mkdir(framesLocation)
            shutil.move(file, framesLocation)

Одно изображение в папке

SCREENSHOT 1

После запуска скрипта, создалась папка и туда закинуло файл

SCREENSHOT 2

Вышло так, что папка осталась и уже пошли качаться новые файлы

SCREENSHOT 3

И сама ошибка

SCREENSHOT 4

Answer 1

[ligvake]

Для того, чтобы shutil.move перезаписывал файлы, нужно указать директорию вместе с именем файла, который вы собираетесь переместить
Нужно поменять строчку с перемещением на

shutil.move(file, os.path.join(framesLocation, file))

Полный скрипт:

import os 
import shutil
from os import path
filename = os.listdir(os.getcwd())
frames = ['.jpeg', '.jpg', '.svg', '.png', '.PNG']
framesLocation = "Snapshots_temp"
for file in filename:
    if os.path.splitext(file)[1] in frames:
        if(path.exists(framesLocation)):
            shutil.move(file, os.path.join(framesLocation, file))
        else:
            os.mkdir(framesLocation)
            shutil.move(file, os.path.join(framesLocation, file))

Другой способ – делать проверку, существует ли файл (нужно не забыть from os.path import exists):

if(not exists(os.path.join(framesLocation, file))):

Тогда получится:

import os 
import shutil
from os import path
from os.path import exists

filename = os.listdir(os.getcwd())
frames = ['.jpeg', '.jpg', '.svg', '.png', '.PNG']
framesLocation = "Snapshots_temp"
for file in filename:
    if os.path.splitext(file)[1] in frames:
        if(not exists(os.path.join(framesLocation, file))):
            if(path.exists(framesLocation)):
                shutil.move(file, framesLocation)
            else:
                os.mkdir(framesLocation)
                shutil.move(file, framesLocation)

Comments

[Elbro]

То, что доктор прописал, спасибо большое!

Answer 2

[Elvis]

Используйте полные пути, а не относительные, тогда будет заменяться.

Answer 3

[soremix]

Не очень понял задачу, но нужно проверять наличие файла, и удалять его в случае чего
os.path.exists(filename)

Comments

[Elbro]

Мне надо удалить файл только в том случае, если он конфликтует.
file = скриншот
framesLocation = временная папка
Повторю задачу:

Качаю картинки 1, 2, 3, 4
Перемещаю их в папку темп
Переименовываю папку

Но бывает что-то падает и картинка остается в исходной папке, что вызывает:
Картинка перемещается в папку темп
Качается новая
Пытается перенестись в папку темп
Выдает ошибку

Я пробовал так, он удалял, но ошибка оставалась

for file in filename:
    if os.path.splitext(file)[1] in frames:
        if(path.exists(file)):
            print(file + 'exist')
            os.remove(file)
if os.path.splitext(file)[1] in frames:
    if(path.exists(framesLocation)):
        shutil.move(file, framesLocation)
    else:
        os.mkdir(framesLocation)
        print('asd')
        shutil.move(file, framesLocation)