Replace this
foreach ($i=0;$i<=$num;$i++) {
%ServerStuatus = ( "$Ip[$i]" => ["$Names[$i]", "unknown"] );
}
With
%ServerStatus = map{
$Ip[$_] => [ $Names[$_], "unknown" ]
} 0 .. $#Ip;
Please note that quoting all your variables that way is
- Unnecessary.
- Is making you code harder to read than it need be, and requiring you to type more.
- Making Perl work harder than it needs to.
- Under some circumstances, can cause errors. Specifically, when passing variables to built-ins that expect pass-by-reference.
Also, you've misspelt %ServerStuatus. Not that I can say anything, my spelling is atrocious. But I always use strict, so if I later spell it correctly, Perl would warn me. Without it, as your code appears to be, it won't be able to.
Examine what is said, not who speaks.
"But you should never overestimate the ingenuity of the sceptics to come up with a counter-argument." -Myles Allen
"Think for yourself!" - Abigail
"Time is a poor substitute for thought"--theorbtwo
"Efficiency is intelligent laziness." -David Dunham
"Memory, processor, disk in that order on the hardware side. Algorithm, algorithm, algorithm on the code side." - tachyon