Skip to content

Comments

Code converted into Object Oriented Approach#1

Open
emekgozluklu wants to merge 4 commits intobasiralab:masterfrom
emekgozluklu:master
Open

Code converted into Object Oriented Approach#1
emekgozluklu wants to merge 4 commits intobasiralab:masterfrom
emekgozluklu:master

Conversation

@emekgozluklu
Copy link

Since the nested function structure is not useful, I changed the code using Object Oriented Approach. It provides:

  • Other people can import and use this class with their datasets
  • No need to pass attributes as arguments. Used class attributes.
  • Methods of the class can be runned seperately for observations.
  • The code is more clean, readable and documentable.

Missing requirements.txt and .gitignore files are added.

nested functions are not usable. using class based approach instead, makes things easier and more useful owing to methods, shared variables etc. Also, people might want to integrate this class to their own projects. It is not possible to do so with nested function structure.
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

Successfully merging this pull request may close these issues.

1 participant