Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A coupling metric that only counts classes which the base class invokes a method #50

Open
aaghamohammadi opened this issue Mar 2, 2020 · 1 comment

Comments

@aaghamohammadi
Copy link
Contributor

Hello,

I find a conflict between your implementation of CBO and the definition of the original CK paper. In the paper, authors say "CBO for a class is a count of the number of other classes to which it is coupled where two classes are coupled if methods declared in one class use methods or instance variables defined by the other class."

@mauricioaniche
Copy link
Owner

Yes. I think that's an outdated definition. To me, if you have a link between class A and class B, there's coupling. That can be realised by, e.g., defining a list of type A, but never invoking A.

But one might want to know the coupling only about methods that are invoked in other classes. I'll rename this issue and leave it for someone who's willing to implement it!

@mauricioaniche mauricioaniche changed the title Conflict Between the CBO Implementation and the Definition of Original CK Paper A coupling metric that only counts classes which the base class invokes a method Mar 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants