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.

Sign up for GitHub

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

Jump to bottom

add feature of measuring extendability. #8

Merged
merged 37 commits into from
Dec 24, 2021
Merged

add feature of measuring extendability. #8

merged 37 commits into from
Dec 24, 2021

Conversation

fatemehkarimi
Copy link
Contributor

No description provided.

fatemehkarimi and others added 29 commits November 18, 2021 13:20
@fatemehkarimi
add class file reader to get streams
a03a16f
@fatemehkarimi
add polymorphism class to measure polymorphism
892b395
@fatemehkarimi
store className and its parent to class javaClass
87b03da
@fatemehkarimi
add method parameters to javaMethod when entering methodDeclration gr…
230600f 
…ammer
@fatemehkarimi
remove logs
c44c93a
@fatemehkarimi
adding javaMethods to their corresponding java class
3bf3800
@fatemehkarimi
calculate polymorphism parameter
669b0cf
@fatemehkarimi
fix typpo in setparents
a37b0f8
@fatemehkarimi
using stack to handle nested classes in polymorphism listener
dfc9276
@fatemehkarimi
ignoring method declaration when it is defined inside an enum
74ca0be
@fatemehkarimi
detecting interfaces defined in a program and their methods in javaIn…
007ba09 
…terface class
@fatemehkarimi
set object of interfaces in classes
9f78124
@fatemehkarimi
check method overloading of interfaces in calcPolymorphism
9cfc814
@fatemehkarimi
remove logs
9949a6d
@fatemehkarimi
add docstrings to classes
67ad09b
@fatemehkarimi
add argument parser
baeaf6f
@fatemehkarimi
extracting parents and interfaces of a class with listener
2088ba1
@fatemehkarimi
calculate number of ancestors for each class
079d276
@niloofarizadyar
Add files via upload
6b5ed18
@fatemehkarimi
measuring inheritence using information extracted from polymorphism
4981db3
@niloofarizadyar
Update extendibility.py
2592d8b
@niloofarizadyar
Add files via upload
4ac219d
@fatemehkarimi
add doc strings
581bf45
@fatemehkarimi
merge coupling to other parameters
9ebd2e4
@fatemehkarimi
remove logs
b5fe3eb
@fatemehkarimi
calculating MFA for inheritance as average of the metric of all classes
cbec2fa
@fatemehkarimi
remove logs
aacf10c
@fatemehkarimi
add modifier of method
1370629
@fatemehkarimi
calculate average of polymorphism of each class as final metric for p…
0c755e2 
…roject
m-zakeri
m-zakeri requested changes Dec 14, 2021
View reviewed changes
Copy link
Owner

@m-zakeri m-zakeri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1- Follow PEP instructions for naming python entities, specifically module names:
https://www.python.org/dev/peps/pep-0008/

2- Merge main.py to extendibility,py for better consistency

3- Ensure fixing issues discussed in presentation session

@m-zakeri
Copy link
Owner

ToDo:
1- Follow PEP instructions for naming python entities, specifically module names:
https://www.python.org/dev/peps/pep-0008/
2- Merge main.py to extendibility.py for better consistency
@fatemehkarimi @niloofarizadyar

@fatemehkarimi
Copy link
Contributor Author

@m-zakeri
I know that camelCase should not be used according to PEP 8, but ANTLR uses camelCase naming convention, and I did so to be consistent with it. do you accept this explanation or I should change the names?

@m-zakeri
Copy link
Owner

@m-zakeri I know that camelCase should not be used according to PEP 8, but ANTLR uses camelCase naming convention, and I did so to be consistent with it. do you accept this explanation or I should change the names?

I think we should not repeat others' mistakes.
Also, please address my second commnet.

@m-zakeri m-zakeri self-requested a review December 18, 2021 08:38
@fatemehkarimi
Copy link
Contributor Author

@m-zakeri The branch is ready.
please merge

@m-zakeri m-zakeri merged commit 7b103af into m-zakeri:main Dec 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m-zakeri m-zakeri Awaiting requested review from m-zakeri

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fatemehkarimi @m-zakeri @niloofarizadyar