Skip to main content
replaced http://pt.stackoverflow.com/ with https://pt.stackoverflow.com/
Fonte Link
  • 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 PythonComo 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.

Fonte Link
Rodrigo Deodoro
  • 1,8mil
  • 1
  • 13
  • 21

  • 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.