Well, it's hard figure out what's really going on from a "paraphrase" but if there is infinite loop, it's probably because constructComplex() calls itself recursively and the conditions that mediate the recusion do not change.

Here's my advice though: use Math::Trig. It has routines that convert from/to polar coordinates and a whole lot more.