Apart from everything that has already been said here: why do you divide the height by 2 to get the radius and then immediately multiply that radius by 2 again, without ever using the radius???
Computers are really fast at calculating, but there is no reason to let them perform unnecessary math operations;)
