Kyle-Pu / Mathematics

Various math formulas coded in Java to help students all across the world!
MIT License
40 stars 59 forks source link

Project Code Analysis #97

Open geogiadim opened 4 years ago

geogiadim commented 4 years ago

Hello everyone,

I tried to make a code analysis for this project. For my analysis I used the enviroment of InteliJ using the CodeMr plug in. I attatched you my results in the pdf file.

codeAnalysis.pdf

LJWxyz commented 4 years ago

lack of cohension and coupling test is meaningless in this project. Say,function a just add two int,do we do:return SOMEDTYPE.value+SOMEDTYPE.value The most meaningful test is probably problematic class or some crucial performance or logic error test. these style test normally does inherit some other developers experience bias and false concept. So.. Contributors,kudos on keeping no prolematic classes zero.

jwheeler31 commented 4 years ago

@geogiadim, I’d recommend just sticking to the math aspect as I attempted to refactor the project about a year ago. The end result was rejected because there were “too many changes.” I don’t want to steal @Kyle-Pu’s project, so I didn’t fork this and recruit developers.