New Ruleset: Object Calisthenics/Solid rules
Open
@manuelpichler is already working on this.
Since Feb 23, 2013.
Enhancement
Feature
New Rule
- Dominant language
- PHP
- Stars
- 2.5k
- Forks
- 361
- Avg merge
- 1d 19h
- Merged PRs (30d)
- 11
Description
- Check for Law Of Demeter violations (how to detect fluent interfaces?)
- Check for more then one nested scope per method
- Usage of else keyword is discouraged
- Usage of boolean flags in method arguments hints at SRP violation.
- Retrieving dependencies through static access (singletons, ...) violates Dependency Inversion Principle
- Multiple instanceof statements in a method/scope hint at violation of open-closed principle
- Interface with more than N methods violates Interface segregation principle
- new Statement in constructor
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.