Assignment to a hash is in list context, so almost all of those param() method calls are in list context. Do you know what that method returns in list context?

Besides that, you're mixing method calls with function calls. That's generally a very bad idea.

Everything should be $q->param( ... ), and you should prefix all of those with scalar.

push @emp, {
                "employer" => scalar $q->param("employer$emp_count"),
                "street"   => scalar $q->param("street$emp_count"),
                ...
};
[download]