Um problema facilmente visível na sua implementação é que a forma que você checa pra ver se o arquivo
journal.txt
existe após a leitura está sujeita a race conditions, vide Como checar se um arquivo existe usando Python.Além disso, o ideal é, não capturar exceções "naked", e sim definindo o tipo específico de exceção que você espera, p.ex.
IOError
,OSError
, etc. Da mesma forma, quando forraise
uma exceção, é bom utilizar uma com um nome específico relacionado ao tipo de erro.Para iterar sobre o dicionário, como uma alternativa, ao invés de iterar sobre o resultado do
enumerate()
, você pode simplesmente iterar sobre a lista:passos = [{'nome':'foo','falhar':True}, {'nome':'bar'}, {'nome':'baz'}] for tarefa in passos: realizar_tarefa_indepotente(tarefa)
A forma que você checa se o valor 'falhar' funciona, mas não é muito explícita. Recomendaria trocar por
if 'falhar' in tarefa.items()
, se você estiver simplesmente checando se ela existe, ouif tarefa.get('falhar')
se você quiser certificar-se que o valor detarefa['falhar']
éTrue
.
Rodrigo Deodoro
- 1,8mil
- 1
- 13
- 21