visitFile

[Ali el ghourassi]

Hello everybody

in  Rule's class sometime we find method visitFile(@Nullable AstNode astNode) this method can analyse one file by one and give the result of every file  but i want analyse all the files in same time with one result  ,there's other method how do that ?

Thanks.

[Ali el ghourassi]

Example

@Override

public void visitFile(@Nullable AstNode astNode) {

lines = Collections.emptyList();

try {

lines = Files.readLines(getContext().getFile(), charset);

[dinesh.bo...@gmail.com]

Hello Ali

In your example you're reading all the lines but of just one file, not of every files.

Can you perhaps explain what you are trying to achieve, and also on which language?

That way we can give you the best solution

[Ali el ghourassi]

Thanks Dinesh for your response

 yes in my example i achieve one file  ,

i am trying to find functions unused ,

for example i can find declaration of function or multiple functions in file and it's  used in other file

so whene i achieve the first file i find the declartion but i can't find the apple of the function in same file so he say the function is unused  

the language is ncl

Le jeudi 28 décembre 2017 09:07:29 UTC, Ali el ghourassi a écrit :

[Ali el ghourassi]

what i am trying to do is to achieve all the files in one time

Le jeudi 28 décembre 2017 09:07:29 UTC, Ali el ghourassi a écrit :

[dinesh.bo...@gmail.com]

I see, so this is on your own custom language plugin.

If you do not need the Abstract Syntax Tree (and so there is no requirement to use SSLR), then you can iterate on all "InputFiles" in your plugin sensor.

See https://github.com/SonarSource/sonar-custom-plugin-example/blob/master/src/main/java/org/sonarsource/plugins/example/rules/CreateIssuesOnJavaFilesSensor.java#L53

If you do need to access the AST, then you can process each file one by one using visitFile() etc.

Your unused function rule should keep track of all functions which have been declared and all functions that have been invoked.

Then, you need at the very end to compute the diff between these two sets to create your unused function issues.

To do so, you can add a method in your check class that you invoke from your plugin only after all files will have been scanned.

Does this make sense?

On Thursday, December 28, 2017 at 10:07:29 AM UTC+1, Ali el ghourassi wrote:

[Ali el ghourassi]

hi Dinesh

but how can i add a method to my check class that will be execute after all files will have been scanned .

Thanks.

Le jeudi 28 décembre 2017 09:07:29 UTC, Ali el ghourassi a écrit :

[Ali el ghourassi]

Hello everybody

i wonder if there is any method of how can scan all files one time .

Thanks

Le jeudi 28 décembre 2017 09:07:29 UTC, Ali el ghourassi a écrit :

[dinesh.bo...@sonarsource.com]

No there is not such method provided out of the box.

The way to go is to keep a state between the analysis of multiple files, and process it at the end.

If you think about it, that's what a method that scans multiple files "at once" would end up doing behind the scene anyway

As this is your plugin, you have control over your code, and you should be able to find a way to do this :-)

On Thursday, December 28, 2017 at 10:07:29 AM UTC+1, Ali el ghourassi wrote: