search

NicholasNeto / string_Calculator_Kata

0 stars 0 forks source link

ENCONTRA DELIMITADOR #9

Open NicholasNeto opened 6 years ago

NicholasNeto commented 6 years ago

Eu fiz esta nova function chamada de " encontraDelimitador , mas achei ela muito ruim ... queria deixar mais simples , tentar não usar o for estas coisas mas ta meio dificil

Ticas para refatorar ?

f8b9b4b0747d7080e04931a2cf8cf9ec0d5721ec

thiagohersan commented 6 years ago

Realmente está meio complicada, tem 3 cópias da lista dentro dessa função: entrada, stringModificada e lista.

Assumindo que o delimitador estará sempre em entrada[2], você sabe que os números começam no entrada[4]. Dá uma olhada em substring e vê se dá para evitar o for loop e tantas variáveis dentro da função.

Seus testes só testam strings com um delimitador e dois números. É bom colocar testes com mais do que 1 delimitador (3 ou mais números). Eu lembro que no seu outro código tinha um bug no step3 onde a soma só funcionava até o primeiro \n e depois ignorava os números.

Uma coisa que não está muito clara na descrição do Kata, então você pode decidir: quando um string tem um delimitador diferente na primeira linha, ele só usa esse delimitador, ou ainda pode misturar , e \n com o delimitador?

Ou seja, esse string é válido: "//*\n1*2*3,4,5,6\n7\n8\n9" ?

thiagohersan commented 6 years ago

Precisa dessa variável? var indexFinal = entrada.length

Que acontece se chamar substring com um só parâmetro?

NicholasNeto commented 6 years ago

O for deu pra evitar , mas a quantidade de variaveis sendo criadas , esta tenso ... Mas não vejo muito ponto pra diminuir mais ..

thiagohersan commented 6 years ago

não precisa diminuir o número de variáveis. O que eu quis dizer é que a função substring pode ser chamada com um único parâmetro.

NicholasNeto commented 6 years ago

feito =)