Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possibilidade de incluir texto manualmente. #49

Closed

Conversation

JessileyOliveira
Copy link

Olá,

Antes de tudo, obrigado pelas ótimas aulas.

Não tenho conhecimento de clean code e nem mesmo de boas praticas, etão adianto desculpas pelos possíveis complicações para ler o código.

O que fiz foi algo muito simples, mas satisfatório para mim (tome dopamina rs) já que meu conhecimento em node é muito baixo.

Basicamente foi incluído uma possibilidade de digitar frases ao invés de buscar no Wikipedia. Busco por frases pois ao colar um texto o readline-sync fazia vários envios e não um único.

Problema: Por mais que eu tentei, não consegui corrigir caracteres acentuados, deixando assim um ponto de interrogação no lugar do carácter.

Novamente agradeço pelos videos, e espero que se não o código, a ideia tenha sido boa.

Possibilidade de escrever content
Finalizado com problemas em caracteres especiais
Copy link

@naurojunior naurojunior left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Muito bacana a ideia, só comentei pra tentar dar umas dicas (e se alguém puder, me corrija também)

@@ -6,6 +6,6 @@ File: `algorithmia.json`

```
{
"apiKey": "sim8As72h56j3mBYqEOSHfLE4k81"
"apiKey": "simbdhBHWbwGDgydYyknwhKzAQo1"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dica, aqui é um README.md (só um texto de exemplo), tua apiKey tem que ficar no credentials. Cuida com o commit de credenciais/senhas e coisas do tipo, porque as pessoas vão ver =)

Eu recomendaria gerar uma nova ApiKey pra evitar problemas

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Opa, realmente, não me atentei em não enviar a api key. Obg

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Altere sua API KEY na origem o quanto antes... se ainda não o fez.

let newPhrase = "";
let completeText = "";
while(newPhrase != "end"){
newPhrase = readline.question('Type the content (type "end" to exit): ')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curti a ideia, eu só recomendaria colocar esse tipo de coisa em outro lugar que não seja o robot de texto, talvez outro robô pra lidar com esse tipo de coisa deixaria o código melhor =)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hun, Pensei aqui pois ele é o robô de texto, então vai cuidar do texto não importante como ele venha. Até mesmo estava pensando em colocar a opção archive, vendo que o readline-sync tem essa possibilidade. (só não estudei ainda). Agradeço a opinião, vou dar uma estudada.

@leodutra
Copy link
Collaborator

Por que fechou?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants