Terminal-ToDo icon indicating copy to clipboard operation
Terminal-ToDo copied to clipboard

Sugestão de Melhora

Open viniciusmesquitac opened this issue 5 years ago • 0 comments

Arquivo: jsonParser

A função addingTaskInFile(goal: Goal) -> String é contra intuitiva pois ela retorna um json em formato de string, quando deveria adicionar a task dentro de um arquivo.

Sugestão:

remover o retorno -> String e escrever e utilizar a função writeInFile dentro do escopo, para finalizar o processo de adição ou alterar o nome para algo do tipo addingTaskAndGetEncodedStringJsonData

A função writeInFile(json: String?) deveria evitar receber o Optional String?, e ela nunca receberá nulo da função addingTaskInFile(goal: Goal) -> String, poderia ser apenas writeInFile(json: String)

viniciusmesquitac avatar Apr 05 '20 03:04 viniciusmesquitac