Sobre atualizar uma foto pessoalmente diria (sendo um chute de minha parte) que as fotos atualizadas nunca devem conter o mesmo nome, se for web, ou então na parte PHP (que é o seu caso) ajustar as URLs baseado no tempo de atualização (sobrescrita da nova foto na antiga após o upload, que é algo que vai obter baseado no próprio sistema operacional, o PHP já contêm isto, que é o filemtime(<localização do arquivo>)
), como exemplo que sugeri em:
Desta forma poderia reaproveitar o mesmo nome e ao mesmo tempo contornar o cache das fotos no navegador para o cliente.
Sobre deletar uma foto do sistema de arquivos você poderia é claro fazer um SELECT
no momento, pegar o caminho da foto, então executar o DELETE
, então se o delete for executado com sucesso pegaria o caminho obtido anteriormente e executaria o unlink
, se o unlink
falhar, executaria o rollback no banco (o que acho que na verdade é dispensável conforme a próxima sugestão), ficaria algo como (é apenas hipotético):
$id = <ID DA FOTO AQUI>;
$foto = null;
$pasta = '/foo/bar/baz/fotos/'; //Pasta aonde se localiza as fotos
$mysqli->autocommit(false); //DESLIGA O autocommit
if ($stmt = $mysqli->prepare('SELECT `id`, `foto` FROM `fotos` WHERE id=?')){
$stmt->bind_param('i', $id);
$stmt->execute();
$result = $stmt->get_result();
while ($row = $result->fetch_assoc()) {
$foto = $row['foto'];
}
$stmt->close();
}
if ($foto && $stmt = $mysqli->prepare('DELETE FROM `fotos` WHERE id=?')){
$stmt->bind_param('i', $id);
$deletado = $stmt->execute();
$stmt->close();
if (!$deletado) {
echo 'Falha ao deletar do banco';
} elseif (unlink($pasta . $foto)) {
//Se falhar ao deletar da pasta então executamos o rollback
$mysqli->rollback();
echo 'Falha ao deletar da pasta';
}
}
Separando os "processos"
Apesar de pensar na possibilidade anterior de "invalidar" (deletar) arquivos de imediato como no exemplo acima, creio que um meio menos "hard" seria deixar este processo ocorrer a parte, pelo fato de um arquivo poder estar sendo usado por sistemas de cache back-end das páginas aonde se apresenta a imagem/resource, se invalidar o arquivo de imediato isso poderia ocasionar efeitos colaterais, então a sugestão seria simplesmente criar um "processo" (não relacionado a threads) a parte que verifique arquivos que não estão mais em uso, assim você poderia eliminar vários arquivos de forma "agendada" consumindo coisas especificas apenas uma vez:
- Como a própria execução do script, que ao invés de eliminar um a um eliminaria vários
- Gravação e leitura do disco apenas de uma vez (apesar disto ser muito relativo e eu realmente não posso afirmar que vai ser melhor apagar vários ao invés de um a um)
A principal vantagem neste caso é que você teria uma garantia que só pegaria o que realmente não consta no banco, claro que se você tiver um sistema que reaproveita nomes de fotos isso com certeza poderia ocasionar um problema de "condição corrida" ou apenas um problema de "inconsistência", então pra isto funcionar bem as fotos nunca devem conter nomes repetidos, mesmo que uma antiga já tenha sido removida
Supondo que executou diversos DELETE
s durante a "jornada de trabalho" (de quem usa o seu sistema), então em um horário mais ocioso algo com um CRON ou algo semelhante, executaria o processo de limpeza, pegando um range de arquivos e checando se existem no banco com NOT IN()
, então o que retornou poderia executar o unlink()
, se forem muitas fotos talvez estabelecer um limite e salvar o ultimo checado, e executando novamente o processo a partir da próxima após o "range".
Não posso afirmar com certeza que é a melhor saída, mas em questões de cache de páginas que ainda precisariam de recursos antigos seria uma forma de minimizar o problema, isso depende muito do que você pretende fazer.