khaledosman / terasoft-12

Automatically exported from code.google.com/p/terasoft-12
0 stars 0 forks source link

edit Environment3 class design #231

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
attributes and methods related to the game such as user, distanceBar, 
hasCollidedWithBall, ballShot, GenerateSolvable(), IsSolvable() and HasScored() 
would be better moved to class Experiment3

Original issue reported on code.google.com by bishoyba...@gmail.com on 11 May 2012 at 11:44

GoogleCodeExporter commented 9 years ago
Personally, i agree

Original comment by q3a...@gmail.com on 13 May 2012 at 4:11

GoogleCodeExporter commented 9 years ago
I disagree. Experiment 3 is a screen, and these methods are unrelated to it.

Original comment by ams.sa...@gmail.com on 17 May 2012 at 11:08

GoogleCodeExporter commented 9 years ago
these methods are not related to Environment3 they are related more to the
games logic so they belong to the Experimen3 class

Original comment by bishoyba...@gmail.com on 17 May 2012 at 11:17

GoogleCodeExporter commented 9 years ago
HasScored() at least which is my method, is related to the environment as it 
calculates depending on the friction for example of the environment. None of 
them are related to the class Experiment3 as it is a screen.

Original comment by ams.sa...@gmail.com on 17 May 2012 at 11:56