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

Sugestões. #12

Closed
Luizerz opened this issue Apr 1, 2022 · 1 comment
Closed

Sugestões. #12

Luizerz opened this issue Apr 1, 2022 · 1 comment

Comments

@Luizerz
Copy link

Luizerz commented Apr 1, 2022

Eae Millena 🙃,

Primeiramente, parabéns pelo código, super organizado e bem legível, gostei principalmente da forma que você organizou as pastas mas, gostaria de fazer umas considerações/sugestões.

O seu README.md se encontra em uma pasta que não é a "raiz" do seu projeto logo, ele não esta sendo exibido por padrão no GitHub.

achei algumas inconsistência durante a execução do programa foi:

Captura de Tela 2022-04-01 às 13 31 04

Captura de Tela 2022-04-01 às 13 32 02

Analisando o código descobri que sua função não permite que o link do site tenha números e como eu estava usando o (https://dl.acm.org/doi/fullHtml/10.1145/3015456) ele não deixava eu adicionar, a maneira mais fácil de contornar isso é não utilizar a verificação de numeral.

let inteiros = CharacterSet.decimalDigits
let decimalRange = text.rangeOfCharacter(from: inteiros)

Pois pelo o que eu consegui entender voce não quer permitir a entrada de números na url/link mas, a meu ver é responsabilidade do usuário colocar um link valido.

No mais, achei seu programa extremamente importante e necessário, eu como acadêmico faço muita referencia em alguns trabalhos, mas as vezes não consigo fazer certo 😅. Logo, quando vi sua proposta fiquei super feliz xD.

@mill3na
Copy link
Owner

mill3na commented Apr 1, 2022

Eae Millena 🙃,

Primeiramente, parabéns pelo código, super organizado e bem legível, gostei principalmente da forma que você organizou as pastas mas, gostaria de fazer umas considerações/sugestões.

O seu README.md se encontra em uma pasta que não é a "raiz" do seu projeto logo, ele não esta sendo exibido por padrão no GitHub.

achei algumas inconsistência durante a execução do programa foi:

Captura de Tela 2022-04-01 às 13 31 04 Captura de Tela 2022-04-01 às 13 32 02

Analisando o código descobri que sua função não permite que o link do site tenha números e como eu estava usando o (https://dl.acm.org/doi/fullHtml/10.1145/3015456) ele não deixava eu adicionar, a maneira mais fácil de contornar isso é não utilizar a verificação de numeral.

let inteiros = CharacterSet.decimalDigits
let decimalRange = text.rangeOfCharacter(from: inteiros)

Pois pelo o que eu consegui entender voce não quer permitir a entrada de números na url/link mas, a meu ver é responsabilidade do usuário colocar um link valido.

No mais, achei seu programa extremamente importante e necessário, eu como acadêmico faço muita referencia em alguns trabalhos, mas as vezes não consigo fazer certo 😅. Logo, quando vi sua proposta fiquei super feliz xD.

Luiz do céu!! Como eu deixei isso passaaaar??? Kkkkkkkk
Vou atualizar, novamente! Estava seguindo uma sugestão de outro issue e não me atentei a isso. Muitíssimo obrigada por me avisar!!

Obrigada pelo feedback!!

@mill3na mill3na closed this as completed Apr 1, 2022
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

No branches or pull requests

2 participants