[Redgie]

Do you actually call the function at any point?

Also, you do not need to use else if, as they can only meet one condition of the same variable.