Skip to content
This repository has been archived by the owner on Oct 31, 2019. It is now read-only.

Refactoring/Cleanup #107

Open
danielchristopher1 opened this issue May 5, 2015 · 0 comments
Open

Refactoring/Cleanup #107

danielchristopher1 opened this issue May 5, 2015 · 0 comments

Comments

@danielchristopher1
Copy link
Contributor

I decided to take a look through our code. There were some things I noticed that we should probably fix.

  • Take out the isAvailable() method out of OpenCVUtils, it shouldn't belong to a util class for OpenCV image processing.
  • Probably need more javadocs.
  • Code to pull json data from files is messy and over complicated
  • Some of our member variables should be put into their base interface/ class.
    Ex:
    private Set set; rather than: private LinkedHashSet set
  • We are using alot of linked data structures, thats probably not needed for most cases.
  • General cleanup would be nice
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant