Fix motivating pr #96
Conversation
| public void cleanText(){ | ||
| DFPBaseSample inst = new DFPBaseSample(); | ||
| DFPBaseSample inst = new DFPBaseSample("the the dog"); | ||
| inst.normalizeWhiteSpace(); //Left | ||
| inst.removeComments(); | ||
| inst.removeDuplicateWords(); //Right |
There was a problem hiding this comment.
Acho que aqui precisaria de uma linha a mais com this.text = inst.text, ou então usar o this em todas, já que está manipulando um objeto novo
There was a problem hiding this comment.
mais que isso, @nathaliafab , faz sentido criar o objeto aí? na versão que você encaminhou não tinha isso, até onde lembro. @galilasmb não copiasse o código de @nathaliafab ?
There was a problem hiding this comment.
Com o this não funciona, temos que saber qual objeto está manipulando, por isso usamos o objeto instanciado em vez do this.
There was a problem hiding this comment.
como assim o this não funciona, @galilasmb ? deveria sim ser possível analisar esse método com this. se não é possível, há algo a ser ajustado na implementação da análise.
There was a problem hiding this comment.
Posso testar mais, mas na última vez ele não conseguia achar o points-to.
There was a problem hiding this comment.
@pauloborba @galilasmb usando spark realmente o this não funciona. Precisa de um new. Usando o cha, funcionou aqui para mim.
Em um exemplo em que temos this.chamadaDeMetodo() esse método não é acessado pela análise que usa spark. Pelo menos em OA, funciona assim atualmente.
|
Mas aí ele deveria assumir que pode ser qualquer instância.
Discute isso com Matheus
… On 8 Oct 2024, at 10:54, Galileu Santos ***@***.***> wrote:
@galilasmb commented on this pull request.
In src/test/java/br/unb/cic/analysis/samples/DFPBaseSample.java <#96 (comment)>:
> public void cleanText(){
- DFPBaseSample inst = new DFPBaseSample();
+ DFPBaseSample inst = new DFPBaseSample("the the dog");
inst.normalizeWhiteSpace(); //Left
inst.removeComments();
inst.removeDuplicateWords(); //Right
Posso testar mais, mas na última vez ele não conseguia achar o points-to.
—
Reply to this email directly, view it on GitHub <#96 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAJGYJDWMWMSOXSCWRHN52LZ2PP25AVCNFSM6AAAAABL3FUYKCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNJUGU3TQNBUHE>.
You are receiving this because your review was requested.
|
No description provided.