FileVisitor improvements

[micheleorselli]

some more refactorings before tackling the @throws tag dependency collection

1. refactored to avoid nested conditionals
2. preferred checks like $node->class instanceof Node\Name\FullyQualified instead of method_exists($node->class, 'toString'). This allows to skip checks on parent:: self:: static:: and native types since a Node\Name\FullyQualified refers always to a class

[codecov]

Codecov Report

Attention: Patch coverage is 96.05263% with 3 lines in your changes missing coverage. Please review.

Project coverage is 97.75%. Comparing base (556c3f1) to head (7302707).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
src/Analyzer/FileVisitor.php	96.05%	3 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #512      +/-   ##
============================================
+ Coverage     97.47%   97.75%   +0.27%     
+ Complexity      631      615      -16     
============================================
  Files            80       80              
  Lines          1785     1778       -7     
============================================
- Hits           1740     1738       -2     
+ Misses           45       40       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[AlessandroMinoccheri]

LGTM

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud