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 for raise 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, ou if tarefa.get('falhar') se você quiser certificar-se que o valor de tarefa['falhar'] é True.
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 for raise 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, ou if tarefa.get('falhar') se você quiser certificar-se que o valor de tarefa['falhar'] é True.
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 for raise 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, ou if tarefa.get('falhar') se você quiser certificar-se que o valor de tarefa['falhar'] é True.
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 for raise 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, ou if tarefa.get('falhar') se você quiser certificar-se que o valor de tarefa['falhar'] é True.