Alteração na comparção das resposta para retirar espaços em branco#2
Alteração na comparção das resposta para retirar espaços em branco#2Michel-MSouza wants to merge 1 commit into
Conversation
rwehresmann
left a comment
There was a problem hiding this comment.
Obrigado pelo interesse no projeto, @Michel-MSouza ! Solicitei apenas uma pequena alteração, assim que acertarmos isso farei o merge.
| results << { | ||
| test_case: test_case, | ||
| correct: result == test_case.output, | ||
| correct: result.gsub(/\s+/, '') == test_case.output.gsub(/\s+/, ''), |
There was a problem hiding this comment.
O problema dessa regex é que ela remove TODOS os espaços de uma string (" ola mundo " se torna "olamundo"). Não consigo pensar numa situação em que isso ocasione um erro agora, mas é melhor não fazermos assim. Podemos utilizar o strip (result.strip), ele servirá melhor para o nosso caso.
|
Boa noite Rodrigo,
Realmente retirar todos os espaços em brancos pode causar eventuais
problemas, mas por enquanto servirá como solução temporária. Eu ainda não
estou bem familiarizado com ruby então ainda preciso estudarr sobre
strip(result.strip).
2018-05-09 14:46 GMT-03:00 Rodrigo Walter Ehresmann <
notifications@github.com>:
… ***@***.**** requested changes on this pull request.
Obrigado pelo interesse no projeto, @Michel-MSouza
<https://github.com/Michel-MSouza> ! Solicitei apenas uma pequena
alteração, assim que acertarmos isso farei o merge.
------------------------------
In app/classes/answer_tester.rb
<#2 (comment)>
:
> @@ -18,7 +18,7 @@ def test
results << {
test_case: test_case,
- correct: result == test_case.output,
+ correct: result.gsub(/\s+/, '') == test_case.output.gsub(/\s+/, ''),
O problema dessa regex é que ela remove TODOS os espaços de uma string ("
ola mundo " se torna "olamundo"). Não consigo pensar numa situação em que
isso ocasione um erro agora, mas é melhor não fazermos assim. Podemos
utilizar o strip (result.strip), ele servirá melhor para o nosso caso.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AeBMKyJ5bDNobYIKqQav2hZ_Jmp3b_sbks5twytZgaJpZM4TyAxv>
.
|
|
Certo, @Michel-MSouza . Quando for olhar, o |
|
Boa noite Rodrigo,
Estava olhando o código e não estou conseguindo achar a parte que
criptografa o nome do usuário no rank. Você poderia me indicar qual a
classe responsável por isso?
Att,
Michel
2018-05-14 21:17 GMT-03:00 Rodrigo Walter Ehresmann <
notifications@github.com>:
… Certo, @Michel-MSouza <https://github.com/Michel-MSouza> . Quando for
olhar, o strip é bem simples. Seu funcionamento é como exemplificado
abaixo:
" olá mundo ".strip
=> "olá mundo"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AeBMK7wAwLTzyIP2gvV1MRnzdJEWPoWvks5tyh6GgaJpZM4TyAxv>
.
|
|
@Michel-MSouza é a classe Fica salvo no campo |
|
Bom dia Rodrigo,
2018-05-30 14:23 GMT-03:00 Rodrigo Walter Ehresmann <
notifications@github.com>:
… @Michel-MSouza <https://github.com/Michel-MSouza> é a classe User:
https://github.com/rwehresmann/farma_alg_reborn/
blob/master/app/models/user.rb
Fica salvo no campo anonymous_id, que é preenchido automaticamente assim
que os dados de um usuário são validados (before_validation callback).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AeBMK_J5I5RPBOszqKIC10iN1cwuoB2Vks5t3tWmgaJpZM4TyAxv>
.
|
|
Bom dia Rodrigo,
Perdão pela mensagem anterior, foi um erro da minha parte. Rodrigo gostaria
de saber se você ainda possui as listas de exercícios escritas pelo
professor Direne. Se você tiver elas poderia me passar? O professor Marcos
sugeriu que eu as utilizasse no farma-alg.
Att,
Michel
2018-06-06 10:29 GMT-03:00 Michel Melo de Souza <mms15@inf.ufpr.br>:
… Bom dia Rodrigo,
2018-05-30 14:23 GMT-03:00 Rodrigo Walter Ehresmann <
***@***.***>:
> @Michel-MSouza <https://github.com/Michel-MSouza> é a classe User:
> https://github.com/rwehresmann/farma_alg_reborn/blob/master/
> app/models/user.rb
>
> Fica salvo no campo anonymous_id, que é preenchido automaticamente assim
> que os dados de um usuário são validados (before_validation callback).
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#2 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AeBMK_J5I5RPBOszqKIC10iN1cwuoB2Vks5t3tWmgaJpZM4TyAxv>
> .
>
|
|
Encaminhei no seu email @Michel-MSouza |
No description provided.