# You assign values to pluralized versions of variables:
($ips,$times,$locationold,$titles)=split('×',$lines);

# Then you *use* singular versions of those variables:
$line="$ip×$time×$place×$title";
[download]

${$location}{online}
${$loc}{online}
[download]

First off, you would really benefit from use strict. I know that gets said a lot around here, but it's also true. You appear to be using symbolic references (i.e. terms such as ${$locationold}{online}++ in this context), also Not a Good Thing, since apart from anything else they're easily mistaken for hard refs, and it's not obvious what they're pointing to unless you're very careful.

I've not dug into the code perhaps as far as I should, but a couple of things stand out. The $place and $title variables are used only once, suggesting a typo (use strict helps here. Really. It saves me on a daily basis). Also, I'm not sure about what you're splitting on. You say it should be '|+|', but your first split is using 'x' as the delimiter.

There may be fundamental misconceptions in the rest of the code, or it may well be fine. I suppose I might be inclined to use separate %exists hashes for each set of terms you're using it for, in case of overlap (unlikely, but not impossible). I may also have seriously misunderstood something :-)

Anyway, hopefully this will give you a start. Good hunting,

Tim

Update: I just realised, the line $line="$ipx$timex$placex$title"; probably won't work either, because you don't have variables named $ipx, $timex, $placex etc. Writing it

$line=$ip."x".$time."x".$place."x".$title;
[download]

Hey Kage we spoke yesterday on the Chatterbox, and I downloaded and attempted to debug your code. first off there are several statements like this:
${$locationold}{online}++;

doesn't $locationold contain a url string? Why are we dereferencing it? I'm really confused by this. If you're not an expert perl programmer, I think you are overcomplicating things.

also you need to read the perlopentut man page. In this script you try to open variables that haven't been initialized that can clobber you script, and your opens should really be followed by an or die() for this reason. The param function from CGI.pm is defined but never used... You use the same filehandle over and over through different opens, which is a Bad Thing.

Anyway, I applaud your effort, but I suggest in the future, when posting here, you add comments to your script before every major block explaining what you want to do. Indent your loops, and explain what errors the interpreter gave you. There are a lot of people on this site, who are much better programmers than me, and very helpful people to boot, but neither they nor i can help much if we cant understand what you are trying to do.

Fun fact:

foreach $loctitle (@titleb){
$online++;
}
[download]